canonical-ubuntu-qa team mailing list archive

[Brian Murray <mp+460129@xxxxxxxxxxxxxxxxxx>]

Brian Murray has proposed merging autopkgtest-cloud:c-t-i-flavor-change into autopkgtest-cloud:master.

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

For more details, see:
https://code.launchpad.net/~ubuntu-release/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/460129

We should use the autopkgtest flavor when creating test instances as it will confirm that the flavor is available and that flavor should be available in every data center. (Also cpu4-ram8-disk50 doesn't exist in bos03.)

Additionally, creating only a couple of instances has been sufficient to troubleshoot issues so I think that should be the default. (I routinely modify the script to reduce the number of instances being created.)
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of autopkgtest-cloud:c-t-i-flavor-change into autopkgtest-cloud:master.

diff --git a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/create-test-instances b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/create-test-instances
index 26ef5f3..674dde3 100755
--- a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/create-test-instances
+++ b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/create-test-instances
@@ -12,6 +12,6 @@ IMAGE=$(openstack image list --format csv -c Name --quote none | grep "adt/ubunt
 # NET_ID is the prop-proposed-migration network from `openstack network list`
 NET_ID=$(openstack network list --format csv -c Name --quote none | grep proposed-migration)
 
-for i in $(seq 1 10); do
-    openstack server create --image "${IMAGE}" --flavor cpu4-ram8-disk50 --nic net-id="${NET_ID}" -- "creation-test-$ARCH-$i"
+for i in $(seq 1 2); do
+    openstack server create --image "${IMAGE}" --flavor autopkgtest --nic net-id="${NET_ID}" -- "creation-test-$ARCH-$i"
 done