Skip to content

Commit

Permalink
[installer] enable protected_secrets by default
Browse files Browse the repository at this point in the history
  • Loading branch information
kylos101 committed Oct 7, 2022
1 parent e564cb4 commit 955e278
Show file tree
Hide file tree
Showing 8 changed files with 43 additions and 24 deletions.
3 changes: 3 additions & 0 deletions install/installer/cmd/testdata/render/aws-setup/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,6 @@ objectStorage:
credentials:
kind: secret
name: aws-storage
experimental:
workspace:
enableProtectedSecrets: true
16 changes: 9 additions & 7 deletions install/installer/cmd/testdata/render/aws-setup/output.golden

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions install/installer/cmd/testdata/render/azure-setup/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,6 @@ objectStorage:
credentials:
kind: secret
name: azure-storage
experimental:
workspace:
enableProtectedSecrets: true
16 changes: 9 additions & 7 deletions install/installer/cmd/testdata/render/azure-setup/output.golden

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions install/installer/cmd/testdata/render/gcp-setup/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,6 @@ objectStorage:
serviceAccount:
kind: secret
name: gcp-storage
experimental:
workspace:
enableProtectedSecrets: true
16 changes: 9 additions & 7 deletions install/installer/cmd/testdata/render/gcp-setup/output.golden

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion install/installer/pkg/components/server/configmap.go
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,9 @@ func configmap(ctx *common.RenderContext) ([]runtime.Object, error) {

defaultFeatureFlags := []NamedWorkspaceFeatureFlag{}
_ = ctx.WithExperimental(func(cfg *experimental.Config) error {
if cfg.Workspace != nil && cfg.Workspace.EnableProtectedSecrets {
if cfg == nil || cfg.Workspace == nil {
defaultFeatureFlags = append(defaultFeatureFlags, NamedWorkspaceFeatureProtectedSecrets)
} else if cfg.Workspace != nil && cfg.Workspace.EnableProtectedSecrets {
defaultFeatureFlags = append(defaultFeatureFlags, NamedWorkspaceFeatureProtectedSecrets)
}
return nil
Expand Down

0 comments on commit 955e278

Please sign in to comment.