| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-18 | |||
| 17:49:51 | stephenfin | yeah, that's it | |
| 17:49:58 | stephenfin | whoops :) | |
| #openstack-nova - 2021-06-19 | |||
| 01:03:59 | opendevreview | Merged openstack/nova master: db: Remove 'nova.db.sqlalchemy.utils' https://review.opendev.org/c/openstack/nova/+/786292 | |
| 01:04:24 | opendevreview | Merged openstack/nova master: db: Reintroduce validation of shadow table schema https://review.opendev.org/c/openstack/nova/+/796814 | |
| 01:04:41 | opendevreview | Merged openstack/nova stable/wallaby: Honor [neutron]http_retries in the manual client https://review.opendev.org/c/openstack/nova/+/794186 | |
| 13:28:32 | opendevreview | Merged openstack/nova stable/wallaby: Stop leaking ceph df cmd in RBD utils https://review.opendev.org/c/openstack/nova/+/791939 | |
| 13:28:50 | opendevreview | Merged openstack/nova master: db: Remove unused DB methods https://review.opendev.org/c/openstack/nova/+/786293 | |
| 18:48:11 | opendevreview | Merged openstack/nova stable/wallaby: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791577 | |
| #openstack-nova - 2021-06-20 | |||
| 15:10:18 | opendevreview | Merged openstack/nova stable/stein: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764308 | |
| #openstack-nova - 2021-06-21 | |||
| 01:46:34 | opendevreview | jiang guodao proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 01:55:06 | swp20 | gmann: hi, our cyborg-tempest-plugin test failed, seems there are no accelerator resources on the zuul server, please help us: https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/796977 | |
| 03:23:56 | opendevreview | jiang guodao proposed openstack/nova stable/victoria: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787252 | |
| 05:58:29 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 05:58:30 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 05:58:31 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 05:58:32 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 05:58:33 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 07:09:51 | opendevreview | Eric Xie proposed openstack/nova master: Fix error '404 Not Found' https://review.opendev.org/c/openstack/nova/+/797233 | |
| 08:32:34 | slaweq | elodilles hi, can You +W patch https://review.opendev.org/c/openstack/nova/+/787252 ? It was rebased recently | |
| 08:34:19 | elodilles | slaweq: +W'd | |
| 08:36:41 | elodilles | I hope it really merges now... the gate is in surprisingly bad shape according to the rechecks on this patch :/ | |
| 08:37:55 | elodilles | despite that a number of gate improvements were merged :X | |
| 08:56:15 | lyarwood | sean-k-mooney / gibi / artom / bauzas ; https://review.opendev.org/c/openstack/nova/+/797142 - can you all take a look at this when you get a chance? Blocks the already in progress backport of https://review.opendev.org/q/I11fb5d3ada7f27b39c183157ea73c8b72b4e672e | |
| 08:56:21 | lyarwood | oh and morning btw :) | |
| 08:56:30 | bauzas | fer sur | |
| 08:56:44 | bauzas | and bonjour | |
| 09:27:22 | slaweq | elodilles thx a lot | |
| 09:37:20 | elodilles | slaweq: np | |
| 10:52:28 | stephenfin | sean-k-mooney: It's working for real this time now https://review.opendev.org/c/openstack/nova/+/797142 | |
| 10:53:32 | sean-k-mooney | stephenfin: yep reviewing it again | |
| 10:54:16 | sean-k-mooney | i did not have a chance to test that since i added the detection mechanium in a real mixed env only with the ci | |
| 10:54:35 | sean-k-mooney | its annoying that this sliped though but im about to +1 that | |
| 10:55:16 | stephenfin | lyarwood: I'd rather avoid addressing those nits on ^, since I've already prepared the backports (with a -1 to prevent anyone merging yet) and don't want to drag more work on myself :-D | |
| 10:55:31 | stephenfin | I could do a follow-up but they seem too minor to bother, for once | |
| 10:56:21 | sean-k-mooney | i dont think the new=... | |
| 10:56:24 | sean-k-mooney | is needed | |
| 10:56:31 | sean-k-mooney | its defenetly nice to prefer to do that | |
| 10:56:38 | sean-k-mooney | if the mock is not required in the test | |
| 10:57:04 | sean-k-mooney | but we didnt do that until about a year or so ago because we did not know we could | |
| 10:57:22 | sean-k-mooney | so its not inconsitnet with thte exsiting test just not the optimal way | |
| 11:05:09 | stephenfin | yeah, my thinking too | |
| 11:05:22 | stephenfin | I'd respin if I hadn't already started on the backports, but ... | |
| 11:05:32 | stephenfin | backports appear to be working as expected too | |
| 11:05:45 | stephenfin | I'll tackle stable/ussuri and stable/train after lunch | |
| 11:06:27 | sean-k-mooney | yeah!!! i really wasnt looking forward to debuging that if it didint | |
| 11:07:20 | sean-k-mooney | /yeah/yay/ | |
| 11:22:26 | sean-k-mooney | by the way http://paste.openstack.org/show/806817/ | |
| 11:22:36 | sean-k-mooney | we have q35 testing right? | |
| 11:23:48 | sean-k-mooney | actully this is more userful http://paste.openstack.org/show/806818/ | |
| 11:24:50 | sean-k-mooney | wait i wonder did i uppercase Q35 instead of q35 | |
| 11:41:57 | sean-k-mooney | lyarwood: stephenfin bauzas https://bugs.launchpad.net/nova/+bug/1933097 this is what i get form deploying openstack at home, more work | |
| 12:24:12 | bauzas | sean-k-mooney: *me* facepalms | |
| 12:28:11 | sean-k-mooney | i never write it with an uppercase but this time i did... | |
| 12:37:07 | opendevreview | Merged openstack/nova master: api: Log correct client IP if load balancer in use https://review.opendev.org/c/openstack/nova/+/786766 | |
| 12:50:13 | sean-k-mooney | bauzas: can you take a look at https://review.opendev.org/q/topic:%22bug%252F1910466%22+(status:open%20OR%20status:merged) | |
| 12:50:30 | bauzas | ack, will look | |
| 12:50:42 | sean-k-mooney | we suspect the customer might start escalating that soonish so just trying to get ahead fo that | |
| 13:09:06 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797291 | |
| 13:46:34 | lyarwood | stephenfin: yeah no issues | |
| 13:46:50 | lyarwood | sean-k-mooney: haha nice, I did think about that when I was doing the nova-manage stuff that does validate that | |
| 14:11:45 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797316 | |
| 14:13:09 | stephenfin | sean-k-mooney: ^ will wait and see how the stable/ussuri change goes (I can't run all tests locally in my env) but that _should_ be it | |
| 14:17:30 | sean-k-mooney | ok good | |
| 14:17:40 | sean-k-mooney | did you hit any issue with python2 compatiablity | |
| 14:17:53 | sean-k-mooney | or other funcitonal test issue | |
| 14:19:22 | opendevreview | Merged openstack/nova master: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/769601 | |
| 14:22:56 | stephenfin | sean-k-mooney: not that I can tell. The modified tests did pass | |
| 14:23:04 | stephenfin | (I could run those) | |
| 14:23:42 | sean-k-mooney | i was ok cool from thje commit it looks like the conflict were not as bad as i was fearing | |
| 14:23:42 | stephenfin | gibi: Are you around today? Care to look at https://review.opendev.org/c/openstack/nova/+/797142 if so? | |
| 14:24:06 | sean-k-mooney | s/i was// | |
| 16:11:44 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797291 | |
| 16:20:11 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797316 | |
| 17:51:36 | sean-k-mooney | stephenfin: https://zuul.opendev.org/t/openstack/build/73f734b9ffe745bd833ffe90f324d6d8/log/controller/logs/screen-n-cpu.txt#7643-7653 | |
| 17:51:55 | sean-k-mooney | stephenfin: its still using <interface type=bridge> | |
| 17:52:22 | sean-k-mooney | stephenfin: it should be <interface type=ethernet> | |
| 17:59:41 | sean-k-mooney | stephenfin: this is how i orginally set teh flag by the way https://review.opendev.org/c/openstack/nova/+/602432/29..30/nova/objects/migrate_data.py | |
| 18:02:05 | sean-k-mooney | i was trying to use udate which updates in place and returns None instead of the updated dict which was a bug | |
| 18:04:42 | sean-k-mooney | this is the only other thing that chaged that jumped out at me https://review.opendev.org/c/openstack/nova/+/602432/29..30/nova/conductor/tasks/live_migrate.py#b403 | |
| 18:04:52 | sean-k-mooney | but i think that is logically the same thing | |
| 18:06:48 | sean-k-mooney | so we still have the same probelm that i was havign with that patch orginally when i put in the code to detect if the dest was patched it always disables the code even after fixing the profile update issue | |
| 18:07:04 | sean-k-mooney | even with https://review.opendev.org/c/openstack/nova/+/797142 | |
| 18:07:53 | sean-k-mooney | i can try and take a look at this with you tomorow if you like but i think we need a functional test that assert the xml has interface type=ethernet | |
| #openstack-nova - 2021-06-22 | |||
| 08:37:04 | zigo | Since Bullseye, live migration is broken on my system, with libvirt not listening on the qemu+tcp port (it's using socket activation). Does anyone know how to configure the new libvirt correctly for this? | |
| 08:58:09 | stephenfin | sean-k-mooney1: you're looking at the wrong XML, I think | |
| 08:59:47 | stephenfin | sean-k-mooney1: You can see the update happening here https://zuul.opendev.org/t/openstack/build/73f734b9ffe745bd833ffe90f324d6d8/log/compute1/logs/screen-n-cpu.txt#7468-7476 | |
| 09:02:09 | bauzas | do someone know whether we use in our jobs the singleconductor devstack value ? https://github.com/openstack/devstack/blob/master/lib/nova#L537 ? | |
| 09:02:42 | bauzas | I was expecting us to create a superconductor by default even for a devstack AIO | |
| 09:09:06 | bauzas | nevermind, found it | |
| 09:09:15 | bauzas | this is just legacy | |
| 09:09:45 | bauzas | I can't see a single job running with a singleconductor | |
| 09:15:34 | stephenfin | bauzas: maybe remove it so? | |
| 09:15:41 | stephenfin | *we should | |
| 09:15:47 | bauzas | yup, think so | |
| 10:07:34 | sean-k-mooney1 | stephenfin the xml should always be ethernet | |
| 10:08:40 | sean-k-mooney1 | stephenfin: it is useing ethernet in that segment which is good | |
| 10:08:54 | sean-k-mooney1 | but it should spawn with it too | |
| 10:09:51 | sean-k-mooney1 | stephenfin: ok so we need to determin why its only taking effect on migration and not on inital spawn | |
| 10:10:12 | sean-k-mooney1 | that narrows it down at least | |
| 10:14:54 | stephenfin | sean-k-mooney: We never set the 'delegate_create' attribute of the VIF object anywhere except as part of a migration | |