-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add support for PDB and TopologySpreadConstraints #9
base: main
Are you sure you want to change the base?
Add support for PDB and TopologySpreadConstraints #9
Conversation
Hello @micrOsman |
Hello @Narekmat, thank you for the review. The version has been increased to 0.7.3 and the app version increased to the current one. |
Hello, Is there anything I can do to help implement these changes? |
@micrOsman hey, sorry, figured out I haven't subscribed to this particular repo for some reason. We'll take a look |
@micrOsman please check this. |
Narekmat could you please run one more build-and-test job? |
What changes did you make? (Give an overview)
This PR is created to add support for Pod Disruption Budget, Pod Topology Spread Constraints and Pod Priority Class. The reason is to enhance the availability, reliability, and resiliency of the Kafka-UI, especially during updates and unexpected situations.
Fixes #3
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)
It was tested locally using a local cluster and another remote cluster.
Values added to value.yaml file used for my local tests were the following:
This is the command used to execute the Helm Chart:
helm install kafka-ui -f .\kafka-ui\values.yaml .\kafka-ui\ -n kafka
Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)
Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)