Skip test_stack_update_with_replacing_userdata

This test case is frequently failing because of a known libvirt issue
in Ubuntu Jammy. We already disabled one functional test case, but will
disable this test case as well, to reduce failure rate of CI runs and
unblock gate.

The existing skip for a different test case[1] is re-implemented using
the proper configuration knob.

This also fixes tox.ini to adapt to new tox 4.0.
 - Update how passenv is defined because space-separated list is no
   longer allowed. Also the values are not case sensitive.
 - skipdist=True breaks installation so is removed.

[1] https://review.opendev.org/c/openstack/heat/+/866545

Co-Authored-By: Rabi Mishra <ramishra@redhat.com>
Related-Bug: #1998274
Story: 2010487
Task: 47056
Change-Id: I915dc83ccde6b6b8497642857292f9974fd84e98

devstack/lib/heat

@@ -438,8 +438,12 @@ function configure_tempest_for_heat {
     # Skip CfnInitIntegrationTest as latest fedora images don't have heat-cfntools
     iniset $TEMPEST_CONFIG heat_plugin skip_scenario_test_list 'AutoscalingLoadBalancerTest, AutoscalingLoadBalancerv2Test, \
 	    SoftwareConfigIntegrationTest, AodhAlarmTest, CfnInitIntegrationTest'
-    # Skip LoadBalancerv2Test as deprecated neutron-lbaas service is not enabled
-    iniset $TEMPEST_CONFIG heat_plugin skip_functional_test_list 'LoadBalancerv2Test, NotificationTest'
+    # Skip LoadBalancerv2Test as deprecated neutron-lbaas service is not enabled.
+    # Also Skip a few tests failing because of a known libvirt issue in Jammy
+    # https://bugs.launchpad.net/nova/+bug/1998274
+    iniset $TEMPEST_CONFIG heat_plugin skip_functional_test_list 'LoadBalancerv2Test, NotificationTest, \
+        UpdateStackTest.test_stack_update_with_replacing_userdata, \
+        CancelUpdateTest.test_cancel_update_server_with_port'
 
     openstack flavor show m1.heat_int || openstack flavor create m1.heat_int --ram 512 --disk 10
     openstack flavor show m1.heat_micro || openstack flavor create m1.heat_micro --ram 128 --disk 1

heat_integrationtests/functional/test_cancel_update.py

@@ -10,8 +10,6 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import testtools
-
 from heat_integrationtests.functional import functional_base
 
 
@@ -48,7 +46,6 @@ resources:
         if not self.conf.minimal_instance_type:
             raise self.skipException("No minimal flavor configured to test.")
 
-    @testtools.skip('Bug 1998274')
     def test_cancel_update_server_with_port(self):
         parameters = {'InstanceType': self.conf.minimal_instance_type,
                       'ImageId': self.conf.minimal_image_ref,

tox.ini

@@ -2,7 +2,6 @@
 envlist = py38,py39,pep8
 ignore_basepython_conflict = True
 minversion = 3.18.0
-skipsdist = True
 
 [testenv]
 basepython = python3
@@ -17,7 +16,10 @@ commands =
   stestr run {posargs}
   stestr slowest
 
-passenv = http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY
+passenv =
+  http_proxy
+  https_proxy
+  no_proxy
 
 [testenv:pep8]
 commands =