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 Mar 27, 2020
1 parent 3ee890a commit 4b4455a
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 @@ -81,8 +81,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 4b4455a

Please sign in to comment.