| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-20 | |||
| 15:25:09 | opendevreview | Merged openstack/nova master: fup: Mock all unix connectors in os-brick fixture https://review.opendev.org/c/openstack/nova/+/804233 | |
| 15:25:39 | lyarwood | \o/ | |
| 16:16:56 | opendevreview | Merged openstack/nova master: VmWare: Fix deprecation warning in unit tests https://review.opendev.org/c/openstack/nova/+/805024 | |
| 16:17:12 | opendevreview | Merged openstack/nova master: docs: Explicitly state lack of support for node renaming https://review.opendev.org/c/openstack/nova/+/795607 | |
| 16:17:28 | opendevreview | Merged openstack/nova master: api: Improve extra spec validator help texts https://review.opendev.org/c/openstack/nova/+/782412 | |
| 16:21:58 | lyarwood | gmann: https://review.opendev.org/c/openstack/devstack/+/805219 - would you mind taking a look at this now that the nova part is in the gate | |
| 16:22:06 | lyarwood | https://review.opendev.org/c/openstack/nova/+/766043 for reference | |
| 16:29:19 | opendevreview | Lee Yarwood proposed openstack/nova master: nova-manage: Introduce volume show, refresh, get_connector commands https://review.opendev.org/c/openstack/nova/+/800634 | |
| 16:36:20 | gmann | lyarwood: sure. checking | |
| 16:45:18 | lyarwood | stephenfin: https://review.opendev.org/c/openstack/nova/+/801607/ - any idea why this isn't +W? | |
| 16:45:25 | lyarwood | and the change above it I think | |
| 17:17:40 | opendevreview | Merged openstack/nova master: scheduler: Merge 'FilterScheduler' into base class https://review.opendev.org/c/openstack/nova/+/773643 | |
| 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 | |