-
Notifications
You must be signed in to change notification settings - Fork 143
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
How is the k8s inventory plugin being used? #31
Comments
@tima I can imagine as following -
|
Hello @tima, this looks like a good candidate for the Bullhorn. |
Hi, I also wonder if this is an anti pattern or not. I have a use case where an app config is store in a secret for an Ansible based operator. The "native" options available in Kubernetes imply pod restart when config resource is changed. However, using Ansible, I could template the config resource in Kubernetes and keep in sync the respective file in all replicas, no pod restart needed. For that, the inventory plugin will be helpful. For me, it seems an anti pattern from Kubernetes perspective. However, isn’t Ansible potential being limited without it? Wouldn’t Ansible become a template manager for Kubernetes? |
Sorry I didn't see your comment sooner @jobcespedes. Wouldn't one of the |
They do the work, along with add_host. However, once you have more than one use case, it does not feel right to use Ansible tasks to build inventories and it does not help reusability. |
In ansible-collections/community.kubernetes#217, @fabianvf made some modifications to the k8s inventory plugin that we believe improve its overall operation, cuts down on the "noise" and just makes more sense. This begs the question -- how is this plugin being used? Does this refactoring make sense? What is the impact, if any, of the changes? Are there other improvements to be made to help users?
ADDENDUM: Is this plugin needed at all or is it supporting an anti-pattern (Kubernetes should be managing containers in its cluster) that it should be deprecated and removed?
The text was updated successfully, but these errors were encountered: