-
Notifications
You must be signed in to change notification settings - Fork 433
Publish roles and playbooks as Ansible galaxy collection #1756
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @Lirt! |
Hi @Lirt. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Hey! Thanks for the PR. Whilst this looks fine on the face of it, we are pretty low on maintainers and will potentially be losing another one soon. The only problem I have is that by adding in the Galaxy support, it will introduce another area of expectation around support of the feature. I personally wouldn't have the time to keep on top of this as well as all of the providers we currently support too. It's also only moving a subset of the roles over too which then brings in questions around what should be considered a Galaxy collection and what should be considered native to the repo. I'll have to see how other maintainers feel about this as obviously this is one persons opinion 😄 but right now I'm not sure we could merge this in and provide it the support it would need moving forward and I don't want to see it just start to rot as a result. |
Adding my thoughts to this... We generally want to only encourage a single way of using image-builder to ensure we can support it as much as possible. Officially this is via the container image we provide but we also understand that a lot of people run it directly by checking out the repo and running Make so we also support that. We have had similar discussions in the past about other ways of using image-builder (such as from within EKS-Anywhere) and we've said we can't support it as we don't have the skills, time or resources to do so. If we had more test coverage across the different providers and configurations to the point where we had more confidence in changes this might be a different discussion but right now it's too much of a burden on the maintainers and contributors unfortunately. |
Hi, thanks for looking at the PR. This change shouldn't add extra maintenance burden. It only packages the ansible part into an ansible-native artifact. Any new role or playbook that is added under galaxy repository
All of the roles are included in the collection (no subset). You can check it in my fork:
The MR required some renaming changes (generally playbooks must use absolute paths) and some file moves, but it was very small. People would need to get used to reference roles in playbooks with absolute paths. Let me know what do you think. |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Publish roles and playbooks as Ansible galaxy collection
This PR sets up CI that publishes ansible roles and playbooks as Ansible Galaxy Collection.
By pushing image-builder roles into galaxy we allow users to use them without cloning full git repository.
Related issues
Additional context
./playbooks
in the galaxy dir.kubernetes_sigs
has to be set up by a maintainer with access tokubernetes_sigs
group. Possibly other namespace names could be used.