add instance groups

This commit is contained in:
sean-m-ssullivan 2021-04-03 14:00:56 -05:00
parent f35ce93fb1
commit f9d2db696b
6 changed files with 107 additions and 9 deletions

View File

@ -61,6 +61,11 @@ options:
description:
- Credentials to be used by hosts belonging to this inventory when accessing Red Hat Insights API.
type: str
instance_groups:
description:
- list of Instance Groups for this Organization to run on.
type: list
elements: str
state:
description:
- Desired state of the resource.
@ -104,6 +109,7 @@ def main():
variables=dict(type='dict'),
kind=dict(choices=['', 'smart'], default=''),
host_filter=dict(),
instance_groups=dict(type="list", elements='str'),
insights_credential=dict(),
state=dict(choices=['present', 'absent'], default='present'),
)
@ -158,12 +164,20 @@ def main():
if insights_credential is not None:
inventory_fields['insights_credential'] = module.resolve_name_to_id('credentials', insights_credential)
association_fields = {}
instance_group_names = module.params.get('instance_groups')
if instance_group_names is not None:
association_fields['instance_groups'] = []
for item in instance_group_names:
association_fields['instance_groups'].append(module.resolve_name_to_id('instance_groups', item))
# We need to perform a check to make sure you are not trying to convert a regular inventory into a smart one.
if inventory and inventory['kind'] == '' and inventory_fields['kind'] == 'smart':
module.fail_json(msg='You cannot turn a regular inventory into a "smart" inventory.')
# If the state was present and we can let the module build or update the existing inventory, this will return on its own
module.create_or_update_if_needed(inventory, inventory_fields, endpoint='inventories', item_type='inventory')
module.create_or_update_if_needed(inventory, inventory_fields, endpoint='inventories', item_type='inventory', associations=association_fields,)
if __name__ == '__main__':

View File

@ -86,6 +86,11 @@ options:
description:
- Execution Environment to use for the JT.
type: str
instance_groups:
description:
- list of Instance Groups for this Organization to run on.
type: list
elements: str
forks:
description:
- The number of parallel or simultaneous processes to use while executing the playbook.
@ -366,6 +371,7 @@ def main():
vault_credential=dict(),
credentials=dict(type='list', elements='str'),
execution_environment=dict(),
instance_groups=dict(type="list", elements='str'),
forks=dict(type='int'),
limit=dict(),
verbosity=dict(type='int', choices=[0, 1, 2, 3, 4], default=0),
@ -564,6 +570,12 @@ def main():
for item in notifications_error:
association_fields['notification_templates_error'].append(module.resolve_name_to_id('notification_templates', item))
instance_group_names = module.params.get('instance_groups')
if instance_group_names is not None:
association_fields['instance_groups'] = []
for item in instance_group_names:
association_fields['instance_groups'].append(module.resolve_name_to_id('instance_groups', item))
on_change = None
new_spec = module.params.get('survey_spec')
if new_spec is not None:

View File

@ -45,6 +45,11 @@ options:
default: "present"
choices: ["present", "absent"]
type: str
instance_groups:
description:
- list of Instance Groups for this Organization to run on.
type: list
elements: str
notification_templates_started:
description:
- list of notifications to send on start
@ -108,6 +113,7 @@ def main():
description=dict(),
default_environment=dict(),
max_hosts=dict(type='int', default="0"),
instance_groups=dict(type="list", elements='str'),
notification_templates_started=dict(type="list", elements='str'),
notification_templates_success=dict(type="list", elements='str'),
notification_templates_error=dict(type="list", elements='str'),
@ -124,7 +130,6 @@ def main():
description = module.params.get('description')
default_ee = module.params.get('default_environment')
max_hosts = module.params.get('max_hosts')
# instance_group_names = module.params.get('instance_groups')
state = module.params.get('state')
# Attempt to look up organization based on the provided name
@ -136,6 +141,12 @@ def main():
# Attempt to look up associated field items the user specified.
association_fields = {}
instance_group_names = module.params.get('instance_groups')
if instance_group_names is not None:
association_fields['instance_groups'] = []
for item in instance_group_names:
association_fields['instance_groups'].append(module.resolve_name_to_id('instance_groups', item))
notifications_start = module.params.get('notification_templates_started')
if notifications_start is not None:
association_fields['notification_templates_started'] = []

View File

@ -8,6 +8,7 @@
inv_name1: "AWX-Collection-tests-tower_inventory-inv1-{{ test_id }}"
inv_name2: "AWX-Collection-tests-tower_inventory-inv2-{{ test_id }}"
cred_name1: "AWX-Collection-tests-tower_inventory-cred1-{{ test_id }}"
group_name1: "AWX-Collection-tests-tower_instance_group-group1-{{ test_id }}"
- block:
- name: Create an Insights Credential
@ -25,17 +26,30 @@
that:
- "result is changed"
- name: Create an Inventory
tower_inventory:
name: "{{ inv_name1 }}"
organization: Default
insights_credential: "{{ result.id }}"
- name: Create an Instance Group
tower_instance_group:
name: "{{ group_name1 }}"
state: present
register: result
- assert:
that:
- "result is changed"
- name: Create an Inventory
tower_inventory:
name: "{{ inv_name1 }}"
organization: Default
insights_credential: "{{ result.id }}"
instance_groups:
- "{{ group_name1 }}"
state: present
register: result
- assert:
that:
- "result is changed"
- name: Test Inventory module idempotency
tower_inventory:
name: "{{ result.id }}"
@ -158,6 +172,11 @@
- "{{ inv_name2 }}"
- "copy_{{ inv_name1 }}"
- name: Delete the instance groups
tower_instance_group:
name: "{{ group_name1 }}"
state: absent
- name: Delete Insights Credential
tower_credential:
name: "{{ cred_name1 }}"

View File

@ -14,6 +14,7 @@
lab1: "AWX-Collection-tests-tower_job_template-lab1-{{ test_id }}"
email_not: "AWX-Collection-tests-tower_job_template-email-not-{{ test_id }}"
webhook_not: "AWX-Collection-tests-tower_notification_template-wehbook-not-{{ test_id }}"
group_name1: "AWX-Collection-tests-tower_instance_group-group1-{{ test_id }}"
- name: Create a Demo Project
tower_project:
@ -48,6 +49,16 @@
name: "{{ lab1 }}"
organization: Default
- name: Create an Instance Group
tower_instance_group:
name: "{{ group_name1 }}"
state: present
register: result
- assert:
that:
- "result is changed"
- name: Add email notification
tower_notification_template:
name: "{{ email_not }}"
@ -81,7 +92,11 @@
project: "{{ proj1 }}"
inventory: Demo Inventory
playbook: hello_world.yml
credentials: ["{{ cred1 }}", "{{ cred2 }}"]
credentials:
- "{{ cred1 }}"
- "{{ cred2 }}"
instance_groups:
- "{{ group_name1 }}"
job_type: run
state: present
register: jt1_result
@ -384,6 +399,11 @@
organization: Default
state: absent
- name: Delete the instance groups
tower_instance_group:
name: "{{ group_name1 }}"
state: absent
- name: Delete webhook notification
tower_notification_template:
name: "{{ webhook_not }}"

View File

@ -1,7 +1,12 @@
---
- name: Generate a test ID
set_fact:
test_id: "{{ lookup('password', '/dev/null chars=ascii_letters length=16') }}"
- name: Generate an org name
set_fact:
org_name: "AWX-Collection-tests-tower_organization-org-{{ lookup('password', '/dev/null chars=ascii_letters length=16') }}"
org_name: "AWX-Collection-tests-tower_organization-org-{{ test_id }}"
group_name1: "AWX-Collection-tests-tower_instance_group-group1-{{ test_id }}"
- name: Make sure {{ org_name }} is not there
tower_organization:
@ -28,10 +33,22 @@
that:
- "result is not changed"
- name: Create an Instance Group
tower_instance_group:
name: "{{ group_name1 }}"
state: present
register: result
- assert:
that:
- "result is changed"
- name: "Pass in all parameters"
tower_organization:
name: "{{ org_name }}"
description: "A description"
instance_groups:
- "{{ group_name1 }}"
register: result
- assert:
@ -48,6 +65,11 @@
that:
- "result is changed"
- name: Delete the instance groups
tower_instance_group:
name: "{{ group_name1 }}"
state: absent
- name: "Remove the organization"
tower_organization:
name: "{{ org_name }}"