mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
5a1ee4e3ee
[PR #7790/787fa462 backport][stable-8] fix(modules/gitlab_runner): Use correct argument to list all runners (#8032)
fix(modules/gitlab_runner): Use correct argument to list all runners (#7790)
* fix(modules/gitlab_runner): Use correct argument to list all runners
python-gitlab 4.0.0 removed support for the `as_list=False` parameter.
This functionality is now available as `iterator=True`.
Without this change, the module actually only retrieves the first
20 results, which can lead to non-idempotent behavior, such as
registering a runner again.
* Add changelog entry (#7790)
* gitlab_runner: Check python-gitlab version when listing runners
* gitlab: Add list_all_kwargs variable to module_utils
* refactor(gitlab modules): use list_all_kwargs where it helps (#7790)
I did not change every instance of all=True or all=False, only those
which could obviously benefit from simplifying:
* Code using `all=True` but then searching for any items that match a
condition (no need to collect the full list).
* Code that basically reimplements `all=True` with manual pagination.
(These could be changed to `all=True`, but `list_all_kwargs` also
sets per_page to 100, to gather data faster.)
* gitlab_instance_variable: Use list_all_kwargs
* Add new changelog entry for gitlab module changes (#7790)
(cherry picked from commit 787fa46217
)
Co-authored-by: patchback[bot] <45432694+patchback[bot]@users.noreply.github.com>
Co-authored-by: Mike Wadsten <mikewadsten@gmail.com>
360 lines
12 KiB
Python
360 lines
12 KiB
Python
#!/usr/bin/python
|
|
# -*- coding: utf-8 -*-
|
|
|
|
# Copyright (c) 2023, Benedikt Braunger (bebr@adm.ku.dk)
|
|
# Based on code:
|
|
# Copyright (c) 2020, Florent Madiot (scodeman@scode.io)
|
|
# Copyright (c) 2019, Markus Bergholz (markuman@gmail.com)
|
|
# GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt)
|
|
# SPDX-License-Identifier: GPL-3.0-or-later
|
|
from __future__ import absolute_import, division, print_function
|
|
__metaclass__ = type
|
|
|
|
DOCUMENTATION = r'''
|
|
module: gitlab_instance_variable
|
|
short_description: Creates, updates, or deletes GitLab instance variables
|
|
version_added: 7.1.0
|
|
description:
|
|
- Creates a instance variable if it does not exist.
|
|
- When a instance variable does exist, its value will be updated if the values are different.
|
|
- Support for instance variables requires GitLab >= 13.0.
|
|
- Variables which are not mentioned in the modules options, but are present on the GitLab instance,
|
|
will either stay (O(purge=false)) or will be deleted (O(purge=true)).
|
|
author:
|
|
- Benedikt Braunger (@benibr)
|
|
requirements:
|
|
- python >= 2.7
|
|
- python-gitlab python module
|
|
extends_documentation_fragment:
|
|
- community.general.auth_basic
|
|
- community.general.gitlab
|
|
- community.general.attributes
|
|
|
|
attributes:
|
|
check_mode:
|
|
support: full
|
|
diff_mode:
|
|
support: none
|
|
|
|
options:
|
|
state:
|
|
description:
|
|
- Create or delete instance variable.
|
|
default: present
|
|
type: str
|
|
choices: ["present", "absent"]
|
|
purge:
|
|
description:
|
|
- When set to V(true), delete all variables which are not mentioned in the task.
|
|
default: false
|
|
type: bool
|
|
variables:
|
|
description:
|
|
- A list of dictionaries that represents CI/CD variables.
|
|
default: []
|
|
type: list
|
|
elements: dict
|
|
suboptions:
|
|
name:
|
|
description:
|
|
- The name of the variable.
|
|
type: str
|
|
required: true
|
|
value:
|
|
description:
|
|
- The variable value.
|
|
- Required when O(state=present).
|
|
type: str
|
|
masked:
|
|
description:
|
|
- Whether variable value is masked or not.
|
|
type: bool
|
|
default: false
|
|
protected:
|
|
description:
|
|
- Whether variable value is protected or not.
|
|
type: bool
|
|
default: false
|
|
variable_type:
|
|
description:
|
|
- Whether a variable is an environment variable (V(env_var)) or a file (V(file)).
|
|
type: str
|
|
choices: [ "env_var", "file" ]
|
|
default: env_var
|
|
'''
|
|
|
|
|
|
EXAMPLES = r'''
|
|
- name: Set or update some CI/CD variables
|
|
community.general.gitlab_instance_variable:
|
|
api_url: https://gitlab.com
|
|
api_token: secret_access_token
|
|
purge: false
|
|
variables:
|
|
- name: ACCESS_KEY_ID
|
|
value: abc1312cba
|
|
- name: SECRET_ACCESS_KEY
|
|
value: 1337
|
|
masked: true
|
|
protected: true
|
|
variable_type: env_var
|
|
|
|
- name: Delete one variable
|
|
community.general.gitlab_instance_variable:
|
|
api_url: https://gitlab.com
|
|
api_token: secret_access_token
|
|
state: absent
|
|
variables:
|
|
- name: ACCESS_KEY_ID
|
|
'''
|
|
|
|
RETURN = r'''
|
|
instance_variable:
|
|
description: Four lists of the variablenames which were added, updated, removed or exist.
|
|
returned: always
|
|
type: dict
|
|
contains:
|
|
added:
|
|
description: A list of variables which were created.
|
|
returned: always
|
|
type: list
|
|
sample: ['ACCESS_KEY_ID', 'SECRET_ACCESS_KEY']
|
|
untouched:
|
|
description: A list of variables which exist.
|
|
returned: always
|
|
type: list
|
|
sample: ['ACCESS_KEY_ID', 'SECRET_ACCESS_KEY']
|
|
removed:
|
|
description: A list of variables which were deleted.
|
|
returned: always
|
|
type: list
|
|
sample: ['ACCESS_KEY_ID', 'SECRET_ACCESS_KEY']
|
|
updated:
|
|
description: A list pre-existing variables whose values have been set.
|
|
returned: always
|
|
type: list
|
|
sample: ['ACCESS_KEY_ID', 'SECRET_ACCESS_KEY']
|
|
'''
|
|
|
|
from ansible.module_utils.basic import AnsibleModule
|
|
from ansible.module_utils.api import basic_auth_argument_spec
|
|
from ansible_collections.community.general.plugins.module_utils.gitlab import (
|
|
auth_argument_spec, gitlab_authentication, ensure_gitlab_package, filter_returned_variables, list_all_kwargs
|
|
)
|
|
|
|
|
|
class GitlabInstanceVariables(object):
|
|
|
|
def __init__(self, module, gitlab_instance):
|
|
self.instance = gitlab_instance
|
|
self._module = module
|
|
|
|
def list_all_instance_variables(self):
|
|
return list(self.instance.variables.list(**list_all_kwargs))
|
|
|
|
def create_variable(self, var_obj):
|
|
if self._module.check_mode:
|
|
return True
|
|
var = {
|
|
"key": var_obj.get('key'),
|
|
"value": var_obj.get('value'),
|
|
"masked": var_obj.get('masked'),
|
|
"protected": var_obj.get('protected'),
|
|
"variable_type": var_obj.get('variable_type'),
|
|
}
|
|
|
|
self.instance.variables.create(var)
|
|
return True
|
|
|
|
def update_variable(self, var_obj):
|
|
if self._module.check_mode:
|
|
return True
|
|
self.delete_variable(var_obj)
|
|
self.create_variable(var_obj)
|
|
return True
|
|
|
|
def delete_variable(self, var_obj):
|
|
if self._module.check_mode:
|
|
return True
|
|
self.instance.variables.delete(var_obj.get('key'))
|
|
return True
|
|
|
|
|
|
def compare(requested_variables, existing_variables, state):
|
|
# we need to do this, because it was determined in a previous version - more or less buggy
|
|
# basically it is not necessary and might results in more/other bugs!
|
|
# but it is required and only relevant for check mode!!
|
|
# logic represents state 'present' when not purge. all other can be derived from that
|
|
# untouched => equal in both
|
|
# updated => name and scope are equal
|
|
# added => name and scope does not exist
|
|
untouched = list()
|
|
updated = list()
|
|
added = list()
|
|
|
|
if state == 'present':
|
|
existing_key_scope_vars = list()
|
|
for item in existing_variables:
|
|
existing_key_scope_vars.append({'key': item.get('key')})
|
|
|
|
for var in requested_variables:
|
|
if var in existing_variables:
|
|
untouched.append(var)
|
|
else:
|
|
compare_item = {'key': var.get('name')}
|
|
if compare_item in existing_key_scope_vars:
|
|
updated.append(var)
|
|
else:
|
|
added.append(var)
|
|
|
|
return untouched, updated, added
|
|
|
|
|
|
def native_python_main(this_gitlab, purge, requested_variables, state, module):
|
|
|
|
change = False
|
|
return_value = dict(added=list(), updated=list(), removed=list(), untouched=list())
|
|
|
|
gitlab_keys = this_gitlab.list_all_instance_variables()
|
|
before = [x.attributes for x in gitlab_keys]
|
|
|
|
existing_variables = filter_returned_variables(gitlab_keys)
|
|
|
|
for item in requested_variables:
|
|
item['key'] = item.pop('name')
|
|
item['value'] = str(item.get('value'))
|
|
if item.get('protected') is None:
|
|
item['protected'] = False
|
|
if item.get('masked') is None:
|
|
item['masked'] = False
|
|
if item.get('variable_type') is None:
|
|
item['variable_type'] = 'env_var'
|
|
|
|
if module.check_mode:
|
|
untouched, updated, added = compare(requested_variables, existing_variables, state)
|
|
|
|
if state == 'present':
|
|
add_or_update = [x for x in requested_variables if x not in existing_variables]
|
|
for item in add_or_update:
|
|
try:
|
|
if this_gitlab.create_variable(item):
|
|
return_value['added'].append(item)
|
|
|
|
except Exception:
|
|
if this_gitlab.update_variable(item):
|
|
return_value['updated'].append(item)
|
|
|
|
if purge:
|
|
# refetch and filter
|
|
gitlab_keys = this_gitlab.list_all_instance_variables()
|
|
existing_variables = filter_returned_variables(gitlab_keys)
|
|
|
|
remove = [x for x in existing_variables if x not in requested_variables]
|
|
for item in remove:
|
|
if this_gitlab.delete_variable(item):
|
|
return_value['removed'].append(item)
|
|
|
|
elif state == 'absent':
|
|
# value does not matter on removing variables.
|
|
# key and environment scope are sufficient
|
|
for item in existing_variables:
|
|
item.pop('value')
|
|
item.pop('variable_type')
|
|
for item in requested_variables:
|
|
item.pop('value')
|
|
item.pop('variable_type')
|
|
|
|
if not purge:
|
|
remove_requested = [x for x in requested_variables if x in existing_variables]
|
|
for item in remove_requested:
|
|
if this_gitlab.delete_variable(item):
|
|
return_value['removed'].append(item)
|
|
|
|
else:
|
|
for item in existing_variables:
|
|
if this_gitlab.delete_variable(item):
|
|
return_value['removed'].append(item)
|
|
|
|
if module.check_mode:
|
|
return_value = dict(added=added, updated=updated, removed=return_value['removed'], untouched=untouched)
|
|
|
|
if len(return_value['added'] + return_value['removed'] + return_value['updated']) > 0:
|
|
change = True
|
|
|
|
gitlab_keys = this_gitlab.list_all_instance_variables()
|
|
after = [x.attributes for x in gitlab_keys]
|
|
|
|
return change, return_value, before, after
|
|
|
|
|
|
def main():
|
|
argument_spec = basic_auth_argument_spec()
|
|
argument_spec.update(auth_argument_spec())
|
|
argument_spec.update(
|
|
purge=dict(type='bool', required=False, default=False),
|
|
variables=dict(type='list', elements='dict', required=False, default=list(), options=dict(
|
|
name=dict(type='str', required=True),
|
|
value=dict(type='str', no_log=True),
|
|
masked=dict(type='bool', default=False),
|
|
protected=dict(type='bool', default=False),
|
|
variable_type=dict(type='str', default='env_var', choices=["env_var", "file"])
|
|
)),
|
|
state=dict(type='str', default="present", choices=["absent", "present"]),
|
|
)
|
|
|
|
module = AnsibleModule(
|
|
argument_spec=argument_spec,
|
|
mutually_exclusive=[
|
|
['api_username', 'api_token'],
|
|
['api_username', 'api_oauth_token'],
|
|
['api_username', 'api_job_token'],
|
|
['api_token', 'api_oauth_token'],
|
|
['api_token', 'api_job_token'],
|
|
],
|
|
required_together=[
|
|
['api_username', 'api_password'],
|
|
],
|
|
required_one_of=[
|
|
['api_username', 'api_token', 'api_oauth_token', 'api_job_token']
|
|
],
|
|
supports_check_mode=True
|
|
)
|
|
ensure_gitlab_package(module)
|
|
|
|
purge = module.params['purge']
|
|
state = module.params['state']
|
|
|
|
variables = module.params['variables']
|
|
|
|
if state == 'present':
|
|
if any(x['value'] is None for x in variables):
|
|
module.fail_json(msg='value parameter is required in state present')
|
|
|
|
gitlab_instance = gitlab_authentication(module)
|
|
|
|
this_gitlab = GitlabInstanceVariables(module=module, gitlab_instance=gitlab_instance)
|
|
|
|
changed, raw_return_value, before, after = native_python_main(this_gitlab, purge, variables, state, module)
|
|
|
|
# postprocessing
|
|
for item in after:
|
|
item['name'] = item.pop('key')
|
|
for item in before:
|
|
item['name'] = item.pop('key')
|
|
|
|
untouched_key_name = 'key'
|
|
if not module.check_mode:
|
|
untouched_key_name = 'name'
|
|
raw_return_value['untouched'] = [x for x in before if x in after]
|
|
|
|
added = [x.get('key') for x in raw_return_value['added']]
|
|
updated = [x.get('key') for x in raw_return_value['updated']]
|
|
removed = [x.get('key') for x in raw_return_value['removed']]
|
|
untouched = [x.get(untouched_key_name) for x in raw_return_value['untouched']]
|
|
return_value = dict(added=added, updated=updated, removed=removed, untouched=untouched)
|
|
|
|
module.exit_json(changed=changed, instance_variable=return_value)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
main()
|