| *** dmitriis is now known as Guest22 | 00:56 | |
| gmann | bauzas: did you get chance to get placement project created? You need to ask one of the member of "OpenStack Administrators" in LP to create the placement and assign it to "nova-driver" group to further maintain that. project https://launchpad.net/~openstack-admins/+members#active | 02:12 |
|---|---|---|
| gmann | bauzas: may be fungi can help on this ^^ | 02:12 |
| gmann | please let me know once done, I will create the BP there for my placement spec | 02:13 |
| opendevreview | Ghanshyam proposed openstack/placement master: Modify the placement API policies defaults and scope_type https://review.opendev.org/c/openstack/placement/+/865618 | 03:29 |
| opendevreview | Nobuhiro MIKI proposed openstack/nova-specs master: Add maxphysaddr support for Libvirt https://review.opendev.org/c/openstack/nova-specs/+/861033 | 03:30 |
| opendevreview | Ghanshyam proposed openstack/placement master: Modify the placement API policies defaults and scope_type https://review.opendev.org/c/openstack/placement/+/865618 | 03:51 |
| *** akekane is now known as abhishekk | 04:02 | |
| opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | 04:12 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/train: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/864168 | 07:38 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/train: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/864169 | 07:38 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/train: Block unsupported instance ops with accelerators. https://review.opendev.org/c/openstack/nova/+/865624 | 07:38 |
| Uggla | good morning Nova ! | 07:54 |
| gibi | o/ | 07:57 |
| gibi | Uggla: do you have anything I should review for you? | 07:57 |
| opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | 08:01 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | 08:04 |
| opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | 08:09 |
| bauzas | hello Nova | 08:13 |
| bauzas | gmann: ack, I didn't had time yet, but I can ask indeed | 08:14 |
| bauzas | I was thinking we were having the placement LP | 08:14 |
| gibi | sean-k-mooney: finally the stable/train version of https://review.opendev.org/c/openstack/nova/+/841444 is green, it took a while to land the fix that switched back the jobs to focal | 08:26 |
| sean-k-mooney[m] | ah that was till in flight i saw that it was proposed but had assumed that the job change had landed already | 08:28 |
| gibi | me too but | 08:29 |
| sean-k-mooney[m] | gibi: can you repond to artoms comment on the previous patch https://review.opendev.org/c/openstack/nova/+/841288/3 | 08:29 |
| sean-k-mooney[m] | im technically off today but im not going car shopping for 3 hours so i can take a look at a few things in the meantime | 08:30 |
| gibi | done | 08:32 |
| gibi | I'm not sure I see the issue artom saw | 08:32 |
| sean-k-mooney[m] | i just added some more comments | 08:49 |
| sean-k-mooney[m] | there is one nit but im ok with the contnet im going to quickly reivew the actual fix | 08:49 |
| sean-k-mooney[m] | gibi: ok im +2 on both of those | 08:55 |
| gibi | thanks | 08:55 |
| auniyal | Hi | 09:42 |
| auniyal | please review these - https://review.opendev.org/c/openstack/nova/+/864169 | 09:42 |
| bauzas | sean-k-mooney: so, should we be non-voting nova-next ? | 09:50 |
| * bauzas throwing a stone in the lake | 09:50 | |
| sean-k-mooney[m] | just add the trunk test to the exclude regex | 09:50 |
| sean-k-mooney[m] | for nova next and the nova ovs job | 09:51 |
| bauzas | lemme fire a change | 09:51 |
| bauzas | I'm done with rechecking | 09:51 |
| bauzas | looks like we also have another problem https://zuul.opendev.org/t/openstack/build/6b766705d27d4308999ddd36d763e325 | 09:52 |
| sean-k-mooney[m] | thats just volume detach | 09:55 |
| sean-k-mooney[m] | qemu broke that 2 years ago and never fixed it | 09:55 |
| sean-k-mooney[m] | for some reason the lvm job is showing it more often then everything else | 09:56 |
| gibi | we did fixed detach in tempest so those failures are for something new | 09:56 |
| sean-k-mooney[m] | fix is a stong word but yes we worked around the bootup issue in most cases | 09:57 |
| sean-k-mooney[m] | i still think we were missing one patch | 09:57 |
| gibi | yes but that work around means we should not see this failure | 09:57 |
| bauzas | sean-k-mooney: do we have a launchpad bug tracking the trunk problem ? | 09:58 |
| sean-k-mooney[m] | neutron do its not a nova issue | 09:58 |
| * bauzas is writing .zuul.yaml and I'd like adding a comment | 09:59 | |
| bauzas | ralonsoh: do you have a launchpad bug about the live-migration bug with trunk ports ? | 09:59 |
| bauzas | ralonsoh: context https://zuul.opendev.org/t/openstack/build/c6dee3859e354e6d8ed69c7ac7826761 | 10:00 |
| gibi | bauzas: https://bugs.launchpad.net/nova/+bug/1940425 | 10:04 |
| sean-k-mooney[m] | https://review.opendev.org/c/openstack/tempest/+/842240 is open for waiting before volume attach | 10:04 |
| bauzas | gibi: excellent thanks | 10:05 |
| gibi | sean-k-mooney[m]: good find | 10:06 |
| ralonsoh | bauzas, https://bugs.launchpad.net/neutron/+bug/1997025 | 10:06 |
| sean-k-mooney[m] | i just looked at https://review.opendev.org/q/topic:wait_until_sshable_pingable | 10:06 |
| gibi | maybe gmann has context for that https://review.opendev.org/c/openstack/tempest/+/842240 . I already lost the context | 10:08 |
| gibi | the linked other tempest patch was merged | 10:08 |
| gibi | so maybe we just need to merge https://review.opendev.org/c/openstack/tempest/+/842240 | 10:08 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Skip live_migration_with_trunk tempest test on ovs-hybrid and nova-next https://review.opendev.org/c/openstack/nova/+/865658 | 10:09 |
| bauzas | gibi: sean-k-mooney: zuul needs your help ^ | 10:09 |
| gibi | I've rebased https://review.opendev.org/c/openstack/tempest/+/842240 to get it out from W -1 | 10:09 |
| gibi | bauzas: on it | 10:09 |
| bauzas | I'm trying to figure out the occurrences | 10:09 |
| bauzas | but after 3 days trying to merge my patches, I dare people to tell me the problem is only intermittent :) | 10:10 |
| gibi | probably the frequency changed with the jammy switch | 10:10 |
| gibi | it was intermittent in the past | 10:10 |
| bauzas | oh, surely | 10:11 |
| bauzas | the bug is old | 10:11 |
| bauzas | the frequency changed tho | 10:11 |
| sean-k-mooney[m] | its not something we can fix in nova | 10:11 |
| bauzas | and that's why I'd like to eventually find the number | 10:11 |
| bauzas | sean-k-mooney: don't disagree | 10:11 |
| bauzas | the bug report says it's invalid for nhova | 10:11 |
| ralonsoh | bauzas, gibi since we merged os-vif 3.1.0, the problem is not always reproducible but happens 90% of the executions | 10:11 |
| ralonsoh | I'm finishing https://review.opendev.org/c/openstack/neutron/+/837780/ that shoudl avoid this issue in ML2/OVS | 10:12 |
| ralonsoh | testing patch: https://review.opendev.org/c/openstack/neutron/+/865039/ | 10:12 |
| gibi | ralonsoh: cool, thanks for working on it | 10:12 |
| bauzas | ralonsoh: great, thanks for working on it | 10:14 |
| bauzas | https://zuul.opendev.org/t/openstack/builds?job_name=nova-ovs-hybrid-plug&skip=0 | 10:15 |
| bauzas | that's... interesting | 10:15 |
| sean-k-mooney[m] | we have actully merged things on master in the last few days | 10:16 |
| sean-k-mooney[m] | just not your patch | 10:16 |
| sean-k-mooney[m] | but the failure rate is certenly high enough to be a problem | 10:16 |
| bauzas | yeah, but despite finding occurences of the exact issue, I just see that the number of builds from this job becomes flakey | 10:17 |
| sean-k-mooney[m] | bauzas: +1 on the zuul patch can you make the changes in line | 10:17 |
| bauzas | sean-k-mooney: I wasn't pointing to my own patch and I understand we can continue to merge | 10:17 |
| bauzas | sean-k-mooney: my only concern was the failure rate which is unknown | 10:17 |
| bauzas | and I suspect it to be higher than previously | 10:18 |
| sean-k-mooney[m] | oh i know i have noticed it while reviewing code over the last week or so | 10:18 |
| bauzas | (just to clarify my words above) | 10:18 |
| bauzas | sean-k-mooney: ack for the zuul +1, looking now | 10:18 |
| bauzas | ralonsoh: good spot then, I'll change the regex | 10:19 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Skip live_migration_with_trunk tempest test on ovs-hybrid and nova-next https://review.opendev.org/c/openstack/nova/+/865658 | 10:27 |
| bauzas | gibi: sean-k-mooney: ralonsoh: updated ^ | 10:27 |
| bauzas | please don't argue on the regex :) | 10:28 |
| bauzas | and I preferred to have the full test name | 10:29 |
| bauzas | (in case, a good regex trainer is https://regex101.com/= | 10:29 |
| sean-k-mooney | sure that works i just prefer keping it short but correct | 10:36 |
| sean-k-mooney | its on its way | 10:36 |
| gibi | stephenfin: if you have time the pci in placement series next 7 patches are ready to land and was reviewed by sean-k-mooney[m] already. https://review.opendev.org/c/openstack/nova/+/852771/13 | 10:50 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 11:10 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 11:17 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 11:22 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 11:26 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 11:41 |
| opendevreview | John Garbutt proposed openstack/nova-specs master: Ironic shard_key to replace peer_list https://review.opendev.org/c/openstack/nova-specs/+/862833 | 12:18 |
| fungi | gmann: bauzas: i don't think openstack-admins needs to create projects on lp. anyone can do it | 12:26 |
| fungi | at least i don't think i've ever created lp projects or groups for individual teams in the past, and i assume they always handled that themselves | 12:31 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | 13:17 |
| Uggla | gibi, sorry I have just seen your msg :( | 13:26 |
| Uggla | gibi, if you have time, sure you can continue with the virtiofs serie review. | 13:27 |
| stephenfin | gibi: sorry, was looking at k8s CAPO issues. Will take a look, sure | 13:53 |
| *** tbachman_ is now known as tbachman | 14:18 | |
| bauzas | f*** the zuul patch got hit by the nova-lvm job failure with volume detach | 14:18 |
| bauzas | f*cking Friday | 14:19 |
| bauzas | Black Friday is well suited name | 14:19 |
| sahid | o/ bauzas, dansmith how do you feel about https://review.opendev.org/c/openstack/nova-specs/+/857838 ? the code is ready I hope without too many issues :-) | 14:22 |
| bauzas | sahid: today's holiday for US folks | 14:35 |
| bauzas | sahid: but I can take a look if I have time | 14:35 |
| sahid | bauzas: oh.. did not noticed that | 14:36 |
| bauzas | gibi: https://zuul.opendev.org/t/openstack/builds?job_name=nova-lvm&skip=0 makes me feel nova-lvm is broken on master since a couple of days | 14:36 |
| sahid | sure please, but only if it's to put a +2 ;) | 14:36 |
| opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Proposes cpu power managment in libvirt https://review.opendev.org/c/openstack/nova-specs/+/861591 | 15:13 |
| bauzas | gibi: sean-k-mooney: updated my cpu power mgmt spec | 15:13 |
| gibi | bauzas: ack on both | 15:45 |
| gibi | stephenfin: thanks! | 15:46 |
| gibi | Uggla: ack, I will try to get to it | 15:47 |
| bauzas | gibi: yet again, nova-lvm failed | 15:50 |
| bauzas | https://review.opendev.org/c/openstack/nova/+/865658 | 15:50 |
| gibi | yeah it seems pretty brokenish | 15:50 |
| bauzas | I'm gonna amend the patch above to skip the other test in nova-lvm | 15:50 |
| gibi | yeah we need to. but we will forget about them unfortunately | 15:52 |
| gibi | I rebased https://review.opendev.org/c/openstack/tempest/+/842240 but it failed... | 15:52 |
| Uggla | gibi, thx | 15:52 |
| bauzas | gibi: https://104f3c0137add5b7a6f4-941b17a299be1c85b9ce2467ba19245c.ssl.cf1.rackcdn.com/865658/2/check/nova-lvm/bc3d051/testr_results.html | 15:55 |
| bauzas | that's even failing in a TearDownClass :( | 15:56 |
| bauzas | gibi: so test_rescue module is trampled, I'm proposing to skip it entirely, thoughts ? | 15:57 |
| gibi | sure, skip that fails to unblock the gate | 15:58 |
| bauzas | or shall we wait for your tempest patch to be green ? | 15:58 |
| bauzas | gibi: we could have the zuul file depend-on your tempest patch | 15:59 |
| gibi | that's gmann's patch :) and it does not work at the moment. | 15:59 |
| gibi | I already lost all my context on it | 16:02 |
| bauzas | ok, then I'm about to give up until monday | 16:02 |
| bauzas | when I see the gate and all the changes that were checked since one day, I feel pretty alone in the dark | 16:03 |
| gibi | I'm torn, I would like to help but my priorities are elswhere | 16:05 |
| gibi | I'm looking at the actual detach failure now to at least confirm it is the same issue | 16:05 |
| gibi | updated https://review.opendev.org/c/openstack/tempest/+/842240/ , it is a long shot as I'm not really know what I am doing in tempest :) | 16:23 |
| gibi | if it greens out then I will push a test nova patch depending on it | 16:24 |
| gmann | gibi: hi, sorry I also missed to track that. let me check it today | 18:45 |
| gmann | bauzas: i have created the LP project for placement and assigned maintainer to 'nova-drivers' https://launchpad.net/placement | 19:03 |
| gmann | bauzas: as I am not part of nova-drivers group in LP, i cannot re-assign the 'Driver' from me to 'nova-drivers' or add it as part of 'OpenStack' | 19:05 |
| gmann | or if either of you bauzas gibi dansmith melwitt can change back the Maintainer to me then I can finish the things there and assigned back to 'nova-drivers' | 19:09 |
| gmann | did not that after changing maintainer to nova-driver it would not allow me to do the things | 19:10 |
| gmann | that could have been last step | 19:10 |
| gmann | * did not know that | 19:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!