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

docs: installation: scaling: mention kubernetes core components resou… #1295

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aborrero
Copy link

…rces footprint

Introduce a mention to how many kyverno resources may impact the kubernetes core components regarding their resources footprint.

See also: kyverno/kyverno#10458

Related issue

Proposed Changes

Checklist

  • [] I have read the contributing guidelines.
  • [] I have inspected the website preview for accuracy.
  • [] I have signed off my issue.

…rces footprint

Introduce a mention to how many kyverno resources may impact the
kubernetes core components regarding their resources footprint.

See also: kyverno/kyverno#10458

Signed-off-by: Arturo Borrero Gonzalez <[email protected]>
Copy link

welcome bot commented Jun 20, 2024

Thanks for opening your first Pull Request here! Please check out our Contributing guidelines and confirm that you Signed off.

@realshuting
Copy link
Member

realshuting commented Jun 25, 2024

Hi @chipzoller - can you help review?

Copy link
Contributor

@chipzoller chipzoller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like it's better for the troubleshooting section than here.

@aborrero
Copy link
Author

This seems like it's better for the troubleshooting section than here.

The idea is to prevent folks from having to reach to the troubleshooting section.

I added to the scaling section because I liked the idea of having everything related to the topic of how to operate kyverno at scale on the same page.

Should I have this information here when I started working with kyverno, would have saved me quite a few hours of work.

@chipzoller
Copy link
Contributor

But this is a really, really niche concern. Prior to you, we had never heard of people using truly "thousands" of Kyverno policies let alone the problems they might cause.

@aborrero
Copy link
Author

But this is a really, really niche concern. Prior to you, we had never heard of people using truly "thousands" of Kyverno policies let alone the problems they might cause.

The troubleshooting section is "after the fact" documentation.

I expect anything related to kubernetes ecosystem to be extremely scalable by default. Ideally, handling a few thousand resources would be no big deal for kyverno.

If handling such a load is not big deal, then it should be clearly documented. But not hidden as an "after the fact" information.

It should be front and center, and in that sense, I can't think of a reason why a section called "scalability" would be the wrong place to put this information.

@realshuting realshuting self-requested a review September 23, 2024 13:42
@realshuting realshuting self-assigned this Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants