| *** yadnesh|away is now known as yadnesh | 04:00 | |
| ykarel | dansmith, ralonsoh it's a known issue https://bugs.launchpad.net/devstack/+bug/2002629 | 05:33 |
|---|---|---|
| ykarel | fix https://review.opendev.org/c/openstack/devstack/+/869969 | 05:33 |
| ykarel | gmann, kopecmartin please check ^ once around | 07:12 |
| *** soniya29|rover is now known as soniya29|rover|lunch | 07:38 | |
| *** soniya29|rover|lunch is now known as soniya29|rover | 08:30 | |
| *** jpena|off is now known as jpena | 08:42 | |
| ralonsoh | hi folks, maybe you can help me with this issue | 09:30 |
| ralonsoh | in neutron-tempest-plugin I see the project is installed using the requirements.txt file | 09:30 |
| ralonsoh | but not using the upper-constraints file | 09:30 |
| ralonsoh | https://581502a1694f018b82f7-7fbadfa7985858ab78a3aa61deaba560.ssl.cf1.rackcdn.com/869152/2/check/openstack-tox-pep8/a50ec7c/job-output.txt | 09:30 |
| ralonsoh | the project is editable and tox is installing the requirements file without using the limit upper-constraints | 09:31 |
| ralonsoh | ok, maybe this is because we don't have the requirements.txt file in the deps section | 09:32 |
| *** elodilles_pto is now known as elodilles | 10:19 | |
| *** soniya29|rover is now known as soniya29|rover|afk | 10:21 | |
| opendevreview | Lajos Katona proposed openstack/tempest master: Introduce @serial test execution decorator https://review.opendev.org/c/openstack/tempest/+/821732 | 11:03 |
| *** soniya29|rover|afk is now known as soniya29|rover | 11:09 | |
| *** soniya29|rover is now known as soniya29|rover|brb | 11:45 | |
| *** soniya29|rover|brb is now known as soniya29|rover | 12:01 | |
| sean-k-mooney | lajoskatona: if your going to continue workin on https://review.opendev.org/c/openstack/tempest/+/821732 can you try and get that landded before the end fo this cycle. | 12:10 |
| sean-k-mooney | tempest has been blocking us fixing the bug for over a year now https://review.opendev.org/c/openstack/nova/+/821423 | 12:11 |
| sean-k-mooney | i would really like to be able to get https://bugs.launchpad.net/nova/+bug/1907775 finally fixed | 12:15 |
| sean-k-mooney | i just rechecked https://review.opendev.org/c/openstack/nova/+/821423 so we shoudl see if the latest revsion fo the tempest patch works as intended | 12:15 |
| lajoskatona | sean-k-mooney: I am on it | 12:22 |
| sean-k-mooney | lajoskatona: hopefully im wrong and changing the folder name wont break the tempest patch the way i expect | 12:25 |
| sean-k-mooney | its been a long time since i looked at that properly | 12:25 |
| sean-k-mooney | creating the nested folder stucture shoudl not break anything so its just the zzz_serial cahnge that im concerned about | 12:26 |
| lajoskatona | sean-k-mooney: I fear the same, but locally I havent seen any problem with it, but I can change back if some zuul job find the problem | 12:26 |
| sean-k-mooney | ack | 12:29 |
| sean-k-mooney | the main problem i have with reviewing it is given how little work i don with tempest i dont have any of the context loaded about how this is being used other then zzz_ was being used for ordering and i currently dont have time to dig into to load up the context | 12:31 |
| *** soniya29|rover is now known as soniya29|rover|out | 16:07 | |
| dulek | Hi! Any idea what changed recently so Kuryr UT jobs started to fail? I cannot reproduce this locally: https://4c7502961c4643b6854e-e3236fcfb41e99e283b71fa476352983.ssl.cf5.rackcdn.com/869860/2/check/openstack-tox-py38/2034a97/job-output.txt | 17:05 |
| sean-k-mooney | lajoskatona: bad news is https://review.opendev.org/c/openstack/nova/+/821423 failed good news is it was the functional job | 17:35 |
| sean-k-mooney | lajoskatona: i have not check to see if the serial tests ran however | 17:36 |
| sean-k-mooney | if they did then your new reviesion is likely fine | 17:36 |
| sean-k-mooney | the test are here https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_cf2/821423/4/check/tempest-integrated-compute/cf23689/testr_results.html | 17:37 |
| sean-k-mooney | so the serial test ran the final question is did they run last after all other tests | 17:38 |
| sean-k-mooney | thats what teh zzz_ prefix was for to minmise the test execution overhead | 17:39 |
| sean-k-mooney | they are also last but only becasue ser comes after sce | 17:42 |
| sean-k-mooney | so serial_test is last becasue we dont happen to have a module that is later | 17:43 |
| sean-k-mooney | zzz_ was to force that | 17:43 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/cf236899fbc741058588dc27bac2eb00/log/job-output.txt#21057-21087 | 17:44 |
| *** jpena is now known as jpena|off | 17:46 | |
| lajoskatona | sean-k-mooney: yes I remember the original debate around zzzz_, but with this now we have a proof that technically serial_tests is also ok, and personally I am happy with either way | 17:50 |
| lajoskatona | sean-k-mooney: so I would wait for QA votes :-) | 17:50 |
| sean-k-mooney | ack it might be wort addting a doc for this soemwhere but if other are ok with it form the tempest side | 17:51 |
| sean-k-mooney | it unblocks us on the nova side and we can fix the bug | 17:51 |
| sean-k-mooney | thats why i removed the -1 since it looks like it works fine | 17:51 |
| lajoskatona | sean-k-mooney: thanks for the help | 18:05 |
| gmann | ykarel: ack | 18:37 |
| gmann | lajos, sean-k-mooney will review the serial decorator on priority. apology for delay in review on that. | 18:37 |
| sean-k-mooney | no worries | 18:37 |
| sean-k-mooney | if its making progress that fine | 18:38 |
| opendevreview | Merged openstack/devstack master: [OVN] Ensure socket files are absent in init_ovn https://review.opendev.org/c/openstack/devstack/+/869969 | 22:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!