| opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | 02:22 |
|---|---|---|
| -opendevstatus- NOTICE: The maintenance of the review.opendev.org Gerrit service is now complete and service has been restored. Please alert us in #opendev if you have any issues. Thank you | 03:28 | |
| opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | 04:38 |
| *** dpawlik0 is now known as dpawlik | 05:10 | |
| *** mgoddard- is now known as mgoddard | 06:04 | |
| *** dpawlik5 is now known as dpawlik | 06:50 | |
| *** dpawlik7 is now known as dpawlik | 07:06 | |
| *** akekane_ is now known as abhishekk | 07:07 | |
| *** dpawlik3 is now known as dpawlik | 08:30 | |
| *** dpawlik4 is now known as dpawlik | 08:44 | |
| 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:46 |
| gibi | lyarwood: morning. thanks of the information. | 08:49 |
| bauzas | lyarwood: hope you'll eventually get all sorted as quickly as possible | 08:55 |
| gibi | bauzas: +1 | 08:56 |
| lyarwood | thanks both, yeah hopefully I get everything booked in sooner rather than later | 08:56 |
| *** rpittau|afk is now known as rpittau | 09: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:34 |
| 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:37 |
| *** kopecmartin is now known as kopecmartin|pto | 09:45 | |
| 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 | 09:52 |
| elodilles | lyarwood: :S thanks for the heads up. and as bauzas said, too, hope that it will be sorted out soon. :X | 10:49 |
| lyarwood | thanks :) | 10:58 |
| 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:08 |
| 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:20 |
| 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 |
| 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 |
| 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 |
| kevinz | Thanks a lot! | 11:42 |
| kashyap | kevinz: Hi, a quick question: have you tested it functionally? (As in doing a real migration w/ your patch) | 11:44 |
| kevinz | kashyap: yes definitely | 11:48 |
| 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 |
| kevinz | kashayap: Thanks, really appreciate | 11:48 |
| 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:49 |
| artom | kashyap, kevinz, we have an AArch64 CI, no? Could we not run a multinode job in it to test live migration? | 11:51 |
| kashyap | artom: If we do have such a proper CI, then definitely it helps | 11:51 |
| 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:52 |
| kevinz | kashyap: ack, will add the info | 11:53 |
| artom | kashyap, aha, ack | 11:53 |
| 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 |
| kashyap | Okay, I'll read the libvirt thread and respond back. | 12:03 |
| kevinz | kashyap: OK, np, the libvirt patch has a thread discussion for this | 12:04 |
| kashyap | kevinz: I have a couple of questions, I'll ask it there for the record | 12:04 |
| kashyap | kevinz: I wrote some comments; please see if that makes sense to you. And, thanks gibi for Ccing me | 13:06 |
| sean-k-mooney | kashyap: for what its worth i do think we should be adding the cpu comparision for aarch64 | 13:17 |
| sean-k-mooney | kashyap: although i agree with you that linking to the libvirt patch is not very operator friendly | 13:20 |
| kevinz | kashyap: sean-k-mooney: Thank a lot! I will address the comments :-) | 13:23 |
| kashyap | sean-k-mooney: Like I wrote in the review, this discussion gets muddied by the vague term "CPU comparison" | 13:49 |
| kashyap | There's two things: | 13:49 |
| kashyap | (1) comparing guest CPU to host CPU -- the "traditional" use case of libvirt CPU APIs | 13:49 |
| kashyap | (2) comparing source host CPU to dest host CPU -- the only thing supported for Arm/AArch64 by virtue of this new patch | 13:50 |
| kashyap | Here, for AArch64/Arm, we're talking about the 2nd case, host to host CPU comparison -- that's a reasonable and valid use case, although remains to be seen how effective it is for Arm/AArch64 | 13:51 |
| kashyap | In summary, for AArch64, the source host CPU to dest host CPU comparison is useful only to ensure your guest don't migrate a completely different host. | 13:52 |
| kashyap | kevinz: --^ | 13:52 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | 13:53 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | 13:53 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | 13:53 |
| kashyap | kevinz: I was just thinking on a walk -- I wonder how much do you gain even with host-to-host CPU comparison. For Arm/AArch64, if you care about live migration: you're better off ensuring you have the identical host CPUs and all related virt software in your Arm/AArch64 pool + host-passthrough | 13:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | 13:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | 13:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | 13:54 |
| kashyap | sean-k-mooney: In my two points above; point (1) -- comparing guest CPU to host CPu -- doesn't make sense for reasons noted before and in the review. If you're talking about point (2), then we're in agreement :-) | 13:56 |
| kashyap | kevinz: (And yes, I've read the upstream libvirt thread. Thx for the link) | 13:56 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | 13:58 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 13:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | 14:03 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 14:04 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | 14:04 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | 14:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | 14:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | 14:08 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 14:10 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | 14:16 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | 16:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | 16:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | 16:25 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | 16:26 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | 16:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | 16:29 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 16:30 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | 16:31 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 16:32 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | 16:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | 16:35 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | 16:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | 16:38 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 16:39 |
| opendevreview | Elod Illes proposed openstack/nova stable/stein: Make _rebase_with_qemu_img() generic https://review.opendev.org/c/openstack/nova/+/780784 | 16:40 |
| opendevreview | Elod Illes proposed openstack/nova stable/stein: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757085 | 16:41 |
| *** rpittau is now known as rpittau|afk | 17:38 | |
| opendevreview | Lee Yarwood proposed openstack/nova master: DNM - Test tempest-integrated-compute-centos-8-stream https://review.opendev.org/c/openstack/nova/+/799996 | 18:32 |
| opendevreview | Samuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects https://review.opendev.org/c/openstack/nova/+/801377 | 18:57 |
| opendevreview | Samuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects https://review.opendev.org/c/openstack/nova/+/801377 | 19:13 |
| opendevreview | Samuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects https://review.opendev.org/c/openstack/nova/+/801377 | 19:16 |
| NobodyCam | good afternoon Nova folks, off the wall question, I have a record in the (cell0) instance record that has been orphaned but is causing a phantom instance to appear in server list. my question is what is the best way to clear the phantom instance, update: "updated_at,deleted_at, deleted, vm_state, to indicate deleted or to nuke (delete) the record | 21:18 |
| NobodyCam | s/instance record/instance table/ | 21:18 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!