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 15, 2020
1 parent ad02c23 commit 8c5c047
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 8c5c047

Please sign in to comment.