Monday, 2021-07-19

opendevreviewchengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs  https://review.opendev.org/c/openstack/nova/+/76233002: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 you03:28
opendevreviewchengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs  https://review.opendev.org/c/openstack/nova/+/76233004:38
*** dpawlik0 is now known as dpawlik05:10
*** mgoddard- is now known as mgoddard06:04
*** dpawlik5 is now known as dpawlik06:50
*** dpawlik7 is now known as dpawlik07:06
*** akekane_ is now known as abhishekk07:07
*** dpawlik3 is now known as dpawlik08:30
*** dpawlik4 is now known as dpawlik08:44
lyarwoodelodilles: 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
gibilyarwood: morning. thanks of the information.08:49
bauzaslyarwood: hope you'll eventually get all sorted as quickly as possible08:55
gibibauzas:  +108:56
lyarwoodthanks both, yeah hopefully I get everything booked in sooner rather than later08:56
*** rpittau|afk is now known as rpittau09:31
opendevreviewStephen Finucane proposed openstack/nova master: nova-manage: Introduce bdm show, refresh, get_connector commands  https://review.opendev.org/c/openstack/nova/+/80063409:34
opendevreviewSlawek Kaplonski proposed openstack/nova stable/rocky: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/79142009:37
*** kopecmartin is now known as kopecmartin|pto09:45
opendevreviewArtom Lifshitz proposed openstack/nova master: doc: allow_resize_to_same_host needs nova-api restarted  https://review.opendev.org/c/openstack/nova/+/80127509:52
elodilleslyarwood: :S thanks for the heads up. and as bauzas said, too, hope that it will be sorted out soon. :X10:49
lyarwoodthanks :)10:58
opendevreviewArtom Lifshitz proposed openstack/nova master: WIP: allow service delete even if compute node record is missing  https://review.opendev.org/c/openstack/nova/+/80128511:08
opendevreviewTakashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service  https://review.opendev.org/c/openstack/nova/+/77869611:20
opendevreviewTakashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service  https://review.opendev.org/c/openstack/nova/+/77869611:25
opendevreviewTakashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service  https://review.opendev.org/c/openstack/nova/+/77869611:26
kevinzHi lyarwood: stephenfin: could you help to review the patch https://review.opendev.org/c/openstack/nova/+/763928, to support live migration on arm6411:42
kevinzThanks a lot!11:42
kashyapkevinz: Hi, a quick question: have you tested it functionally?  (As in doing a real migration w/ your patch)11:44
kevinzkashyap: yes definitely11:48
kashyapkevinz: I'm just taking a look; and yes, host CPU compat check indeed doesn't make sense on AArch64.  Thanks for logging it11:48
kevinzkashayap: Thanks, really appreciate11:48
kashyapkevinz: 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 time11:49
artomkashyap, kevinz, we have an AArch64 CI, no? Could we not run a multinode job in it to test live migration?11:51
kashyapartom: If we do have such a proper CI, then definitely it helps11:51
kevinzartom: 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/+/70831711:52
kevinzkashyap: ack, will add the info11:53
artomkashyap, aha, ack11:53
kashyapkevinz: 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 guests12:03
kashyapOkay, I'll read the libvirt thread  and respond back.12:03
kevinzkashyap: OK, np, the libvirt patch has a thread discussion for this12:04
kashyapkevinz: I have a couple of questions, I'll ask it there for the record12:04
kashyapkevinz: I wrote some comments; please see if that makes sense to you.  And, thanks gibi for Ccing me13:06
sean-k-mooneykashyap: for what its worth i do think we should be adding the cpu comparision for aarch6413:17
sean-k-mooneykashyap: although i agree with you that linking to the libvirt patch is not very operator friendly13:20
kevinzkashyap: sean-k-mooney: Thank a lot! I will address the comments :-)13:23
kashyapsean-k-mooney: Like I wrote in the review, this discussion gets muddied by the vague term "CPU comparison"13:49
kashyapThere's two things:13:49
kashyap(1) comparing guest CPU to host CPU -- the "traditional" use case of libvirt CPU APIs13:49
kashyap(2) comparing source host CPU to dest host CPU -- the only thing supported for Arm/AArch64 by virtue of this new patch13:50
kashyapHere, 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/AArch6413:51
kashyapIn 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
kashyapkevinz: --^13:52
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject server create with extended resource req  https://review.opendev.org/c/openstack/nova/+/79361913:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req  https://review.opendev.org/c/openstack/nova/+/79362013:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams  https://review.opendev.org/c/openstack/nova/+/79150313:53
kashyapkevinz: 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-passthrough13:54
opendevreviewBalazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36  https://review.opendev.org/c/openstack/nova/+/79150413:54
opendevreviewBalazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query  https://review.opendev.org/c/openstack/nova/+/79150513:54
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation  https://review.opendev.org/c/openstack/nova/+/79245813:54
kashyapsean-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
kashyapkevinz: (And yes, I've read the upstream libvirt thread.  Thx for the link)13:56
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests  https://review.opendev.org/c/openstack/nova/+/79893013:58
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources  https://review.opendev.org/c/openstack/nova/+/79239413:59
opendevreviewBalazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data  https://review.opendev.org/c/openstack/nova/+/80008514:03
opendevreviewBalazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling  https://review.opendev.org/c/openstack/nova/+/79150614:04
opendevreviewBalazs Gibizer proposed openstack/nova master: Support boot with extended resource request  https://review.opendev.org/c/openstack/nova/+/80008614:04
opendevreviewBalazs Gibizer proposed openstack/nova master: Support move ops with extended resource request  https://review.opendev.org/c/openstack/nova/+/80008714:07
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos  https://review.opendev.org/c/openstack/nova/+/80008814:07
opendevreviewBalazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format  https://review.opendev.org/c/openstack/nova/+/80008914:08
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place  https://review.opendev.org/c/openstack/nova/+/79362114:10
opendevreviewTakashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service  https://review.opendev.org/c/openstack/nova/+/77869614:16
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req  https://review.opendev.org/c/openstack/nova/+/79362016:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams  https://review.opendev.org/c/openstack/nova/+/79150316:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36  https://review.opendev.org/c/openstack/nova/+/79150416:25
opendevreviewBalazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query  https://review.opendev.org/c/openstack/nova/+/79150516:26
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation  https://review.opendev.org/c/openstack/nova/+/79245816:27
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests  https://review.opendev.org/c/openstack/nova/+/79893016:29
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources  https://review.opendev.org/c/openstack/nova/+/79239416:30
opendevreviewBalazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data  https://review.opendev.org/c/openstack/nova/+/80008516:31
opendevreviewBalazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling  https://review.opendev.org/c/openstack/nova/+/79150616:32
opendevreviewBalazs Gibizer proposed openstack/nova master: Support boot with extended resource request  https://review.opendev.org/c/openstack/nova/+/80008616:34
opendevreviewBalazs Gibizer proposed openstack/nova master: Support move ops with extended resource request  https://review.opendev.org/c/openstack/nova/+/80008716:35
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos  https://review.opendev.org/c/openstack/nova/+/80008816:36
opendevreviewBalazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format  https://review.opendev.org/c/openstack/nova/+/80008916:38
opendevreviewBalazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place  https://review.opendev.org/c/openstack/nova/+/79362116:39
opendevreviewElod Illes proposed openstack/nova stable/stein: Make _rebase_with_qemu_img() generic  https://review.opendev.org/c/openstack/nova/+/78078416:40
opendevreviewElod Illes proposed openstack/nova stable/stein: Use absolute path during qemu img rebase  https://review.opendev.org/c/openstack/nova/+/75708516:41
*** rpittau is now known as rpittau|afk17:38
opendevreviewLee Yarwood proposed openstack/nova master: DNM - Test tempest-integrated-compute-centos-8-stream  https://review.opendev.org/c/openstack/nova/+/79999618:32
opendevreviewSamuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects  https://review.opendev.org/c/openstack/nova/+/80137718:57
opendevreviewSamuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects  https://review.opendev.org/c/openstack/nova/+/80137719:13
opendevreviewSamuel proposed openstack/nova master: Add REST and compute APIs to migrate instances between projects  https://review.opendev.org/c/openstack/nova/+/80137719:16
NobodyCamgood 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 record21:18
NobodyCams/instance record/instance table/21:18

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!