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

Remove PodTopologySpreadConstraints #752

Closed
cristiklein opened this issue Dec 8, 2023 · 0 comments · Fixed by #754
Closed

Remove PodTopologySpreadConstraints #752

cristiklein opened this issue Dec 8, 2023 · 0 comments · Fixed by #754
Assignees
@cristiklein
Labels
documentation Improvements or additions to documentation

Comments

@cristiklein
Copy link
Collaborator

Describe the bug

The user demo currently encourages adding podTopologySpreadConstraints.

During the arch meeting on 2023-12-07, we agreed to the following change: To improve application developer experience, we will provide sane cluster build-in defaults. This makes it unnecessary to include podTopologySpreadConstraints in the application Helm chart. However, we will still keep a commented out example, just in case application developers need to tune the build-in defaults.
@cristiklein cristiklein added the documentation Improvements or additions to documentation label Dec 8, 2023
@cristiklein cristiklein self-assigned this Dec 8, 2023
@cristiklein cristiklein mentioned this issue Dec 12, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cristiklein cristiklein

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove topologySpreadConstraints from user-demo elastisys/compliantkubernetes
1 participant
@cristiklein