| opendevreview | Ghanshyam proposed openstack/devstack-plugin-ceph stable/stein: Moving the stable/rocky and older stable jobs to stable/stein https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811510 | 00:25 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/grenade stable/xena: zuul: Add the neutron devstack plugin and enable neutron-trunk service https://review.opendev.org/c/openstack/grenade/+/811511 | 00:49 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/victoria: Enable neutron trunk extension https://review.opendev.org/c/openstack/devstack/+/811518 | 00:57 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/ussuri: Enable neutron trunk extension https://review.opendev.org/c/openstack/devstack/+/811519 | 01:01 |
| opendevreview | Merged openstack/tempest-stress master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/tempest-stress/+/788223 | 02:44 |
| opendevreview | Merged openstack/tempest-stress master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/tempest-stress/+/787077 | 02:44 |
| opendevreview | wangzhiguang proposed openstack/tempest master: delete group type testcase https://review.opendev.org/c/openstack/tempest/+/811525 | 03:03 |
| opendevreview | Merged openstack/patrole master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/patrole/+/787169 | 03:19 |
| opendevreview | Tushar Trambak Gite proposed openstack/coverage2sql master: Switch testing to Yoga testing runtime https://review.opendev.org/c/openstack/coverage2sql/+/811527 | 03:29 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811074 | 03:38 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811075 | 03:38 |
| opendevreview | Ghanshyam proposed openstack/devstack-plugin-ceph stable/xena: Fix job definition as devstack-plugin-ceph is branched https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811348 | 03:42 |
| gmann | melwitt: gouthamr ^^ xena backport for ceph job fix | 03:44 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/wallaby: Enable neutron trunk extension https://review.opendev.org/c/openstack/devstack/+/811457 | 03:47 |
| opendevreview | Ghanshyam proposed openstack/grenade stable/xena: zuul: Add the neutron devstack plugin and enable neutron-trunk service https://review.opendev.org/c/openstack/grenade/+/811511 | 03:53 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/victoria: Enable neutron trunk extension https://review.opendev.org/c/openstack/devstack/+/811518 | 03:59 |
| opendevreview | Ghanshyam proposed openstack/grenade stable/victoria: zuul: Add the neutron devstack plugin and enable neutron-trunk service https://review.opendev.org/c/openstack/grenade/+/811542 | 04:13 |
| gmann | gibi: lyarwood backporting grenade fix in victoria also. Basically I am 1. enabling the extension in devstack(which was missed earlier that is why test was skipped and nova-grenade-multinode passing) 2. testing job on nova side with extension enabled on devstack (depends-on) + grenade fix (devstack's patch depends-on) - https://review.opendev.org/q/topic:%22bug%252F1945346%22+(status:open%20OR%20status:merged) | 04:17 |
| gmann | I will check the result in morning. | 04:18 |
| opendevreview | Merged openstack/tempest master: Fix unused parameters in _create_subnet method https://review.opendev.org/c/openstack/tempest/+/807976 | 04:18 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/ussuri: Enable neutron trunk extension https://review.opendev.org/c/openstack/devstack/+/811519 | 04:24 |
| gmann | I have added the comments on each devstack and grenade patch about seeing the result of the fix | 04:25 |
| gmann | frickler: yoctozepto kopecmartin once you are online and results are happy in this (this is fix for nova gate blocker https://bugs.launchpad.net/grenade/+bug/1945346) https://review.opendev.org/q/topic:%22bug%252F1945346%22+(status:open%20OR%20status:merged) | 04:26 |
| gmann | sequence is little confusing as devstack old need to enable the extension to run the test in stable grenade job, for example. devstack stable/wallaby need to enable the extension and it trigger test to run in stable/xena so devstack stable/wallaby patch has depends-on with the grenade stable/xena fix to pass that test | 04:29 |
| gmann | this sequence is tested in nova stable/* DNM patches | 04:30 |
| opendevreview | Merged openstack/tempest master: Skip snaphost tests unless snapshot feature is enabled https://review.opendev.org/c/openstack/tempest/+/808909 | 04:43 |
| *** jpena|off is now known as jpena | 07:28 | |
| yoctozepto | gmann: what's victoria there for though? | 07:29 |
| opendevreview | Merged openstack/tempest master: Fix IPv6 localhost replacement in Swift container sync test https://review.opendev.org/c/openstack/tempest/+/810681 | 10:22 |
| *** pojadhav- is now known as pojadhav | 10:34 | |
| *** poojajadhav is now known as pojadhav | 11:08 | |
| *** jpena is now known as jpena|lunch | 11:24 | |
| *** pojadhav is now known as pojadhav|afk | 12:15 | |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Test Network QoS minimum packet rate scenarios https://review.opendev.org/c/openstack/tempest/+/806257 | 12:17 |
| *** jpena|lunch is now known as jpena | 12:22 | |
| gmann | frickler: yoctozepto kopecmartin this is for nova-ceph job re-enabling (https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1945358) https://review.opendev.org/q/topic:%22bug%252F1945358%22+(status:open%20OR%20status:merged) | 12:35 |
| yoctozepto | gmann: yeah but I did not understand what happened | 12:57 |
| yoctozepto | what has changed | 12:57 |
| gmann | yoctozepto: you ceph job one? | 12:57 |
| yoctozepto | something in tempest? | 12:58 |
| yoctozepto | gmann: ah, I mean trunk | 12:58 |
| yoctozepto | what's in there | 12:58 |
| gmann | yoctozepto: ohk :) | 12:58 |
| yoctozepto | can you explain it to me? | 12:59 |
| yoctozepto | why do we "fix" down to victoria, yet skip master | 12:59 |
| yoctozepto | I missed the reasoning | 12:59 |
| gmann | yoctozepto: so trunk extensions was missed to enable in devstack stable branch when we cap them and so tempest test test_live_migration_with_trunk was skipped - | 12:59 |
| yoctozepto | ok, that I can buy | 13:00 |
| yoctozepto | but then where does the failure come from? | 13:00 |
| gmann | yoctozepto: master has enable all the extension by default with 'All' so everything run there https://github.com/openstack/devstack/blob/master/lib/tempest#L663 | 13:00 |
| gmann | yoctozepto: and it is failing master nova-grenade-multinode job on master | 13:00 |
| yoctozepto | hmm, but how come we only see this now and not on previous branchings? | 13:01 |
| gmann | while checking it i realize this extension is not enabled in stable branches to job passing because test is skipped | 13:01 |
| gmann | yoctozepto: because tests were skipped and job passing. | 13:01 |
| gmann | when i enbaled it in stable branch you can see test failing there too | 13:01 |
| yoctozepto | yeah, so what changed? grenade got more coverage? | 13:02 |
| gmann | here https://review.opendev.org/c/openstack/nova/+/811513 | 13:02 |
| gmann | I am enabling trunk in stable too - https://review.opendev.org/c/openstack/devstack/+/811457 | 13:02 |
| gmann | so basically if we enable trunk in stable then we need grenade fix on stable otherwise not | 13:03 |
| gmann | question is if we want to increase the trunk test coverage on stable in tempest multinode and grenade multinode job then we need to enable | 13:03 |
| yoctozepto | yeah, that makes sens e | 13:04 |
| yoctozepto | I meant though why we have observed this in yoga | 13:04 |
| yoctozepto | and not when master was xena | 13:04 |
| gmann | ohk, one more thing. xena is also not yet capped the extensions and run with 'All' as of now ( https://review.opendev.org/c/openstack/devstack/+/811485) | 13:05 |
| gmann | so it is failing on master and Xena where trunk extenion is enabled | 13:06 |
| gmann | since what time it started failing in Xena and not in starting I think lyarwood may know exact reasoning | 13:06 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/xena: Cap Volume, Swift, and Volume API extensions list for stable/xena https://review.opendev.org/c/openstack/devstack/+/811485 | 13:12 |
| lyarwood | gmann: sorry was AFK, is the live migration job still failing? There are a few known issues that mean we've moved it to non-voting on stable/xena and master at the moment in Nova | 13:17 |
| lyarwood | gmann: I've just made a note on the PTG pad to ask for help from the libvirt folks in closing these out early in the Yoga cycle | 13:18 |
| gmann | lyarwood: no, its same nova-grenade-multinode job things for trunk | 13:18 |
| yoctozepto | yeah, I'm asking for more background to understand it better :-) | 13:21 |
| opendevreview | Merged openstack/os-performance-tools master: Switch testing to Yoga testing runtime https://review.opendev.org/c/openstack/os-performance-tools/+/811296 | 13:31 |
| gmann | kopecmartin: replied on https://review.opendev.org/c/openstack/tempest/+/810998 not sure what CI is that and what configuration. | 13:57 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Refactor existing QoS min bw tests https://review.opendev.org/c/openstack/tempest/+/806255 | 14:18 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Add QoS minimum packet rate rule client https://review.opendev.org/c/openstack/tempest/+/806256 | 14:18 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Test Network QoS minimum packet rate scenarios https://review.opendev.org/c/openstack/tempest/+/806257 | 14:18 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Consolidate client usage in MinBwAllocationPlacementTest https://review.opendev.org/c/openstack/tempest/+/811733 | 14:19 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Reproduce bug 1943724 https://review.opendev.org/c/openstack/tempest/+/809198 | 14:22 |
| opendevreview | Pavan Kesava Rao proposed openstack/whitebox-tempest-plugin master: [WIP] Add vTPM testcases https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/811737 | 14:46 |
| opendevreview | Merged openstack/grenade master: zuul: Add the neutron devstack plugin and enable neutron-trunk service https://review.opendev.org/c/openstack/grenade/+/811117 | 14:48 |
| opendevreview | Pavan Kesava Rao proposed openstack/whitebox-tempest-plugin master: [WIP] Add vTPM testcases https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/811737 | 14:49 |
| gmann | yoctozepto: kopecmartin meanwhile need review in ceph job fix too https://review.opendev.org/q/topic:%22bug%252F1945358%22+(status:open%20OR%20status:merged) | 15:12 |
| melwitt | gmann: ack will look | 15:14 |
| opendevreview | Merged openstack/stackviz master: Switch testing to Yoga testing runtime https://review.opendev.org/c/openstack/stackviz/+/811297 | 15:18 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Test Network QoS minimum packet rate scenarios https://review.opendev.org/c/openstack/tempest/+/806257 | 15:19 |
| gmann | melwitt: thanks | 15:19 |
| *** dviroel is now known as dviroel|ruck | 15:23 | |
| yoctozepto | gmann: approved | 15:31 |
| yoctozepto | that thing I understood, but the trunk is still confusing me | 15:32 |
| yoctozepto | :D | 15:32 |
| gmann | yoctozepto: thanks. | 15:38 |
| gmann | yoctozepto: ok, on trunk i think i have answer now | 15:40 |
| yoctozepto | spill the beans :D | 15:40 |
| gmann | yoctozepto: it start failing in master when stable/xena was cut. because while xena master stable/wallaby did not run the test as extension was not enabled (https://review.opendev.org/c/openstack/devstack/+/811457) | 15:41 |
| gmann | and when we cut the stable/xena and until now stable/xena still has 'All' extensions enabled | 15:42 |
| gmann | https://review.opendev.org/c/openstack/devstack/+/811485/2/lib/tempest#663 | 15:42 |
| gmann | so it fail only on master (bcz stable/xena enable all extension) and starting when stable/xena is cut | 15:43 |
| gmann | hope i clarify it :) | 15:44 |
| yoctozepto | gmann: so it will get fixed by merging the cap | 15:46 |
| yoctozepto | but you are suggesting to just enable testing it | 15:46 |
| yoctozepto | is that correct now? | 15:46 |
| gmann | yoctozepto: so if we enable the test then we need grenade fix also which is depends-on in devstack patches. | 15:49 |
| gmann | yoctozepto: I am waiting for neutron team opinon on whether enabling test on stable is fine or not | 15:50 |
| gmann | but they would not merge until grenade fix merge | 15:50 |
| *** jpena is now known as jpena|off | 16:38 | |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Fix job definition as devstack-plugin-ceph is branched https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 | 16:41 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/xena: Fix job definition as devstack-plugin-ceph is branched https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811348 | 16:51 |
| opendevreview | Archit Modi proposed openstack/whitebox-tempest-plugin master: DNM: Run whitebox-devstack-ceph-compute-local-ephemeral https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/811193 | 19:36 |
| opendevreview | Merged openstack/devstack stable/wallaby: Re-enable nova-ceph-multistore job https://review.opendev.org/c/openstack/devstack/+/811490 | 21:02 |
| opendevreview | Merged openstack/devstack stable/xena: Re-enable nova-ceph-multistore job https://review.opendev.org/c/openstack/devstack/+/811480 | 21:02 |
| kopecmartin | gmann: I was out of the office for majority of today, I'll check the review first thing in the morning | 21:04 |
| gmann | kopecmartin: no worry, gate blocker are all good/merged now. stable branch one (enabling tests) are pending which are no hurry and waiting for neutron team response first | 21:05 |
| kopecmartin | that's perfect | 21:06 |
| kopecmartin | wau, you made a lot of patches today | 21:06 |
| opendevreview | Merged openstack/devstack stable/victoria: Re-enable nova-ceph-multistore job https://review.opendev.org/c/openstack/devstack/+/811489 | 21:17 |
| *** dviroel|ruck is now known as dviroel|out | 21:34 | |
| gmann | kopecmartin: long week, release time as usual :) | 22:37 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!