| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-25 | |||
| 10:16:42 | sean-k-mooney[m] | but the failure rate is certenly high enough to be a problem | |
| 10:17:11 | 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:20 | sean-k-mooney[m] | bauzas: +1 on the zuul patch can you make the changes in line | |
| 10:17:38 | bauzas | sean-k-mooney: I wasn't pointing to my own patch and I understand we can continue to merge | |
| 10:17:52 | bauzas | sean-k-mooney: my only concern was the failure rate which is unknown | |
| 10:18:09 | bauzas | and I suspect it to be higher than previously | |
| 10:18:26 | sean-k-mooney[m] | oh i know i have noticed it while reviewing code over the last week or so | |
| 10:18:30 | bauzas | (just to clarify my words above) | |
| 10:18:48 | bauzas | sean-k-mooney: ack for the zuul +1, looking now | |
| 10:19:24 | bauzas | ralonsoh: good spot then, I'll change the regex | |
| 10:27:16 | 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:25 | bauzas | gibi: sean-k-mooney: ralonsoh: updated ^ | |
| 10:28:50 | bauzas | please don't argue on the regex :) | |
| 10:29:27 | bauzas | and I preferred to have the full test name | |
| 10:29:52 | bauzas | (in case, a good regex trainer is https://regex101.com/= | |
| 10:36:37 | sean-k-mooney | sure that works i just prefer keping it short but correct | |
| 10:36:54 | sean-k-mooney | its on its way | |
| 10:50:12 | 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 | |
| 11:10:00 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:17:21 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:22:31 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:26:50 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:41:29 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 12:18:25 | 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:26:20 | fungi | gmann: bauzas: i don't think openstack-admins needs to create projects on lp. anyone can do it | |
| 12:31:39 | 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 | |
| 13:17:54 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 13:26:31 | Uggla | gibi, sorry I have just seen your msg :( | |
| 13:27:28 | Uggla | gibi, if you have time, sure you can continue with the virtiofs serie review. | |
| 13:53:10 | stephenfin | gibi: sorry, was looking at k8s CAPO issues. Will take a look, sure | |
| 14:18:56 | bauzas | f*** the zuul patch got hit by the nova-lvm job failure with volume detach | |
| 14:19:05 | bauzas | f*cking Friday | |
| 14:19:25 | bauzas | Black Friday is well suited name | |
| 14:22:11 | 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:35:12 | bauzas | sahid: today's holiday for US folks | |
| 14:35:41 | bauzas | sahid: but I can take a look if I have time | |
| 14:36:03 | sahid | bauzas: oh.. did not noticed that | |
| 14:36:05 | 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:24 | sahid | sure please, but only if it's to put a +2 ;) | |
| 15:13:07 | 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:24 | bauzas | gibi: sean-k-mooney: updated my cpu power mgmt spec | |
| 15:45:51 | gibi | bauzas: ack on both | |
| 15:46:05 | gibi | stephenfin: thanks! | |
| 15:47:50 | gibi | Uggla: ack, I will try to get to it | |
| 15:50:08 | bauzas | gibi: yet again, nova-lvm failed | |
| 15:50:13 | bauzas | https://review.opendev.org/c/openstack/nova/+/865658 | |
| 15:50:21 | gibi | yeah it seems pretty brokenish | |
| 15:50:54 | bauzas | I'm gonna amend the patch above to skip the other test in nova-lvm | |
| 15:52:07 | gibi | yeah we need to. but we will forget about them unfortunately | |
| 15:52:21 | gibi | I rebased https://review.opendev.org/c/openstack/tempest/+/842240 but it failed... | |
| 15:52:33 | Uggla | gibi, thx | |
| 15:55:31 | bauzas | gibi: https://104f3c0137add5b7a6f4-941b17a299be1c85b9ce2467ba19245c.ssl.cf1.rackcdn.com/865658/2/check/nova-lvm/bc3d051/testr_results.html | |
| 15:56:05 | bauzas | that's even failing in a TearDownClass :( | |
| 15:57:43 | bauzas | gibi: so test_rescue module is trampled, I'm proposing to skip it entirely, thoughts ? | |
| 15:58:11 | gibi | sure, skip that fails to unblock the gate | |
| 15:58:27 | bauzas | or shall we wait for your tempest patch to be green ? | |
| 15:59:04 | bauzas | gibi: we could have the zuul file depend-on your tempest patch | |
| 15:59:31 | gibi | that's gmann's patch :) and it does not work at the moment. | |
| 16:02:27 | gibi | I already lost all my context on it | |
| 16:02:49 | bauzas | ok, then I'm about to give up until monday | |
| 16:03:15 | bauzas | when I see the gate and all the changes that were checked since one day, I feel pretty alone in the dark | |
| 16:05:14 | gibi | I'm torn, I would like to help but my priorities are elswhere | |
| 16:05:40 | gibi | I'm looking at the actual detach failure now to at least confirm it is the same issue | |
| 16:23:51 | 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:24:08 | gibi | if it greens out then I will push a test nova patch depending on it | |
| 18:45:25 | gmann | gibi: hi, sorry I also missed to track that. let me check it today | |
| 19:03:59 | gmann | bauzas: i have created the LP project for placement and assigned maintainer to 'nova-drivers' https://launchpad.net/placement | |
| 19:05:12 | 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:09:49 | 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:10:11 | gmann | did not that after changing maintainer to nova-driver it would not allow me to do the things | |
| 19:10:25 | gmann | that could have been last step | |
| 19:20:44 | gmann | * did not know that | |
| #openstack-nova - 2022-11-26 | |||
| 02:32:37 | opendevreview | Ghanshyam proposed openstack/nova master: DNM:testing lvm job https://review.opendev.org/c/openstack/nova/+/865765 | |
| #openstack-nova - 2022-11-27 | |||
| 15:25:58 | opendevreview | Andrei Yachmenev proposed openstack/nova-specs master: Add audio device to libvirt XML by nova libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/865634 | |
| 19:16:43 | opendevreview | Rafael Weingartner proposed openstack/nova master: Nova to honor "cross_az_attach" during server(VM) migrations https://review.opendev.org/c/openstack/nova/+/864760 | |
| #openstack-nova - 2022-11-28 | |||
| 09:46:20 | bauzas | gmann: thanks for having incepted the placement LP resource https://launchpad.net/placement | |
| 09:46:38 | bauzas | gmann: since I don't know exactly what's left, I just set the maintainer back to you, sure | |
| 09:46:47 | bauzas | gmann: ping me once you're done | |
| 10:14:49 | darkhorse | Hi team, In the compute_nodes table, there are 'host' column and 'hypervisor_hostname' column, which looks the same. Will you explain the difference between these fields? | |
| 10:24:38 | gibi | bauzas: so https://review.opendev.org/c/openstack/tempest/+/842240 is not green yet, but gmann pushed a test patch for nova-lvm with it https://review.opendev.org/c/openstack/nova/+/865765 it is also red but with one less failed test case (test_attach_detach_volume is green there now) | |
| 10:25:40 | bauzas | ack | |
| 10:26:26 | bauzas | darkhorse: 'host' is about the name of the nova-compute service | |
| 10:26:36 | bauzas | which is opted in by the conf option | |
| 10:27:00 | bauzas | darkhorse: while "hypervisor_hostname" is provided by the virt driver | |
| 10:30:42 | darkhorse | bauzas: thank you for clarification! | |
| 10:30:50 | bauzas | np | |
| 10:32:06 | bauzas | gibi: sean-k-mooney: could you please review my spec https://review.opendev.org/c/openstack/nova-specs/+/861591 before 3pm UTC today ? :) | |
| 10:34:20 | 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:35:32 | bauzas | gibi: sean-k-mooney: gmann: ^ fwiw, added depends-on gmann's tempest patch on ^ to eventually get it accepted | |
| 10:42:12 | gibi | bauzas: lets wait for gmann to get up and discuss what are our options about the gate regarding the detach issue. If gmann sees a possible fix then I would wait for it, but if he doesn't then I cannot offer any better way forward now than disabling the tests given the amount of time I can spend on this. | |
| 10:43:17 | bauzas | gibi: I need to catch my daughter at school but let's revisit this once gmann's up | |
| 10:43:24 | gibi | yepp | |
| 10:43:55 | gibi | bauzas: I'm +2 on https://review.opendev.org/c/openstack/nova-specs/+/861591 | |
| 10:49:25 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 11:11:41 | bauzas | gibi: I very briefly looked at the failing tests and yeah, we need to continue working on it if we want to merge it | |
| 11:11:53 | bauzas | I can try to help but I don't have a lot of context about it | |
| 11:12:17 | bauzas | also, I wonder *why* we now have a larger number of issues with nova-lvm | |
| 11:12:46 | bauzas | anyway, /me goes afk for lunch | |
| 11:13:59 | kashyap | bauzas: I think not many (if any at all) production users use LVM | |
| 11:14:06 | kashyap | (With Nova, I mean) | |