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

v1.9.1 onwards depends on ansible.windows which does not exist #275

Closed
anshulbehl opened this issue Mar 30, 2021 · 2 comments · Fixed by #276
Closed

v1.9.1 onwards depends on ansible.windows which does not exist #275

anshulbehl opened this issue Mar 30, 2021 · 2 comments · Fixed by #276
Assignees
Labels
bug Something isn't working

Comments

@anshulbehl
Copy link

Bug description
A clear and concise description of what the issue is.
There is a dependency on ansible.windows which does not exist on Automation Hub, hence its failing the building of execution environment
Also any customer who will try to install the collections from Automation Hub will also fail because of the unavailability of the collection, please fix this asap

cc: @tadeboro

@anshulbehl anshulbehl added the bug Something isn't working label Mar 30, 2021
@tadeboro tadeboro self-assigned this Mar 30, 2021
@tadeboro
Copy link
Contributor

Hi @anshulbehl.

We added the ansible.windows dependency when we were preparing the Sensu Go Ansible Collection for execution environments. The reasoning behind our decision was that without Windows collection, EE will fail to execute valid playbooks if users are trying to manage Windows hosts. What I failed to notice at the time was the fact that the ansible.windows collection is not certified (I naively assumed anything in the ansible namespace is certified) and only tested EE build using Ansible Galaxy as the source of collections.

Before the introduction of execution environments, this was not an issue because all of the relevant Windows content was part of Ansible 2.9 and our roles are written with Ansible 2.9 users in mind (we do not use FQCNs in our roles for this reason).

We have a PR ready that removes the hard dependency on the Windows Ansible Collection, but there is a really nasty side-effect of that PR: EEs containing the Sensu Go Ansible Collection just lost the capability to install Sensu Go agents on Windows hosts. So community users actually have access to more functionality than paying customers that rely solely on certified content.

@anshulbehl
Copy link
Author

@tadeboro thanks for handling this, I will keep you updated when or if we certify ansible.windows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants