| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-20 | |||
| 17:17:47 | opendevreview | Merged openstack/nova master: tests: Remove 'stub_out_trusted_certs' https://review.opendev.org/c/openstack/nova/+/765432 | |
| 17:17:59 | opendevreview | Merged openstack/nova stable/wallaby: Retry lvm volume and volume group query https://review.opendev.org/c/openstack/nova/+/796707 | |
| 17:21:54 | opendevreview | Merged openstack/nova master: Add logs when cannot fit numa https://review.opendev.org/c/openstack/nova/+/798187 | |
| 17:22:03 | opendevreview | Merged openstack/nova master: Improve prep_resize reschedule unit test https://review.opendev.org/c/openstack/nova/+/800297 | |
| 18:58:34 | opendevreview | Merged openstack/nova master: VMWare: Use get_hardware_devices throughout https://review.opendev.org/c/openstack/nova/+/805017 | |
| 19:03:58 | gmann | stephenfin: I replied on hostname change, please check if that make sense (basically we should remove the policy check to show it to non-admin after deprecation phase) - https://review.opendev.org/c/openstack/nova/+/778550/10/nova/api/openstack/compute/views/servers.py#414 | |
| 19:41:39 | gibi | gmann: sure, no problem to amend the spec to reflect the reality | |
| 20:14:11 | NobodyCam | Good Afternoon Nova, Crazy question, is there a way to verify a ironic node is read for scheduling... I am checking ironic node status / placement status, but even if both are available I am still hitting `No valid hosts` | |
| 20:23:34 | gmann | gibi: k. here it is https://review.opendev.org/c/openstack/nova-specs/+/805374 | |
| 20:32:22 | opendevreview | Lance Bragstad proposed openstack/nova master: WIP: Update create server policy for system-admins https://review.opendev.org/c/openstack/nova/+/805456 | |
| 20:35:53 | artom | NobodyCam, your best best is probably to trace the instance create request ID in the scheduler with DEBUG enabled, and see why it failed scheduling | |
| 20:36:31 | artom | NobodyCam, you either got no results from the Placement queries, in which case you'll need to check the Placement log, or one of the filters ended up with 0 hosts, in which case the scheduler logs will tell you | |
| 20:36:35 | NobodyCam | its timing.. | |
| 20:37:05 | NobodyCam | node is deleted but scheduler says no | |
| 20:40:40 | NobodyCam | actual error is `Failed to compute_task_build_instances: No valid host was found. There are not enough hosts available.` | |
| 20:41:08 | NobodyCam | scheduler is not returning the host in filtered hosts | |
| 20:41:25 | NobodyCam | I was looking for a way to check that externally | |
| 20:46:45 | NobodyCam | I should add that we force placement to a particular host in our enviroment | |
| 20:47:04 | NobodyCam | Environment even | |
| 21:29:02 | NobodyCam | artom: FYI it's looking like this check works: openstack hypervisor show <NODEUUID> -c disk_available_least -f value > 0 | |
| 21:56:05 | opendevreview | Merged openstack/nova stable/wallaby: Fix request path to query a resource provider by uuid https://review.opendev.org/c/openstack/nova/+/803070 | |
| 21:56:11 | opendevreview | Merged openstack/nova stable/wallaby: Reproducer unit test for bug 1860312 https://review.opendev.org/c/openstack/nova/+/802840 | |
| 22:03:39 | opendevreview | Merged openstack/nova stable/wallaby: Allow deletion of compute service with no compute nodes https://review.opendev.org/c/openstack/nova/+/802841 | |
| 22:03:46 | opendevreview | Merged openstack/nova stable/wallaby: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/802236 | |
| 22:52:59 | opendevreview | Merged openstack/nova stable/ussuri: Honor [neutron]http_retries in the manual client https://review.opendev.org/c/openstack/nova/+/795176 | |
| 22:57:12 | opendevreview | Merged openstack/nova master: Rename vgpu options to mdev https://review.opendev.org/c/openstack/nova/+/801607 | |
| 22:57:24 | opendevreview | Merged openstack/nova master: Change the admin-visible logs for mdev support https://review.opendev.org/c/openstack/nova/+/803378 | |
| 22:57:43 | opendevreview | Merged openstack/nova stable/wallaby: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797652 | |
| 23:08:38 | opendevreview | Merged openstack/nova stable/wallaby: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797653 | |
| 23:19:18 | opendevreview | Merged openstack/nova master: [ironic] correct capabilities for disk images supported https://review.opendev.org/c/openstack/nova/+/773691 | |
| #openstack-nova - 2021-08-21 | |||
| 02:51:14 | opendevreview | Merged openstack/nova stable/ussuri: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785061 | |
| 02:51:20 | opendevreview | Merged openstack/nova master: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/766043 | |
| 02:51:30 | opendevreview | Merged openstack/nova master: Deprecate filters that have been replaced by placement filters https://review.opendev.org/c/openstack/nova/+/745605 | |
| 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 | |