Merge ~jmcvaughn/stsstack-bundles:fix-vault-certificates-relation-check into stsstack-bundles:master

Proposed by James Vaughn
Status: Merged
Merged at revision: 14cd4d96ccc8080d6edf80bf4d276b1bb001bf7b
Proposed branch: ~jmcvaughn/stsstack-bundles:fix-vault-certificates-relation-check
Merge into: stsstack-bundles:master
Diff against target: 13 lines (+1/-1)
1 file modified
openstack/novarc (+1/-1)
Reviewer Review Type Date Requested Status
Jolly Bundlers Pending
Review via email: mp+393643@code.launchpad.net

Commit message

Add missing quotes to Vault certificates relation check

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/openstack/novarc b/openstack/novarc
2index 4aed822..473a82d 100644
3--- a/openstack/novarc
4+++ b/openstack/novarc
5@@ -22,7 +22,7 @@ if [ -n "`juju config keystone ssl_cert`" ]; then
6 export OS_AUTH_PROTOCOL=https
7 elif ((`jq -r '.applications[]| select(."charm-name"=="vault")' $juju_status_json_cache| wc -l`)); then
8 # Vault-based ssl
9- if `jq -r .applications.vault.relations.certificates[] $juju_status_json_cache| grep -q keystone`; then
10+ if `jq -r '.applications.vault.relations.certificates[]' $juju_status_json_cache| grep -q keystone`; then
11 export OS_AUTH_PROTOCOL=https
12 fi
13 else

Subscribers

People subscribed via source and target branches