Skip to content

Commit

Permalink
Merge pull request #282 from wking/tests-run-optional-image-path
Browse files Browse the repository at this point in the history
tests/run: Change required IMAGE_PATH to an optional IMAGE_URL on libvirt
  • Loading branch information
openshift-merge-robot authored Sep 19, 2018
2 parents 5f59fec + bf577f3 commit d0b3f91
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions tests/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ aws)
CONFIGURE_AWS_ROLES=False
;;
libvirt)
[ -z ${IMAGE_PATH+x} ] && (echo "Please set libvirt IMAGE_PATH" >&2; exit 1)
;;
*)
echo "unrecognized backend: ${BACKEND}" >&2
Expand Down Expand Up @@ -100,8 +99,8 @@ python <<-EOF >"${CLUSTER_NAME}.yaml"
if ${CONFIGURE_AWS_ROLES:-False}:
config['aws']['master']['iamRoleName'] = 'tf-tectonic-master-node'
config['aws']['worker']['iamRoleName'] = 'tf-tectonic-worker-node'
elif '${BACKEND}' == 'libvirt':
config['libvirt']['image'] = 'file://${IMAGE_PATH}'
elif '${BACKEND}' == 'libvirt' and '${IMAGE_URL}':
config['libvirt']['image'] = '${IMAGE_URL}'
yaml.safe_dump(config, sys.stdout)
EOF

Expand Down

0 comments on commit d0b3f91

Please sign in to comment.