Friday, 2024-10-25

*** mhen_ is now known as mhen01:39
melwittis there a supported way to get around https://github.com/openstack/tempest/blob/master/tools/tempest-extra-tests-list.txt in jobs like devstack-plugin-nfs-tempest-full? I have a patch that proposes a fix for an NFS volume bug but in order to cover it, tempest.api.compute.admin. VolumesAssistedSnapshotsTest would need to run01:57
gmannmelwitt: you can use the tox env 'full' which run the admin test also. or use the tox env 'all' and define tempest_test_regex to run a set of tests  example https://github.com/openstack/tempest/blob/357fc7ec2dfb1fb27d52cd518375e4d8ec977733/zuul.d/integrated-gate.yaml#L35502:41
gmannmelwitt: this is what 'full' tox env run "API tests in parallel + scenario in serial - slow tests":  https://github.com/openstack/tempest/blob/master/tox.ini#L114-L12002:42
melwittgmann: ok, thanks. the current command line is 'tempest run --regex '(?!.*\[.*\bslow\b.*\])(^tempest\.api)' --exclude-list ./tools/tempest-extra-tests-list.txt --concurrency=4' I wondered if there was a way to adjust it via the job definition02:43
gmannor maybe 'integrated-compute' which exclude keytsone and swift tests and run everything else02:44
gmannmelwitt: i see. we have variable for that let me find02:44
melwittI'm trying to find the easiest way 😂 02:45
gmannmelwitt: this one you can use to exclude the tests tempest_test_exclude_list02:47
gmannmelwitt: not tested but this should work in your case https://paste.openstack.org/show/blwGI2kD4CehFAWjlvv0/02:47
melwittgmann: thanks! I will try it. I wondered if you could make include list and exclude list fight against each other and I wasn't sure which would win02:48
gmannmelwitt: Tempest have separate variable for those and we take care of those before sending it to test runner, here are all "tempest run" options in run-tempest role which we can use in job definition https://github.com/openstack/tempest/tree/master/roles/run-tempest02:50
melwittthanks gmann 02:51
opendevreviewyatin proposed openstack/devstack stable/2024.2: Pull RDO Trunk repos when CentOS Stream official RPM not available  https://review.opendev.org/c/openstack/devstack/+/93338405:07
opendevreviewMerged openstack/devstack master: Add image format enforcement toggle  https://review.opendev.org/c/openstack/devstack/+/93102607:56
opendevreviewMerged openstack/devstack master: Globally skip devstack job for pre-commit config update  https://review.opendev.org/c/openstack/devstack/+/93312507:56
opendevreviewMerged openstack/devstack master: Replace deprecated datetime.utcnow()  https://review.opendev.org/c/openstack/devstack/+/93083908:38
opendevreviewEduardo Olivares proposed openstack/tempest master: Remove tox-py38 jobs  https://review.opendev.org/c/openstack/tempest/+/93339808:59
opendevreviewEduardo Olivares proposed openstack/tempest master: Remove tox-py38 jobs  https://review.opendev.org/c/openstack/tempest/+/93339809:00
opendevreviewEduardo Olivares proposed openstack/tempest master: Measure metadata downtime during live-migration  https://review.opendev.org/c/openstack/tempest/+/93128909:00
eolivaregmann, kopecmartin o/09:01
eolivaretox-py38 jobs are broken. Can you please review this patch? https://review.opendev.org/c/openstack/tempest/+/93339809:02
fricklereolivare: kopecmartin: maybe you'd rather want to override to some stable reqs for that job? (not sure how much effort that would be, though)09:09
eolivarefrickler, it's not my decision, but if it's not supported, we could save some resources/time removing those jobs09:22
kopecmartinwe don't have different set of requirements for older pythons, so removing the job is the right thing to do09:50
clarkbthats not entirely true right? you can use python version markers to have different requirements for different pythons. I don't think this plays into py38 job removal too much but want to calrify you absolutely can do that13:05
*** whoami-rajat_ is now known as whoami-rajat14:21
opendevreviewJoel Capitao proposed openstack/devstack master: Pull RDO Trunk repos when CentOS Stream official RPM not available  https://review.opendev.org/c/openstack/devstack/+/93091315:28
opendevreviewyatin proposed openstack/devstack stable/2024.2: Pull RDO Trunk repos when CentOS Stream official RPM not available  https://review.opendev.org/c/openstack/devstack/+/93338415:42
gmannStop testing py3.8 is not that straight forward in tempest.  If we remove the testing of py3.8 then we have to remove the support of py3.8 and we need to release a tag before that. 16:22
gmannAnother point is that we need to check how it will impact the supported stable branches which has py3.8 as default. I am checking it.16:22
tkajinamso probably we have to create py38-constraints as we did for py3616:34
gmannwe did not create py36 constraint file18:02
gmannor py3818:02
clarkbI think constraints files can have python markers too18:15
clarkbhttps://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L42-L43 ya18:15
gmanno know but I am saying we did not go for this option for py3.6 case because tempest master will not be able to ship new features of oslo/other deps for older python branches. it is better to test them with older tempest.18:18
gmannbut seeing all older supported branches by tempest master support py3.9 so testing can be done on py3.9 or tempest in py3.9 venv is v3.9 not available on the cloud testing env18:19
gmannI think we can drop the py3.8 support from tempest but need to release a new tag and recommend to use tempest in py3.9 venv for older stable branches if they want to test them with Tempest master18:19
clarkbgot it18:20
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Wait for libvirt domain shutdown correctly  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/93345418:33
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Wait for libvirt domain shutdown correctly  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/93345418:36
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Wait for libvirt domain shutdown correctly  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/93345418:44
gmannkopecmartin: I will start the Tempest new release today, not sure if you will be able to review the constraint pin things but it should be ok to do on Monday as people just finished PTG and they are not blocked much. I will send email anyways to give a heads up about tempest constraint pinning is cmg18:49
*** elodilles is now known as elodilles_pto20:28
opendevreviewGhanshyam proposed openstack/tempest master: Remove python 3.8 support  https://review.opendev.org/c/openstack/tempest/+/93339821:10
gmannkopecmartin:  seeing the changes merged between Tempest last release 41.0.0 and now, I think we can declare the 41.0.0 as last supported release for python 3.8. If you think new release is needed then let me know otherwise ^^ I updated the change21:11

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!