summaryrefslogtreecommitdiff
path: root/gate
diff options
context:
space:
mode:
authorZuul <zuul@review.opendev.org>2020-12-16 18:37:40 +0000
committerGerrit Code Review <review@openstack.org>2020-12-16 18:37:40 +0000
commit5d910b6695fc67517b6fe08194ed2754a182f9a9 (patch)
tree7dea3af759305af29c2f2e08c7afcbf854fde791 /gate
parentfc38f822dd11b33da29ecd65beef58ed91aa8423 (diff)
parent528740128afada585cd35af80c4ebfe8af8bbea4 (diff)
downloadnova-5d910b6695fc67517b6fe08194ed2754a182f9a9.tar.gz
Merge "Revert "Temporarily disable parts of heal port allocation test""
Diffstat (limited to 'gate')
-rwxr-xr-xgate/post_test_hook.sh34
1 files changed, 12 insertions, 22 deletions
diff --git a/gate/post_test_hook.sh b/gate/post_test_hook.sh
index 47a8164c57..28ad9b939e 100755
--- a/gate/post_test_hook.sh
+++ b/gate/post_test_hook.sh
@@ -185,11 +185,8 @@ fi
echo "Deleting allocations in placement for the server"
openstack resource provider allocation delete ${server_id}
-# 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
-# echo "Deleting allocation key from the binding:profile of the bandwidth aware port"
-# openstack port unset --binding-profile allocation port-normal-qos
+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.
allocations=$(openstack resource provider allocation show ${server_id} \
@@ -199,15 +196,12 @@ if [[ "$allocations" != "" ]]; then
exit 2
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
-# null_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation")
-# if [[ "$null_rp_uuid" != "" ]]; then
-# echo "Binding profile not updated for the bandwidth aware port."
-# exit 2
-# fi
+null_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation")
+if [[ "$null_rp_uuid" != "" ]]; then
+ echo "Binding profile not updated for the bandwidth aware port."
+ exit 2
+fi
# Make sure our extra key in the binding:profile is still there
my_key=$(get_binding_profile_value port-normal-qos "my_key")
@@ -238,16 +232,12 @@ if [[ "$allocations" == "" ]]; then
exit 2
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
-# bandwidth_allocations=$(echo "$allocations" | grep NET_BW_EGR_KILOBIT_PER_SEC)
-# if [[ "$bandwidth_allocations" == "" ]]; then
-# echo "Failed to heal port allocations."
-# exit 2
-# fi
+bandwidth_allocations=$(echo "$allocations" | grep NET_BW_EGR_KILOBIT_PER_SEC)
+if [[ "$bandwidth_allocations" == "" ]]; then
+ echo "Failed to heal port allocations."
+ exit 2
+fi
# Make sure that the binding:profile.allocation key healed back
healed_rp_uuid=$(get_binding_profile_value port-normal-qos "allocation")