Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meta_dependency_check seemingly not functional when run in Controller #942

Closed
l3acon opened this issue Oct 18, 2024 · 4 comments
Closed
Labels
bug Something isn't working new New issue, this should be removed once reviewed

Comments

@l3acon
Copy link

l3acon commented Oct 18, 2024

Summary

The ansible-galaxy collection verify command seems to require galaxy servers to be configured, which in Controller isn't possible unless it's syncing collections. I think this is a security measure, e.g. when trying to create a custom credential for galaxy Controller responds with Environment variable ANSIBLE_GALAXY_SERVER_AH_TOKEN may affect Ansible configuration so its use is not allowed in credentials.. That is, this works just fine from CLI ansible, but not in AAP Controller.

I am testing APD for use with AAP 2.5, after updating the collections it seems to mostly work, I just have to bypass the meta_dependency_check role by setting controller_dependency_check: false via extra_vars.

Issue Type

  • Bug Report

Ansible, Collection, Controller details

sh-4.4$ ansible-galaxy collection list

# /usr/share/ansible/collections/ansible_collections
Collection                          Version
----------------------------------- -------
amazon.aws                          8.1.0  
ansible.controller                  4.6.1  
ansible.netcommon                   7.0.0  
ansible.posix                       1.5.4  
ansible.utils                       5.0.0  
ansible.windows                     2.4.0  
chocolatey.chocolatey               1.5.1  
cisco.ios                           9.0.0  
cisco.iosxr                         10.0.0 
cisco.nxos                          9.0.0  
community.general                   9.2.0  
community.windows                   2.2.0  
containers.podman                   1.15.4 
infoblox.nios_modules               1.6.1  
infra.ah_configuration              2.0.6  
infra.controller_configuration      2.11.0 
kubernetes.core                     5.0.0  
redhat.insights                     1.3.0  
redhat.openshift                    4.0.0  
redhat.openshift_virtualization     1.5.0  
redhat.rhel_system_roles            1.23.0 
redhat.satellite                    4.0.0  
redhat_cop.controller_configuration 2.3.1
  • ansible installation method: OS Package and EE

OS / ENVIRONMENT

running AAP 2.5 via operator

Desired Behavior

infra.aap_configuration should be usable within AAP Controller, ideally without having to define controller_dependency_check: false

Actual Behavior

Dispatch role fails with error message:

msg: One of awx.awx or ansible.controller must be installed
_ansible_no_log: false
changed: false

When running via CLI (ansible-playbook or navigator) this is not an issue, simply define the AH credentials via environment variable.

STEPS TO REPRODUCE

The following steps are how I discovered this:

A simpler workflow should be able to re-produce this behavior:

  • Include dispatch role
  • Configure JT in AAP Controller
  • Run, see that it returns the aforementioned error.
@l3acon l3acon added bug Something isn't working new New issue, this should be removed once reviewed labels Oct 18, 2024
@sean-m-sullivan
Copy link
Collaborator

We've discussed an update along with our update to the entire collection, will keep this in mind, we still want to have checks, but realize the galaxy connection might not always be there

@sean-m-sullivan
Copy link
Collaborator

@l3acon Set the variable controller_dependency_check to false and it will disable this feature. We are disabling it by default in a future release.

jce-redhat added a commit to ansible/product-demos that referenced this issue Oct 28, 2024
jce-redhat added a commit to ansible/product-demos that referenced this issue Oct 28, 2024
@sean-m-sullivan
Copy link
Collaborator

can you check this now with the 3.0.0 it should be fixed.

@l3acon
Copy link
Author

l3acon commented Nov 14, 2024

Yes, I can confirm this is fixed in 3.0.0

@l3acon l3acon closed this as completed Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new New issue, this should be removed once reviewed
Projects
None yet
Development

No branches or pull requests

2 participants