chore(inventory): Apply suggested fixes from code review

Name changes
Description changes
linking issue to edited out task

#17 #13
This commit is contained in:
2024-08-19 12:41:08 +09:30
committed by Jon
parent 83f12f063d
commit 975f4b6ee5

View File

@ -14,7 +14,7 @@
- name: Inventory host - name: Inventory host
block: block:
- name: Test Packages - name: fetch Packages
ansible.builtin.package_facts: ansible.builtin.package_facts:
manager: auto manager: auto
become: true become: true
@ -45,6 +45,9 @@
when: > when: >
package.value[0].name | regex_search("^[a-k]") package.value[0].name | regex_search("^[a-k]")
# https://github.com/nofusscomputing/ansible_collection_centurion/issues/19
# This teask has been commented out due to the above issue
# - name: Inventory Software [l] # - name: Inventory Software [l]
# ansible.builtin.set_fact: # ansible.builtin.set_fact:
@ -135,20 +138,20 @@
ask_credential_on_launch: true ask_credential_on_launch: true
ask_limit_on_launch: true ask_limit_on_launch: true
concurrent_jobs_enabled: true concurrent_jobs_enabled: true
description: Collect inventory of host machines and publish to centurion_erp description: Inventory host machines and publish to Centurion ERP
execution_environment: "No Fuss Computing EE" execution_environment: "No Fuss Computing EE"
job_type: "run" job_type: "run"
# job_tags: complete # job_tags: complete
labels: labels:
- itsm
- itam
- inventory
- centurion - centurion
- inventory
- itam
- itsm
use_fact_cache: true use_fact_cache: true
credential_types: credential_types:
- name: 'Playbook/Inventory/Centurion' - name: 'Collection/No Fuss Computing/Centurion/API'
description: | description: |
Credentials for authentication to Centurion_erp Credentials for authentication to Centurion ERP
inputs: | inputs: |
fields: fields:
- id: centurion_url - id: centurion_url
@ -170,3 +173,4 @@
CENTURION_API: '{{ centurion_url }}' CENTURION_API: '{{ centurion_url }}'
CENTURION_TOKEN: '{{ centurion_token }}' CENTURION_TOKEN: '{{ centurion_token }}'
CENTURION_VALIDATE_CERTS: '{{ centurion_validate_certs | default(true) }}' CENTURION_VALIDATE_CERTS: '{{ centurion_validate_certs | default(true) }}'