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

Identities management becomes increasingly complex and error-prone; should be revisited/refactored for ease of continuing development #224

Open
comtalyst opened this issue Mar 20, 2024 · 1 comment
Assignees
Labels
area/bootstrap Issues or PRs related to bootstrap area/code-organization Issues or PRs related to code organization area/security Issues or PRs related to security kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Comments

@comtalyst
Copy link
Collaborator

Tell us about your request

The area of identities management, as well as its interface, should be revisited to prevent impending, if not existing, engineering mistakes and security concerns of the codebase.

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?

As the project becomes more mature, the existing assumptions on the uses of identity have evolved, and will continue to do so.
For example, kubelet identity (i.e., user assigned identity) is being used on everything Karpenter needs(?), while its original expectation is just to assign it to the provisioning nodes. This fact should, at least, be clearly reflected in the code (e.g., through proper naming) to prevent unexpected misuses in the case where changes will be introduced in this area, as well as to prevent security breaches.

Are you currently working around this issue?

No

Additional Context

No response

Attachments

No response

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment
@comtalyst comtalyst added kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. area/bootstrap Issues or PRs related to bootstrap area/security Issues or PRs related to security area/code-organization Issues or PRs related to code organization labels Mar 20, 2024
@comtalyst comtalyst self-assigned this Apr 25, 2024
@comtalyst
Copy link
Collaborator Author

Working on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/bootstrap Issues or PRs related to bootstrap area/code-organization Issues or PRs related to code organization area/security Issues or PRs related to security kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Development

No branches or pull requests

1 participant