Friday, 2023-02-03

opendevreviewMerged openstack/tempest master: Move test_dhcp_port_status_active from neutron-tempest-plugin  https://review.opendev.org/c/openstack/tempest/+/82444001:24
*** yadnesh|away is now known as yadnesh04:12
opendevreviewGhanshyam proposed openstack/tempest master: Use project_reader in server test  https://review.opendev.org/c/openstack/tempest/+/87121004:26
opendevreviewAmit Uniyal proposed openstack/tempest master: WIP: Adds test for resize server swap to 0  https://review.opendev.org/c/openstack/tempest/+/85888507:16
opendevreviewAmit Uniyal proposed openstack/tempest master: WIP: Adds test for resize server swap to 0  https://review.opendev.org/c/openstack/tempest/+/85888507:21
*** jpena|off is now known as jpena08:44
plibeau4hello we have issue with rally-tox-cover on https://review.opendev.org/c/openstack/rally-openstack/+/871586/2 do you have idea to fix it?09:25
lpiwowargmann: I have answered your question and thank you for approving the fix! https://review.opendev.org/c/openstack/tempest/+/869440/5//COMMIT_MSG09:35
opendevreviewAmit Uniyal proposed openstack/tempest master: WIP: Adds test for resize server swap to 0  https://review.opendev.org/c/openstack/tempest/+/85888509:45
*** yadnesh is now known as yadnesh|away13:15
sean-k-mooneyjust an FYI i have some fixes for gnocchi installation via devstack both locally and in ci https://review.opendev.org/q/topic:gnocchi-devstack-fixes currently the openstack/telemetry-tempest-plugin patch is blocked by the rust/python cryptograpy issue on train but once that is resolved it woudl be nice to move those forward13:34
fungiearly heads up that debian 12 (bookworm) and ubuntu 23.04 (lunar) are going to disallow `sudo pip install` and anything you install with pip will need to go into venvs/virtualenvs15:40
fungihttps://salsa.debian.org/cpython-team/python3/-/blob/master/debian/README.venv15:41
fungihttps://discuss.python.org/t/10302/6915:41
fungithis is going to mean devstack refactors to rely entirely on venv installation before openstack moves officially to ubuntu 24.04 lts, but ideally much sooner since anyone on the next versions of debian and ubuntu are going to start hitting this in only a few months15:42
sean-k-mooneyfungi: well that means we need to revive/merge the patch that installs everything with devstack in a venv15:42
fungiyes, precisely15:42
sean-k-mooneyfor what its worth the latest version of that patch was working fine for me15:43
sean-k-mooneybut i have not used it in a few months15:43
sean-k-mooneyhttps://review.opendev.org/c/openstack/devstack/+/558930 for context of other who have not seen that15:44
fungiyep, i was thinking specifically of clarkb's implementation15:45
clarkbothers should feel welcome to take that work and update it or abandon it and start over etc15:45
fungiglad to hear it was still usable fairly recently, since it's been sitting there for so long15:45
fungiwe're coming up on its 5-year anniversary15:45
clarkbI expect my ability to drive something like this to merging these days is much more limited15:45
sean-k-mooneywell its been 9 months since i ran it15:45
sean-k-mooneybut i was able to use it to install on an m1 macbook pro using debian bookworm15:46
clarkbsean-k-mooney: thats good to hear15:46
sean-k-mooneyi have since nuked that linux install actully not macbook pro it was a macbook air15:47
sean-k-mooneyas far as i am aware the kvm issues i hit have also been fixed since then15:47
sean-k-mooneyon x86 you should not have issues i just pushed some draft comment i had pending but i have lost all the context on this15:48
sean-k-mooneyfungi: so i take it this would be a goal to compelte in the B cycle15:48
sean-k-mooneythe work and start/resume in devstack before then as the patch is opt in15:48
sean-k-mooneybut we would want to have ci jobs testing with this in 2023.215:49
clarkbsean-k-mooney: ya I think my suggestion would be to start it as soon as someone is able since you can switch to venvs before it becomes necessary15:49
opendevreviewSofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests  https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/74290515:49
sean-k-mooneyincidently the gnocchi devstack fixes i have in https://review.opendev.org/q/topic:gnocchi-devstack-fixes15:50
sean-k-mooneyare actuly due to sudo pip install and the git changes that happend recently15:50
sean-k-mooneyi would still prefer to merge those before the venv work but it would not have been requied if we were installing in a venv15:51
fungisean-k-mooney: i guess if it touches a bunch of separate plugins in projects or something, then the community goals framework is applicable. if it just needs updating in devstack itself, then just get it done, doesn't need coordination with other teams16:24
clarkbfungi: when I last poked at it seriously devstack and grenade were definitely the core bits of it, but it would also break projects (at the time I was trying to make ironic play nice with it)16:25
clarkbfungi: maye what we need is less of a community wide goal and more of a community wide "sorry this is happening and we'll just need to accept it"16:25
fungiand also, like clarkb says, the sooner the better. debian 12 and ubuntu 23.04 are only a few months away and users are going to start using them and expecting devstack to work16:25
clarkbfor when the devstack changes land16:25
sean-k-mooneyit can be done mostlyin devstack16:25
sean-k-mooneythen we just need to opt into it in the ci jobs by setting the flag16:26
sean-k-mooneyand eventually the default can be flipped in devstack and or the jobs16:26
clarkbsean-k-mooney: I think I'd make it the default16:26
clarkbonce it is working16:26
sean-k-mooneyyep me to16:26
fungiyeah, i wouldn't bother with goals/govenance, just make sure it's widely announced16:26
clarkband force people to opt out rather than in16:26
sean-k-mooneyand deprecate teh old way then remvoe it it C16:27
sean-k-mooneyfungi: i would suggest just doing a PSA about it on the mailing list and ensure we talk about it in the PTG where approriate16:27
fungiyeah, i'll put something together for the ml, just wanted to give folks a heads up in here as soon as i noticed it was going forward16:29
dansmithgmann: kopecmartin: the test timeouts lately are kinda weird17:01
dansmiththree minutes between subsequent test results in the section getting run when the timeout happens17:01
*** jpena is now known as jpena|off17:35
opendevreviewVadym Markov proposed openstack/devstack stable/wallaby: Use proper sed separator for paths  https://review.opendev.org/c/openstack/devstack/+/86584118:15
gmanndansmith: kopecmartin : I am thinking if this is adding something in timeout https://review.opendev.org/c/openstack/tempest/+/82173218:28
gmannbut this add serial run tag at test class level and supposed to run all serial test at the end 18:28
gmannbut it can add something to job timeout as running serial tests at the end add up some time to total test run18:30
dansmithah, are those running serial? they have concurrency=4, but yeah, the timing is like they're running one at a time18:30
gmanntwo tests for now run serial and at the end of all test run irrespective of concurrency18:31
dansmithoh is it just two?18:31
gmannoh no, its is more than two as serial tag in base class of aggregate tests18:32
dansmithokay, because I see two run successfully before it times out in the run I'm looking at18:32
gmannthis is taking 187 sec https://zuul.opendev.org/t/openstack/build/9b79f54d3f6c4e05ae6619fdac7ad95a/log/job-output.txt#2573818:36
gmannwhich is last test before timeout and it did not go till serial tests run itself18:37
gmannand this does lot of things so not sure which operation adding more time now https://github.com/openstack/tempest/blob/7c9389a00072dbcf7b4772d156f505cabeba4252/tempest/scenario/test_minimum_basic.py#L16918:37
gmannand this is the test I have seen timeout in multinode job also. 18:38
gmannI feel we should mark it as slow test (to run in tempest-slow job)18:38
gmannand if we still see timeout then increase the job time because of those serial test run addition18:38
gmanndansmith: kopecmartin: what you think?18:39
gmannanother example of this test taking 258 sec - https://zuul.openstack.org/build/084729c5a1fe46e091a17a50441e99d5/log/job-output.txt#3417318:40
dansmithgmann: sure, but there's clearly another test that is running for almost 4 minutes *after* that test18:40
gmannhumm. yeah that is good point18:42
gmannlet's mark it slow and see of we can find another slow tests. 18:43
gmannthough we do not know what added time in what operation in this release or it is due to ubuntu 22.0418:44
dansmithyeah, maybe the long one will run and then we'll timeout and see what it is :)18:44
gmannin this, this test fail (took 258 sec) but job reached to the end and running serial aggregate tests https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_084/872566/1/check/tempest-multinode-full-py3/084729c/controller/logs/stackviz/index.html#/stdin/timeline?test=tempest.serial_tests.scenario.test_aggregates_basic_ops.TestAggregatesBasicOps.test_aggregate_basic_ops18:48
dansmithyeah, that's long19:02
opendevreviewGhanshyam proposed openstack/tempest master: Mark test_minimum_basic_instance_hard_reboot_after_vol_snap_deletion as slow test  https://review.opendev.org/c/openstack/tempest/+/87269119:27
gmanndansmith: kopecmartin ^^ also opened bug to track the timeout https://bugs.launchpad.net/tempest/+bug/200478019:34
kopecmartingmann: thanks, i already +2 and +w it as it's Friday evening for me already and i'm not sure when i'll get back to the computer .. 19:34
dansmithcool19:35
gmannkopecmartin: thanks. have a nice weekend19:35
gmannI am planning to shutdown my PC today EOD otherwise its every weekend end up working story :)19:36
dansmithgmann: ++19:36
kopecmartinhave a nice weekend everyone19:40
kopecmartingmann: yeah , do that 19:40
kopecmartini have all work notifications silenced during weekends (and everyday after 6pm)19:41
gmann++19:41
dansmithaw man, chunked get made it to the gate, but is now failing there20:17
gmannhumm21:06
opendevreviewGhanshyam proposed openstack/tempest master: Mark test_minimum_basic_instance_hard_reboot_after_vol_snap_deletion as slow test  https://review.opendev.org/c/openstack/tempest/+/87269123:40
gmannkopecmartin: ^^ if you are still around. found one more slow tests and need approval23:42

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