openstack/nova
Code Issues Proposed changes

Temporarily disable parts of heal port allocation test

Browse Source 
Due to bug 1894825 the nova-next job fails on master as the allocation
key cannot be deleted from the binding:profile of a neutron port. This
patch temporarily disable this test step while the bug is being fixed
and a new neutronlib is released with the fix.

Change-Id: I4dfebfb5c92dd8a5cdc779aac587e7477cd5fad6
Related-Bug: #1894825
Closes-Bug: #1898035
This commit is contained in:
Balazs Gibizer 2020-10-01 14:01:37 +02:00
parent 2beb18444f
commit d309e3cdf5
1 changed files with 22 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
gate/post_test_hook.sh
View File

@ -185,8 +185,11 @@ fi
echo "Deleting allocations in placement for the server" echo "Deleting allocations in placement for the server"
openstack resource provider allocation delete ${server_id} openstack resource provider allocation delete ${server_id}
echo "Deleting allocation key from the binding:profile of the bandwidth aware port" # NOTE(gibi) disable this step until the bug
openstack port unset --binding-profile allocation port-normal-qos # https://bugs.launchpad.net/neutron/+bug/1894825 in fixed and a new
# neutron-lib version is released
# echo "Deleting allocation key from the binding:profile of the bandwidth aware port"
# openstack port unset --binding-profile allocation port-normal-qos
# Make sure the allocations are gone. # Make sure the allocations are gone.
allocations=$(openstack resource provider allocation show ${server_id} \ allocations=$(openstack resource provider allocation show ${server_id} \
@ -196,12 +199,15 @@ if [[ "$allocations" != "" ]]; then
exit 2 exit 2
fi fi
# NOTE(gibi) disable this step until the bug
# https://bugs.launchpad.net/neutron/+bug/1894825 in fixed and a new
# neutron-lib version is released
# Make sure that the binding:profile.allocation key is gone # Make sure that the binding:profile.allocation key is gone
null_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation") # null_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation")
if [[ "$null_rp_uuid" != "" ]]; then # if [[ "$null_rp_uuid" != "" ]]; then
echo "Binding profile not updated for the bandwidth aware port." # echo "Binding profile not updated for the bandwidth aware port."
exit 2 # exit 2
fi # fi
# Make sure our extra key in the binding:profile is still there # Make sure our extra key in the binding:profile is still there
my_key=$(get_binding_profile_value port-normal-qos "my_key") my_key=$(get_binding_profile_value port-normal-qos "my_key")
@ -232,12 +238,16 @@ if [[ "$allocations" == "" ]]; then
exit 2 exit 2
fi fi
# NOTE(gibi) disable this step until the bug
# https://bugs.launchpad.net/neutron/+bug/1894825 in fixed and a new
# neutron-lib version is released
# Make sure that the allocations contains bandwidth as well # Make sure that the allocations contains bandwidth as well
bandwidth_allocations=$(echo "$allocations" | grep NET_BW_EGR_KILOBIT_PER_SEC) # bandwidth_allocations=$(echo "$allocations" | grep NET_BW_EGR_KILOBIT_PER_SEC)
if [[ "$bandwidth_allocations" == "" ]]; then # if [[ "$bandwidth_allocations" == "" ]]; then
echo "Failed to heal port allocations." # echo "Failed to heal port allocations."
exit 2 # exit 2
fi # fi
# Make sure that the binding:profile.allocation key healed back # Make sure that the binding:profile.allocation key healed back
healed_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation") healed_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation")