Friday, 2021-02-19

*** luksky has quit IRC00:22
*** martinkennelly has quit IRC00:30
*** LinPeiWen has joined #openstack-nova00:35
openstackgerritJorhson Deng proposed openstack/nova master: Reject to attach and detach volume while task_state is not None  https://review.opendev.org/c/openstack/nova/+/75891800:47
*** mlavalle has quit IRC01:13
*** rcernin has quit IRC01:29
*** macz_ has joined #openstack-nova01:30
*** zzzeek has quit IRC01:30
*** zzzeek has joined #openstack-nova01:34
*** macz_ has quit IRC01:34
*** rcernin has joined #openstack-nova01:35
*** mkrai has joined #openstack-nova02:15
openstackgerritMerged openstack/nova master: Add regression test for bug 1914777  https://review.opendev.org/c/openstack/nova/+/77530702:18
openstackbug 1914777 in OpenStack Compute (nova) "Possible race condition between n-cpu and n-api when deleting a building instance" [High,In progress] https://launchpad.net/bugs/1914777 - Assigned to melanie witt (melwitt)02:18
openstackgerritMerged openstack/nova stable/ussuri: tools: Allow check-cherry-picks.sh to be disabled by an env var  https://review.opendev.org/c/openstack/nova/+/76602402:18
*** sapd1 has joined #openstack-nova02:22
*** zenkuro has quit IRC02:37
*** rcernin has quit IRC02:38
*** rcernin has joined #openstack-nova02:39
*** rcernin has quit IRC02:50
*** rcernin has joined #openstack-nova03:03
*** sapd1 has quit IRC03:07
*** sapd1 has joined #openstack-nova03:10
*** rcernin has quit IRC03:17
*** rcernin has joined #openstack-nova03:26
*** yonglihe has joined #openstack-nova03:30
*** sapd1 has quit IRC03:34
*** dviroel has quit IRC03:35
*** rcernin has quit IRC03:37
*** sapd1 has joined #openstack-nova03:45
*** spatel has joined #openstack-nova03:45
*** rcernin has joined #openstack-nova03:52
*** flaviof has quit IRC04:01
*** TheJulia has quit IRC04:01
*** vdrok has quit IRC04:02
*** vdrok has joined #openstack-nova04:03
*** TheJulia has joined #openstack-nova04:04
*** flaviof has joined #openstack-nova04:04
*** sapd1_x has joined #openstack-nova04:15
*** spatel has quit IRC04:18
*** spatel has joined #openstack-nova04:20
*** __ministry has joined #openstack-nova04:23
*** hemanth_n has joined #openstack-nova04:29
*** hemanth_n has quit IRC04:42
openstackgerritmelanie witt proposed openstack/nova master: Handle instance = None in _local_delete_cleanup  https://review.opendev.org/c/openstack/nova/+/77530804:47
*** macz_ has joined #openstack-nova04:52
*** macz_ has quit IRC04:57
*** ratailor has joined #openstack-nova05:07
*** zoharm has joined #openstack-nova05:11
*** gyee has quit IRC05:23
*** sapd1 has quit IRC05:28
*** mkrai has quit IRC05:32
*** mkrai has joined #openstack-nova05:32
*** spatel has quit IRC05:58
*** benj_ has quit IRC06:12
*** links has joined #openstack-nova06:16
*** swp20 has quit IRC06:18
*** benj_ has joined #openstack-nova06:19
*** mkrai has quit IRC06:23
*** iurygregory_ has joined #openstack-nova06:26
*** elod has quit IRC06:26
*** iurygregory has quit IRC06:27
*** elod has joined #openstack-nova06:28
*** elod has quit IRC06:34
*** elod has joined #openstack-nova06:54
*** ratailor has quit IRC06:57
*** ratailor has joined #openstack-nova07:01
openstackgerritMerged openstack/nova stable/ussuri: libvirt: Only ask tpool.Proxy to autowrap vir* classes  https://review.opendev.org/c/openstack/nova/+/76122307:02
*** sapd1 has joined #openstack-nova07:06
*** slaweq has joined #openstack-nova07:16
*** luksky has joined #openstack-nova07:21
*** rcernin has quit IRC07:29
*** ralonsoh has joined #openstack-nova07:35
openstackgerritYongli He proposed openstack/nova master: smartnic support  https://review.opendev.org/c/openstack/nova/+/75894407:41
*** dklyle has quit IRC07:46
yonglihesean-k-mooney:  review invite, thanks: https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged)07:57
yonglihegibi:  review invite,  thanks. https://review.opendev.org/q/topic:%22bp%252Fsriov-smartnic-support%22+(status:open%20OR%20status:merged)08:02
*** __ministry has quit IRC08:03
gibiyonglihe: I will try08:05
*** rpittau|afk is now known as rpittau08:06
yonglihethanks, it's kind late for this release .  lots of depend-on items, good news is that's all got merged(neutron and cyborg).08:06
gibiyonglihe: ack, honestly I have a lot of thing to review so I might not be able to get to it this week08:10
yongliheack -:)08:12
*** amodi has quit IRC08:12
*** khomesh24 has joined #openstack-nova08:18
*** __ministry has joined #openstack-nova08:21
*** amodi has joined #openstack-nova08:23
*** ihti[m] has quit IRC08:26
*** masterpe has quit IRC08:26
*** ociuhandu has joined #openstack-nova08:31
*** andrewbonney has joined #openstack-nova08:35
*** tosky has joined #openstack-nova08:40
*** ociuhandu has quit IRC08:41
*** xek has joined #openstack-nova08:43
*** ociuhandu has joined #openstack-nova08:45
*** mgoddard has quit IRC08:48
*** ociuhandu has quit IRC08:50
*** lpetrut has joined #openstack-nova09:04
lyarwoodelod: https://review.opendev.org/c/openstack/nova/+/761224  - would you mind hitting this today, downstream finally ran into it so I'm getting chased around with a pitchfork. ^_^09:07
*** ociuhandu has joined #openstack-nova09:08
elodlyarwood: sure, looking :)09:14
*** ociuhandu has quit IRC09:21
*** masterpe has joined #openstack-nova09:21
*** ociuhandu has joined #openstack-nova09:22
openstackgerritJorhson Deng proposed openstack/nova master: Reject to attach and detach volume while task_state is not None  https://review.opendev.org/c/openstack/nova/+/75891809:23
bauzasgibi: stephenfin: eek, now I added nova/network/neutron.py in the mypy files, looks to me I need to add *all* the type hints for all the methods /o\09:24
bauzasgibi: stephenfin: context, Zuul is -1 for pep8 https://review.opendev.org/c/openstack/nova/+/77397609:24
* bauzas is a sad panda again :(09:25
elodlyarwood: patch looks good, +2+W'd09:26
*** ociuhandu has quit IRC09:27
melwittbauzas: mypy is now your life09:27
bauzas:(09:28
bauzasI was already loud about mypy09:28
bauzasso i won't tell more09:28
melwittit's ok, I was just trying to be funny09:28
bauzasbut next time I'm asked to provide type hints for a module, I'll lookup whether the file is already checked09:28
bauzasmelwitt: no worries, I prefer american pies09:29
lyarwoodbauzas: -1 nit did you mean bauzas(self) -> 'SadPanda': ?09:29
melwittlol09:30
bauzasexcellent09:30
bauzasAttibutError : object 'SadPanda' does not exist09:30
bauzasbut yeah, mypy is like a cold sea, you just step into it by the toe first09:31
bauzasor you'll get cold09:32
bauzasbut at least I can reproduce it locally \o/09:35
* bauzas goes into fixing typing issues that he didn't wrote09:35
*** ociuhandu has joined #openstack-nova09:35
*** martinkennelly has joined #openstack-nova09:36
*** swp20 has joined #openstack-nova09:40
swp20stephenfin: morning09:41
swp20for the noVNC patch, can we add TLS in the follow change? as the bp's goal is add the password to get vnc url.09:42
*** dtantsur|afk is now known as dtantsur09:42
*** derekh has joined #openstack-nova09:45
*** brinzhang has joined #openstack-nova09:46
openstackgerritSylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments  https://review.opendev.org/c/openstack/nova/+/77397609:51
openstackgerritSylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter  https://review.opendev.org/c/openstack/nova/+/74906809:51
bauzasstephenfin: gibi: eventually gave up on fixing neutron issues09:52
bauzas(for mypy)09:52
openstackgerritWenping Song proposed openstack/nova master: Nova supports password encrypted VNC  https://review.opendev.org/c/openstack/nova/+/62233609:56
openstackgerritLucian Petrut proposed openstack/nova master: hyper-v rbd volume support  https://review.opendev.org/c/openstack/nova/+/76355010:00
*** k_mouza has joined #openstack-nova10:01
brinzhangstephenfin, swp20: yes, this spec we just want to add the vnc authenticate10:06
brinzhang by password, if we want to add the x509vnc support, we can do that later, rather than in this spec10:06
brinzhangstephenfin, swp20: as your point in https://review.opendev.org/c/openstack/nova/+/622336/30/nova/console/rfb/auths.py#3010:08
stephenfinbrinzhang: swp20: I need to think about it more and read melwitt's replies. I don't understand why someone would opt for basic VNC auth when X509+VNC is an option. Using just basic VNC auth makes this feature effectively useless for deployments with encryption, which is virtually all TripleO deployments at least10:14
stephenfinbrinzhang: Surely you folks aren't going to recommend people use VNC and drop encryption?10:15
*** LinPeiWen has quit IRC10:15
lyarwoodstephenfin / gibi ; https://review.opendev.org/c/openstack/nova/+/774897/ - Would you mind looking at this again today and highlighting which approach you think I should take here (o.vo vs sqla)? I'm not sure how I can test this on an actual env at scale before we merge tbh.10:22
lyarwoodurgh lc-- pip-- how does it pass locally and then fail in the gate10:28
stephenfinsure10:29
brinzhangstephenfin: I know you want to encrypt identities and files, but this is not my original intention of designing this feature. For the users I have seen, vnc password authentication is sufficient.10:32
brinzhangstephenfin: but as you said, using x509+vnc is an optional, we can try10:33
*** jangutter_ has joined #openstack-nova10:34
brinzhangs/optional/option/10:37
*** jangutter has quit IRC10:38
*** jangutter_ has quit IRC10:39
*** jangutter has joined #openstack-nova10:39
*** Underknowledge has joined #openstack-nova10:45
*** dviroel has joined #openstack-nova10:51
*** jangutter_ has joined #openstack-nova10:51
*** dtantsur is now known as dtantsur|brb10:55
*** jangutter has quit IRC10:55
gibibauzas: have you tried the routed net patches in a devstack? I think it is cannot be activated as the extension name is wrong in the code https://review.opendev.org/c/openstack/nova/+/773976/9/nova/network/constants.py#3211:01
gibibauzas: I can give a try to adding  mypy needs to neutron.py11:02
*** ihti[m] has joined #openstack-nova11:08
*** ihti[m] has quit IRC11:08
*** ociuhandu has quit IRC11:17
*** ociuhandu has joined #openstack-nova11:18
*** k_mouza has quit IRC11:20
*** k_mouza has joined #openstack-nova11:20
*** ociuhandu has quit IRC11:23
*** ihti[m] has joined #openstack-nova11:24
*** tkajinam has quit IRC11:25
openstackgerritMerged openstack/nova stable/train: tools: Allow check-cherry-picks.sh to be disabled by an env var  https://review.opendev.org/c/openstack/nova/+/76602511:59
*** NobodyCam has quit IRC12:08
*** rpittau has quit IRC12:08
*** johnsom has quit IRC12:08
*** masayukig has quit IRC12:08
*** flaviof has quit IRC12:08
*** vdrok has quit IRC12:08
*** TheJulia has quit IRC12:08
*** k_mouza_ has joined #openstack-nova12:11
*** k_mouza_ has quit IRC12:12
*** k_mouza_ has joined #openstack-nova12:12
*** k_mouza has quit IRC12:15
*** khomesh24 has quit IRC12:16
*** flaviof has joined #openstack-nova12:18
*** vdrok has joined #openstack-nova12:18
*** NobodyCam has joined #openstack-nova12:18
*** TheJulia has joined #openstack-nova12:19
*** LinPeiWen has joined #openstack-nova12:19
*** masayukig has joined #openstack-nova12:19
*** Luzi has joined #openstack-nova12:19
*** rpittau has joined #openstack-nova12:21
*** brinzhang has quit IRC12:22
*** johnsom has joined #openstack-nova12:23
*** dtantsur|brb is now known as dtantsur12:29
*** mgoddard has joined #openstack-nova12:30
openstackgerritLee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent  https://review.opendev.org/c/openstack/nova/+/72076912:39
*** LinPeiWen has quit IRC12:45
*** zenkuro has joined #openstack-nova12:46
*** iurygregory_ is now known as iurygregory12:49
*** links has quit IRC12:49
*** mlavalle has joined #openstack-nova12:54
*** ociuhandu has joined #openstack-nova12:55
*** links has joined #openstack-nova12:59
*** ratailor has quit IRC13:01
*** knikolla has quit IRC13:06
*** ociuhandu has quit IRC13:06
*** ociuhandu has joined #openstack-nova13:07
*** knikolla has joined #openstack-nova13:07
*** ociuhandu has quit IRC13:11
bauzasgibi: unfortunately no, I don't have an env with multiple nodes at hand13:14
bauzas...13:14
*** rcernin has joined #openstack-nova13:26
*** k_mouza_ has quit IRC13:28
*** k_mouza has joined #openstack-nova13:28
*** rcernin has quit IRC13:30
*** sapd1 has quit IRC13:34
openstackgerritLucian Petrut proposed openstack/nova master: hyper-v rbd volume support  https://review.opendev.org/c/openstack/nova/+/76355013:42
*** tbachman has quit IRC13:42
*** ociuhandu has joined #openstack-nova13:44
*** khomesh24 has joined #openstack-nova13:55
lyarwooddansmith / gmann / gibi ; https://review.opendev.org/c/openstack/tempest/+/771499 - This came up during the meeting yesterday, IMHO we should go ahead with this now if multinode provides the same coverage13:57
*** rcernin has joined #openstack-nova14:01
*** spatel has joined #openstack-nova14:02
*** sapd1 has joined #openstack-nova14:03
*** ociuhandu has quit IRC14:07
*** ociuhandu has joined #openstack-nova14:08
stephenfinbauzas: It should be pretty easy to set one up with DevStack and VMs, if you have something with 32 GB of RAM14:09
stephenfinIf not, sean-k-mooney might be able to lend you a hand?14:10
*** ociuhandu has quit IRC14:12
gibibauzas: ack, I will try to validate as much of the assumption the functional test cases makes as possible in a devstack14:16
gibiright now I saw the problem with the extension name14:16
gibiand strugling with neutron to produce the assumed placement config (PRs, inventories, aggregates) and nova config (aggregates) for the networks14:17
gibilyarwood: I'm OK to drop the base grenade14:19
sean-k-mooneyi see my name14:20
sean-k-mooneyreading14:20
sean-k-mooneybauzas: i have got 3-4 nodes on my p50 laptop before but if you need nodes i think i still have space on my home cloud14:21
sean-k-mooneybauzas: i can give you a log in and you can install away14:21
bauzasstephenfin: gibi: sean-k-mooney: my laptop (a t470s) only has 16GB of RAM14:22
dansmithlyarwood: cool with me!14:22
sean-k-mooneybauzas: i have found the min ram size for contolers is about 6G and comptue is about 2-414:22
bauzasI also have two mini optiplexs but they also have 16GB14:22
bauzasand I'm waiting for my new laptop since beginning of december, but the provider is running out of stock till' March :)14:23
bauzasthings are going crazy those days with CPUs and GPUs14:23
bauzassame with bikes fwiw14:23
*** k_mouza_ has joined #openstack-nova14:23
*** gibi__ has joined #openstack-nova14:24
gibi__hm, my raspberry hosting my irc client just died14:24
*** ociuhandu has joined #openstack-nova14:24
sean-k-mooneybauzas: looking at my cloud i currently have about 35GB free if you need to spin up a couple of 8G vms14:25
*** gibi has quit IRC14:25
bauzassean-k-mooney: well, spinning up a devstack env with multiple nodes would take me a couple of days14:26
bauzasif someone can verify the segments extension for me, this should be enough IMHO14:26
gibi__bauzas: I'm on that14:26
bauzasgibi__: <314:26
bauzasI'm enough confident in my functional testing14:27
*** k_mouza has quit IRC14:27
bauzasbut yeah, this (the segments extension name) wasn't verified14:27
gibi__bauzas: yes, I also think that the funct test are plenty, I just need to verify the assumptions the func test makes about neutron14:27
sean-k-mooneybauzas: ok i can get you a 2 node deployment in my cloud in a hour or so.14:28
gibi__like the name of the extension14:28
*** ociuhandu has quit IRC14:28
sean-k-mooneybauzas: oh you just want that14:28
sean-k-mooneyi cans show you were its defiend14:28
*** ociuhandu has joined #openstack-nova14:28
sean-k-mooneyhttps://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/segment.py#L3214:28
bauzashttps://docs.openstack.org/neutron/latest/contributor/internals/segments.html14:29
bauzasit says SEGMENT with capitals14:29
bauzassean-k-mooney: sure, but 'segment' looks to work too https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/segment.py#L2914:30
bauzasthat's what I provided14:30
sean-k-mooneyyes segment is the alis14:30
sean-k-mooneyjust ignore case14:30
bauzasand https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/segment.py#L36 tells me 'segment' too14:30
bauzasgibi__: ^14:30
sean-k-mooneythats the resouce name14:31
sean-k-mooneyfor the api endpoint14:31
sean-k-mooneybauzas: the only one im sure its not is all uppercase14:31
bauzashuh14:31
*** khomesh24 has quit IRC14:32
sean-k-mooneyhttp://paste.openstack.org/show/802824/14:32
sean-k-mooneyi dont have the segments extion enabled14:32
gibi__this is the output of the extension list http://paste.openstack.org/show/802810/14:32
sean-k-mooneybut the Name will be Segment and the alias will be segment14:32
gibi__the nova code checks the keys in the segments dict14:33
sean-k-mooneythe alias are always  lowercase14:33
gibi__so yes, it is Segment what we need14:33
sean-k-mooneyoh mybe not14:33
bauzas...14:34
bauzaswe're discussing about keys using CamelCase ?14:34
*** Luzi has quit IRC14:34
bauzasco'on Neutron !14:34
sean-k-mooneybauzas: the name is not used in code14:34
bauzasokay, so 'segment' then ?14:35
sean-k-mooneythe alisa is whats used internally14:35
*** rcernin has quit IRC14:35
bauzaswhy is this so complicated to know which extension name to use ?14:35
sean-k-mooneythe name is the huma readable name more or less you would use the alis in the config files for example14:35
sean-k-mooneywhere are you using it14:35
bauzasgibi__: tbh, I copied the semantics from multi_provider_network extension14:35
sean-k-mooneythere are two filed you can check for either14:36
bauzasthis was a single name and with all non-capitals14:36
bauzassean-k-mooney: context https://review.opendev.org/c/openstack/nova/+/773976/9/nova/network/constants.py#3214:36
*** tbachman has joined #openstack-nova14:36
sean-k-mooneythanks jsut went lookign for that14:36
sean-k-mooneybauzas: stephenfin has a patch to normallyise you use of names to the alias14:37
gibi__I've tried, in devstack. I enabled the segment service_plugin in neutron and the pre-filter did not triggered as it looked for the name 'segments' but the extensions dict in nova contains 'Segments'14:37
gibi__bah14:38
gibi__segment and Segment14:38
gibi__singular14:38
bauzasweirdo14:38
gibi__I change the constant to Segment locally and it now triggers the prefilter14:38
bauzasI'll need to respin the patch either way14:38
gibi__but now I'm stuck troubleshooting neutron segment plugin as it does not create the placement RP inventory and aggregate14:38
sean-k-mooneybauzas: https://review.opendev.org/c/openstack/nova/+/70579214:39
gibi__I will let you know where am I when I leave for the weekend14:39
sean-k-mooneybauzas: currently we use the name so you would need Segment for your patch14:40
*** ociuhandu has quit IRC14:40
bauzasok, let me respin then14:40
bauzasgibi__: thanks for checking14:40
gibi__no problemo14:40
*** ociuhandu has joined #openstack-nova14:40
bauzasgibi__: https://docs.openstack.org/neutron/latest/admin/config-routed-networks.html14:41
*** tbachman has quit IRC14:42
gibi__bauzas: ack14:46
*** gibi__ has quit IRC14:46
*** gibi has joined #openstack-nova14:46
*** ociuhandu has quit IRC14:49
*** __ministry has quit IRC14:50
openstackgerritSylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments  https://review.opendev.org/c/openstack/nova/+/77397614:51
openstackgerritSylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter  https://review.opendev.org/c/openstack/nova/+/74906814:52
gibibauzas, sean-k-mooney: another finding http://paste.openstack.org/show/802828/ in this simple net config the segments plugin in neutron is not triggered as the subnet does not show any segment_id, but in the other hand the network has a segment, and the segment list shows the segment id14:56
gibithis will be a problem for us14:56
gibias in this case nova things there is a segment (based on segments list) but the segmetns plugin in neturon did not created any aggregates as the subnet does not show a segment_id14:56
bauzaswait14:57
* bauzas goes looking at pastebin14:57
sean-k-mooneygibi: i think you need to be an admin to see it14:58
bauzasgibi: I'll enter some bug triage meeting in a second (and I have to host it), could we postpone our discussion by one hour ?14:59
gibibauzas: sure, I will be her14:59
gibisean-k-mooney: I'm admin14:59
sean-k-mooneygibi: can you try listing the filed explictly?14:59
gibisean-k-mooney: also verifyed that in this case the subnet has no segment_id from the neutron code perspective14:59
gibisean-k-mooney: sure14:59
bauzasgibi: my wife will look at me as we have curfew at 6pm, so I won't be able to stay late as we need to visit supermarkets but we can continue for 10 mins14:59
sean-k-mooneyor doign it with debug14:59
sean-k-mooneyosc may not render it14:59
sean-k-mooneyby default that is15:00
sean-k-mooneyjust reading your pastbin  but need to jump to same meeting as bauzas  brb15:00
gibisean-k-mooney: ack, have a nice meeting :)15:01
gibihttp://paste.openstack.org/show/802829/15:01
gibiso the REST API returns segment_id null15:01
stephenfinsean-k-mooney: I've about three patches that bauzas would benefit from, IMO, but I obviously don't want to make a review chain even longer /o\15:01
stephenfin(the secure boot one is 12 patches long already)15:02
*** ociuhandu has joined #openstack-nova15:03
sean-k-mooneystephenfin: ya i know just point it out to show we currently use the Name field15:03
sean-k-mooneyso Segment15:03
sean-k-mooneynot the alias field15:04
stephenfinah, yes, good point15:04
stephenfinwho ever thought that was a good idea...15:04
stephenfin*whoever15:04
sean-k-mooneywhomever?15:04
sean-k-mooneyclearly the most important question of the day15:05
stephenfinwhoever15:06
stephenfinI will give it to whomever wants it15:06
*** ociuhandu has quit IRC15:06
stephenfinyou can sub whomever with him, her, us etc.15:07
*** ociuhandu has joined #openstack-nova15:07
sean-k-mooneyimperitive vs questioning ?15:07
stephenfinWhoever wants it can have it15:07
stephenfinyou can sub whoever for he, she, we, etc.15:07
*** rpittau is now known as rpittau|afk15:07
stephenfinso if e.g. "him" would make sense, choose whomever, else choose whoever15:08
stephenfinat least I think that's how it works :)15:08
stephenfinYup, Google agrees with me https://www.grammarly.com/blog/2-effective-ways-to-deal-with-whoeverwhomever-confusion/15:08
openstackgerritArtom Lifshitz proposed openstack/nova master: pci: implement the `socket` NUMA affinity policy  https://review.opendev.org/c/openstack/nova/+/77277915:10
*** lpetrut has quit IRC15:15
gibisean-k-mooney, bauzas: I think I figured out another interesting thing. A subnet can be created without providing a segment id but also with --network-segment the user can provide a segment_id for a subnet. In the former case the segments plugin ignore the subnet and no placement and nova aggregates are created. But in the latter case the segment plugin acts on the subnet. However in both cases nova15:17
gibiconsiders the network having a a segment as the segments list command returns the segment regardless of how the subnet was created.15:17
*** ociuhandu has quit IRC15:26
*** ociuhandu has joined #openstack-nova15:26
openstackgerritArtom Lifshitz proposed openstack/nova master: pci: implement the `socket` NUMA affinity policy  https://review.opendev.org/c/openstack/nova/+/77277915:29
*** ociuhandu has quit IRC15:32
*** dklyle has joined #openstack-nova15:47
*** ociuhandu has joined #openstack-nova15:57
bauzasgibi: I'm back15:58
bauzaswrapping up on your comments15:59
*** ociuhandu has quit IRC16:00
*** ociuhandu has joined #openstack-nova16:00
bauzasgibi: what you said last doesn't look an issue to me, right?16:07
bauzasit's more the operator who messed up16:07
gibibauzas: no, both config is valid and both network is a valid network16:09
gibiI think neutron considers net3 as a non routed network while net4 as a routed network16:10
gibibut now consider both as routed and try to find the aggergate for net316:10
gibiand fails16:10
gibis/but now/but nova/16:10
bauzasgibi: you refer to http://paste.openstack.org/show/802828/ ?16:11
gibisorry I refer to https://review.opendev.org/c/openstack/nova/+/773976/10/nova/scheduler/utils.py#137016:12
*** k_mouza_ has quit IRC16:12
sean-k-mooneygibi: yes subnets dont need to have segment ids16:12
*** k_mouza has joined #openstack-nova16:13
sean-k-mooneygibi: but a network requires that all subnets either have them or none of them do16:13
sean-k-mooneygibi: we shoudl have support for both routed and non routed netowrks in teh same cloud16:13
gibisean-k-mooney: ^^ yeah this is what my example in the comment tries out16:14
sean-k-mooneyno what i mean is i pointed this out already and bauzas added a func test to make sure this works if i am not rembereing16:14
gibiso we can have one network with a segment and with subnet without that segment, and also another network with a segmetn and a subnet that has that segment16:15
sean-k-mooneygibi: we will check every network but if a netwrok does not have segment it wont break anything16:15
sean-k-mooneywe are ment to catch InvalidRoutedNetworkConfiguration in the prefilter16:15
gibisean-k-mooney: yes but if the network has a segment but the subnet does not16:15
*** ihti[m] has quit IRC16:15
bauzasgibi: and you're passing a port ?16:15
sean-k-mooneysegments are mapped to subnets16:16
sean-k-mooneynot to networks16:16
gibisean-k-mooney: but nova queries segment list16:16
bauzasthat ^16:16
gibifor networks16:16
bauzasgibi: for net3, it won't return segments, right?16:16
sean-k-mooneyno we get the segment on a network and look those up16:16
bauzashttps://review.opendev.org/c/openstack/nova/+/773976/10/nova/scheduler/utils.py#135616:16
sean-k-mooneythe network itself never has a segment16:17
gibihttps://review.opendev.org/c/openstack/nova/+/773976/10/nova/network/neutron.py#351016:17
sean-k-mooneyand all subnets in a singel networ will  eithe rhave segments or not16:17
sean-k-mooneyneutorn does not allow you to mix within a netowrk16:17
gibibauzas: for net3 the segment list will return a segment16:17
bauzasbecause of provider-segment ?16:18
gibiI guess becuase it is a provider network16:18
sean-k-mooneynet3 in the test not http://paste.openstack.org/show/802828/ right16:18
sean-k-mooneysorry need to try an pull up the same thing your looking at16:18
* bauzas is confused16:19
sean-k-mooneysame16:19
bauzasI don't see how neutron can end up creating segments by the network16:19
gibisean-k-mooney: you see in that paste that for net3 the segmetn list returns a segment16:19
sean-k-mooneywhat is net 316:19
sean-k-mooneythe first vxlan one16:19
sean-k-mooneysorry vlan16:20
sean-k-mooney 0437d333-a597-4d50-867e-d1bee7e4fc59 | None | a75a232f-c1df-4b71-9597-92acc8e83dad | vlan         |     103 |  <-- this is net316:20
gibisean-k-mooney: give me a sec, I put up a full reproduction printouit16:20
bauzasgibi: looks to me the segmentation_id on the network is a VLAN ID16:20
gmannlyarwood: ack, looks ok to me, I will merge that.16:20
bauzaswhich makes no sense on a L316:20
bauzasbut whatever16:20
sean-k-mooneygibi:  provider:segmentation_id is not the same as the segment-id16:20
bauzasmy point ^16:21
sean-k-mooney provider:segmentation_id is the vlan id16:21
bauzas\o/16:21
bauzasI'm a network expert !16:21
bauzas:D16:21
bauzasgibi: that being said, does https://review.opendev.org/c/openstack/nova/+/773976/10/nova/network/neutron.py#3510 returns you some segments for net3 ?16:22
bauzasit shouldn't16:22
sean-k-mooneygibi: so ya the neutron plugin is slightly wrong16:22
bauzasgibi: could you actually issue the CLI call against net3 ?16:22
sean-k-mooneybut it wont break us16:22
gibihttp://paste.openstack.org/show/802832/16:22
sean-k-mooneywe are doing the right thing and only looking a subnet segment_ids16:23
* bauzas clicks but has to wait, DSL here16:23
sean-k-mooneygibi: yep that is not a routed network16:23
bauzasholy fuck16:23
gibiso in that paste you see that a simple net create and subnet create result in a segment returned by the the segment list command16:23
bauzassean-k-mooney: but that's not normal16:23
bauzassegment list returns the VLAN ID as segment16:24
bauzaswhich is horribly wrong16:24
sean-k-mooneyya that proably a bug16:24
sean-k-mooneybut we are not using that api16:24
bauzaswe do16:24
sean-k-mooneyso its not an issue16:24
bauzassean-k-mooney: https://review.opendev.org/c/openstack/nova/+/773976/10/nova/network/neutron.py#351016:24
gibiyes nova does use segment list16:24
bauzaswe do use segment list16:24
sean-k-mooneyoh ... yes i see16:24
bauzasgibi: could you try hitting the API directly?16:24
sean-k-mooneywell there is a way to work around it16:25
sean-k-mooney provider:segmentation_id will always be ints16:25
gibi(interestingly the segments plugin in neutron does not relay on the segment list of the network but rely on the segment_id attribute of the subnet only)16:25
bauzasgibi: I'll show you the API call16:25
sean-k-mooneyand segment_ids will alwasy be uuids16:25
gibibauzas: I can do that16:25
bauzasgibi: this is so fucked16:25
bauzasgibi: please issue https://review.opendev.org/c/openstack/nova/+/773976/3/nova/network/neutron.py#352316:25
sean-k-mooneygibi: yes so there is a bug in neutron but we can workaround it16:25
bauzassean-k-mooney: oh surely we can workaround by uuid.isuuid()16:26
bauzasbut this is crap16:26
sean-k-mooneyactully we might be miss reading this16:26
sean-k-mooney+--------------------------------------+------+--------------------------------------+--------------+---------+16:27
sean-k-mooney| ID                                   | Name | Network                              | Network Type | Segment |16:27
sean-k-mooney+--------------------------------------+------+--------------------------------------+--------------+---------+16:27
sean-k-mooney| 92924c05-5419-4d7a-a557-dc465a10e26c | None | bcc9a2e3-3b8e-4e4b-a3f9-0072676693d0 | vlan         |     103 |16:27
sean-k-mooney+--------------------------------------+------+--------------------------------------+--------------+---------+16:27
sean-k-mooneyso 92924c05-5419-4d7a-a557-dc465a10e26c woudl be the segment_id16:27
sean-k-mooneywe set in the subnet16:27
sean-k-mooneyand 103 is the provider:segmenation_id16:27
sean-k-mooneyi think thats whats happening16:28
bauzasyeah16:28
bauzasI'm horrified16:28
bauzasthe whole semantics of a segment is borked16:28
sean-k-mooneywell kind of there was ment to be a rest api and then then decided to misuse physnets16:29
sean-k-mooneya segment in neutron terms is very closely realted to a phsynet16:29
gibihere is the raw REST API Call http://paste.openstack.org/show/802834/16:29
sean-k-mooneygibi: right but again routed networks is a misnomer16:30
sean-k-mooneyits the subnets that are routed not the networks16:30
bauzasgibi: /me cries16:30
gibiabout the possible solution; if we can also gather the segment ids via the subnets of the network then we would be in sync with the neutron's segments plugin16:30
sean-k-mooneybauzas: lets just avoid this api16:30
sean-k-mooneybauzas: instead look up the subnets for a network then look up each of the segments16:31
bauzassean-k-mooney: sure, it's not reliable and I thank gibi for catching this weird bug16:31
gibiso instead of list segments --network we can do list subnets --network and the interate the subnets and gather segment_id from each16:31
sean-k-mooneyyep16:31
bauzasyup16:31
gibisean-k-mooney: exactly16:31
bauzason it16:31
sean-k-mooneythat will work just less effecient16:31
gibiyepp16:31
sean-k-mooneyand then we can see if neutron can fix the former api16:31
bauzaswe would only leave one single method in the network.neutron API16:32
bauzasget_segments_from_subnet()16:32
sean-k-mooneyi would keep the network on and just have it delegate16:32
bauzasthis is actually error-proof as, as sean-k-mooney said, segments are always related to subnets, not networks16:32
bauzasand getting segments from the network are actually a shortcut16:33
sean-k-mooneyyep it was16:33
sean-k-mooneybut it should have been a valid one16:33
bauzassean-k-mooney: well, I prefer to stick with one single method in neutron.py and be smarter in the utils module16:33
gibiI don't want to push you into more despair but I also see isses with the actual inventory handling on neutron side. When a port is created in the routed net the segments plugin expected to chang the IP inventory in placement as the port now reserved an ip address. However for some reason when the plugin gets the port created event it sees a port without fixed ip and then decides that this port does16:33
gibinot consume IP16:33
sean-k-mooneyhttps://docs.openstack.org/api-ref/network/v2/?expanded=list-segments-detail#list-segments16:34
gibiit might be due to the fact that nova creates the port with ip_allocation=deferred16:34
sean-k-mooneybauzas: that works for me too16:34
bauzasgibi: call my president and ask him to open the skilift, and then I won't be despressed16:34
sean-k-mooneygibi: i dont think they have implement that yet have they16:35
sean-k-mooneyips in placment16:35
gibibauzas: sorry, I have no power over France16:35
gibisean-k-mooney: they did16:35
sean-k-mooneyi suggested this a few cycle ago but never saw it get done16:35
sean-k-mooneyok cool16:35
gibisean-k-mooney: it just not the good way16:35
bauzasgibi: ask your president to invade my country then16:35
gibibauzas: we are weak16:35
gibibauzas: you have nukes man16:36
sean-k-mooneygibi: its not adding an ip because its has ip-allocation=defer16:36
gibisean-k-mooney: they only implemented via inventory handling not allocation handling16:36
bauzasgibi: you were ruling Central Europe for more than 400 years, you can certainly do things16:36
sean-k-mooneyso the ip is only consumed when the port is bound16:36
gibibauzas: a that time we had horses and bows, that does not work well against nukes16:36
gibi:D16:36
bauzasback to work, focusing16:37
gibisean-k-mooney: yepp, but when the port is bound then the segment plugin should detect that and update placement16:37
sean-k-mooneyyes16:37
gibisean-k-mooney: I have to check this codepath ^^16:37
sean-k-mooneyso the inentory of ips is not accurate16:37
bauzaswait16:38
sean-k-mooneyalthough really nova should be including an ip requrest no?16:38
bauzasI'm not used to the placement resources16:38
sean-k-mooneyin the placment query16:38
sean-k-mooneyi think that is what they are expecting16:38
gibisean-k-mooney: the proper solution should be IP resource request in the port for nova16:38
gibisean-k-mooney: but that is missing16:38
bauzasIP addresses are modeled on which RPs ?16:38
sean-k-mooneygibi: yep that is what i was thinking16:38
bauzasgibi: I personnally feel that routed networks never tried to consume things16:38
sean-k-mooneybauzas: a neutron resouce provider with mis_share_via_aggreate in teh segment aggreate16:39
gibisean-k-mooney: yes, inventory handling is done in neutron and they update the reserved value on the inventory16:39
sean-k-mooneyew16:39
gibiit is not perfect :)16:39
gibiOK, it is a hack16:39
gibibut it sort of could work16:39
sean-k-mooneyya16:39
bauzasI'm lost16:39
sean-k-mooneyi think this is next cycles problem16:39
sean-k-mooneybauzas: i think we can ignore that for now16:40
gibisean-k-mooney: definetly. I don't want to do anything with the resource allocation now16:40
gibijust want to see that the segments plugin works end to end as is today16:40
sean-k-mooneybauzas: tl;dr neuton update reserved value in ip address inventoy instead of adding a resouce request but lets not fix that this cycle16:40
gibi^^ yes16:41
*** gyee has joined #openstack-nova16:41
sean-k-mooneybauzas: so for now i gues the plan is. drop the get segmens by networks funcion in neutorn.py update the utils funciton to look up the subnets then look up the segments16:42
bauzasOK, I'm then on track16:42
sean-k-mooneyand preten the neutron api and ip issue are not a thing untill next cycle16:42
bauzasyup, again, routed networks is just a scheduling feature, not a consuming feature16:43
sean-k-mooneygibi: correct me if im wrong but neturon can fix the ip thing entirely on there end right. just stop updated reserved and ask for the ip class in the port resouce request16:43
bauzasmeaning, a trick to hack the destination, not a trick to consume resources on the destination16:43
sean-k-mooneybauzas: it was ment to do both16:43
gibibauzas: correect16:43
sean-k-mooneyevenually16:43
gibifor now16:43
bauzasand I know there are those two aspects in placement, but these are separate16:43
sean-k-mooneybauzas: right but both are modeled16:44
bauzassean-k-mooney: agreed, but with placement, you can do both16:44
bauzashence the confusion16:44
bauzasand agreed, this is a pure technical limitation16:44
bauzaswe could be smarter (c)16:44
gibisean-k-mooney: about the fix in a future cycle. yes, neutron should include an IP resource request to the port for a port that was expected to consume an ip address.16:45
bauzasand hopefully, can fire the new revision before we all duck out16:45
gibisean-k-mooney: the question for me how to decide if an ip_allocation=deferred port will consume an ipv4 address or not16:45
gibiit depends on which subnet / segment the port will be bound to16:45
sean-k-mooneygibi: well it will always consume an ip but ipv4 vs v6 is tricky if it has both16:45
gibibut binding is too late, at that point everyting is already consumed in placement16:45
sean-k-mooneyfor port that are cretaed on a subnet its simper for one created on a netwrok i guess they will have to limit route netwok to one ip class?16:46
sean-k-mooneye.g. ipv4 or ipv6 only16:46
gibisean-k-mooney: yes, if there are ipv4 subnet on one segment connected to host A and an ipv6 subent on another segment connected to host B then the resource request cannot be decided before the binding16:46
sean-k-mooneygibi: perhaps the port need to have a ip version field as part of the request16:47
sean-k-mooneye.g. when you do the prot create16:47
sean-k-mooneyif you want ot ues routed networks and not specify a fix ip or subnet require an ip version to be set16:48
gibihm, but how to fill that resevation when the port is created by nova?16:48
gibiso the answer is routed nets only work with port :)16:48
sean-k-mooneyya maybe16:48
gibithis would not be the first feature that only works with precreated ports :)16:48
sean-k-mooneyi dont know but future us problem not now us problem right :)16:48
gibisure16:48
* gibi goes back looking at the segments plugin port_update codepatch16:49
sean-k-mooneytrue sriov/qos16:49
sean-k-mooneygibi: there is alwasy the long awatied "create ports in the conductor" feature16:49
sean-k-mooneyalthough that proably wont help much16:50
gibiit is awaited yes, and it won't help here much yes :)16:51
bauzassean-k-mooney: I'm tempted to directly get the segments when calling the list of subnets16:59
bauzassince the segment ID can be part of the result16:59
bauzashttps://docs.openstack.org/api-ref/network/v2/index.html?expanded=list-subnets-detail#subnets16:59
bauzasthis would prevent extra HTTP roundtrips17:00
sean-k-mooneyyes17:00
sean-k-mooneythat will work if you can filter that by network which i assume you can17:00
sean-k-mooneyya network_id is optional17:01
bauzastechnically, one is a query parameter, the other is a result field17:01
bauzasand yeah, we can filter using network_id17:01
sean-k-mooneyyep so that works17:01
bauzasso we can say something like list_networks(network_id=myid, fields=['segment_id']17:01
sean-k-mooneyyou can loop over the subnets in python with a list comprehention and produce a list of segment ids17:02
bauzasthis would give us the list of segments the network relates to17:02
bauzasthere is the fields param that does me for free :)17:02
sean-k-mooneytrue17:02
bauzasso this would actually be a rewrite of get_segment_ids_for_network()17:03
bauzasand I would keep the method17:03
bauzasat least its signature17:03
bauzastempting17:03
sean-k-mooneyso really its just GET /subnet?network_id=<id>&fileds=segment_id17:03
bauzasyup17:03
bauzasgibi: opinions on it ?17:03
sean-k-mooneymaybe keep both funciton in neutorn.py then17:04
sean-k-mooneyjust  change the url17:04
bauzasyah17:05
sean-k-mooneyok so you just need to update https://review.opendev.org/c/openstack/nova/+/773976/10/nova/network/neutron.py#351017:06
sean-k-mooneythat sound good to me17:06
sean-k-mooneygibi: that is a really good catch by the way17:06
gibi"GET /subnet?network_id=<id>&fileds=segment_id" looks good to me17:07
bauzasgibi: can you please tell me what gives you the segment_id field on a non-configured subnet ?17:08
bauzasthe field is empty or unexisting ?17:08
bauzasthe API docs is not describing it17:08
gibibauzas: trying ...17:09
bauzasthanks17:09
bauzasI really need to get some env17:09
* bauzas is ashamed by that17:09
*** sapd1_y has joined #openstack-nova17:09
sean-k-mooneyhttp://paste.openstack.org/show/802840/17:10
*** k_mouza_ has joined #openstack-nova17:11
gibibauzas: the above GET results in http://paste.openstack.org/show/802841/ if the subnet has a segment_id set17:11
sean-k-mooneyso it looks like if the extnsion is not there its an error17:11
gibibauzas: will check the no segment_id case in a sec17:11
sean-k-mooneybut we have the check for that so it should be fine17:11
*** jraju__ has joined #openstack-nova17:11
sean-k-mooneygibi: im expecting an empty list17:12
*** ralonsoh_ has joined #openstack-nova17:12
bauzassean-k-mooney: your paste probably doesn't have the extension17:12
sean-k-mooneyso proably  {"subnets":[]}17:12
sean-k-mooneybauzas: it does not17:12
sean-k-mooneybauzas:which is why i said we just need to keep the extension test17:13
sean-k-mooney*check17:13
bauzassean-k-mooney: yup, I just changed one liner17:13
bauzasie. the Neutron client call17:13
gibibauzas: and this is the case when the subnet has no segment_id set http://paste.openstack.org/show/802842/17:13
*** hemna_ has joined #openstack-nova17:13
*** obre_ has joined #openstack-nova17:13
sean-k-mooneyoh its null17:13
sean-k-mooney{"subnets":[{"segment_id":null}]}17:14
bauzasperfect17:14
gibiso you get a list of subnets and for each subnet you get a segment_id which is either null or a uuid17:14
bauzasthe client will then return None17:14
bauzasright, it's not a list17:14
openstackgerritLee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent  https://review.opendev.org/c/openstack/nova/+/72076917:14
bauzasbecause there is a 1:1 mapping segment/subnet17:14
gibiyepp17:14
sean-k-mooneywell its a dict os with a list of dicts17:14
bauzasall good, touching my UTs and I'm done17:15
sean-k-mooneyone per subnet on the network17:15
bauzasyeah, I also changed the comprehension list17:15
bauzasbut we're all good17:15
openstackgerritStephen Finucane proposed openstack/nova master: tests: Add 'nova.virt.libvirt.utils.get_arch' stub to fixture  https://review.opendev.org/c/openstack/nova/+/77490417:17
openstackgerritStephen Finucane proposed openstack/nova master: tests: Remove unnecessary mock  https://review.opendev.org/c/openstack/nova/+/77490517:17
openstackgerritStephen Finucane proposed openstack/nova master: tests: Poison os.uname  https://review.opendev.org/c/openstack/nova/+/77541517:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Parse the 'os' element from domainCapabilities  https://review.opendev.org/c/openstack/nova/+/67379017:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Stop passing around virt_type, caps  https://review.opendev.org/c/openstack/nova/+/77568917:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Add missing type hints  https://review.opendev.org/c/openstack/nova/+/77568817:17
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start parsing 'os_secure_boot'  https://review.opendev.org/c/openstack/nova/+/68262817:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Switch to libvirt's firmware auto-selection  https://review.opendev.org/c/openstack/nova/+/77569117:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Report secure boot support to scheduler  https://review.opendev.org/c/openstack/nova/+/77569017:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Add firmware auto-configuration support  https://review.opendev.org/c/openstack/nova/+/77667817:17
openstackgerritStephen Finucane proposed openstack/nova master: compute: Report COMPUTE_SECURITY_UEFI_SECURE_BOOT  https://review.opendev.org/c/openstack/nova/+/77667917:17
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Translate secure boot requests to trait  https://review.opendev.org/c/openstack/nova/+/77668017:17
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Wire up 'os_secure_boot' property  https://review.opendev.org/c/openstack/nova/+/77668117:17
openstackgerritStephen Finucane proposed openstack/nova master: tests: Add functional tests for UEFI, secure boot  https://review.opendev.org/c/openstack/nova/+/77668217:17
openstackgerritStephen Finucane proposed openstack/nova master: tests: Remove duplicated 'start_compute' helper  https://review.opendev.org/c/openstack/nova/+/77668317:17
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document UEFI secure boot feature  https://review.opendev.org/c/openstack/nova/+/77668417:17
-openstackstatus- NOTICE: All jobs are failing with POST_FAILURE due to a backward incompatible change made in the swift log upload libarary role. Working on a fix now.17:17
stephenfinwell gdi, didn't I time that well17:18
bauzassean-k-mooney: gibi: someone fancy reporting a neutron bug against the segments API ?17:18
*** k_mouza has quit IRC17:18
*** links has quit IRC17:18
*** zenkuro has quit IRC17:18
*** ralonsoh has quit IRC17:18
*** sapd1_x has quit IRC17:18
*** hemna has quit IRC17:18
*** obre has quit IRC17:18
bauzasI'm adding a fat comment explaining this is borked and ideally I should mention the bug report17:18
*** zenkuro has joined #openstack-nova17:19
sean-k-mooneybauzas: ya although there wont be a performace hit so less of an issue17:19
stephenfinAnyone want to give me a one line about what I've missed RE: some segment issue?17:20
gibibauzas, sean-k-mooney: I'm not sure I understand why returning a segment from a network is a bad thing especially as the API is created to list segments per network17:20
sean-k-mooneystephenfin: /segments?network=<id> does not give you the routed segement for the subnets on the network17:21
gibistephenfin: routed net patch had issue with collecting segment_ids from neutron17:21
*** k_mouza has joined #openstack-nova17:21
*** links has joined #openstack-nova17:21
*** 07IAAULXE has joined #openstack-nova17:21
*** ralonsoh has joined #openstack-nova17:21
*** hemna has joined #openstack-nova17:21
*** obre has joined #openstack-nova17:21
sean-k-mooneygibi: the segmentation id is ment to be admin only17:21
gibibauzas, sean-k-mooney: so I let you report a bug17:21
sean-k-mooneygibi: the provider:segmenat_id that is17:21
stephenfinhmm, so how do we get it? Can we?17:21
*** hack-char has quit IRC17:22
stephenfinI'm guessing we need to do the filtering ourselves?17:22
sean-k-mooneystephenfin: ya via the subnets endpoint17:22
stephenfinAh17:22
*** k_mouza has quit IRC17:22
*** links has quit IRC17:22
*** 07IAAULXE has quit IRC17:22
*** ralonsoh has quit IRC17:22
*** hemna has quit IRC17:22
*** obre has quit IRC17:22
openstackgerritLee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent  https://review.opendev.org/c/openstack/nova/+/72076917:22
sean-k-mooneyso /subnets?network_id=<id>&fields=segment_id17:23
sean-k-mooneystephenfin: i think we are all good bauzas is updating test now17:23
bauzasI'm 99% done17:24
bauzasjust running UTs17:24
bauzasfor regression testing17:24
bauzasall good, uploading17:24
*** hack-char has joined #openstack-nova17:24
bauzasstephenfin: that reminds me, we haven't discussed on the exception handling you'd like17:25
bauzascould you tl;dr your concerns ?N17:25
stephenfinI'd like us to go back to raising a generic NovaException-derived exception, and simply capture/transform that into something scheduler-specific inside the pre-filter17:26
*** ociuhandu_ has joined #openstack-nova17:27
bauzasack ok17:27
bauzasso, leave the exceptions be neutron-somehow specific and just catch and reraise in the prefilter, gotcha17:28
stephenfinyup, please :)17:28
bauzasI can do it17:28
bauzaswhoaw, that's now 2 mins I'm uploading the series17:28
* bauzas loves his ADSL connection17:28
bauzas800kps up17:28
bauzaskbps even17:28
bauzaswith small caps for the 'b'17:29
openstackgerritSylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments  https://review.opendev.org/c/openstack/nova/+/77397617:29
bauzaswow17:29
openstackgerritSylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter  https://review.opendev.org/c/openstack/nova/+/74906817:29
bauzasgibi: you're free to leave, it's late your time17:29
bauzashere, I'm stuck at home until 6am17:30
bauzasstephenfin: working on the FUP17:30
gibibauzas, sean-k-mooney: I will file a bug on the inventory handling as I think I understand why inventory is not updated when the deferred port is bound, there is some logic error in the code17:30
gibihere https://opendev.org/openstack/neutron/src/branch/master/neutron/services/segments/plugin.py#L446-L45017:30
*** ociuhandu has quit IRC17:30
*** ociuhandu_ has quit IRC17:31
*** bbowen has quit IRC17:31
*** efried has quit IRC17:36
*** efried has joined #openstack-nova17:36
gibibauzas, sean-k-mooney: the ip allocation bug https://bugs.launchpad.net/neutron/+bug/191627617:55
openstackLaunchpad bug 1916276 in neutron "IP address inventory handling for routed net does not work for ip_allocation=deferrred " [Undecided,New]17:55
gibisean-k-mooney: could you please file a bug for the segment listing bug, I don't feel to have the knowledge to do that17:56
-openstackstatus- NOTICE: The change to the upload role has been reverted. Jobs started since the revert appear to be functioning normally. You can recheck changes that failed for builds started between 16:12 and 17:13UTC reporting POST_FAILURE safely now.17:58
bauzasgibi: sean-k-mooney: I spotted an issues with my functest as list_subnets mock doesn't filter17:58
bauzasso I'll need to respin17:58
bauzasgibi: thanks for the bug report17:58
*** derekh has quit IRC18:01
*** andrewbonney has quit IRC18:04
sean-k-mooneyill try and do it on monday. i need to get a few things done today before i finish and its getting a little late18:06
*** dtantsur is now known as dtantsur|afk18:10
gibisean-k-mooney: no problem, and thank you18:11
gibistephenfin: does this make sense to you? https://review.opendev.org/c/openstack/nova/+/772381/5/nova/virt/libvirt/driver.py#25718:13
*** zoharm has quit IRC18:14
stephenfingibi: It does. I hadn't paid attention to the detach-specific nature of the handler18:15
gibistephenfin: do you mean the device specific nature?18:16
stephenfinYes, sorry :)18:16
gibinp it is late Friday :)18:16
gibistephenfin: OK, I can do that generalization18:16
stephenfincool :)18:16
*** k_mouza_ has quit IRC18:17
gibithanks for the suggestion :)18:17
gibibut now I leave for the weekend18:18
gibihave a nice one everyone!18:18
gibio/18:18
stephenfino/18:18
* stephenfin goes too18:18
*** mlavalle has quit IRC18:18
*** mlavalle has joined #openstack-nova18:20
bauzasI'm also done now \o18:23
bauzasjust uploading the series18:23
openstackgerritSylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments  https://review.opendev.org/c/openstack/nova/+/77397618:23
openstackgerritSylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter  https://review.opendev.org/c/openstack/nova/+/74906818:23
openstackgerritSylvain Bauza proposed openstack/nova master: FUP: Catch and reraise routed nets exception  https://review.opendev.org/c/openstack/nova/+/77669418:23
bauzasit's 7.30pm for a friday18:23
bauzaswhat a hell18:23
bauzasstephenfin: your call dude https://review.opendev.org/c/openstack/nova/+/77669418:24
* bauzas rushes off18:24
kukaczhi, I'm searching if there's a method to set (even manually) qemu's l2-cache-size on a qcow2-backed nova instance. any ideas?18:24
*** kashyap has quit IRC18:39
admin0 anyone seen this before:  deployment script fails due to it being unable to contact metadata service -- url_helper.py[DEBUG]: Calling 'http://169.254.169.254/openstack' failed [3/-1s]: request error [HTTPConnectionPool(host='169.254.169.254', port=80): Max retries exceeded with url: /openstack (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f513eef1cf8>: Failed to establish a new connection: [Errno 113] No route to18:50
admin0host .. but after its booted up, it can curl that service fine18:50
*** luksky has quit IRC19:12
*** luksky has joined #openstack-nova19:12
*** jraju__ has quit IRC19:18
*** gmann is now known as gmann_afk19:19
*** supamatt has joined #openstack-nova19:24
*** zenkuro has quit IRC19:36
*** amodi has quit IRC19:36
*** ttx has quit IRC19:36
*** johanssone has quit IRC19:36
*** ganso has quit IRC19:36
*** tobberydberg has quit IRC19:36
*** jhesketh has quit IRC19:36
*** yoctozepto has quit IRC19:36
*** sapd1_y has quit IRC19:36
*** slaweq has quit IRC19:36
*** pmannidi has quit IRC19:36
*** zimmerry has quit IRC19:36
*** mgariepy has quit IRC19:36
*** bnemec has quit IRC19:36
*** irclogbot_2 has quit IRC19:36
*** zzzeek has quit IRC19:36
*** gouthamr has quit IRC19:36
*** kevinbenton has quit IRC19:36
*** openstackgerrit has quit IRC19:36
*** adrianc has quit IRC19:36
*** gmann_afk has quit IRC19:36
*** rm_work has quit IRC19:36
*** tobias-urdin has quit IRC19:36
*** gryf has quit IRC19:36
*** efried has quit IRC19:36
*** mgoddard has quit IRC19:36
*** ircuser-1 has quit IRC19:36
*** artom has quit IRC19:36
*** adriant has quit IRC19:36
*** fyx has quit IRC19:36
*** aarents has quit IRC19:36
*** lxkong has quit IRC19:36
*** zigo has quit IRC19:36
*** stephenfin has quit IRC19:36
*** mnasiadka has quit IRC19:36
*** gary_perkins has quit IRC19:36
*** Underknowledge has quit IRC19:36
*** eandersson has quit IRC19:36
*** tbarron|out has quit IRC19:36
*** noonedeadpunk has quit IRC19:36
*** jrosser has quit IRC19:36
*** csatari has quit IRC19:36
*** cgoncalves has quit IRC19:36
*** dpawlik has quit IRC19:36
*** owalsh has quit IRC19:36
*** masayukig has quit IRC19:36
*** arne_wiebalck has quit IRC19:36
*** nicolasbock has quit IRC19:36
*** mnaser has quit IRC19:36
*** mugsie has quit IRC19:36
*** jkulik has quit IRC19:36
*** jmlowe has quit IRC19:36
*** PrinzElvis has quit IRC19:36
*** CeeMac has quit IRC19:36
*** eharney has quit IRC19:36
*** dosaboy has quit IRC19:36
*** osmanlicilegi has quit IRC19:36
*** ftarasenko has quit IRC19:36
*** StevenK has quit IRC19:36
*** melwitt has quit IRC19:36
*** Corwin has quit IRC19:36
*** dansmith has quit IRC19:36
*** valleedelisle has quit IRC19:36
*** knikolla has quit IRC19:36
*** johnsom has quit IRC19:36
*** TheJulia has quit IRC19:36
*** flaviof has quit IRC19:36
*** dviroel has quit IRC19:36
*** cz3 has quit IRC19:36
*** bbezak has quit IRC19:36
*** tinwood has quit IRC19:36
*** Hazelesque has quit IRC19:36
*** jroll has quit IRC19:36
*** mlavalle has quit IRC19:36
*** obre_ has quit IRC19:36
*** gyee has quit IRC19:36
*** swp20 has quit IRC19:36
*** martinkennelly has quit IRC19:36
*** kukacz has quit IRC19:36
*** masterpe has quit IRC19:36
*** xek has quit IRC19:36
*** logan- has quit IRC19:36
*** lyarwood has quit IRC19:36
*** f0o has quit IRC19:36
*** Jeffrey4l has quit IRC19:36
*** yankcrime has quit IRC19:36
*** ChanServ has quit IRC19:36
*** luksky has quit IRC19:36
*** hack-char has quit IRC19:36
*** hemna_ has quit IRC19:36
*** ralonsoh_ has quit IRC19:36
*** gibi has quit IRC19:36
*** sapd1 has quit IRC19:36
*** jangutter_ has quit IRC19:36
*** benj_ has quit IRC19:36
*** smcginnis has quit IRC19:36
*** kevinz has quit IRC19:36
*** trident has quit IRC19:36
*** rpittau|afk has quit IRC19:36
*** NobodyCam has quit IRC19:36
*** vdrok has quit IRC19:36
*** iurygregory has quit IRC19:36
*** yonglihe has quit IRC19:36
*** MrClayPole has quit IRC19:36
*** dasp has quit IRC19:36
*** janno_ has quit IRC19:36
*** lifeless has quit IRC19:36
*** markmcclain has quit IRC19:36
*** fnordahl has quit IRC19:36
*** dtantsur|afk has quit IRC19:36
*** raorn has quit IRC19:36
*** fungi has quit IRC19:36
*** supamatt has quit IRC19:36
*** dklyle has quit IRC19:36
*** ccstone has quit IRC19:36
*** ianw has quit IRC19:36
*** sean-k-mooney has quit IRC19:36
*** lbragstad has quit IRC19:36
*** coreycb has quit IRC19:36
*** guilhermesp has quit IRC19:36
*** bauzas has quit IRC19:36
*** tristanC has quit IRC19:36
*** DinaBelova has quit IRC19:36
*** amotoki has quit IRC19:36
*** zer0c00l has quit IRC19:36
*** mjturek has quit IRC19:36
*** atmark has quit IRC19:36
*** etp has quit IRC19:36
*** johnthetubaguy has quit IRC19:36
*** priteau has quit IRC19:36
*** xarlos has quit IRC19:36
*** dtruong has quit IRC19:36
*** pots has quit IRC19:36
*** brtknr has quit IRC19:36
*** ebbex has quit IRC19:36
*** donnyd has quit IRC19:36
*** purplerbot has quit IRC19:36
*** frickler has quit IRC19:36
*** tosky has quit IRC19:36
*** dave-mccowan has quit IRC19:36
*** sorrison has quit IRC19:36
*** artom has joined #openstack-nova19:40
*** supamatt has joined #openstack-nova19:40
*** luksky has joined #openstack-nova19:40
*** mlavalle has joined #openstack-nova19:40
*** efried has joined #openstack-nova19:40
*** hack-char has joined #openstack-nova19:40
*** zenkuro has joined #openstack-nova19:40
*** obre_ has joined #openstack-nova19:40
*** hemna_ has joined #openstack-nova19:40
*** ralonsoh_ has joined #openstack-nova19:40
*** sapd1_y has joined #openstack-nova19:40
*** gyee has joined #openstack-nova19:40
*** dklyle has joined #openstack-nova19:40
*** gibi has joined #openstack-nova19:40
*** sapd1 has joined #openstack-nova19:40
*** knikolla has joined #openstack-nova19:40
*** johnsom has joined #openstack-nova19:40
*** rpittau|afk has joined #openstack-nova19:40
*** masayukig has joined #openstack-nova19:40
*** TheJulia has joined #openstack-nova19:40
*** NobodyCam has joined #openstack-nova19:40
*** vdrok has joined #openstack-nova19:40
*** flaviof has joined #openstack-nova19:40
*** jangutter_ has joined #openstack-nova19:40
*** dviroel has joined #openstack-nova19:40
*** Underknowledge has joined #openstack-nova19:40
*** swp20 has joined #openstack-nova19:40
*** martinkennelly has joined #openstack-nova19:40
*** masterpe has joined #openstack-nova19:40
*** xek has joined #openstack-nova19:40
*** tosky has joined #openstack-nova19:40
*** amodi has joined #openstack-nova19:40
*** slaweq has joined #openstack-nova19:40
*** iurygregory has joined #openstack-nova19:40
*** benj_ has joined #openstack-nova19:40
*** yonglihe has joined #openstack-nova19:40
*** zzzeek has joined #openstack-nova19:40
*** arne_wiebalck has joined #openstack-nova19:40
*** tbarron|out has joined #openstack-nova19:40
*** xarlos has joined #openstack-nova19:40
*** dave-mccowan has joined #openstack-nova19:40
*** zimmerry has joined #openstack-nova19:40
*** priteau has joined #openstack-nova19:40
*** ccstone has joined #openstack-nova19:40
*** eandersson has joined #openstack-nova19:40
*** ircuser-1 has joined #openstack-nova19:40
*** gouthamr has joined #openstack-nova19:40
*** kevinbenton has joined #openstack-nova19:40
*** ianw has joined #openstack-nova19:40
*** nicolasbock has joined #openstack-nova19:40
*** jhesketh has joined #openstack-nova19:40
*** cz3 has joined #openstack-nova19:40
*** mnaser has joined #openstack-nova19:40
*** dtruong has joined #openstack-nova19:40
*** mugsie has joined #openstack-nova19:40
*** openstackgerrit has joined #openstack-nova19:40
*** pmannidi has joined #openstack-nova19:40
*** sean-k-mooney has joined #openstack-nova19:40
*** noonedeadpunk has joined #openstack-nova19:40
*** lxkong has joined #openstack-nova19:40
*** jkulik has joined #openstack-nova19:40
*** MrClayPole has joined #openstack-nova19:40
*** jmlowe has joined #openstack-nova19:40
*** pots has joined #openstack-nova19:40
*** adrianc has joined #openstack-nova19:40
*** sorrison has joined #openstack-nova19:40
*** ttx has joined #openstack-nova19:40
*** logan- has joined #openstack-nova19:40
*** tobberydberg has joined #openstack-nova19:40
*** ganso has joined #openstack-nova19:40
*** johanssone has joined #openstack-nova19:40
*** irclogbot_2 has joined #openstack-nova19:40
*** bnemec has joined #openstack-nova19:40
*** mgariepy has joined #openstack-nova19:40
*** gryf has joined #openstack-nova19:40
*** rm_work has joined #openstack-nova19:40
*** tobias-urdin has joined #openstack-nova19:40
*** gmann_afk has joined #openstack-nova19:40
*** lbragstad has joined #openstack-nova19:40
*** brtknr has joined #openstack-nova19:40
*** PrinzElvis has joined #openstack-nova19:40
*** CeeMac has joined #openstack-nova19:40
*** atmark has joined #openstack-nova19:40
*** eharney has joined #openstack-nova19:40
*** adriant has joined #openstack-nova19:40
*** guilhermesp has joined #openstack-nova19:40
*** coreycb has joined #openstack-nova19:40
*** ebbex has joined #openstack-nova19:40
*** jrosser has joined #openstack-nova19:40
*** fyx has joined #openstack-nova19:40
*** csatari has joined #openstack-nova19:40
*** bauzas has joined #openstack-nova19:40
*** kukacz has joined #openstack-nova19:40
*** yankcrime has joined #openstack-nova19:40
*** lyarwood has joined #openstack-nova19:40
*** dasp has joined #openstack-nova19:40
*** cgoncalves has joined #openstack-nova19:40
*** aarents has joined #openstack-nova19:40
*** mjturek has joined #openstack-nova19:40
*** zigo has joined #openstack-nova19:40
*** f0o has joined #openstack-nova19:40
*** janno_ has joined #openstack-nova19:40
*** smcginnis has joined #openstack-nova19:40
*** bbezak has joined #openstack-nova19:40
*** stephenfin has joined #openstack-nova19:40
*** Jeffrey4l has joined #openstack-nova19:40
*** dpawlik has joined #openstack-nova19:40
*** dosaboy has joined #openstack-nova19:40
*** osmanlicilegi has joined #openstack-nova19:40
*** lifeless has joined #openstack-nova19:40
*** yoctozepto has joined #openstack-nova19:40
*** mnasiadka has joined #openstack-nova19:40
*** markmcclain has joined #openstack-nova19:40
*** donnyd has joined #openstack-nova19:40
*** gary_perkins has joined #openstack-nova19:40
*** tinwood has joined #openstack-nova19:40
*** owalsh has joined #openstack-nova19:40
*** tristanC has joined #openstack-nova19:40
*** DinaBelova has joined #openstack-nova19:40
*** purplerbot has joined #openstack-nova19:40
*** ftarasenko has joined #openstack-nova19:40
*** etp has joined #openstack-nova19:40
*** johnthetubaguy has joined #openstack-nova19:40
*** raorn has joined #openstack-nova19:40
*** fungi has joined #openstack-nova19:40
*** ChanServ has joined #openstack-nova19:40
*** frickler has joined #openstack-nova19:40
*** dtantsur|afk has joined #openstack-nova19:40
*** trident has joined #openstack-nova19:40
*** tepper.freenode.net sets mode: +o ChanServ19:40
*** Hazelesque has joined #openstack-nova19:40
*** jroll has joined #openstack-nova19:40
*** dansmith has joined #openstack-nova19:40
*** valleedelisle has joined #openstack-nova19:40
*** Corwin has joined #openstack-nova19:40
*** zer0c00l has joined #openstack-nova19:40
*** kevinz has joined #openstack-nova19:40
*** amotoki has joined #openstack-nova19:40
*** fnordahl has joined #openstack-nova19:40
*** StevenK has joined #openstack-nova19:40
*** melwitt has joined #openstack-nova19:40
*** mgoddard has joined #openstack-nova19:41
*** slaweq has quit IRC19:51
*** hamalq has joined #openstack-nova19:58
*** ralonsoh_ has quit IRC20:06
*** zzzeek has quit IRC20:28
*** zzzeek has joined #openstack-nova20:30
*** rcernin has joined #openstack-nova20:32
*** zimmerry has quit IRC20:40
*** rcernin has quit IRC20:40
*** zimmerry has joined #openstack-nova20:40
*** ociuhandu has joined #openstack-nova20:41
*** gibi has quit IRC20:55
*** bbowen has joined #openstack-nova21:19
*** haleyb has quit IRC21:36
*** haleyb has joined #openstack-nova21:40
*** xek has quit IRC22:12
openstackgerritMerged openstack/nova master: libvirt: Record the machine_type of instances in system_metadata  https://review.opendev.org/c/openstack/nova/+/76753322:15
*** artom has quit IRC22:30
*** rcernin has joined #openstack-nova22:59
*** spatel has quit IRC23:16
*** gmann_afk is now known as gmann23:21
*** rcernin has quit IRC23:26
*** ociuhandu has quit IRC23:27
*** hamalq has quit IRC23:50

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