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

Commit

Permalink
Upgrade bootstrap-secrets before anything else
Browse files Browse the repository at this point in the history
So if there is a node pending for bootstrapping, it will be allowed to
bootstrap first, before we proceed with other upgrades.

Closes #949

Signed-off-by: Mateusz Gozdek <[email protected]>
  • Loading branch information
invidian committed Dec 2, 2020
1 parent e06bfab commit 57d771c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions pkg/platform/platform.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,10 @@ const (
// platforms.
func CommonControlPlaneCharts(includeKubeletChart bool) []helm.LokomotiveChart {
charts := []helm.LokomotiveChart{
{
Name: "bootstrap-secrets",
Namespace: "kube-system",
},
{
Name: "pod-checkpointer",
Namespace: "kube-system",
Expand All @@ -59,10 +63,6 @@ func CommonControlPlaneCharts(includeKubeletChart bool) []helm.LokomotiveChart {
Name: "lokomotive",
Namespace: "lokomotive-system",
},
{
Name: "bootstrap-secrets",
Namespace: "kube-system",
},
}

if includeKubeletChart {
Expand Down
2 changes: 1 addition & 1 deletion pkg/platform/platform_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,12 +49,12 @@ func TestAppendVersionTag(t *testing.T) {

func TestCommonControlPlaneChartsOrder(t *testing.T) {
expectedOrder := []string{
"bootstrap-secrets",
"pod-checkpointer",
"kube-apiserver",
"kubernetes",
"calico",
"lokomotive",
"bootstrap-secrets",
"kubelet",
}

Expand Down

0 comments on commit 57d771c

Please sign in to comment.