Friday, 2021-05-14

« Thursday, 2021-05-13 Index Saturday, 2021-05-15 »
*** sapd1_x has quit IRC00:48
*** k_mouza has joined #openstack-nova00:51
*** k_mouza has quit IRC00:56
*** gyee has quit IRC01:21
*** LinPeiWen77 has joined #openstack-nova01:21
*** k_mouza has joined #openstack-nova01:42
*** k_mouza has quit IRC01:46
*** amodi has quit IRC01:53
openstackgerritmelanie witt proposed openstack/placement master: Microversion 1.37: API support for consumer types  https://review.opendev.org/c/openstack/placement/+/67944103:04
openstackgerritmelanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache  https://review.opendev.org/c/openstack/placement/+/67948603:04
*** rcernin has quit IRC03:04
openstackgerritQiu Fossen proposed openstack/nova-specs master: Allow migrating PMEM's data  https://review.opendev.org/c/openstack/nova-specs/+/78556303:09
*** rcernin has joined #openstack-nova03:22
*** macz_ has joined #openstack-nova03:32
*** psachin has joined #openstack-nova03:33
*** macz_ has quit IRC03:37
*** martinkennelly has quit IRC03:41
*** brinzhang0 has quit IRC03:46
*** hemanth_n has joined #openstack-nova03:48
*** benj_ has quit IRC04:01
*** benj_ has joined #openstack-nova04:03
*** hemanth_n has quit IRC04:08
*** brinzhang0 has joined #openstack-nova04:25
openstackgerritmelanie witt proposed openstack/placement master: Microversion 1.37: API support for consumer types  https://review.opendev.org/c/openstack/placement/+/67944104:43
openstackgerritmelanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache  https://review.opendev.org/c/openstack/placement/+/67948604:43
*** tamas_erdei has joined #openstack-nova05:00
*** terdei has quit IRC05:02
*** ratailor has joined #openstack-nova05:04
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Note env OS_PLACEMENT_API_VERSION support  https://review.opendev.org/c/openstack/osc-placement/+/79135605:04
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Note env OS_PLACEMENT_API_VERSION support  https://review.opendev.org/c/openstack/osc-placement/+/79135605:05
*** avolkov has joined #openstack-nova05:06
*** macz_ has joined #openstack-nova05:33
*** macz_ has quit IRC05:38
*** swp20 has joined #openstack-nova05:46
*** rcernin has quit IRC05:55
*** rcernin has joined #openstack-nova05:57
*** lpetrut has joined #openstack-nova06:01
openstackgerritchengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs  https://review.opendev.org/c/openstack/nova/+/76233006:05
*** slaweq has joined #openstack-nova06:06
openstackgerritYongli He proposed openstack/nova master: Smartnic support - cyborg drive  https://review.opendev.org/c/openstack/nova/+/77136206:21
openstackgerritYongli He proposed openstack/nova master: smartnic support - new vnic type  https://review.opendev.org/c/openstack/nova/+/77136306:21
openstackgerritYongli He proposed openstack/nova master: smartnic support  https://review.opendev.org/c/openstack/nova/+/75894406:22
openstackgerritYongli He proposed openstack/nova master: smartnic support - reject server move and suspend  https://review.opendev.org/c/openstack/nova/+/77991306:22
openstackgerritYongli He proposed openstack/nova master: smartnic support - functional tests  https://review.opendev.org/c/openstack/nova/+/78014706:22
*** brinzhang_ has joined #openstack-nova06:31
*** whoami-rajat_ has joined #openstack-nova06:32
*** swp2081 has joined #openstack-nova06:32
*** swp20 has quit IRC06:33
*** brinzhang0 has quit IRC06:34
*** dklyle has quit IRC06:52
openstackgerritSlawek Kaplonski proposed openstack/nova stable/victoria: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/78725206:58
yonglihegibi,  please remove  that '-2', thanks, https://review.opendev.org/c/openstack/nova/+/77136206:58
gibiyonglihe: thanks for the headsup, I reset my vote06:59
openstackgerritSlawek Kaplonski proposed openstack/nova stable/ussuri: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/78725307:01
openstackgerritSlawek Kaplonski proposed openstack/nova stable/train: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/78731607:02
*** andrewbonney has joined #openstack-nova07:13
*** macz_ has joined #openstack-nova07:34
*** macz_ has quit IRC07:38
*** k_mouza has joined #openstack-nova07:43
*** tosky has joined #openstack-nova07:47
*** k_mouza has quit IRC07:47
*** rcernin has quit IRC07:56
*** rcernin has joined #openstack-nova07:56
*** lucasagomes has joined #openstack-nova08:00
fricklerslaweq: IIUC ^^ should also be applicable to stein, since that's where the req'd changes made it into neutron, right? not sure if nova still updates that branch, but I'd take a look at doing that locally, then08:01
slaweqfrickler: ok, I will check it08:02
*** yoctozepto6 is now known as yoctozepto08:03
openstackgerritSlawek Kaplonski proposed openstack/nova stable/stein: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/79140208:05
*** swp2081 has quit IRC08:05
*** swp20 has joined #openstack-nova08:07
*** derekh has joined #openstack-nova08:10
*** rcernin has quit IRC08:32
*** mgoddard has quit IRC08:43
*** mgoddard has joined #openstack-nova08:43
openstackgerritSlawek Kaplonski proposed openstack/nova stable/rocky: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/79142008:59
openstackgerritSlawek Kaplonski proposed openstack/nova stable/queens: [neutron] Get only ID and name of the SGs from Neutron  https://review.opendev.org/c/openstack/nova/+/79142109:00
*** macz_ has joined #openstack-nova09:35
*** macz_ has quit IRC09:40
*** ociuhandu has joined #openstack-nova09:45
*** links has joined #openstack-nova09:54
*** k_mouza has joined #openstack-nova10:10
*** ociuhandu has quit IRC10:15
*** ociuhandu has joined #openstack-nova10:17
*** ociuhandu has quit IRC10:22
*** ociuhandu has joined #openstack-nova10:32
*** swp20 has quit IRC10:42
*** martinkennelly has joined #openstack-nova10:49
*** k_mouza has quit IRC11:33
sean-k-mooneystephenfin: https://review.opendev.org/c/openstack/nova/+/791135/1/nova/compute/manager.py#8616 you told me not to use save_and_reraise_exception() anymore at one point right?11:35
sean-k-mooneyor am i imaginging that11:35
*** macz_ has joined #openstack-nova11:36
stephenfinyeah, it's generally not necessary11:37
stephenfinthe only reason to use it is if you're doing some complicated things that might raise their own exception11:38
stephenfinbut even then there's a way to make sure you raise the correct thing11:38
stephenfinI don't recall off the top of my head though11:39
*** k_mouza has joined #openstack-nova11:39
*** ratailor has quit IRC11:39
*** macz_ has quit IRC11:40
sean-k-mooneyok the instance.save im doing could raise so i guess i should use it sicne i dont really feel like wrapign that in a try except11:42
sean-k-mooneyi have generally been avoiding using it thoug when doing simple things like logging11:43
*** k_mouza has quit IRC11:44
*** whoami-rajat_ is now known as whoami-rajat11:51
*** swp20 has joined #openstack-nova11:56
openstackgerritLee Yarwood proposed openstack/nova master: Add regression test for bug #1928063  https://review.opendev.org/c/openstack/nova/+/79065811:58
openstackbug 1928063 in OpenStack Compute (nova) "SEV enabled instance unable to hard reboot" [Undecided,In progress] https://launchpad.net/bugs/1928063 - Assigned to Lee Yarwood (lyarwood)11:58
openstackgerritLee Yarwood proposed openstack/nova master: image_meta: Provide image_ref as the id when fetching from instance  https://review.opendev.org/c/openstack/nova/+/79065911:58
openstackgerritLee Yarwood proposed openstack/nova master: hardware: Use image_meta.id within get_mem_encryption_constraint  https://review.opendev.org/c/openstack/nova/+/79066011:58
*** k_mouza has joined #openstack-nova12:02
*** mlavalle has joined #openstack-nova12:08
*** jawad_axd has joined #openstack-nova12:12
*** brinzhang0 has joined #openstack-nova12:13
*** brinzhang_ has quit IRC12:16
*** mgariepy has quit IRC12:21
*** mgariepy has joined #openstack-nova12:35
*** lpetrut has quit IRC12:36
*** jawad_axd has quit IRC12:36
*** ociuhandu has quit IRC12:36
*** jawad_axd has joined #openstack-nova12:36
*** ociuhandu has joined #openstack-nova12:41
*** raildo has joined #openstack-nova12:44
openstackgerritMerged openstack/nova master: mypy: Add type annotations to 'nova.pci'  https://review.opendev.org/c/openstack/nova/+/67620912:48
*** lpetrut has joined #openstack-nova12:48
*** ociuhandu has quit IRC13:01
*** ociuhandu has joined #openstack-nova13:02
*** brinzhang_ has joined #openstack-nova13:06
*** ociuhandu has quit IRC13:07
*** brinzhang0 has quit IRC13:09
openstackgerritLee Yarwood proposed openstack/nova stable/wallaby: Add regression test for bug #1922053  https://review.opendev.org/c/openstack/nova/+/79147713:16
openstackbug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,Fix released] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood)13:16
openstackgerritLee Yarwood proposed openstack/nova stable/wallaby: api: Reject requests to force up computes when `done` evacuation records exist  https://review.opendev.org/c/openstack/nova/+/79147813:16
*** brinzhang0 has joined #openstack-nova13:18
*** psachin has quit IRC13:20
*** brinzhang_ has quit IRC13:21
*** swp20 has quit IRC13:23
*** efried has quit IRC13:25
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: Remove self.server dependency form _evacuate_server  https://review.opendev.org/c/openstack/nova/+/79148013:31
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: Use _evacuate_server helper in func test  https://review.opendev.org/c/openstack/nova/+/79148113:31
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1922053  https://review.opendev.org/c/openstack/nova/+/79148213:31
openstackbug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,Fix released] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood)13:31
openstackgerritLee Yarwood proposed openstack/nova stable/victoria: api: Reject requests to force up computes when `done` evacuation records exist  https://review.opendev.org/c/openstack/nova/+/79148313:31
*** efried has joined #openstack-nova13:32
*** ociuhandu has joined #openstack-nova13:32
*** hoonetorg has quit IRC13:32
*** ociuhandu has quit IRC13:32
*** andrewbogott has left #openstack-nova13:33
*** ociuhandu has joined #openstack-nova13:33
*** macz_ has joined #openstack-nova13:37
*** ociuhandu has quit IRC13:38
*** ociuhandu has joined #openstack-nova13:38
*** macz_ has quit IRC13:41
*** hoonetorg has joined #openstack-nova13:44
*** jawad_axd has quit IRC13:58
*** ociuhandu has quit IRC14:18
*** martinkennelly has quit IRC14:39
*** mlavalle has quit IRC14:41
openstackgerritmelanie witt proposed openstack/nova master: Reject open redirection in the console proxy  https://review.opendev.org/c/openstack/nova/+/79129714:45
*** ociuhandu has joined #openstack-nova14:45
*** dklyle has joined #openstack-nova14:53
melwittgibi, dansmith: added a reno ^ I didn't know if/how I should use fancy markup in it so I didn't14:55
*** ociuhandu has quit IRC14:56
gibimelwitt: thanks, will check back soon14:56
melwittty14:58
*** macz_ has joined #openstack-nova14:58
dansmithmelwitt: seems alright to me.. will let gibi +W if he's around and wants to review14:59
dansmithbut otherwise I'd be fine with it14:59
melwittsounds cool, thanks15:00
openstackgerritBalazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP  https://review.opendev.org/c/openstack/nova/+/78720515:11
openstackgerritBalazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture  https://review.opendev.org/c/openstack/nova/+/78720615:11
*** ociuhandu has joined #openstack-nova15:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Support the new port resource_request format  https://review.opendev.org/c/openstack/nova/+/78720815:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix RequestLevelParams persistence handling in RequestSpec  https://review.opendev.org/c/openstack/nova/+/79150215:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams  https://review.opendev.org/c/openstack/nova/+/79150315:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36  https://review.opendev.org/c/openstack/nova/+/79150415:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query  https://review.opendev.org/c/openstack/nova/+/79150515:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling  https://review.opendev.org/c/openstack/nova/+/79150615:11
*** raildo has quit IRC15:11
*** mlavalle has joined #openstack-nova15:11
*** tkajinam has quit IRC15:14
*** ociuhandu has quit IRC15:16
openstackgerritMerged openstack/osc-placement master: Add functional-py39 tox target  https://review.opendev.org/c/openstack/osc-placement/+/76091215:20
*** ociuhandu has joined #openstack-nova15:21
openstackgerritMerged openstack/osc-placement master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/osc-placement/+/76071015:22
openstackgerritMerged openstack/osc-placement master: Add py38 package metadata  https://review.opendev.org/c/openstack/osc-placement/+/75698515:22
openstackgerritMerged openstack/osc-placement master: Verify result for inventory set --dry-run  https://review.opendev.org/c/openstack/osc-placement/+/68029515:22
openstackgerritMerged openstack/osc-placement master: Add "--resource-class" to  allocation unset  https://review.opendev.org/c/openstack/osc-placement/+/69762015:22
gibimelwitt: is it OK for you if I touch the reno a bit to add some formatting?15:25
melwittgibi: sure, go ahead15:25
gibiOK15:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject open redirection in the console proxy  https://review.opendev.org/c/openstack/nova/+/79129715:28
*** __ministry has quit IRC15:40
*** __ministry has joined #openstack-nova15:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix RequestLevelParams persistence handling in RequestSpec  https://review.opendev.org/c/openstack/nova/+/79150215:44
openstackgerritBalazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP  https://review.opendev.org/c/openstack/nova/+/78720515:44
*** ociuhandu has quit IRC15:45
openstackgerritBalazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture  https://review.opendev.org/c/openstack/nova/+/78720615:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams  https://review.opendev.org/c/openstack/nova/+/79150315:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36  https://review.opendev.org/c/openstack/nova/+/79150415:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query  https://review.opendev.org/c/openstack/nova/+/79150515:48
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling  https://review.opendev.org/c/openstack/nova/+/79150615:49
openstackgerritBalazs Gibizer proposed openstack/nova master: Support the new port resource_request format  https://review.opendev.org/c/openstack/nova/+/78720815:50
*** lpetrut has quit IRC16:10
*** macz_ has quit IRC16:12
*** lucasagomes has quit IRC16:26
*** gyee has joined #openstack-nova16:29
openstackgerritMerged openstack/os-vif master: Creating oslo.config.opts entry_points for plugins  https://review.opendev.org/c/openstack/os-vif/+/78964516:32
openstackgerritMerged openstack/os-vif master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/os-vif/+/78973416:32
*** k_mouza has quit IRC16:39
*** k_mouza has joined #openstack-nova16:59
*** k_mouza has quit IRC17:04
*** k_mouza has joined #openstack-nova17:31
*** k_mouza has quit IRC17:36
*** andrewbonney has quit IRC17:41
*** ociuhandu has joined #openstack-nova18:01
*** amodi has joined #openstack-nova18:06
*** ociuhandu has quit IRC18:07
*** links has quit IRC18:37
*** tbachman has quit IRC18:53
*** tbachman has joined #openstack-nova18:53
*** whoami-rajat has quit IRC19:21
*** avolkov has quit IRC19:35
*** k_mouza has joined #openstack-nova20:17
*** k_mouza has quit IRC20:21
*** rcernin has joined #openstack-nova20:39
*** macz_ has joined #openstack-nova21:03
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: Regression test for changing the MTU during a live migration  https://review.opendev.org/c/openstack/nova/+/79123521:03
*** macz_ has quit IRC21:07
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: Regression test for changing the MTU during a live migration  https://review.opendev.org/c/openstack/nova/+/79123521:08
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: Don't update the MTU during a live migration  https://review.opendev.org/c/openstack/nova/+/79155321:08
artomsean-k-mooney, you're probably off by now, but ^^ is proof that we suck21:08
*** macz_ has joined #openstack-nova21:08
*** tbachman has quit IRC21:14
sean-k-mooneyartom: not really its not ment to be updated21:20
sean-k-mooneysimply not updating the mtu wehn we regenerate the xml is not a vaild option21:21
sean-k-mooneywell its an option but its still incorrect21:21
sean-k-mooneywe discussi that approch in the ptg and i tought we rejected it21:22
sean-k-mooneysince the vm will not have working network connectivy if we do thi21:23
sean-k-mooneygeneve and vxlan for that matter do not allow packet fragmentation so if the guest had a larger mtu before and they send a full mtu sized packet it will be dropped21:24
sean-k-mooneyif it had a smaller mtu before the it will be able to transmit fine but i belive the recive path will be broken although it might work21:25
artomsean-k-mooney, true - I can change the fix's approach21:26
artomBut the point is, we were still happily trying to update the XML with the new MTU21:26
artomDespite specifically having code in there with comments saying we can't do that21:26
artomWe missed the case of an MTU already being set21:26
sean-k-mooneythat was because of a differnt upgrade issue21:26
sean-k-mooneythat was for when we first added mtu support21:27
sean-k-mooneyfor jumbo frames21:27
artomAh, yeah, I don't know the history21:27
artomAnyways, I'm just happy I got my head around it21:28
artomI could never reconcile the "we just need a better error message" with the error that libvirt was throwing. We were clearly doing *something* wrong21:28
sean-k-mooneyas i said in the downstream bz the neutron extinon is broken by desing21:29
sean-k-mooneyand im suprised we support in our down stream product21:29
sean-k-mooneywhen writeble mtus was first intoduced it was for just one out of tree network backend21:30
sean-k-mooneyand no integration was added with nova21:30
artomRight, I get that part21:30
artomNeutron added a thing that Nova was never modified to support21:30
artomWell... Yeah, I see where you're coming from21:30
artomIt's not a bug if Neutron unilaterally changed the data on us21:31
sean-k-mooneyyes and that was not intended to work with ovn or ml2/ovs at the time21:31
sean-k-mooneyat somepoint it started to be used with those i guesss21:31
artomYeah21:32
artomAnyways, sorry for the noise :) As I said, just happy I groked it21:32
artom*Fully* groked it21:32
* artom is halfway through his beer21:32
sean-k-mooneyno worries21:33
sean-k-mooneyhttps://bugs.launchpad.net/neutron/+bug/167163421:34
openstackLaunchpad bug 1671634 in neutron "[RFE] Allow to set MTU for networks" [Wishlist,Fix released] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)21:34
sean-k-mooneyhttps://review.opendev.org/c/openstack/neutron/+/48351821:34
sean-k-mooneythis is when it was intoduced21:34
*** amodi has quit IRC21:34
sean-k-mooneyartom: and this is where i added the mtu  https://review.opendev.org/c/openstack/nova/+/553072 to fix https://bugs.launchpad.net/nova/+bug/174749621:37
openstackLaunchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Medium,In progress] - Assigned to sean mooney (sean-k-mooney)21:37
sean-k-mooneyartom: anyway enjoy your weekend i was loging in to start backing up file before i resintall my home server21:39
artomsean-k-mooney, o/ you too man21:40
*** rcernin has quit IRC21:40
*** macz_ has quit IRC21:45
*** macz_ has joined #openstack-nova21:49
*** rcernin has joined #openstack-nova22:06
*** rcernin has quit IRC22:11
*** rcernin has joined #openstack-nova22:23
*** rcernin has quit IRC22:27
*** rcernin has joined #openstack-nova22:28
*** rcernin has quit IRC22:33
*** dpawlik has quit IRC22:45
*** tbachman has joined #openstack-nova22:52
*** dpawlik7 has joined #openstack-nova22:53
*** rcernin has joined #openstack-nova23:05
*** derekh has quit IRC23:08
*** rcernin has quit IRC23:09
*** tosky has quit IRC23:11
*** macz_ has quit IRC23:14
*** derekh has joined #openstack-nova23:14
*** rcernin has joined #openstack-nova23:35
*** rcernin has quit IRC23:40
*** mlavalle has quit IRC23:46
*** derekh has quit IRC23:46
« Thursday, 2021-05-13 Index Saturday, 2021-05-15 »

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