Skip to content

Commit 3ff3d8c

Browse files
committed
feat(v3): pass isEmbeddedClusterV3 to adminconsole chart (#2723)
1 parent 7be449c commit 3ff3d8c

File tree

9 files changed

+15
-1
lines changed

9 files changed

+15
-1
lines changed

api/internal/managers/kubernetes/infra/install.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,7 @@ func (m *infraManager) getAddonInstallOpts(license *kotsv1beta1.License, ki kube
157157
EmbeddedConfigSpec: m.getECConfigSpec(),
158158
EndUserConfigSpec: m.getEndUserConfigSpec(),
159159
ProxySpec: ki.ProxySpec(),
160+
IsV3: true,
160161
}
161162

162163
// TODO: no kots app install for now

api/internal/managers/linux/infra/install.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,7 @@ func (m *infraManager) getAddonInstallOpts(ctx context.Context, license *kotsv1b
298298
K0sDataDir: rc.EmbeddedClusterK0sSubDir(),
299299
OpenEBSDataDir: rc.EmbeddedClusterOpenEBSLocalSubDir(),
300300
ServiceCIDR: rc.ServiceCIDR(),
301+
IsV3: true,
301302
}
302303

303304
return opts

pkg/addons/adminconsole/adminconsole.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ type AdminConsole struct {
2323
IsMultiNodeEnabled bool
2424
Proxy *ecv1beta1.ProxySpec
2525
AdminConsolePort int
26+
IsV3 bool
2627

2728
// Linux specific options
2829
ClusterID string

pkg/addons/adminconsole/values.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ func (a *AdminConsole) GenerateHelmValues(ctx context.Context, kcli client.Clien
5050
copiedValues["isHA"] = a.IsHA
5151
copiedValues["isMultiNodeEnabled"] = a.IsMultiNodeEnabled
5252
copiedValues["isAirgap"] = a.IsAirgap
53+
copiedValues["isEmbeddedClusterV3"] = a.IsV3
5354

5455
if domains.ReplicatedAppDomain != "" {
5556
copiedValues["replicatedAppEndpoint"] = netutils.MaybeAddHTTPS(domains.ReplicatedAppDomain)

pkg/addons/adminconsole/values_test.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ func TestGenerateHelmValues_Target(t *testing.T) {
9595
IsMultiNodeEnabled: false,
9696
Proxy: nil,
9797
AdminConsolePort: 8080,
98+
IsV3: true,
9899

99100
ClusterID: "123",
100101
ServiceCIDR: "10.0.0.0/24",
@@ -106,10 +107,10 @@ func TestGenerateHelmValues_Target(t *testing.T) {
106107
values, err := adminConsole.GenerateHelmValues(context.Background(), nil, ecv1beta1.Domains{}, nil)
107108
require.NoError(t, err, "GenerateHelmValues should not return an error")
108109

109-
assert.Contains(t, values, "embeddedClusterID")
110110
assert.Equal(t, "123", values["embeddedClusterID"])
111111
assert.Equal(t, dataDir, values["embeddedClusterDataDir"])
112112
assert.Equal(t, filepath.Join(dataDir, "k0s"), values["embeddedClusterK0sDir"])
113+
assert.Equal(t, true, values["isEmbeddedClusterV3"])
113114

114115
assert.Contains(t, values["extraEnv"], map[string]interface{}{
115116
"name": "SSL_CERT_CONFIGMAP",
@@ -128,6 +129,7 @@ func TestGenerateHelmValues_Target(t *testing.T) {
128129
IsMultiNodeEnabled: false,
129130
Proxy: nil,
130131
AdminConsolePort: 8080,
132+
IsV3: true,
131133
}
132134

133135
values, err := adminConsole.GenerateHelmValues(context.Background(), nil, ecv1beta1.Domains{}, nil)
@@ -136,6 +138,7 @@ func TestGenerateHelmValues_Target(t *testing.T) {
136138
assert.NotContains(t, values, "embeddedClusterID")
137139
assert.NotContains(t, values, "embeddedClusterDataDir")
138140
assert.NotContains(t, values, "embeddedClusterK0sDir")
141+
assert.Equal(t, true, values["isEmbeddedClusterV3"])
139142

140143
for _, env := range values["extraEnv"].([]map[string]interface{}) {
141144
assert.NotEqual(t, "SSL_CERT_CONFIGMAP", env["name"], "SSL_CERT_CONFIGMAP environment variable should not be set")

pkg/addons/highavailability.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -251,6 +251,7 @@ func (a *AddOns) EnableAdminConsoleHA(ctx context.Context, opts EnableHAOptions)
251251
DataDir: opts.DataDir,
252252
K0sDataDir: opts.K0sDataDir,
253253
AdminConsolePort: opts.AdminConsolePort,
254+
IsV3: false,
254255
}
255256
if err := ac.Upgrade(ctx, a.logf, a.kcli, a.mcli, a.hcli, a.domains, a.addOnOverrides(ac, opts.EmbeddedConfigSpec, opts.EndUserConfigSpec)); err != nil {
256257
return errors.Wrap(err, "upgrade admin console")

pkg/addons/install.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ type InstallOptions struct {
2828
EndUserConfigSpec *ecv1beta1.ConfigSpec
2929
KotsInstaller adminconsole.KotsInstaller
3030
ProxySpec *ecv1beta1.ProxySpec
31+
IsV3 bool
3132

3233
// Linux only options
3334
ClusterID string
@@ -52,6 +53,7 @@ type KubernetesInstallOptions struct {
5253
EndUserConfigSpec *ecv1beta1.ConfigSpec
5354
KotsInstaller adminconsole.KotsInstaller
5455
ProxySpec *ecv1beta1.ProxySpec
56+
IsV3 bool
5557
}
5658

5759
func (a *AddOns) Install(ctx context.Context, opts InstallOptions) error {
@@ -160,6 +162,7 @@ func GetAddOnsForInstall(opts InstallOptions) []types.AddOn {
160162
DataDir: opts.DataDir,
161163
K0sDataDir: opts.K0sDataDir,
162164
AdminConsolePort: opts.AdminConsolePort,
165+
IsV3: opts.IsV3,
163166

164167
Password: opts.AdminConsolePwd,
165168
TLSCertBytes: opts.TLSCertBytes,
@@ -195,6 +198,7 @@ func GetAddOnsForKubernetesInstall(opts KubernetesInstallOptions) []types.AddOn
195198
IsMultiNodeEnabled: opts.IsMultiNodeEnabled,
196199
Proxy: opts.ProxySpec,
197200
AdminConsolePort: opts.AdminConsolePort,
201+
IsV3: opts.IsV3,
198202

199203
Password: opts.AdminConsolePwd,
200204
TLSCertBytes: opts.TLSCertBytes,

pkg/addons/upgrade.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,7 @@ func (a *AddOns) getAddOnsForUpgrade(meta *ectypes.ReleaseMetadata, opts Upgrade
118118
DataDir: opts.DataDir,
119119
K0sDataDir: opts.K0sDataDir,
120120
AdminConsolePort: opts.AdminConsolePort,
121+
IsV3: false,
121122
})
122123

123124
return addOns, nil

tests/dryrun/install_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ func testDefaultInstallationImpl(t *testing.T) {
101101
"embeddedClusterID": in.Spec.ClusterID,
102102
"embeddedClusterDataDir": "/var/lib/embedded-cluster",
103103
"embeddedClusterK0sDir": "/var/lib/embedded-cluster/k0s",
104+
"isEmbeddedClusterV3": false,
104105
})
105106
assertHelmValuePrefixes(t, adminConsoleOpts.Values, map[string]string{
106107
"images.kotsadm": "fake-replicated-proxy.test.net/anonymous",

0 commit comments

Comments
 (0)