| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-21 | |||
| 07:29:57 | opendevreview | Merged openstack/nova-specs master: Update attachment_id spec to include implementation changes https://review.opendev.org/c/openstack/nova-specs/+/805374 | |
| 08:02:59 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | |
| 08:21:01 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add missing [oslo_reports] options https://review.opendev.org/c/openstack/nova/+/804568 | |
| 12:33:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port resource request tests https://review.opendev.org/c/openstack/nova/+/801815 | |
| 12:33:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 12:33:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 12:33:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 12:33:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect port-resource-request-groups neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 12:33:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 12:33:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 12:36:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 12:43:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 12:43:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 12:43:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 12:45:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 12:47:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 12:47:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | |
| 12:47:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 12:52:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 12:54:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 12:55:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 12:56:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 12:58:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 12:59:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| 12:59:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reno for qos-minimum-guaranteed-packet-rate https://review.opendev.org/c/openstack/nova/+/805046 | |
| #openstack-nova - 2021-08-22 | |||
| 10:16:01 | opendevreview | Merged openstack/nova master: Remove the redundant code for attach interface https://review.opendev.org/c/openstack/nova/+/796208 | |
| 10:23:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 10:23:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 10:23:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 10:23:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 10:23:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 10:28:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 10:31:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| 10:32:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reno for qos-minimum-guaranteed-packet-rate https://review.opendev.org/c/openstack/nova/+/805046 | |
| #openstack-nova - 2021-08-23 | |||
| 02:38:27 | opendevreview | Brin Zhang proposed openstack/nova master: Refactor check and exception https://review.opendev.org/c/openstack/nova/+/749052 | |
| 07:26:22 | opendevreview | Merged openstack/nova master: fup: Move _migrate_stub into LibvirtMigrationMixin https://review.opendev.org/c/openstack/nova/+/804232 | |
| 08:39:28 | gibi | stephenfin: hi! I had to rebase the pps series as cyborg smartnic support landed in the meantime causing conflicts. If you have time then re-review would be appreciated | |
| 08:39:59 | gibi | lyarwood: I know that it is a big thing to ask but I would need a second core on the pps series. I'm happy to trade reviews on anything | |
| 09:01:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Avoid unbound instance_uuid var during delete https://review.opendev.org/c/openstack/nova/+/805605 | |
| 10:11:11 | qiujunting_ | sean-k-mooney | |
| 10:15:23 | qiujunting_ | sean:https://review.opendev.org/c/openstack/nova/+/802225/1/nova/compute/manager.py#5615 this should be done in prep_resize, but in the spec, Copy the PMEM data from old PMEM device to new PMEM device at migtate_disk_and_power_off. https://review.opendev.org/c/openstack/nova-specs/+/785563/14/specs/xena/approved/allow-migrate-pmem-data.rst#65 | |
| 11:03:40 | lyarwood | gibi: of course, sorry just back online, got held up coming back from a weekend away because of $baby. | |
| 11:06:24 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Drop references to non-filter scheduler drivers https://review.opendev.org/c/openstack/nova/+/773645 | |
| 11:06:24 | opendevreview | Stephen Finucane proposed openstack/nova master: scheduler: Merge driver into manager https://review.opendev.org/c/openstack/nova/+/773644 | |
| 11:06:25 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Merge 'test_utils', 'test_scheduler_utils' https://review.opendev.org/c/openstack/nova/+/773646 | |
| 11:06:25 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated aliases https://review.opendev.org/c/openstack/nova/+/773647 | |
| 11:06:34 | gibi | lyarwood: hi. no worries and thanks | |
| 11:39:42 | opendevreview | sean mooney proposed openstack/os-vif master: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/805223 | |
| 12:25:53 | opendevreview | sean mooney proposed openstack/os-vif master: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/805625 | |
| 12:37:13 | opendevreview | Lee Yarwood proposed openstack/nova master: nova-manage: Introduce volume show, refresh, get_connector commands https://review.opendev.org/c/openstack/nova/+/800634 | |
| 12:54:20 | opendevreview | Lee Yarwood proposed openstack/nova master: api: Introduce microversion 2.89 adjusting os-volume_attachments https://review.opendev.org/c/openstack/nova/+/804275 | |
| 12:55:01 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805628 | |
| 12:55:27 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805629 | |
| 12:55:43 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805630 | |
| 13:01:00 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805631 | |
| 13:22:29 | sean-k-mooney | gibi: can you take a look at https://review.opendev.org/c/openstack/project-config/+/787523/ for the review priorty lable | |
| 13:23:03 | sean-k-mooney | i think that should implement it correctly now if you want any changes let me know | |
| 13:23:52 | sean-k-mooney | sorry to have put that off for so long but it would be nice to have it working before we hit FF so we can use it for FF and RC issues | |
| 13:29:44 | gibi | sean-k-mooney: thanks for working on that I left feedback | |
| 13:30:54 | gibi | I agree that we should try to use that | |
| 13:31:00 | gibi | for the incoming FF | |
| 13:31:40 | sean-k-mooney | ack you suggestions make sense let me update it quickly | |
| 13:32:00 | gibi | ok | |
| 13:35:09 | sean-k-mooney | ok that should do it https://review.opendev.org/c/openstack/project-config/+/787523 | |
| 13:38:11 | opendevreview | Vadym Markov proposed openstack/nova master: Fix for concurrent DB update issue https://review.opendev.org/c/openstack/nova/+/804985 | |
| 14:02:59 | opendevreview | Lee Yarwood proposed openstack/nova master: workarounds: Remove disable_native_luksv1 https://review.opendev.org/c/openstack/nova/+/805647 | |
| 14:03:00 | opendevreview | Lee Yarwood proposed openstack/nova master: workarounds: Remove rbd_volume_local_attach https://review.opendev.org/c/openstack/nova/+/805648 | |
| 14:04:42 | fungi | melwitt: gibi: someone commented on https://launchpad.net/bugs/1927677 (OSSA-2021-002, CVE-2021-3654) that the fix is incomplete... any chance you have time to take a closer look and reopen the bug if so? | |
| 14:07:28 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 14:08:44 | gibi | fungi: ack | |
| 14:09:22 | gibi | fungi, melwitt: I can look at it tomorrow so melwitt if you have time still today then feel free to check it out | |
| 14:14:01 | fungi | i don't know that it's urgent, just didn't want it to slip through the cracks as it's comments on a closed bug | |
| 14:14:12 | fungi | but thanks! | |
| 14:15:07 | gibi | fungi: thanks for the headsup it would not notice it otherwise | |
| 14:16:20 | fungi | no worries, it's exactly why i continue to monitor comments on presumed fixed vulnerability reports | |
| 14:21:48 | sean-k-mooney | so they are reporting it does not work with 3 slashes | |
| 14:22:03 | sean-k-mooney | but with 2 or more then 4 | |
| 14:23:03 | sean-k-mooney | https://launchpadlibrarian.net/554904339/python3.patch is more general | |
| 14:23:47 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 14:24:07 | fungi | sean-k-mooney: yep, hoping someone can test and confirm the assertion that the vulnerability is still exploitable with odd numbers of slashes | |
| 14:24:27 | sean-k-mooney | let me see if we have a func test for this | |
| 14:24:36 | sean-k-mooney | if we do i can try it with 3 slasshes | |
| 14:24:48 | fungi | thanks a bunch | |
| 14:24:53 | fungi | that's a huge help | |
| 14:26:44 | sean-k-mooney | we dont seam to have a func test but we do have a unit test https://review.opendev.org/c/openstack/nova/+/803091 ill see if i can modify that quickly | |
| 14:29:43 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805628 | |
| 14:30:29 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805629 | |
| 14:30:57 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805630 | |
| 14:31:41 | sean-k-mooney | testtools.matchers._impl.MismatchError: 'Error code: 400' not in 'HTTP/1.1 301 Moved Permanently\r\nServer: WebSockify Python/3.8.10\r\nDate: Mon, 23 Aug 2021 14:31:26 GMT\r\nLocation: /example.com/%2F../\r\n\r\n' | |
| 14:31:55 | sean-k-mooney | fungi: ^ so ya that seams to confirm that /// is not blocked | |
| 14:32:10 | sean-k-mooney | ill quickly apply there patch and see if that correct it | |
| 14:33:37 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805631 | |
| 14:34:04 | sean-k-mooney | gibi: fungi there patch seams to work for both cases. shall i submit it to gerrit | |
| 14:34:42 | gibi | sean-k-mooney: yes please | |
| 14:34:48 | gibi | or wait | |
| 14:35:07 | gibi | do we need any special treatment due to this being a secuirty bug? | |
| 14:35:20 | gibi | I guess not but fungi might knows better | |
| 14:35:27 | sean-k-mooney | well its public and the report and patch were submitted in the clear | |
| 14:35:45 | sean-k-mooney | so i think at this point we treat it as a public security bug | |
| 14:36:08 | sean-k-mooney | if the report was made privatly then yes | |