Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Commit

Permalink
cli/cmd/cluster-apply.go: don't try to upgrade managed platforms
Browse files Browse the repository at this point in the history
If we deploy on managed Kubernetes, it won't have controlplane installed
as Helm releases, so we shouldn't try to upgrade it. The upgrade will be
most likely driven by Terraform.

Refs #215 #216

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
  • Loading branch information
invidian committed Apr 24, 2020
1 parent 72ae0f2 commit 9861002
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cli/cmd/cluster-apply.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,8 @@ func runClusterApply(cmd *cobra.Command, args []string) {
ctxLogger.Fatalf("Verify cluster: %v", err)
}

// Do controlplane upgrades only if cluster already exists.
if exists {
// Do controlplane upgrades only if cluster already exists and it is not a managed platform.
if exists && !p.Meta().Managed {
fmt.Printf("\nEnsuring that cluster controlplane is up to date.\n")

cu := controlplaneUpdater{
Expand Down

0 comments on commit 9861002

Please sign in to comment.