Thursday, 2021-01-21

*** jamesmcarthur has joined #openstack-meeting-300:11
*** artom has quit IRC00:11
*** macz_ has quit IRC00:30
*** jamesmcarthur has quit IRC00:40
*** mlavalle has quit IRC00:49
*** baojg has joined #openstack-meeting-301:36
*** ricolin_ has joined #openstack-meeting-302:14
*** hemanth_n has joined #openstack-meeting-302:18
*** ricolin_ has quit IRC04:20
*** macz_ has joined #openstack-meeting-304:21
*** ricolin has joined #openstack-meeting-304:24
*** macz_ has quit IRC04:25
*** yamamoto_ has joined #openstack-meeting-304:51
*** yamamoto has quit IRC04:52
*** ricolin has quit IRC05:11
*** ricolin has joined #openstack-meeting-305:24
*** psahoo has joined #openstack-meeting-306:34
*** psahoo_ has joined #openstack-meeting-306:58
*** psahoo has quit IRC07:02
*** lpetrut has joined #openstack-meeting-307:10
*** eolivare has joined #openstack-meeting-307:39
*** ralonsoh has joined #openstack-meeting-307:41
*** slaweq has joined #openstack-meeting-307:58
*** e0ne has joined #openstack-meeting-307:58
*** tosky has joined #openstack-meeting-308:16
*** lpetrut_ has joined #openstack-meeting-309:34
*** tosky has quit IRC09:36
*** tosky_ has joined #openstack-meeting-309:36
*** lpetrut has quit IRC09:37
*** yamamoto_ has quit IRC09:39
*** tosky_ is now known as tosky09:43
*** yamamoto has joined #openstack-meeting-310:22
*** yamamoto has quit IRC10:28
*** e0ne has quit IRC11:28
*** baojg has quit IRC11:41
*** yamamoto has joined #openstack-meeting-311:56
*** ricolin has quit IRC12:06
*** raildo has joined #openstack-meeting-312:38
*** yamamoto has quit IRC12:40
*** hemanth_n has quit IRC13:02
*** tosky has quit IRC13:06
*** tosky has joined #openstack-meeting-313:06
*** ricolin has joined #openstack-meeting-313:08
*** yamamoto has joined #openstack-meeting-313:13
*** yamamoto has quit IRC13:18
*** yamamoto has joined #openstack-meeting-313:18
*** hemanth_n has joined #openstack-meeting-314:21
*** macz_ has joined #openstack-meeting-314:25
*** markmcclain has quit IRC14:26
*** artom has joined #openstack-meeting-314:28
*** macz_ has quit IRC14:30
*** markmcclain has joined #openstack-meeting-314:46
*** irclogbot_2 has quit IRC15:21
*** irclogbot_2 has joined #openstack-meeting-315:22
*** hemanth_n has quit IRC15:25
*** elod has joined #openstack-meeting-315:33
*** lpetrut_ has quit IRC15:42
*** psahoo_ has quit IRC15:51
*** mlavalle has joined #openstack-meeting-315:51
*** macz_ has joined #openstack-meeting-315:59
*** macz_ has quit IRC15:59
gibi#startmeeting nova16:00
openstackMeeting started Thu Jan 21 16:00:06 2021 UTC and is due to finish in 60 minutes.  The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: nova)"16:00
openstackThe meeting name has been set to 'nova'16:00
lyarwood\o16:00
dansmitho/16:00
stephenfino/16:00
*** macz_ has joined #openstack-meeting-316:00
gibio/16:01
bauzas\o16:01
gibi#topic Bugs (stuck/critical)16:01
*** openstack changes topic to "Bugs (stuck/critical) (Meeting topic: nova)"16:01
gibiwe have one critical bug #link https://bugs.launchpad.net/nova/+bug/191260716:01
openstackLaunchpad bug 1912607 in OpenStack Compute (nova) "test_attach_cloned_encrypted_volume fails in nova-ceph-multistore job permanently " [Critical,In progress] - Assigned to Lee Yarwood (lyarwood)16:01
gibithere is a patch to fix it #link https://review.opendev.org/c/openstack/nova/+/77177716:01
dansmith++16:02
gibibut I saw some dicussion on #openstack-nova16:02
gmanno/16:02
gibidansmith, lyarwood does ^^ enough?16:02
lyarwoodgibi: yeah that's enough16:02
dansmithyup16:02
gibicool16:02
lyarwoodgibi: I'm going to try to clean things up after16:02
gibilyarwood: thanks16:02
lyarwoodgibi: but for now that's all we needd16:02
gibiis there any other bug we need to dicuss today?16:03
gibibtw #link 18 new untriaged bugs (+2 since the last meeting): #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New16:03
*** sean-k-mooney has joined #openstack-meeting-316:04
gibi#topic Gate status16:04
*** openstack changes topic to "Gate status (Meeting topic: nova)"16:04
gibigate on master is blocked due to #link https://bugs.launchpad.net/nova/+bug/191260716:04
openstackLaunchpad bug 1912607 in OpenStack Compute (nova) "test_attach_cloned_encrypted_volume fails in nova-ceph-multistore job permanently " [Critical,In progress] - Assigned to Lee Yarwood (lyarwood)16:04
gibiis there any other gate issue we should be aware of?16:04
lyarwoodI've seen https://bugs.launchpad.net/nova/+bug/1912310 crop up a few times in the last week16:04
openstackLaunchpad bug 1912310 in OpenStack Compute (nova) "libvirt.libvirtError: unable to connect to server at " [Undecided,New]16:04
lyarwoodin the live migration jobs, yet to debug any further16:05
gibithanks for the headsup16:06
gibi#topic Release Planning16:07
*** openstack changes topic to "Release Planning (Meeting topic: nova)"16:07
gibitoday is Milestone 216:07
gibiWe have spec freeze today. Spec that does not have two +2s at EOB today will need to be moved to X16:07
gibiWe have 14 approved bp and we completed 1 so far.16:08
gibiis there any open spec that needs special attention?16:08
gibiI see https://review.opendev.org/c/openstack/nova-specs/+/765901 being close16:08
gibiadd port scoped numa affinity spec ^^16:09
gibithere is an open issue on it conserning the precedence order of the policy in the port and in the flavor16:09
gibiso it needs more eyes today, or it will be bumped to X16:10
gibiany other M2 or release related topic to discuss?16:11
gibi#topic Runway status16:12
*** openstack changes topic to "Runway status (Meeting topic: nova)"16:12
gibi#link https://etherpad.opendev.org/p/nova-runways-wallaby16:13
sean-k-mooneyi put that in the opens discsuion section by the way16:13
sean-k-mooneybut ya we can move on16:13
gibisean-k-mooney: OK, then we can go back to that16:13
gibiin the open discussion16:13
gibiso runways16:13
gibi#link https://blueprints.launchpad.net/nova/+spec/cyborg-shelve-and-unshelve  : has been completed \o/16:13
gibi#link https://blueprints.launchpad.net/nova/+spec/nova-support-webvnc-with-password-anthentication : has been updated and needs re-review16:13
gibiI will do a review round on it as I had reviewed it previously16:14
gibi#link https://blueprints.launchpad.net/nova/+spec/compact-db-migrations-wallaby : first half of the patches has been merged, the rest needs a second core to approve16:14
dansmithI +2d all of them,16:15
gibiI guess we will eat these patches slowly but surely I see dansmith and bauzas is being involved16:15
dansmithI thought you did too16:15
dansmithdid you mean a third?16:15
gibihm, I can go back and upgrade my votes16:15
bauzasgibi: I'll continue16:15
gibibut it is better to have a third eye16:15
gibibauzas: thanks16:15
bauzasbut this takes a bit of time16:15
gibibauzas: sure it is16:16
bauzasand I'm conservative about DB changes16:16
dansmithas we all should be :)16:16
gibiagree16:16
gibi#link https://blueprints.launchpad.net/nova/+spec/modernize-os-hypervisors-api : the api code landed, the python-novaclient patch and the policy patch needs some work16:16
bauzasI totally forgot to look at the series16:17
gibias far as I see the ball is at stephenfin's side in the open patches16:17
stephenfinyup, gotta respin those16:17
gibicool16:17
gibiand last but no least :)16:18
gibi#link https://blueprints.launchpad.net/nova/+spec/support-interface-attach-with-qos-ports : stephenfin is +2 all over it, so it needs a second core to look at16:18
bauzasI can do it16:19
gibiI can trade reviews with any cores if needed16:19
* bauzas clicks16:19
gibibauzas: thanks16:19
gibianything else related almost-done features?16:19
sean-k-mooneygibi: i was +1 on that in the past too right16:20
gibisean-k-mooney: yes I think so16:21
sean-k-mooneyi am +1 on qos attach in general so it would be nice ot see that approved16:21
sean-k-mooneyoh the spec is approved16:21
gibisean-k-mooney: yeah this was about the implementation :) which is also quite ready16:22
sean-k-mooneyill try and reveiw the  code today16:22
gibithanks16:22
gibi#topic Stable Branches16:22
*** openstack changes topic to "Stable Branches (Meeting topic: nova)"16:22
gibionly stable/rocky is blocked (patch to unblock: https://review.opendev.org/766492 )16:22
gibithe rest looks OK16:22
gibiEOM16:22
gibiany other stable thing to bring up?16:22
elodmeanwhile that patch is +2+W'd by lyarwood , thx for that!16:23
elodnothing else I think16:23
lyarwoodyup that's all I had16:23
gibicool16:23
gibi#topic Sub/related team Highlights16:23
*** openstack changes topic to "Sub/related team Highlights (Meeting topic: nova)"16:23
gibiLibvirt (bauzas)16:23
bauzasnothing to report, sir.16:23
gibiOK16:23
lyarwoodversion bump is posted16:23
gibilyarwood: I will review that in the coming days16:24
lyarwoodwe also have a few changes dropping UML and Xen if anyone has time16:24
lyarwoodhttps://review.opendev.org/c/openstack/nova/+/75470016:24
gibi#topic Open discussion16:25
*** openstack changes topic to "Open discussion (Meeting topic: nova)"16:25
gibi(stephenfin) Approve specless blueprint https://blueprints.launchpad.net/nova/+spec/smarter-usb-devices16:25
gibiThe name is misleading. This is both a bugfix (don't add USB controllers unless needed, which is already merged) and add support for a 'hw_input_bus' image metadata property16:26
gibiThis was already discussed previously. melwitt was ambivalent and deferred to others http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018713.html16:26
gibialso impl patches are available https://review.opendev.org/q/topic:bp/smarter-usb-devices16:26
stephenfinI think that summarizes it quite nicely. Just checking that no one is against me continuing with that16:27
gibiany objections?16:27
sean-k-mooneyno just notig that the second half is not backporable16:28
sean-k-mooneywhere as the first might be16:28
sean-k-mooneyso make sure they are in seperate patches16:28
stephenfinyup, they are16:28
sean-k-mooneycool16:29
*** e0ne has joined #openstack-meeting-316:29
gibiI hear no objection so I will approve the bp after the meeting16:29
stephenfinta16:30
gibi(sean) https://review.opendev.org/c/openstack/nova-specs/+/765901 precedence of port numa affinity policy16:30
gibiso the numa affinity policy can be provided in the flavor16:31
gibiand the spec proposes the possibility to provide it in the port as well16:31
gibithe spec proposes also that the precedence is port > flavor16:31
gibiwhich make sense from the perspective that the port is more fine grained16:31
gibiand therefore allow customization of a flavor based policy16:32
gibibut the flavor based policy is admin only by default16:32
gibiwhile the port based policy is not admin only16:32
stephenfinthat seems a bit misleading16:32
gibistephenfin: correct me please16:33
stephenfinit's admin-only because flavor extra specs are admin-only16:33
artomWe have the same policy in image meta as well, what's the preference there?16:33
artomSeeing as image is user and flavor is admin...16:33
sean-k-mooneyif image != flavor then erorr16:33
* bauzas drops \o16:33
gibibauzas: \o16:33
sean-k-mooneyother wise we use image or flavor with same precednce16:33
stephenfinbut there's no special reason we couldn't otherwise allow non-admins configure NUMA policies16:33
artomsean-k-mooney, that seems... pointless? So if flavor doesn't have anything, image can't set it?16:33
sean-k-mooneywelll no16:34
stephenfinartom: if the flavor is set16:34
stephenfinif both are set and they don't match16:34
artomstephenfin, so could we apply the same logic to port?16:34
stephenfin(we've a bad history of flip-flopping on how to resolve image-flavour mismatches)16:34
artomOnly use it if flavor doesn't set anything?16:34
stephenfinwe could, but that would kill a certain amount of the value of the feature16:35
sean-k-mooneyif we do that you cant set a different polciy for alis based passthough16:35
stephenfinyou want to be able to say "all ports *except* this one have NUMA affinity*16:35
stephenfin*"16:35
sean-k-mooneythat too16:35
stephenfinas well as "all ports *except* this one don't have NUMA affinity"16:35
stephenfins/NUMA affinity/strict NUMA affinity/16:36
artomIf that's the use case I don't see how port can be anything other than "final word"16:36
sean-k-mooneythe extra specs also only applie to sriov/pci devices16:36
sean-k-mooneyits is ignored for numa aware vswitches16:36
sean-k-mooneyif that is configreued you have stict affintiy for all numa instnaces16:36
stephenfingibi: If there were a "negative" NUMA affinity policy then I'd agree that something user configurable (port policies) wouldn't be allowed override what the admin states16:37
stephenfinbecause that would prevent the admin "preserving" resources16:37
stephenfinbut we don't provide that because it would be actively harmful :)16:37
artomstephenfin, I guess the question could be - would there ever be a use case for an admin reserving a PCI device for a specific NUMA node?16:38
sean-k-mooneyyou cant do that today16:38
gibistephenfin: so nova always prefer affinity regardless of the policy but doesn't fail if it cannot affine it if the policy allows suboptimal config16:38
stephenfinartom: we don't provide any way to do that16:38
stephenfingibi: yes16:38
gibistephenfin: then you convinced me16:38
gibiI'm OK with the currently proposed precedence16:39
sean-k-mooneymelwitt: thoughts?16:39
stephenfinif the user insists on optimal config and we can't give it to them, well, that's on them16:39
artomstephenfin, ok, re-phrasing - if the admin sets a `required` policy because for some reason they only want PCI devices to be used by VMs on the same NUMA nodes...16:39
artomWould there ever be a use case where the user overriding that with a port policy would be harmful?16:39
sean-k-mooneyone thing that imporant here is the admins policy will still apply to all other ports and passthough devices16:40
stephenfinartom: while possible, I don't think that's a realistic scenario16:40
sean-k-mooneythe policy on the port only applies to that port16:40
artomstephenfin, I mean, I'd tend to agree - I'm playing devil's advocate here16:40
stephenfinno, the user would just get worse performance16:40
stephenfinoh yeah, I know :)16:40
gibigetting worse performance when explicitly asked for is OK in my eyes16:41
*** lpetrut has joined #openstack-meeting-316:42
stephenfinyeah, and nova will already try to give better performance, so all the non-admin user could do is insist on it16:42
gibiI will upgrade my vote on the spec but hold the +W so that melwitt can comment.16:42
sean-k-mooneyya one usecase that dirves that is the nfv case where eveything should be strict but they dont care about the managment interface16:42
stephenfinwhich would simply fail if it couldn't be met16:42
stephenfincool16:42
gibiis there anything else for today?16:42
sean-k-mooneynot from me16:43
gibiOK16:43
gibithanks for joining today. Enjoy the rest of your day!16:43
sean-k-mooneyo/16:44
gibi#endmeeting16:44
*** sean-k-mooney has left #openstack-meeting-316:44
elodo/16:44
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"16:44
openstackMeeting ended Thu Jan 21 16:44:13 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:44
openstackMinutes:        http://eavesdrop.openstack.org/meetings/nova/2021/nova.2021-01-21-16.00.html16:44
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/nova/2021/nova.2021-01-21-16.00.txt16:44
openstackLog:            http://eavesdrop.openstack.org/meetings/nova/2021/nova.2021-01-21-16.00.log.html16:44
*** elod has left #openstack-meeting-316:49
*** lpetrut has quit IRC16:50
*** jamesmcarthur has joined #openstack-meeting-316:51
*** lpetrut has joined #openstack-meeting-317:40
*** diablo_rojo has quit IRC17:41
*** ralonsoh has quit IRC17:52
*** lpetrut has quit IRC17:55
*** e0ne has quit IRC18:33
*** _mlavalle_1 has joined #openstack-meeting-319:00
*** mlavalle has quit IRC19:03
*** haleyb has quit IRC19:27
*** haleyb has joined #openstack-meeting-319:28
*** jamesmcarthur has quit IRC19:52
*** slaweq has quit IRC19:55
*** jamesmcarthur has joined #openstack-meeting-320:03
*** irclogbot_2 has quit IRC20:14
*** tosky has quit IRC20:14
*** tosky has joined #openstack-meeting-320:15
*** irclogbot_3 has joined #openstack-meeting-320:16
*** eolivare has quit IRC20:17
*** raildo has quit IRC21:41
*** jamesmcarthur_ has joined #openstack-meeting-321:45
*** jamesmcarthur has quit IRC21:49
*** jamesmcarthur has joined #openstack-meeting-322:02
*** jamesmcarthur_ has quit IRC22:05
*** jamesmcarthur_ has joined #openstack-meeting-322:51
*** jamesmcarthur has quit IRC22:55
*** jamesmcarthur has joined #openstack-meeting-323:02
*** yamamoto has quit IRC23:04
*** yamamoto has joined #openstack-meeting-323:04
*** jamesmcarthur_ has quit IRC23:05
*** jamesmcarthur has quit IRC23:12
*** jamesmcarthur has joined #openstack-meeting-323:16
*** jamesmcarthur has quit IRC23:16
*** jamesmcarthur has joined #openstack-meeting-323:16
*** jamesmcarthur has quit IRC23:55
*** jamesmcarthur has joined #openstack-meeting-323:59

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