Skip to content

Commit

Permalink
Change name of summon-sidecar sample app enable flag
Browse files Browse the repository at this point in the history
  • Loading branch information
Samir Shetty committed May 19, 2021
1 parent fb1f084 commit 6ef73bb
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion bin/test-workflow/7_app_deploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ deploy_sidecar_app() {
fi

helm install sidecar-app . -n "$TEST_APP_NAMESPACE_NAME" --debug --wait \
--set authn-k8s.enabled=true \
--set app-summon-sidecar.enabled=true \
--set global.conjur.conjurConnConfigMap="conjur-connect-configmap" \
--set app-summon-sidecar.conjur.authnLogin="$CONJUR_AUTHN_LOGIN_PREFIX/test-app-summon-sidecar"
popd > /dev/null
Expand Down
2 changes: 1 addition & 1 deletion helm/app-deploy/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ dependencies:
- name: app-summon-sidecar
repository: "file://charts/app-summon-sidecar"
version: ">= 0.0.1"
condition: authn-k8s.enabled
condition: app-summon-sidecar.enabled
2 changes: 1 addition & 1 deletion helm/app-deploy/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ global:
# associated sample application container) will be deployed to the
# same application Namespace. The default (app-summon-sidecar) is to enable only an authn-k8s
# sidecar container. Uncomment authenticator types as desired.
authn-k8s:
app-summon-sidecar:
enabled: true

secretless-broker:
Expand Down

0 comments on commit 6ef73bb

Please sign in to comment.