Skip to content

[Telemetry Manager] add priority class #17157

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

Closed
a-thaler opened this issue Mar 24, 2023 · 1 comment
Closed

[Telemetry Manager] add priority class #17157

a-thaler opened this issue Mar 24, 2023 · 1 comment
Assignees
Labels
area/telemetry Issues or PRs related to the telemetry module
Milestone

Comments

@a-thaler
Copy link
Contributor

a-thaler commented Mar 24, 2023

Description
By default the module components should be scheduled with a higher priority then user workload to assure that system functionality is working preferred over custom workload. For that custom priority classes are needed. The user can explicit overrule the priority if he has reasons.
So introduce a priority class and assign it to all telemetry components.

Criteria

  • As part of the managers kustomize files two priorityClass gets installed
  • The manager and all subsequent components are setting these classes
  • Current helm chart should already ship the new priorityClasses as well and decouble from the global setting
  • An env test assures that the classes are set always
  • Verify upgrade flow if fluent-bit takes new priority class
  • Verify migration to modules work without hiccup

Related
kyma-project/community#768

@rakesh-garimella
Copy link
Contributor

  • The priority class has been added. They are passed in helm and also in the kustomize
  • The migration test and upgrade test looks okay.
  • Env tests have been added

@a-thaler a-thaler added this to the 2.15 milestone May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/telemetry Issues or PRs related to the telemetry module
Projects
None yet
Development

No branches or pull requests

2 participants