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.

Sign up for GitHub

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

Jump to bottom

Deprecate the k8s inventory plugin #570

Open
tima opened this issue Jan 19, 2023 · 2 comments
Open

Deprecate the k8s inventory plugin #570

tima opened this issue Jan 19, 2023 · 2 comments
Labels
planned This issue is planned to be taken up in a feature release type/admin Administrative task regarding this repo

Comments

@tima
Copy link
Collaborator

tima commented Jan 19, 2023

SUMMARY

Over 2 years ago I asked in #31 if this inventory plugin was 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. Based on the feedback (or lack thereof) I think it should be deprecated and removed.

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

plugins/inventory/k8s.py
@tima tima added planned This issue is planned to be taken up in a feature release type/admin Administrative task regarding this repo labels Jan 19, 2023
@tima tima mentioned this issue Feb 23, 2023
Merged
@twingg
Copy link

twingg commented Mar 28, 2023

Why is it being deprecated?

@tima
Copy link
Collaborator Author

tima commented Jun 1, 2023

It is supporting an anti-pattern. Kubernetes should be managing containers in its cluster not Ansible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
planned This issue is planned to be taken up in a feature release type/admin Administrative task regarding this repo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tima @twingg