openstack/puppet-openstack-integration
Code Issues Proposed changes

Switch tempest to use RPM by default and disable trove

Browse Source 
1) Switch tempest to be installed from RPM instead of source

Currently, tempest is deployed from source by default. This is
problematic as some changes in master trunk can be incompatible
with packages in promoted repo.

This patch switchs p-o-i to deploy tempest from the rpm package
used in the promoted repo, what should fix this kind of scenarios.

2) Disable trove until integration with keystone v3 is fixed upstream

Currently, trove doesn't work fine with keystone API v3. It's
being worked in https://review.openstack.org/#/c/507087/ in
upstream project.

3) Disable tempest tests for object_storage when running with ubuntu
as they are failing with the current version of tempest in the uca repo.

Until it's fixed, let's disable it from p-o-i to unblock promotion.

Change-Id: I385e2f34dd925bdc2fdbc1553d49dce104f566f2
This commit is contained in:
Alfredo Moralejo 2017-10-10 19:05:08 +02:00
parent 6f245df88c
commit 7994398c62
2 changed files with 37 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
fixtures/scenario003.pp
View File

@ -30,7 +30,10 @@ case $::osfamily {
# NOTE(mnaser): We need to figure out why Murano won't accept credentials # NOTE(mnaser): We need to figure out why Murano won't accept credentials
# and how to get it to work with Keystone V3. # and how to get it to work with Keystone V3.
$murano_enabled = false $murano_enabled = false
$trove_enabled = true # TODO (amoralej): trove doesn't work fine with keystone V3. Until it's
# fixed in https://review.openstack.org/#/c/507087/, let's disable it
# so that it doesn't block promotion.
$trove_enabled = false
} }
default: { default: {
fail("Unsupported osfamily (${::osfamily})") fail("Unsupported osfamily (${::osfamily})")

56
run_tests.sh
View File

@ -33,7 +33,7 @@ export DISTRO=$(lsb_release -c -s)
# export TEMPEST_VERSION=${TEMPEST_VERSION:-'382a2065f3364a36c110bfcc6275a0f8f6894773'} # export TEMPEST_VERSION=${TEMPEST_VERSION:-'382a2065f3364a36c110bfcc6275a0f8f6894773'}
export TEMPEST_VERSION=${TEMPEST_VERSION:-'master'} export TEMPEST_VERSION=${TEMPEST_VERSION:-'master'}
# For installing Tempest from RPM keep TEMPEST_FROM_SOURCE to false # For installing Tempest from RPM keep TEMPEST_FROM_SOURCE to false
export TEMPEST_FROM_SOURCE=${TEMPEST_FROM_SOURCE:-true} export TEMPEST_FROM_SOURCE=${TEMPEST_FROM_SOURCE:-false}
# Cirros Image directory # Cirros Image directory
export IMG_DIR=${IMG_DIR:-'/tmp/openstack/image'} export IMG_DIR=${IMG_DIR:-'/tmp/openstack/image'}
@ -98,34 +98,43 @@ print_header 'Clone Tempest, plugins & pre-cache CirrOS'
# the local directory but works needs to be added into puppet to properly find # the local directory but works needs to be added into puppet to properly find
# the path. # the path.
if [ -e /usr/zuul-env/bin/zuul-cloner ] && [ "${TEMPEST_FROM_SOURCE}" = true ] ; then # If it's running in tempest gate, we need to deploy it from source to test the
/usr/zuul-env/bin/zuul-cloner --workspace /tmp --cache-dir /opt/git \ # patch.
git://git.openstack.org openstack/tempest if [[ "${ZUUL_PROJECT}" = "openstack/tempest" ]]; then
TEMPEST_FROM_SOURCE=true
fi
if [ -e /usr/zuul-env/bin/zuul-cloner ] ; then
# For ubuntu we always need to deploy tempest-horizon from source
if uses_debs; then if uses_debs; then
/usr/zuul-env/bin/zuul-cloner --workspace /tmp --cache-dir /opt/git \ /usr/zuul-env/bin/zuul-cloner --workspace /tmp --cache-dir /opt/git \
git://git.openstack.org openstack/tempest-horizon git://git.openstack.org openstack/tempest-horizon
fi fi
if [ "${TEMPEST_FROM_SOURCE}" = true ]; then
# Pin Tempest to TEMPEST_VERSION unless we're running inside the /usr/zuul-env/bin/zuul-cloner --workspace /tmp --cache-dir /opt/git \
# openstack/tempest gate. git://git.openstack.org openstack/tempest
if [[ "${ZUUL_PROJECT}" != "openstack/tempest" ]]; then # Pin Tempest to TEMPEST_VERSION unless we're running inside the
pushd /tmp/openstack/tempest # openstack/tempest gate.
git reset --hard $TEMPEST_VERSION if [[ "${ZUUL_PROJECT}" != "openstack/tempest" ]]; then
popd pushd /tmp/openstack/tempest
git reset --hard $TEMPEST_VERSION
popd
fi
fi fi
elif [ "${TEMPEST_FROM_SOURCE}" = true ]; then else
# remove existed checkout before clone # For ubuntu we always need to deploy tempest-horizon from source
$SUDO rm -rf /tmp/openstack/tempest
$SUDO rm -rf /tmp/openstack/tempest-horizon
# We're outside the gate, just do a regular git clone
git clone git://git.openstack.org/openstack/tempest /tmp/openstack/tempest
if uses_debs; then if uses_debs; then
$SUDO rm -rf /tmp/openstack/tempest-horizon
git clone git://git.openstack.org/openstack/tempest-horizon /tmp/openstack/tempest-horizon git clone git://git.openstack.org/openstack/tempest-horizon /tmp/openstack/tempest-horizon
fi fi
pushd /tmp/openstack/tempest if [ "${TEMPEST_FROM_SOURCE}" = true ]; then
git reset --hard origin/$TEMPEST_VERSION $SUDO rm -rf /tmp/openstack/tempest
popd git clone git://git.openstack.org/openstack/tempest /tmp/openstack/tempest
pushd /tmp/openstack/tempest
git reset --hard origin/$TEMPEST_VERSION
popd
fi
fi fi
# NOTE(pabelanger): We cache cirros images on our jenkins slaves, check if it # NOTE(pabelanger): We cache cirros images on our jenkins slaves, check if it
@ -236,7 +245,7 @@ print_header 'Prepare Tempest'
# We need to fix it in puppet-tempest, as a workaround we are changing the mode # We need to fix it in puppet-tempest, as a workaround we are changing the mode
# of tempest workspace and run tempest command using root. # of tempest workspace and run tempest command using root.
$SUDO touch /tmp/openstack/tempest/test-whitelist.txt $SUDO touch /tmp/openstack/tempest/test-whitelist.txt
$SUDO chown -R "$(id -u):$(id -g)" /tmp/openstack/tempest/test-whitelist.txt $SUDO chown -R "$(id -u):$(id -g)" /tmp/openstack/tempest/
# install from source now on ubuntu until packaged # install from source now on ubuntu until packaged
if uses_debs; then if uses_debs; then
@ -296,7 +305,8 @@ echo "test_create_bgpvpn" >> /tmp/openstack/tempest/test-whitelist.txt
echo "test_create_show_list_update_delete_l2gateway" >> /tmp/openstack/tempest/test-whitelist.txt echo "test_create_show_list_update_delete_l2gateway" >> /tmp/openstack/tempest/test-whitelist.txt
if uses_debs; then if uses_debs; then
EXCLUDES="--regex=^(?!mistral_tempest_tests.tests.api.v2.test_executions.ExecutionTestsV2.test_get_list_executions.*$)(?!ceilometer.tests.tempest.api.test_telemetry_notification_api.TelemetryNotificationAPITest.test_check_glance_v2_notifications.*$).*" # TODO (amoralej) tempest tests for object_storage are not working in master with current version of tempest in uca (16.1.0).
EXCLUDES="--regex=^(?!mistral_tempest_tests.tests.api.v2.test_executions.ExecutionTestsV2.test_get_list_executions.*$)(?!ceilometer.tests.tempest.api.test_telemetry_notification_api.TelemetryNotificationAPITest.test_check_glance_v2_notifications.*$)(?!tempest.api.object_storage.*$).*"
else else
EXCLUDES="--regex=^(?!tempest.scenario.gnocchi.test.live_assert_vcpus_metric_is_really_expurged.test_request.*$)(?!tempest.scenario.gnocchi.test.live_assert_no_delete_metrics_have_the_gabbilive_policy.test_request.*$).*" EXCLUDES="--regex=^(?!tempest.scenario.gnocchi.test.live_assert_vcpus_metric_is_really_expurged.test_request.*$)(?!tempest.scenario.gnocchi.test.live_assert_no_delete_metrics_have_the_gabbilive_policy.test_request.*$).*"
fi fi