canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+454819@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:fix_make_lxd_secgroup_traceback into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/454819
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:fix_make_lxd_secgroup_traceback into autopkgtest-cloud:master.

diff --git a/mojo/make-lxd-secgroup b/mojo/make-lxd-secgroup
index 14cb740..e249c2c 100755
--- a/mojo/make-lxd-secgroup
+++ b/mojo/make-lxd-secgroup
@@ -16,7 +16,7 @@ LXD_IPS=$(juju status --format=json autopkgtest-lxd-worker | jq --monochrome-out
 HAPROXY_IPS=$(juju status --format=json haproxy-lxd-armhf | jq --monochrome-output --raw-output '.applications["haproxy-lxd-armhf"].units | map(.["public-address"])[]')
 
 if ! openstack security group rule list lxd >/dev/null 2>/dev/null; then
-        openstack security group create lxd "Let the LXD workers communicate with the LXD VMs"
+        openstack security group create lxd --description "Let the LXD workers communicate with the LXD VMs"
 fi
 
 for ip in ${LXD_IPS} ${HAPROXY_IPS}; do