Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Commit

Permalink
Update ingress gateways with bootstrapACLs removal
Browse files Browse the repository at this point in the history
  • Loading branch information
lkysow authored and kschoche committed Jun 18, 2020
1 parent 83e48fb commit 61419ea
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion templates/server-acl-init-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
{{- if (and $serverEnabled .Values.externalServers.enabled) }}{{ fail "only one of server.enabled or externalServers.enabled can be set" }}{{ end -}}
{{- if (or $serverEnabled .Values.externalServers.enabled) }}
{{- if and .Values.global.acls.createReplicationToken (not .Values.global.acls.manageSystemACLs) }}{{ fail "if global.acls.createReplicationToken is true, global.acls.manageSystemACLs must be true" }}{{ end -}}
{{- if .Values.global.bootstrapACLs }}{{ fail "global.bootstrapACLs is removed, use global.acls.manageSystemACLs instead" }}{{ end -}}
{{- if .Values.global.bootstrapACLs }}{{ fail "global.bootstrapACLs was removed, use global.acls.manageSystemACLs instead" }}{{ end -}}
{{- if .Values.global.acls.manageSystemACLs }}
{{- /* We don't render this job when server.updatePartition > 0 because that
means a server rollout is in progress and this job won't complete unless
Expand Down
2 changes: 1 addition & 1 deletion test/unit/server-acl-init-job.bats
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ load _helpers
-x templates/server-acl-init-job.yaml \
--set 'global.bootstrapACLs=true' .
[ "$status" -eq 1 ]
[[ "$output" =~ "global.bootstrapACLs is removed, use global.acls.manageSystemACLs instead" ]]
[[ "$output" =~ "global.bootstrapACLs was removed, use global.acls.manageSystemACLs instead" ]]
}

@test "serverACLInit/Job: does not set -create-client-token=false when client is enabled (the default)" {
Expand Down

0 comments on commit 61419ea

Please sign in to comment.