Skip to content

Commit d2cb234

Browse files
committed
Set qcow image file location with right cirros version
This commit ensures we also set the qcow image location in the tempest config when we update qcow version. The tempest config has a default value for img_file (which is incorrect) but before we can remove the defaults in tempest we need to ensure devstack is using it properly first. The only reason the tests weren't failing here is because tempest falls back to using uec images (which devstack was correctly setting config for) if qcow isn't found. The img_dir was also hardcoded assuming a uec image, however if qcow is intended to be used you'll need to be able to override that, which is added as part of this commit. Change-Id: I05af346b3c9be9560dc8846dd1f437cfbb2d5005
1 parent 5cb3d6b commit d2cb234

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

lib/tempest

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -401,10 +401,12 @@ function configure_tempest {
401401
fi
402402

403403
# Scenario
404-
iniset $TEMPEST_CONFIG scenario img_dir "$FILES/images/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec"
404+
SCENARIO_IMAGE_DIR=${SCENARIO_IMAGE_DIR:-$FILES/images/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec}
405+
iniset $TEMPEST_CONFIG scenario img_dir $SCENARIO_IMAGE_DIR
405406
iniset $TEMPEST_CONFIG scenario ami_img_file "cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-blank.img"
406407
iniset $TEMPEST_CONFIG scenario ari_img_file "cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-initrd"
407408
iniset $TEMPEST_CONFIG scenario aki_img_file "cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-vmlinuz"
409+
iniset $TEMPEST_CONFIG scenario img_file "cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-disk.img"
408410

409411
# Large Ops Number
410412
iniset $TEMPEST_CONFIG scenario large_ops_number ${TEMPEST_LARGE_OPS_NUMBER:-0}

0 commit comments

Comments
 (0)