-
Notifications
You must be signed in to change notification settings - Fork 453
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
Disable etcd backup compaction by default #5144
Disable etcd backup compaction by default #5144
Conversation
@timuthy Can you please fix the failing tests? |
@dguendisch, @mvladev, @ialidzhikov, @plkokanov, @timuthy, @vpnachev, @schrodit, @danielfoehrKn, @BeckerMax, @timebertt, @hendrikKahl, @kris94, @voelzmo, @stoyanr This pullrequest was referenced by @rfranzke in duplicate pullrequest #5147. |
4e8467c
to
eb6ab38
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Earlier, paramters in `etcdConfig` were hard coded and now the very same values became the default. However, these defaults are only active if we don't overwrite them via the values in the Helm chart.
Through #5147 another problem has drawn my attention: We set very low worker values in the Helm chart which become the "new" default when applying Gardenlet via this chart. For instance, if an operator applied Gardenlet earlier, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
* Disable etcd backup compaction by default * Disable etcdConfig in Gardenlet chart Earlier, paramters in `etcdConfig` were hard coded and now the very same values became the default. However, these defaults are only active if we don't overwrite them via the values in the Helm chart.
* Disable etcd backup compaction by default * Disable etcdConfig in Gardenlet chart Earlier, paramters in `etcdConfig` were hard coded and now the very same values became the default. However, these defaults are only active if we don't overwrite them via the values in the Helm chart.
How to categorize this PR?
/area control-plane
/kind enhancement
What this PR does / why we need it:
This PR disables the etcd backup compaction feature by default. It should consciously be enabled by operators because the feature comes with certain cost implications (e.g. network traffic) and requires machines with a rather large root disk.
Release note: