| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-14 | |||
| 10:14:54 | opendevreview | Jorhson Deng proposed openstack/nova master: recheck the attachment_id after the reschedule successful https://review.opendev.org/c/openstack/nova/+/796209 | |
| 10:19:41 | lyarwood | gibi: https://listman.redhat.com/archives/libvir-list/2021-June/msg00337.html FWIW | |
| 10:20:30 | gibi | lyarwood: awesome | |
| 10:20:33 | gibi | thanks | |
| 10:20:49 | gibi | I'm just writing the unit test for the nova patch | |
| 10:35:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle OPERATION_FAILED error during detach https://review.opendev.org/c/openstack/nova/+/796255 | |
| 10:41:03 | gibi | lyarwood: ^^ | |
| 10:41:10 | lyarwood | ack looking | |
| 10:45:01 | lyarwood | LGTM | |
| 11:00:16 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796257 | |
| 11:00:17 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796258 | |
| 11:01:01 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796259 | |
| 11:01:02 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796260 | |
| 11:05:26 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796262 | |
| 11:05:27 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: Detach is broken for multi-attached fs-based volumes https://review.opendev.org/c/openstack/nova/+/796263 | |
| 11:05:28 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796264 | |
| 11:09:39 | stephenfin | lyarwood: elodilles_afk: Could you folks stick https://review.opendev.org/c/openstack/os-vif/+/765967 and its relatives on the stable review list for this week? | |
| 11:09:47 | lyarwood | ack | |
| 11:09:51 | stephenfin | ta | |
| 11:10:16 | lyarwood | didn't we say that the os-vif cores were going to end up as stable cores? | |
| 11:10:31 | lyarwood | ah lunch is ready, I'll get to this afterwards brb | |
| 11:10:37 | stephenfin | I thought we did but we haven't made the change yet | |
| 11:10:40 | stephenfin | nw | |
| 11:18:58 | elodilles | stephenfin: sure, will review as soon as I get there :) | |
| 11:39:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 11:39:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 11:39:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 11:39:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 11:40:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect extended_resource_request neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 11:44:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 11:46:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 11:48:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 11:49:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 11:49:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 11:52:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 11:53:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 11:54:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 11:54:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 11:55:34 | gibi | lyarwood: are you working on adding a retry to nova.virt.libvirt.storage.lvm.get_volume_group_info due to https://bugs.launchpad.net/nova/+bug/1931710 ? If not then I can jump on that (I just so that error again in a fresh run)( | |
| 11:56:01 | lyarwood | gibi: I haven't started so if you have the time please go ahead :) | |
| 11:56:08 | gibi | then I go and do it | |
| 11:56:11 | gibi | :) | |
| 12:17:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Retry volume group query https://review.opendev.org/c/openstack/nova/+/796269 | |
| 12:17:11 | gibi | lyarwood: here it is ^^ | |
| 12:37:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Retry lvm volume and volume group query https://review.opendev.org/c/openstack/nova/+/796269 | |
| 12:38:30 | gibi | realized that I saw failures both in case of lvs and vgs queries | |
| 12:40:05 | lyarwood | Yeah indeed sorry the UI didn't refresh in time | |
| 12:40:53 | lyarwood | fun | |
| 12:41:23 | lyarwood | ^ in nova-multi-cell | |
| 12:41:27 | lyarwood | and | |
| 12:41:41 | lyarwood | in tox-py-36 | |
| 12:42:33 | gibi | oh nice | |
| 12:42:48 | gibi | it was release monday out there it seems :D | |
| 12:44:17 | gibi | but what is the conflict in the stevedore case? stevedore >=1.20.0 and stevedore === 3.3.0 are not conflicting | |
| 12:44:27 | gibi | or am I missing something? | |
| 12:44:39 | lyarwood | tbh both are confusing | |
| 12:50:15 | lyarwood | yeah I can't reproduce the numpy websockify one | |
| 13:06:36 | sean-k-mooney | lyarwood: how are you finding the matix bridge? i likke the way image sharing work on the irc side | |
| 13:07:06 | sean-k-mooney | its nicer on the matrix side of cource since its inline but the urls work well for irc users too | |
| 13:07:08 | lyarwood | I've hardly noticed it thus far that's a great thing | |
| 13:07:36 | sean-k-mooney | still using the web interface or have you tried the weechat plugin | |
| 13:08:54 | sean-k-mooney | gibi: lyarwood if you want a third person to test just let me know | |
| 13:09:56 | lyarwood | sean-k-mooney: I'm using element | |
| 13:11:08 | sean-k-mooney | gibi: lyarwood oh wile i rememebr while i rememeber https://review.opendev.org/c/openstack/ovsdbapp/+/795789 seams to work but terry is fixing it properly in ovs. | |
| 13:12:01 | sean-k-mooney | im going to also see if i can find an os-vif only solution that we could backport | |
| 13:12:08 | gibi | cool | |
| 13:12:19 | gibi | let me know if you need a reviewer to let the solution merges | |
| 13:12:53 | sean-k-mooney | ack will do im not going to get to working on it today but its on my todo list for tomorrow | |
| 13:13:22 | sean-k-mooney | im going to try and finish the off path accleartor spec review today and start on the next revision or the pci in placemnet one | |
| 13:13:35 | gibi | ack | |
| 13:14:36 | sean-k-mooney | i spoke to artom about it last week and im going to split the pci one the first spec will just be for non neutron devices and the second will cover the netruon aspect im also goign to file a but for a different semi feature semi bug which ill bring up in either this weeks meetign or next | |
| 13:15:07 | gibi | the split works for me | |
| 13:15:34 | gibi | just try not to do something for the non neutron devices that will be a dead en for the neutron ones | |
| 13:26:58 | sean-k-mooney | ack | |
| 13:41:18 | opendevreview | Merged openstack/nova master: Revert "Removing mypy to fix the nova CI" https://review.opendev.org/c/openstack/nova/+/795973 | |
| 14:34:51 | stephenfin | lyarwood++ Good catch on the libvirt device removal thing | |
| 14:35:08 | stephenfin | and gibi++ too | |
| 14:35:30 | stephenfin | gibi: I've a small nit on https://review.opendev.org/c/openstack/nova/+/796255. Do you want to address now (since CI is rerunning anyway) or in a follow-up? | |
| 17:22:10 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Replace 'nova boot' with 'openstack server create' https://review.opendev.org/c/openstack/nova/+/794007 | |
| 17:40:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle OPERATION_FAILED error during detach https://review.opendev.org/c/openstack/nova/+/796255 | |
| 17:40:47 | gibi | lyarwood, stephenfin: fixed up ^^ | |
| 17:44:56 | opendevreview | Merged openstack/nova master: Add debug log for device detach libvirt error https://review.opendev.org/c/openstack/nova/+/796043 | |
| 20:55:45 | opendevreview | Mohammed Naser proposed openstack/nova master: Allow X-OpenStack-Nova-API-Version header in CORS https://review.opendev.org/c/openstack/nova/+/796319 | |
| 20:57:26 | opendevreview | Mohammed Naser proposed openstack/nova master: Allow X-OpenStack-Nova-API-Version header in CORS https://review.opendev.org/c/openstack/nova/+/796319 | |
| #openstack-nova - 2021-06-15 | |||
| 00:36:40 | opendevreview | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 00:47:05 | opendevreview | Merged openstack/nova master: Test the NotificationFixture https://review.opendev.org/c/openstack/nova/+/758450 | |
| 00:47:45 | opendevreview | Merged openstack/nova master: Move fake_notifier impl under NotificationFixture https://review.opendev.org/c/openstack/nova/+/758451 | |
| 02:33:51 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 02:33:52 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 02:33:54 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 02:33:55 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 02:33:55 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 07:04:49 | gibi | lyarwood, stephenfin: good morning! another pip conflict http://paste.openstack.org/show/806617/ in https://f4c7a70c63adb2c6ee00-366864060e164744cd046650ee1657de.ssl.cf1.rackcdn.com/794639/6/gate/nova-ceph-multistore/cbf9aff/job-output.txt | |
| 07:08:21 | gibi | lyarwood: did you managed to find some hint yesterday about the websockify or the the stevedore case? | |
| 07:09:17 | gibi | this feels pretty random | |
| 07:31:18 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 07:31:18 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 08:04:59 | lyarwood | gibi: no I couldn't work it out, is this another hit? | |
| 08:36:29 | lyarwood | sean-k-mooney: https://bugs.launchpad.net/nova/+bug/1912310/comments/4 btw looks like more evidence of the ovsdbapp polling bug #1929446 the client here fails to send some keepalives to libvirtd on the dest so it kills the migration and domain | |
| 08:43:01 | lyarwood | gibi: https://zuul.opendev.org/t/openstack/build/cbf9affc449848f9910eaab857c5df74/log/job-output.txt#29072 another example | |