-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Enable endpoint slices when upgrading through CLI (#4864)
## What/How @adleong pointed out in #4780 that when enabling slices during an upgrade, the new value does not persist in the `linkerd-config` ConfigMap. I took a closer look and it seems that we were never overwriting the values in case they were different. * To fix this, I added an if block when validating and building the upgrade options -- if the current flag value differs from what we have in the ConfigMap, then change the ConfigMap value. * When doing so, I made sure to check that if the cluster does not support `EndpointSlices` yet the flag is set to true, we will error out. This is done similarly (copy&paste similarily) to what's in the install part. * Additionally, I have noticed that the helm ConfigMap template stored the flag value under `enableEndpointSlices` field name. I assume this was not changed in the initial PR to reflect the changes made in the protocol buffer. The API (and thus the CLI) uses the field name `endpointSliceEnabled` instead. I have changed the config template so that helm installations will use the same field, which can then be used in the destination service or other components that may implement slice support in the future. Signed-off-by: Matei David <matei.david.35@gmail.com>
- version-2.17
- version-2.16
- version-2.15
- stable-2.14.10
- stable-2.14.9
- stable-2.14.8
- stable-2.14.7
- stable-2.14.6
- stable-2.14.5
- stable-2.14.4
- stable-2.14.3
- stable-2.14.2
- stable-2.14.1
- stable-2.14.0
- stable-2.13.7
- stable-2.13.6
- stable-2.13.5
- stable-2.13.4
- stable-2.13.3
- stable-2.13.2
- stable-2.13.1
- stable-2.13.0
- stable-2.12.6
- stable-2.12.5
- stable-2.12.4
- stable-2.12.3
- stable-2.12.2
- stable-2.12.1
- stable-2.12.0
- stable-2.12.0-rc2
- stable-2.11.5
- stable-2.11.4
- stable-2.11.3
- stable-2.11.2
- stable-2.11.1
- stable-2.11.0
- stable-2.10.2
- stable-2.10.1
- stable-2.10.0
- stable-2.9.5
- stable-2.9.4
- stable-2.9.3
- stable-2.9.2
- stable-2.9.1
- stable-2.9.0
- edge-25.1.2
- edge-25.1.1
- edge-24.11.8
- edge-24.11.7
- edge-24.11.6
- edge-24.11.5
- edge-24.11.4
- edge-24.11.3
- edge-24.11.2
- edge-24.11.1
- edge-24.10.5
- edge-24.10.4
- edge-24.10.3
- edge-24.10.2
- edge-24.10.1
- edge-24.9.3
- edge-24.9.2
- edge-24.9.1
- edge-24.8.3
- edge-24.8.2
- edge-24.8.1
- edge-24.7.5
- edge-24.7.4
- edge-24.7.3
- edge-24.7.2
- edge-24.7.1
- edge-24.6.4
- edge-24.6.3
- edge-24.6.2
- edge-24.6.1
- edge-24.5.5
- edge-24.5.4
- edge-24.5.3
- edge-24.5.2
- edge-24.5.1
- edge-24.4.5
- edge-24.4.4
- edge-24.4.3
- edge-24.4.2
- edge-24.4.1
- edge-24.3.5
- edge-24.3.4
- edge-24.3.3
- edge-24.3.2
- edge-24.3.1
- edge-24.2.5
- edge-24.2.4
- edge-24.2.3
- edge-24.2.2
- edge-24.2.1
- edge-24.1.3
- edge-24.1.2
- edge-24.1.1
- edge-23.12.4
- edge-23.12.3
- edge-23.12.2
- edge-23.12.1
- edge-23.11.4
- edge-23.11.3
- edge-23.11.2
- edge-23.11.1
- edge-23.10.4
- edge-23.10.3
- edge-23.10.2
- edge-23.10.1
- edge-23.9.4
- edge-23.9.3
- edge-23.9.2
- edge-23.9.1
- edge-23.8.3
- edge-23.8.2
- edge-23.8.1
- edge-23.7.3
- edge-23.7.2
- edge-23.7.1
- edge-23.6.3
- edge-23.6.2
- edge-23.6.1
- edge-23.5.3
- edge-23.5.2
- edge-23.5.1
- edge-23.4.3
- edge-23.4.2
- edge-23.4.1
- edge-23.3.4
- edge-23.3.3
- edge-23.3.2
- edge-23.3.1
- edge-23.2.3
- edge-23.2.2
- edge-23.2.1
- edge-23.1.2
- edge-23.1.1
- edge-22.12.1
- edge-22.11.3
- edge-22.11.2
- edge-22.11.1
- edge-22.10.3
- edge-22.10.2
- edge-22.10.1
- edge-22.9.2
- edge-22.9.1
- edge-22.8.3
- edge-22.8.2
- edge-22.8.1
- edge-22.7.3
- edge-22.7.2
- edge-22.7.1
- edge-22.6.2
- edge-22.6.1
- edge-22.5.3
- edge-22.5.2
- edge-22.5.1
- edge-22.4.1
- edge-22.3.5
- edge-22.3.4
- edge-22.3.3
- edge-22.3.2
- edge-22.3.1
- edge-22.2.4
- edge-22.2.3
- edge-22.2.2
- edge-22.2.1
- edge-22.1.5
- edge-22.1.4
- edge-22.1.3
- edge-22.1.2
- edge-22.1.1
- edge-21.12.4
- edge-21.12.3
- edge-21.12.2
- edge-21.12.1
- edge-21.11.4
- edge-21.11.3
- edge-21.11.2
- edge-21.11.1
- edge-21.10.3
- edge-21.10.2
- edge-21.10.1
- edge-21.9.5
- edge-21.9.4
- edge-21.9.3
- edge-21.9.2
- edge-21.9.1
- edge-21.8.4
- edge-21.8.3
- edge-21.8.2
- edge-21.8.1
- edge-21.7.5
- edge-21.7.4
- edge-21.7.3
- edge-21.7.2
- edge-21.7.1
- edge-21.6.5
- edge-21.6.4
- edge-21.6.3
- edge-21.6.2
- edge-21.6.1
- edge-21.5.3
- edge-21.5.2
- edge-21.5.1
- edge-21.4.5
- edge-21.4.4
- edge-21.4.3
- edge-21.4.2
- edge-21.4.1
- edge-21.3.4
- edge-21.3.3
- edge-21.3.2
- edge-21.3.1
- edge-21.2.4
- edge-21.2.3
- edge-21.2.2
- edge-21.2.1
- edge-21.1.4
- edge-21.1.3
- edge-21.1.2
- edge-21.1.1
- edge-20.12.4
- edge-20.12.3
- edge-20.12.2
- edge-20.12.1
- edge-20.11.5
- edge-20.11.4
- edge-20.11.3
- edge-20.11.2
- edge-20.11.1
- edge-20.10.6
- edge-20.10.5
- edge-20.10.4
- edge-20.10.3
- edge-20.10.2
- edge-20.10.1
- edge-20.9.4
- edge-20.9.3
- edge-20.9.2
- edge-20.9.1
- edge-20.8.4
1 parent
8a2cb12
commit 7ed904f
Showing
27 changed files
with
181 additions
and
170 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.