| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-16 | |||
| 10:58:41 | opendevreview | Balazs Gibizer proposed openstack/placement master: Bump os-resource-classes requirements https://review.opendev.org/c/openstack/placement/+/796595 | |
| 11:22:52 | gibi | kashyap: I'm just loking at https://blueprints.launchpad.net/nova/+spec/remove-support-for-floppy-disks which seem to be in a limbo state as it is targeted to xena but it was never set to approved. Do you happen to remember if there was agreemet about that in Xena timeframe? | |
| 11:29:00 | sean-k-mooney | at a minium we proably want to deprecate in xena | |
| 11:29:34 | sean-k-mooney | i dont recalls if we said we would remvoe or not in xena | |
| 12:27:06 | gibi | sean-k-mooney: my problem is that bp state is in limbo so I'm not sure we ever approve this bp | |
| 12:27:16 | gibi | I hope kashyap remembers :D | |
| 13:04:27 | kashyap | gibi: Hi, was away for lunch | |
| 13:06:19 | kashyap | gibi: I only recall a "general agreement" (no active objections on the list - from more than a year ago) | |
| 13:06:40 | kashyap | gibi: Yes, deprecate for sure in Xena | |
| 13:07:10 | gibi | kashyap: do you have intention to working on that bp before Milestone 3 (Sept 3)? | |
| 13:07:16 | kashyap | gibi: Yes | |
| 13:07:47 | kashyap | gibi: Can we remove it in Xena, even? Who uses floppy drives anyway | |
| 13:07:57 | kashyap | I'm expecting a "well, actually" here :D | |
| 13:12:28 | gibi | so there was generic agreement around it so I guees I made a mistake not recording that properly in the bp early in xena | |
| 13:12:43 | gibi | so I will move it to approved now | |
| 13:15:50 | gibi | kashyap: as of the implementation I think we need deprecation first | |
| 13:16:36 | kashyap | gibi: Yeah fair enough | |
| 13:27:18 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova-specs master: Integration With Off-path Network Backends https://review.opendev.org/c/openstack/nova-specs/+/787458 | |
| 14:08:11 | kashyap | gibi: Thanks; I saw the changes | |
| 14:42:07 | dmitriis | gibi, sean-k-mooney: re-proposed https://review.opendev.org/c/openstack/nova-specs/+/787458 for Yoga. There's some progress with Neutron and OVN. | |
| 14:42:07 | dmitriis | Neutron: the RFE https://bugs.launchpad.net/neutron/+bug/1932154 got rfe-approved after the discussion in the drivers meeting https://meetings.opendev.org/irclogs/%23openstack-neutron/%23openstack-neutron.2021-07-09.log.html. We also had a +1 from Brian Haley on the spec https://review.opendev.org/c/openstack/neutron-specs/+/788821 (auto-cleared | |
| 14:42:07 | dmitriis | after a re-upload with nit fixes). | |
| 14:42:07 | dmitriis | OVN: Things moved forward in this thread https://patchwork.ozlabs.org/project/ovn/patch/20210509140305.1910796-1-frode.nordahl@canonical.com/#2699171 and some OVN-side implementation details have changed but nothing Nova needs to care about. We are currently waiting for feedback from Ilya Maximets (hopefully next week) and had some feedback from | |
| 14:42:08 | dmitriis | others already. | |
| 14:44:24 | gibi | dmitriis: thanks for the update. I hope we can make this happen in the next cycle | |
| 16:50:59 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 16:50:59 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Remove psycopg2 warning filter https://review.opendev.org/c/openstack/placement/+/800911 | |
| 16:51:00 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 16:51:00 | opendevreview | Stephen Finucane proposed openstack/placement master: tests: Silence noisy tests https://review.opendev.org/c/openstack/placement/+/801101 | |
| 16:51:01 | opendevreview | Stephen Finucane proposed openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:51:02 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 16:51:03 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 16:51:04 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 16:51:05 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 16:51:05 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 16:51:07 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 17:00:17 | gibi | stephenfin: I know it is Friday afternoon, so it is just for the future... I think the first 15 patches of the pps series is ready for review. This includes all the prerequisit work up until the first successful boot. The patch that actually enables the boot still needs work | |
| 17:00:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump os-resource-classes to 1.1.0 https://review.opendev.org/c/openstack/nova/+/800976 | |
| 17:00: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 | |
| 17:00:40 | stephenfin | excellent | |
| 17:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 17:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 17:00:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect extended_resource_request neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 17:00:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 17:00:45 | stephenfin | I can start at that Monday | |
| 17:00:50 | gibi | stephenfin: thank you | |
| 17:04:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 17:04:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 17:07:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 17:07:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 17:10:25 | opendevreview | Jay Faulkner proposed openstack/nova stable/victoria: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/800873 | |
| 17:11:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 17:11:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 17:11:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 17:16:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | |
| 17:18:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 17:18:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 17:26:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 17:26:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 17:29:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 17:31:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 21:58:06 | opendevreview | melanie witt proposed openstack/nova master: Correct reno for archiving task_log database records https://review.opendev.org/c/openstack/nova/+/801144 | |
| #openstack-nova - 2021-07-17 | |||
| 06:58:50 | opendevreview | Merged openstack/nova stable/stein: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/791402 | |
| #openstack-nova - 2021-07-19 | |||
| 02:22:15 | opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 04:38:56 | opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 08:46:21 | lyarwood | elodilles: gibi: \o morning, btw I'm not away for 2 weeks now. Surgery was unfortunately cancelled at the last minute and hopefully rescheduled for sometime in the next 4 weeks, until then I'm back as normal. | |
| 08:49:11 | gibi | lyarwood: morning. thanks of the information. | |
| 08:55:38 | bauzas | lyarwood: hope you'll eventually get all sorted as quickly as possible | |
| 08:56:01 | gibi | bauzas: +1 | |
| 08:56:26 | lyarwood | thanks both, yeah hopefully I get everything booked in sooner rather than later | |
| 09:34:31 | opendevreview | Stephen Finucane proposed openstack/nova master: nova-manage: Introduce bdm show, refresh, get_connector commands https://review.opendev.org/c/openstack/nova/+/800634 | |
| 09:37:07 | opendevreview | Slawek Kaplonski proposed openstack/nova stable/rocky: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/791420 | |
| 09:52:41 | opendevreview | Artom Lifshitz proposed openstack/nova master: doc: allow_resize_to_same_host needs nova-api restarted https://review.opendev.org/c/openstack/nova/+/801275 | |
| 10:49:33 | elodilles | lyarwood: :S thanks for the heads up. and as bauzas said, too, hope that it will be sorted out soon. :X | |
| 10:58:50 | lyarwood | thanks :) | |
| 11:08:22 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: allow service delete even if compute node record is missing https://review.opendev.org/c/openstack/nova/+/801285 | |
| 11:20:01 | opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | |
| 11:25:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | |
| 11:26:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | |
| 11:42:34 | kevinz | Hi lyarwood: stephenfin: could you help to review the patch https://review.opendev.org/c/openstack/nova/+/763928, to support live migration on arm64 | |
| 11:42:41 | kevinz | Thanks a lot! | |
| 11:44:18 | kashyap | kevinz: Hi, a quick question: have you tested it functionally? (As in doing a real migration w/ your patch) | |
| 11:48:04 | kevinz | kashyap: yes definitely | |
| 11:48:08 | kashyap | kevinz: I'm just taking a look; and yes, host CPU compat check indeed doesn't make sense on AArch64. Thanks for logging it | |
| 11:48:45 | kevinz | kashayap: Thanks, really appreciate | |
| 11:49:47 | kashyap | kevinz: Okay; when you've done such tests, it can help make your case if you also explicitly mention it on the change - something for next time | |
| 11:51:27 | artom | kashyap, kevinz, we have an AArch64 CI, no? Could we not run a multinode job in it to test live migration? | |
| 11:51:47 | kashyap | artom: If we do have such a proper CI, then definitely it helps | |
| 11:52:45 | kevinz | artom: The arm64 CI now is not sufficient stable now so the devstack test and nova test ci is not merged into upstream. FYI: https://review.opendev.org/c/openstack/devstack/+/708317 | |
| 11:53:24 | kevinz | kashyap: ack, will add the info | |
| 11:53:41 | artom | kashyap, aha, ack | |
| 12:03:09 | kashyap | kevinz: Oh, wait - based on that new libvir-list thread, you're selectively adding CPU comparison for AArch64, while in the past we discouraged it, because QEMU AArch64 devs themselves suggested to use 'host-passthrough' for KVM+AArch64 guests | |
| 12:03:19 | kashyap | Okay, I'll read the libvirt thread and respond back. | |
| 12:04:03 | kevinz | kashyap: OK, np, the libvirt patch has a thread discussion for this | |
| 12:04:20 | kashyap | kevinz: I have a couple of questions, I'll ask it there for the record | |
| 13:06:45 | kashyap | kevinz: I wrote some comments; please see if that makes sense to you. And, thanks gibi for Ccing me | |
| 13:17:30 | sean-k-mooney | kashyap: for what its worth i do think we should be adding the cpu comparision for aarch64 | |
| 13:20:23 | sean-k-mooney | kashyap: although i agree with you that linking to the libvirt patch is not very operator friendly | |
| 13:23:11 | kevinz | kashyap: sean-k-mooney: Thank a lot! I will address the comments :-) | |
| 13:49:32 | kashyap | sean-k-mooney: Like I wrote in the review, this discussion gets muddied by the vague term "CPU comparison" | |