Tuesday, 2021-02-09

*** tbachman has quit IRC00:01
*** sorrison has joined #openstack-nova00:02
*** tbachman has joined #openstack-nova00:03
*** pmannidi has quit IRC00:15
*** pmannidi has joined #openstack-nova00:18
*** tosky has quit IRC00:20
*** openstackgerrit has quit IRC00:49
*** mlavalle has quit IRC00:53
*** rchurch has quit IRC00:57
*** rchurch has joined #openstack-nova00:59
*** openstackstatus has quit IRC01:20
*** openstack has joined #openstack-nova01:22
*** ChanServ sets mode: +o openstack01:22
*** pmannidi has quit IRC01:34
*** pmannidi has joined #openstack-nova01:37
*** rcernin has quit IRC02:01
*** zzzeek has quit IRC02:22
*** zzzeek has joined #openstack-nova02:23
*** rcernin has joined #openstack-nova02:25
*** rcernin has quit IRC02:30
*** k_mouza has joined #openstack-nova02:35
*** k_mouza has quit IRC02:39
*** dviroel has quit IRC02:43
*** sapd1 has quit IRC02:46
*** xarlos has quit IRC02:50
*** pmannidi has quit IRC02:54
*** jangutter_ has quit IRC02:54
*** pmannidi has joined #openstack-nova02:57
*** rcernin has joined #openstack-nova03:04
*** TheJulia has quit IRC03:26
*** PrinzElvis has quit IRC03:27
*** CeeMac has quit IRC03:27
*** TheJulia has joined #openstack-nova03:28
*** PrinzElvis has joined #openstack-nova03:29
*** CeeMac has joined #openstack-nova03:31
*** spatel has joined #openstack-nova03:33
*** k_mouza has joined #openstack-nova03:34
*** k_mouza has quit IRC03:39
*** khomesh24 has joined #openstack-nova03:43
*** rcernin has quit IRC03:50
*** pmannidi has quit IRC04:01
*** pmannidi has joined #openstack-nova04:02
*** ratailor has joined #openstack-nova04:17
*** mkrai has joined #openstack-nova04:27
*** rcernin has joined #openstack-nova04:28
*** rcernin has quit IRC04:40
*** rcernin has joined #openstack-nova04:41
*** pmannidi has quit IRC05:05
*** pmannidi has joined #openstack-nova05:08
*** whoami-rajat__ has joined #openstack-nova05:13
*** k_mouza has joined #openstack-nova05:35
*** k_mouza has quit IRC05:40
*** zenkuro has joined #openstack-nova05:44
*** khomesh24 has quit IRC06:10
*** gyee has quit IRC06:11
*** pmannidi has quit IRC06:15
*** pmannidi has joined #openstack-nova06:18
*** openstackgerrit has joined #openstack-nova06:39
openstackgerritYongli He proposed openstack/nova master: smartnic support  https://review.opendev.org/c/openstack/nova/+/75894406:39
*** vishalmanchanda has joined #openstack-nova06:56
*** mkrai has quit IRC06:57
*** spatel has quit IRC07:01
*** rcernin has quit IRC07:15
*** pmannidi has quit IRC07:25
*** pmannidi has joined #openstack-nova07:28
*** rcernin has joined #openstack-nova07:32
*** dklyle has quit IRC07:35
*** ralonsoh has joined #openstack-nova07:36
*** k_mouza has joined #openstack-nova07:37
*** k_mouza has quit IRC07:41
*** rcernin has quit IRC07:43
*** mkrai has joined #openstack-nova07:53
*** rpittau|afk is now known as rpittau07:59
gibidansmith: thanks for the help with the gerrit groups07:59
gibisean-k-mooney: thanks for the answer on the multiple fixed ip question07:59
sean-k-mooneyo/07:59
sean-k-mooneyi hope it made sense07:59
gibiyeah it made08:00
gibione port is one l2 domain even if it has multiple ips08:00
*** mkrai has quit IRC08:00
gibithe trunk port case is interesting but I agree to keep it out of scope08:00
sean-k-mooneyya unless you put a vswitch in between and do some bridging/trunking 1 port can be at most 1 l2 domain08:01
sean-k-mooneyif this was real hardar you could only plug one cable into it after all08:01
gibiyepp08:01
gibithose nasty limitation of the physical world08:02
gibilike speed of light08:02
gibiand the pauli exclusion principle08:03
*** slaweq has joined #openstack-nova08:03
sean-k-mooney:)08:09
sean-k-mooneyim +1 on the first two patches and the last 1 and have a question in the 3rd one but nothing minor08:09
sean-k-mooneyso hopefully we can get that feature done this week08:10
*** rcernin has joined #openstack-nova08:10
gibiI hope too, it was a long drag08:11
*** tesseract has joined #openstack-nova08:13
*** andrewbonney has joined #openstack-nova08:19
*** rcernin has quit IRC08:20
bauzasgibi: sean-k-mooney: hola08:23
*** xarlos has joined #openstack-nova08:23
sean-k-mooneybauzas: o/08:24
bauzasyeah, having IPs from different segments for the same port looked to me weird08:24
bauzasbut, alas.08:24
bauzasI've been teached that there are no limits to the creativity of a fooled brain, if this brain works on networking08:24
gibi:)08:25
bauzasanyway, there is a FIXME, I think we're good with this08:25
sean-k-mooneyyep08:26
bauzassean-k-mooney: you said you have a question in the patch, looking08:26
sean-k-mooneymultiple ips is fine but multiple ips from different segments not so much08:26
sean-k-mooneypatch 308:26
sean-k-mooneythe get segments by subnet function if i rememebr correctly08:26
bauzassean-k-mooney: multiple IPs is totally fine08:27
bauzasthink of v4 and v608:27
bauzasbut, indeed, from two different segments, really ?08:27
sean-k-mooneythere are cases where it is not. e.g. if they are form different segments on the same port and its not a trunk08:27
bauzasfor those cases, please refer to my first comment of the morning here08:28
sean-k-mooneyhttps://review.opendev.org/c/openstack/nova/+/773976/3/nova/network/neutron.py#354308:28
bauzasI'm totally OK with excluding those usecases at first08:28
sean-k-mooneyyep08:28
bauzasas I don't pretend to be able to solve all the world problems, including starvation and climate change08:28
bauzaseven if I appreciate people think I could act this way08:29
sean-k-mooneybasically my question on the patch is are we sure its valid  to assume that if the segment extention is enabled all networks have segments08:30
sean-k-mooneyim not sure that is correct so im not sure raisign an exception when we dont find them is correct08:30
bauzasif not, this would be an operational issue, nope ?08:30
sean-k-mooneyunless you are catching that08:30
sean-k-mooneynot nessisarly08:30
bauzashttps://docs.openstack.org/neutron/latest/admin/config-routed-networks.html08:30
bauzasthe docs says you have to segment08:31
sean-k-mooneyit would depend on if you had a mix of tenant networks and routed provider networks in teh same cloud08:31
bauzasI need coffee08:31
sean-k-mooneyin any one netwok you have to either have all subnets connect to segments or none08:31
bauzasbecause I think I'll suffer headache very soon08:31
bauzassean-k-mooney: back up a littler08:32
bauzassean-k-mooney: if you configure routed networks08:32
sean-k-mooneyif you have 2 networks 1 can be routed and the other not08:32
bauzasright08:32
bauzastechnically, you can have net1 with segments and net2 with not08:32
sean-k-mooneyyep08:32
bauzashmmm08:32
sean-k-mooneyso i dont think we should be raising excpeiton if we dont find segments or networks08:33
bauzassean-k-mooney: indeed08:33
sean-k-mooneyunless you catch it08:33
bauzasthat's a good call08:33
bauzasthat said08:33
bauzaswe only call this method on the port or network that was provided by the user08:33
bauzasbut, as this is a user, they have no context whether the net is routed or not08:34
sean-k-mooneymostly correct08:34
sean-k-mooneythey dont08:34
sean-k-mooneybut routed networks can only be created by admins08:34
bauzasI know08:34
sean-k-mooneyso the end user just knows its a shared network08:34
bauzasright08:34
sean-k-mooneyor one the admin made for them08:34
bauzasso, you can end up with indeed asking for net2 which isn't configured08:35
sean-k-mooneyif they create there own networks in addtion to that they will not be routed08:35
bauzasand in this case, we don't wanna fail08:35
sean-k-mooneyyep08:35
bauzaswe just won't require aggregates08:35
sean-k-mooneywe want to say oke we found no segments so this is not routed so all hosts are vlaid08:35
sean-k-mooneyso you could catch the exception at the top of the filter08:35
sean-k-mooneyand return true with no update to the placment query08:36
bauzasyes and no, I have the choice08:36
*** mkrai has joined #openstack-nova08:36
sean-k-mooneyyes you do08:36
bauzaseither we consider this utils method should raise an exception as this is purely for routed networks08:36
bauzas(and then, agreed, we have to catch the exception in the filter)08:37
sean-k-mooneyor it returns []/None08:37
sean-k-mooneyas if the extention was disabled08:37
bauzasor, we consider this method as general, and yeah None or an empty list sounds reasonable08:37
bauzasyeah, that's why I'm on the latter side08:37
sean-k-mooneyyep i personaly prefer avoiding excpetions08:37
sean-k-mooneysince this is not exceptional behavior08:38
sean-k-mooneyits a perfectly vaild outcome08:38
sean-k-mooneyfor what its worth this is the only really question i had left on the seriese the rest looks good to me08:38
bauzaslast call, singular/plural on the output08:38
sean-k-mooneywell for get a segment by subnet its singualar as i noted in the comment08:39
sean-k-mooneynetworks can be plural08:39
sean-k-mooneyyou can retun both as list as you do now08:39
sean-k-mooneyor you can simply the other path as i suggested08:39
bauzaswell, ok08:39
*** xek has joined #openstack-nova08:40
bauzasgibi: I won't provide a FUP, please bear with me with a second review loop when I'm done with sean-k-mooney's comments08:40
gibibauzas: ack08:41
sean-k-mooneygibi: do you have any preferecne for ^ i dont really so ill leave it to bauzas to desice between the two choiced excption/return []/None and list or no list for get segment by subnet08:42
gibino problemo08:42
bauzasno exception, I decided :)08:42
gibisean-k-mooney: I have to run, but I will read back before lunch08:42
sean-k-mooneybauzas: works for me08:42
gibibauzas: as you see fit :)08:42
sean-k-mooneybauzas: if you are removing the excpetion you might want to make the LOG.error a LOG.debug08:43
sean-k-mooneyanyway cool im going to get coffee08:44
*** khomesh24 has joined #openstack-nova08:48
bauzassean-k-mooney: quick call, I'm touching the get_network too but what gives you /segments?network_id=net1 if net1 isn't configured ?08:55
bauzasI suppose an empty list ?08:55
bauzascontext https://review.opendev.org/c/openstack/nova/+/773976/3/nova/network/neutron.py#352308:56
bauzasI don't have a routed networks env at hand08:57
*** ralonsoh has quit IRC09:02
sean-k-mooneysure we can have a call just need to get my headset09:03
sean-k-mooneyoh your asking what is the result09:03
bauzasyup09:04
sean-k-mooneyi belive its an empty list09:04
bauzasI assume we serve a 200 with an empty list09:04
bauzashttps://docs.openstack.org/api-ref/network/v2/index.html?expanded=list-subnets-detail,show-port-details-detail,show-subnet-details-detail,list-segments-detail#list-segments09:04
bauzasit's just a filtering09:05
bauzasfiltering*09:05
bauzasanyway, will upload and we'll see09:05
sean-k-mooneythe only error code it can return is 401 which is unautrised09:05
sean-k-mooneyso ya 200 with empty list09:05
bauzas(saying this, because I'll stick with LOG.error on something else but a 200)09:06
bauzason the neutron method09:06
sean-k-mooneyhttps://docs.openstack.org/api-ref/network/v2/index.html?expanded=list-segments-detail#list-segments09:06
sean-k-mooneylooking at the doc its either a 200 or 401 and a 401 would not make sense here09:06
bauzassure09:07
bauzaseither way, will upload the patch once I'm done with fixing tests09:07
sean-k-mooneyyep its a list https://github.com/openstack/neutron/blob/master/neutron/services/segments/db.py#L167-L16909:10
*** tosky has joined #openstack-nova09:10
*** pmannidi has quit IRC09:13
*** ociuhandu has joined #openstack-nova09:15
*** pmannidi has joined #openstack-nova09:16
*** martinkennelly has joined #openstack-nova09:17
*** ociuhandu has quit IRC09:20
kashyapIs it just me, or the (relatively) new Gerrit is much slower?09:22
kashyapI've noticed it over the last couple of weeks -- when I hit "Send" on a comment, it takes more than 4-5 seconds to commit09:23
kashyaps/commit/submit/09:23
sean-k-mooneyits slower09:24
sean-k-mooneyits to do with storing the data in git apparently09:24
sean-k-mooneyits proably more the case with nova then other repos09:25
sean-k-mooneysince nova git repo is much bigger09:25
*** derekh has joined #openstack-nova09:26
*** zenkuro has quit IRC09:33
*** zzzeek has quit IRC09:33
*** zenkuro has joined #openstack-nova09:33
*** zzzeek has joined #openstack-nova09:34
*** songwenping_ has joined #openstack-nova09:37
*** swp20 has quit IRC09:40
sean-k-mooneywe dont have the pci whitelist tags stored in the nova db... only in memory in the compute node09:40
kashyapsean-k-mooney: I see, noted09:42
sean-k-mooneyi tought they were stored in extra_info ? did i imaging that?09:42
*** dtantsur|afk is now known as dtantsur09:45
openstackgerritSylvain Bauza proposed openstack/nova master: Add net and utils methods for getting routed netks and segments  https://review.opendev.org/c/openstack/nova/+/77397609:48
openstackgerritSylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter  https://review.opendev.org/c/openstack/nova/+/74906809:48
songwenping_sean-k-mooney: morning, i have manually set the [vnc]/auth_schema=["vnc"] and created a vm with a console without a password and it works in our product, but i have no idea about the tempest to test, would you give some specific guide, please?09:50
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
bauzasgibi: uploaded the last revision, with just a cosmetic change on the fixtures (using the uuid sentinelĂ  + setattr usage instead of copy/pasting 5 times compute create09:51
sean-k-mooneysongwenping_: if you can confirm it works manually i think that is fine for now09:54
sean-k-mooneysongwenping_: for the tempest test we woudl just set it in the zuul job we can do that as a follow up change09:55
songwenping_cool.09:55
sean-k-mooneyonce the irc logs catch up ill link to this in a comment and change too a +1 once i re review09:55
sean-k-mooneyi think you had areassed my other comments however09:55
songwenping_thanks :)09:56
sean-k-mooneybauzas: you didn tfix the exception in the case of networks09:57
sean-k-mooneybauzas: https://review.opendev.org/c/openstack/nova/+/773976/5/nova/network/neutron.py#351009:57
sean-k-mooneyoh actully09:57
sean-k-mooneywill if resp: be true or false09:58
sean-k-mooneyi guess it will eveauate to true09:58
sean-k-mooneyeven if it gets a 200 with an empty list09:58
sean-k-mooneyso ya that should be fine09:58
sean-k-mooneybauzas: ah you have a test for it too cool https://review.opendev.org/c/openstack/nova/+/773976/5/nova/tests/unit/network/test_neutron.py#629309:59
bauzassean-k-mooney: yup10:02
bauzaslike I said earlier, I left the exception raise for networks10:02
bauzasas this would be an unknown reason10:02
sean-k-mooneybauzas: ya i got confused because i needed to look at v3 vs v510:02
bauzasnp10:02
sean-k-mooneyim not a huge fan of setattr but im +1 on both of those changes10:09
stephenfinanother "Multiple possible networks found" gate failure :-(10:10
stephenfinwill this compact DB migration series ever land10:10
sean-k-mooneyright before we swap to etcd :P10:11
sean-k-mooneystephenfin: on a side note the pci filter is massivly less efficent then i tought it was. if i fix that is it a bug or a feature10:12
stephenfindepends on the inefficiency10:12
stephenfinand impact on end users10:12
sean-k-mooneywe dont store the physnet or other tags in the db just in memroy. i tought they were in the extra info column10:12
sean-k-mooneybut no10:12
sean-k-mooneyso when we are doing neutron sriov the filter does not know fi the VF is a nic, gpu, qat device10:13
sean-k-mooneyit just know its a vf10:13
sean-k-mooneywe only check that on the compute node10:13
sean-k-mooneyi could jut stick that in the extra info json blob and then we could check it in the filter10:14
sean-k-mooneyor we coudl track them in plamcnet next cycle but i kind of want to fix this for everyone not running master10:16
stephenfinI'd say push the patch and let's see in the review what everyone else thinks10:17
sean-k-mooneyya ok. not today though i was checking the db for my neutron port numa poicy seriese10:18
sean-k-mooneyjust noticed this wehn i was looking at the pci devices table10:18
sean-k-mooneythe pci request has {"count": 1, "spec": [{"physical_network":"public"}] we have nothing to match that against in the pci device table so really that is all that is missing10:24
*** ratailor_ has joined #openstack-nova10:25
gibibauzas: ack, I will check10:28
*** ratailor has quit IRC10:28
*** pmannidi has quit IRC10:34
*** pmannidi has joined #openstack-nova10:38
sean-k-mooneyyou know when you see something odd in nova and then you investiate and all you find its a world full of sad pandas10:46
sean-k-mooneythat is what the test coverage for this piece of the pci tracker is like10:47
sean-k-mooneyhttps://github.com/openstack/nova/blob/4827a90a02ef91e777b1fcaa946c20c54dd0a29a/nova/tests/unit/pci/test_stats.py#L164-L16810:47
lyarwoodSad Pandas, all the way down.10:48
stephenfingibi: It is not at all a priority, but while it's fresh in your mind I've addressed all but one of your comments on https://review.opendev.org/c/openstack/nova/+/676209 and am researching the final one now10:49
sean-k-mooneylyarwood: indeed10:49
*** mkrai has quit IRC10:52
*** dviroel has joined #openstack-nova10:59
*** xek has quit IRC11:06
*** coreycb has quit IRC11:10
*** guilhermesp has quit IRC11:10
*** nicolasbock has quit IRC11:11
*** masayukig has quit IRC11:11
*** cz3 has quit IRC11:11
*** masayukig has joined #openstack-nova11:11
*** coreycb has joined #openstack-nova11:11
*** nicolasbock has joined #openstack-nova11:11
*** guilhermesp has joined #openstack-nova11:11
*** cz3 has joined #openstack-nova11:11
gibistephenfin: ack, will check this afternoon11:12
* bauzas wonders why english folks use pandas for telling they're sad11:14
bauzasfor us, we say "I'm a sad clown"11:15
*** zzzeek has quit IRC11:19
*** zzzeek has joined #openstack-nova11:19
*** Luzi has joined #openstack-nova11:23
*** k_mouza has joined #openstack-nova11:25
sean-k-mooneybauzas: because pandas are cute so when they are sad its really bad. also its a meme11:29
sean-k-mooneybauzas: sad panda https://i.kym-cdn.com/photos/images/newsfeed/000/092/706/Sad_Panda_Chibi_by_mongrelssister.png11:38
sean-k-mooneyhappy panda https://i.pinimg.com/originals/99/2a/35/992a358dcff70ca9bfcb4b227c08333c.png11:39
sean-k-mooneynow back to unit test land...11:39
lyarwoodbauzas: also helpful reminder, sean-k-mooney isn't English, stephenfin is but that's another story. ./me runs11:42
*** tbachman has quit IRC11:42
sean-k-mooneyhaha11:42
*** tbachman has joined #openstack-nova11:42
gibilol11:44
gibiI feel there is an intersting story behind this11:45
sean-k-mooneygibi: stephenfin lived in essex for a bit when he was young before moving back to live in clare11:49
lyarwoodand loves being called English11:49
lyarwood /s11:49
gibi:D11:49
sean-k-mooneygibi: but calling an irish person english is like calling lyarwood welsh11:49
* lyarwood ragequits11:50
lyarwoodwell tbh it's likely worse given recent history11:50
*** pmannidi has quit IRC11:50
lyarwoodanyway11:50
* lyarwood gets back to docs11:50
*** pmannidi has joined #openstack-nova11:52
*** zul has quit IRC11:54
gibibauzas: I'm +2 on the route net series11:59
*** whoami-rajat__ has quit IRC12:08
sean-k-mooneystephenfin: ok so the tags are not store in the pci_devices table but they are in the PciDevicePoolList in teh compute nodes table that is used to populate the host_state object12:08
*** andrewbonney has quit IRC12:08
*** masayukig has quit IRC12:08
*** coreycb has quit IRC12:08
*** guilhermesp has quit IRC12:08
*** nicolasbock has quit IRC12:09
*** vishalmanchanda has quit IRC12:09
*** cz3 has quit IRC12:09
sean-k-mooneystephenfin: so its not a performcne issue12:09
sean-k-mooneyi was just expecting it to be in both12:09
*** dviroel has quit IRC12:09
*** whoami-rajat__ has joined #openstack-nova12:10
*** andrewbonney has joined #openstack-nova12:10
*** dviroel has joined #openstack-nova12:10
*** masayukig has joined #openstack-nova12:10
*** coreycb has joined #openstack-nova12:10
*** guilhermesp has joined #openstack-nova12:10
*** vishalmanchanda has joined #openstack-nova12:10
*** nicolasbock has joined #openstack-nova12:11
*** cz3 has joined #openstack-nova12:12
kashyaplyarwood: Reading back ... I parsed bauzas' "english speaking" as "anglosphere"12:12
* kashyap gets back to the kitchen to make a meal, before he falls off the chair12:13
lyarwoodthank you captain obvious ;)12:14
sean-k-mooneykashyap: so did we :) but there was an opertunity for a joke to be made12:15
lyarwoodand as a new dad I need the practice12:16
*** xek has joined #openstack-nova12:17
kashyapsean-k-mooney: Hehe, figured as much; don't want to rob you of a good tease12:17
sean-k-mooneyi dont know you were fairly on point. maybe it was too good and you need to recalibrate to dad joke levels12:18
kashyapHeh12:19
kashyapsean-k-mooney: Do you ever sleep, BTW?  ... I almost see your scrollback at all hours.  Unless my IRC client is tricking me12:21
sean-k-mooneyi didnt sleep much last night. went to sleep aroudn 2-3 work at about half 5 got up at 8 and started working around 8:30.12:25
sean-k-mooneyso ill be finishing early today12:25
*** rcernin has joined #openstack-nova12:27
*** rcernin has quit IRC12:31
*** ratailor_ has quit IRC12:40
gibikashyap: I left feedback in https://review.opendev.org/c/openstack/nova/+/774240/12:41
kashyapsean-k-mooney: Ouch; get some good rest12:45
kashyapgibi: Will look after lunch; thank you12:45
*** sorrison has quit IRC12:48
*** sorrison has joined #openstack-nova12:49
*** whoami-rajat__ has quit IRC12:57
*** CeeMac has quit IRC12:58
*** CeeMac has joined #openstack-nova13:01
*** iurygregory has quit IRC13:04
*** pmannidi has quit IRC13:06
*** iurygregory has joined #openstack-nova13:06
*** whoami-rajat__ has joined #openstack-nova13:07
*** pmannidi has joined #openstack-nova13:09
kashyapgibi: In your review, which test job you quote that unit test failure from?13:16
gibibauzas: could you hit https://review.opendev.org/c/openstack/nova/+/756530 at some point? that is the last real change in the qos changes in Wallaby13:16
gibikashyap: tox -e py3813:16
gibikashyap: to get the debug log you need to define OS_DEBUG=1 in your env and make the test fail13:16
kashyapgibi: Ah, thanks!13:20
sean-k-mooneyor just do "tox -e py3"13:24
sean-k-mooneythen you dont need to care13:24
sean-k-mooneyit will use your systems default python13:24
sean-k-mooneykashyap: just be aware that OS_DEBUG at least for functional test will be very very verbose when it fails13:25
kashyapsean-k-mooney: Noted :)  Just unit tests13:25
*** ociuhandu has joined #openstack-nova13:28
kashyapsean-k-mooney: Have you got a latest minimal DevStack (without Cinder)?13:29
kashyapsean-k-mooney: I have this ... can this be cutdown any further? - https://paste.centos.org/view/c9c4ecdf13:29
bauzasgibi: I was thinking about it, LOL.13:32
bauzasit's my next todo13:32
gibibauzas: thanks13:32
*** ociuhandu has quit IRC13:32
sean-k-mooneykashyap: its not a minimal config but this is the bases i work off https://paste.centos.org/view/c57cf6a413:34
sean-k-mooneykashyap: you should not need FORCE=yes or the neutron settings your adding13:35
sean-k-mooneyunless its conflicting with your home/work network13:35
sean-k-mooneykashyap: you also done need the data dirs stuff but you are using a non default location so not minimal but i guess its what you need13:37
bauzasgibi: no spec for the attach/detach case, right? https://review.opendev.org/c/openstack/nova/+/756530/21//COMMIT_MSG13:37
bauzasI was thinking about the workflow, do we have other resources for attach/detach ?13:37
gibibauzas: there is spec for the qos series13:38
kashyapsean-k-mooney: Reading ...13:38
bauzasI know13:38
bauzashttps://review.opendev.org/c/openstack/nova-specs/+/75547713:38
kashyapsean-k-mooney: The data dirs stuff -- that is intentional; as I want it to be in a non-default location13:38
bauzasokay okay, will lookup13:38
kashyapsean-k-mooney: Thanks!13:38
gibibauzas: then I don't get your question13:38
bauzasgibi: I didn't remember whether we discussed about the workflow13:39
bauzasbut yeah we did13:39
gibibauzas: there is only neutron resource request and the PCI request it handled for interface attach13:39
*** Luzi has quit IRC13:43
*** k_mouza has quit IRC13:47
*** k_mouza_ has joined #openstack-nova13:47
*** k_mouza_ has quit IRC13:49
*** k_mouza has joined #openstack-nova13:49
*** mlavalle has joined #openstack-nova13:58
*** spatel has joined #openstack-nova13:59
*** nweinber has joined #openstack-nova14:03
*** khomesh24 has quit IRC14:09
*** pmannidi has quit IRC14:13
*** khomesh24 has joined #openstack-nova14:14
*** pmannidi has joined #openstack-nova14:18
stephenfinlyarwood: You've done the next patch in the series but any chance you could hit this? https://review.opendev.org/c/openstack/nova/+/741663/14:43
*** stand has joined #openstack-nova14:51
lyarwoodops my bad, sure14:52
openstackgerritStephen Finucane proposed openstack/placement master: policy: Deprecate 'admin_api' rule  https://review.opendev.org/c/openstack/placement/+/77233414:53
openstackgerritStephen Finucane proposed openstack/placement master: policy: Add note about keystone's expansion of roles  https://review.opendev.org/c/openstack/placement/+/77275214:53
openstackgerritStephen Finucane proposed openstack/placement master: Move policy deprecation to base rules  https://review.opendev.org/c/openstack/placement/+/77278414:53
openstackgerritStephen Finucane proposed openstack/placement master: policy: Add releasenote for RBAC work  https://review.opendev.org/c/openstack/placement/+/77465414:53
stephenfingmann: ^14:53
bauzasgibi: had an open question with some nits too https://review.opendev.org/c/openstack/nova/+/75653014:53
stephenfinrelease note fresh out of the oven14:53
bauzastl;dr: should we delete the QoS allocations on interface detach ?14:53
gmannstephenfin: ack. mostly rebase right14:54
stephenfinyeah, I just reapplied +W14:54
stephenfinthe releasenote is the only real change14:54
gmannok14:56
gmannstephenfin: lgtm 1 comment on adding link to keystone doc https://review.opendev.org/c/openstack/placement/+/77465415:05
lyarwooddoes anyone know if there's an easy way to list instances by cell via an existing cli?15:07
lyarwoodcontext being a nova-status check I've added lists cell uuids and not instance uuids which isn't that useful15:07
lyarwoodbefore I go and write something to list instances by cell uuid or instances that fail my check I wanted to ensure something didn't already exist15:08
lyarwoodI can't see anything obvious in osc15:08
openstackgerritStephen Finucane proposed openstack/placement master: policy: Add releasenote for RBAC work  https://review.opendev.org/c/openstack/placement/+/77465415:08
stephenfingmann: done ^15:08
lyarwoodor novaclient15:08
stephenfinlyarwood: A quick search through api-ref suggest we don't expose anything about cells v2 to the REST API15:09
lyarwoodstephenfin: gah, well I guess I'm writing a nova-manage command then15:10
lyarwoodor rewriting this status check to list instance uuids15:10
lyarwoodbut given the table output that's going to look awful15:10
*** pmannidi has quit IRC15:24
*** pmannidi has joined #openstack-nova15:26
*** hack-char has quit IRC15:29
*** hack-char has joined #openstack-nova15:29
kashyapsean-k-mooney: That 'DEVSTACK_PARALLEL' thing in your config - is that merged, or were you testing something from Git?15:36
sean-k-mooneyoh no thats not merged yet15:36
sean-k-mooneyi was testing dans patch15:36
sean-k-mooneykashyap: https://review.opendev.org/c/openstack/devstack/+/77150515:37
sean-k-mooneyit works pretty well15:37
sean-k-mooneytested it on ubuntu and fedora15:37
*** dklyle has joined #openstack-nova15:38
kashyapsean-k-mooney: Thanks; also, remind me again, 'g-reg' can be avoided right?15:38
*** khomesh24 has quit IRC15:41
*** k_mouza_ has joined #openstack-nova15:43
*** k_mouza has quit IRC15:47
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Add support for microversion v2.88  https://review.opendev.org/c/openstack/python-novaclient/+/77057315:47
sean-k-mooneykashyap: yes15:53
sean-k-mooneydo i still have it in min15:53
sean-k-mooneyyou just need thet glance api now i think15:53
*** spatel has quit IRC15:59
*** macz_ has joined #openstack-nova16:02
*** macz_ has quit IRC16:02
*** macz_ has joined #openstack-nova16:03
kashyapsean-k-mooney: You don't, though.  I just kept in as a old habit16:03
kashyapThx16:03
kashyaplyarwood: Aside - have you tried DevStack on F33 yet?16:03
openstackgerritStephen Finucane proposed openstack/os-vif master: Resolve dependency issues  https://review.opendev.org/c/openstack/os-vif/+/77469416:06
stephenfinsean-k-mooney: ^16:06
stephenfinas promised16:06
stephenfinit sounds like the general consensus has been to remove these lower-constraint jobs entirely though. They're gone in most of oslo land now as well as neutron16:07
*** hemna has quit IRC16:07
lyarwoodkashyap: I have but couldn't get it to work16:08
lyarwoodkashyap: even when forcing py3816:08
*** hemna has joined #openstack-nova16:08
kashyaplyarwood: Okay ... thought so; F32?16:08
lyarwoodkashyap: iirc virtualenv tries to use py39 in places and that breaks16:08
* kashyap goes with F3216:08
lyarwoodkashyap: yeah f32 should work16:08
kashyapThx16:09
kashyapsean-k-mooney: On your word, I'm trying the _PARALLEL thing ... hope it won't turn into a yak :D16:10
kashyapI'll report back my result, if it works16:10
sean-k-mooneystephenfin: lets try keeping it for this cycle16:10
sean-k-mooneystephenfin: if it break agian before the end of the cycle we can drop it16:11
sean-k-mooneystephenfin: our lower constratis job is a littel different since we actully run our unit tests and docs16:12
stephenfinlyarwood: Is there any way to map a migration request (via '/servers/{server_id}/action (migrate)') to a migration record ('/servers/{server_id}/migrations')16:14
stephenfinI can't see any. It's weird that the former doesn't return an ID in the success case16:14
*** spatel has joined #openstack-nova16:14
sean-k-mooneystephenfin: tthe request id might be usable if you are looking at logs16:15
stephenfinI'm thinking of OSC16:15
stephenfinhttps://review.opendev.org/c/openstack/osc-lib/+/77387716:15
stephenfinRather than polling the server status, I'd like to map the migration request to the record as above and check the status of that16:16
stephenfinbut there doesn't appear to be any way to do that :-\16:16
sean-k-mooney/servers/{server_id}/migrations16:18
sean-k-mooneyso you can list the migrates for a server16:18
sean-k-mooneyand get the last one16:19
stephenfinYeah, I was hoping to avoid that16:19
sean-k-mooneymaybe via the server events?16:19
stephenfinHmm, that could work16:19
stephenfinI think the issue is that the Migration object isn't created until we land on a compute noe16:20
stephenfin*node16:20
stephenfinand we use a cast so can't wait for that16:20
stephenfinsounds like a potential usability RFE for Xena16:20
* lyarwood reads back16:20
lyarwoodyeah I did wonder why we didn't list the request-id in https://docs.openstack.org/api-ref/compute/?expanded=show-migration-details-detail#show-migration-details tbh16:22
lyarwoodso you could then lookup by request-id or something16:22
dansmithstephenfin: we didn't originally expose any details of migrations to users other than a resize I think, which is why we don't have a lot of stuff baked around making that trackable by a user16:22
lyarwooddefinitely something to improve in X16:22
sean-k-mooneywell the issue is that we return nothing for the action16:23
lyarwoodwe should always have the request-id16:23
sean-k-mooneylyarwood: sure but we dont save request ids normally16:23
dansmithstephenfin: there's also the migration context on the instance, so we can stash things like that there before we create the migration if that helps16:23
dansmithlike that being the req-id16:23
sean-k-mooneydansmith: ya we do but there is not realy a good way to acess that form the api16:24
lyarwoodwell that's my point sean-k-mooney, stashing it in the migration record16:24
dansmithsean-k-mooney: no, I mean as support for making the connection possible between the migration request and the actual migration16:24
sean-k-mooneywe coudl certenly improt the api respocne to make that eaiser ya16:26
gibibauzas: thanks for the review, I will reply hopefully tomorrow16:26
bauzasgibi: not sure about my question if it's a problem, honestly16:27
bauzasgibi: wdyt about this ?16:27
sean-k-mooneydansmith: just looking at what currently avaiable however its non trivial for osc to figure out in a way that will work with existing clouds16:27
*** rcernin has joined #openstack-nova16:27
dansmithyes, clearly16:27
gibibauzas: the detach case has already implemented couple of cycles ago16:28
gibilet me dig out the lunk16:28
gibilink16:28
dansmithI was just saying I think the lack of obvious linkage is because we haven't always exposed much, and was suggesting to stephenfin that the migration context could likely be used to stash things before we've created the migration record if indeed that happens late as he says (I don't really remember16:28
*** k_mouza_ has quit IRC16:29
gibibauzas: the detach has functional test coverage already on master https://github.com/openstack/nova/blob/941f4624638090b27e9e18601cde0529fcdbd22f/nova/tests/functional/test_servers.py#L613416:29
*** k_mouza has joined #openstack-nova16:29
gibibauzas: it was implemented in https://review.opendev.org/c/openstack/nova/+/622421 in stein16:30
bauzasgibi: ah ok16:31
bauzasgibi: all good then, just answer my nits16:31
gibibauzas: will do16:31
*** rcernin has quit IRC16:32
* bauzas needs to go off for supermarket duties, curfew in 30 mins :(16:32
* bauzas actually rushes16:32
*** sapd1 has joined #openstack-nova16:36
*** sapd1 has quit IRC16:43
*** pmannidi has quit IRC16:48
kashyapmelwitt: Hi16:49
kashyapmelwitt: I'm talking to libvirt block layer dev on that bug you filed16:50
kashyapmelwitt: Do you have a link to the libvirtd.log from where you posted this snippet? -- https://launchpadlibrarian.net/521114497/bug1914282_log_excerpts.txt16:50
*** pmannidi has joined #openstack-nova16:50
kashyapmelwitt: In the bug itself you posted the "ready": true bit ... but right after that we should see for an event called BLOCK_JOB_COMPLETED16:50
kashyapmelwitt: Ah, ignore me; it's the 'controller' dir16:52
*** slaweq is now known as slaweq|away16:56
*** spatel has quit IRC17:07
*** spatel has joined #openstack-nova17:08
*** LinPeiWen has quit IRC17:17
*** gyee has joined #openstack-nova17:20
*** tesseract has quit IRC17:28
openstackgerritStephen Finucane proposed openstack/nova master: Drop support for custom schedulers  https://review.opendev.org/c/openstack/nova/+/77363717:29
openstackgerritStephen Finucane proposed openstack/nova master: conf: Drop '[scheduler] periodic_task_interval'  https://review.opendev.org/c/openstack/nova/+/77363817:29
openstackgerritStephen Finucane proposed openstack/nova master: conf: Clean up docs for scheduler options  https://review.opendev.org/c/openstack/nova/+/77363917:30
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Remove 'USES_ALLOCATION_CANDIDATES'  https://review.opendev.org/c/openstack/nova/+/77364017:30
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Remove 'hosts_up'  https://review.opendev.org/c/openstack/nova/+/77364117:30
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove FakeScheduler (for realz)  https://review.opendev.org/c/openstack/nova/+/77364217:30
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Merge 'FilterScheduler' into base class  https://review.opendev.org/c/openstack/nova/+/77364317:30
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Merge driver into manager  https://review.opendev.org/c/openstack/nova/+/77364417:30
openstackgerritStephen Finucane proposed openstack/nova master: docs: Drop references to non-filter scheduler drivers  https://review.opendev.org/c/openstack/nova/+/77364517:30
openstackgerritStephen Finucane proposed openstack/nova master: tests: Merge 'test_utils', 'test_scheduler_utils'  https://review.opendev.org/c/openstack/nova/+/77364617:30
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove deprecated aliases  https://review.opendev.org/c/openstack/nova/+/77364717:30
*** k_mouza_ has joined #openstack-nova17:41
*** k_mouza has quit IRC17:44
*** dtantsur is now known as dtantsur|afk17:50
*** nweinber has quit IRC17:55
*** nweinber has joined #openstack-nova17:55
*** pmannidi has quit IRC17:56
*** rpittau is now known as rpittau|afk17:56
*** pmannidi has joined #openstack-nova17:58
*** derekh has quit IRC18:04
*** xek has quit IRC18:14
*** k_mouza_ has quit IRC18:20
openstackgerritMerged openstack/nova master: db: Compact Pike database migrations  https://review.opendev.org/c/openstack/nova/+/75908718:34
openstackgerritStephen Finucane proposed openstack/nova master: Use neutronclient's port binding APIs  https://review.opendev.org/c/openstack/nova/+/70629518:35
*** zenkuro has quit IRC18:49
*** andrewbonney has quit IRC18:52
*** zenkuro has joined #openstack-nova19:02
*** pmannidi has quit IRC19:12
*** pmannidi has joined #openstack-nova19:16
*** macz_ has quit IRC19:39
*** macz_ has joined #openstack-nova19:40
*** pmannidi has quit IRC20:21
*** pmannidi has joined #openstack-nova20:23
*** dklyle has quit IRC20:39
*** dklyle has joined #openstack-nova20:39
artomCI jobs for https://review.opendev.org/c/openstack/nova/+/772779/6 seem to not have picked up the os-traits dependency at https://review.opendev.org/c/openstack/os-traits/+/77170520:52
artomAnd I don't understand what's going on...20:52
artomIf I `pip install -e .` my os-traits patch into the tox venv locally, functional tests pass...20:54
lyarwoodouch, did you not get the trait in before the lib freeze?21:02
*** whoami-rajat__ has quit IRC21:03
lyarwoodah wait that isn't for a while21:03
lyarwoodwhy did I think that was at M221:03
artomLib freeze is normally later, no?21:03
artomAh, it's the week before M3 actually21:04
artomShould un-WIP it then21:06
artomThough the CI thing is still baffling21:06
artomWhere do I even start lookingÉ21:06
artom?21:06
lyarwoodyeah iirc we don't pull libs into the tox envs for func and unit so you need an actual release to start using them there21:06
lyarwooddepends-on libs that is21:06
openstackgerritArtom Lifshitz proposed openstack/os-traits master: Add COMPUTE_SOCKET_PCI_NUMA_AFFINITY trait  https://review.opendev.org/c/openstack/os-traits/+/77170521:06
* lyarwood really needs to push his ephemeral encryption stuff along this week21:07
lyarwoodjust need to finish the machine type stuff first that I've had to rewrite most of today after finding the hyperv driver uses hw_machine_type as well /o\21:07
artomlyarwood, release or just landed in master?21:07
lyarwoodartom: release and bump requirements I think21:07
* lyarwood thinks that had to happen for his stable rescue stuff in the past21:08
openstackgerritArtom Lifshitz proposed openstack/nova master: libvirt: start tracking NUMACell.socket for hosts  https://review.opendev.org/c/openstack/nova/+/76681621:10
openstackgerritArtom Lifshitz proposed openstack/nova master: extra specs/image props: add `socket` PCI NUMA affinity  https://review.opendev.org/c/openstack/nova/+/77274821:10
openstackgerritArtom Lifshitz proposed openstack/nova master: Add `socket` PCI NUMA affinity policy request prefilter  https://review.opendev.org/c/openstack/nova/+/77274921:10
openstackgerritArtom Lifshitz proposed openstack/nova master: Track host NUMA topology in PCI manager  https://review.opendev.org/c/openstack/nova/+/77414921:10
openstackgerritArtom Lifshitz proposed openstack/nova master: WIP: pci: implement the `socket` NUMA affinity policy  https://review.opendev.org/c/openstack/nova/+/77277921:10
* artom un-WIPs what he can21:10
*** pmannidi has quit IRC21:31
*** pmannidi has joined #openstack-nova21:34
gmanngibi: melwitt stephenfin multiple network fix is merged, you can recheck failure if there is any https://review.opendev.org/c/openstack/tempest/+/77016921:34
gmannlet's see if we see any other test failing for that error.21:34
lyarwood\o/21:36
*** zenkuro has quit IRC21:44
openstackgerritGhanshyam proposed openstack/nova master: DNM:try l-c with direct deps  https://review.opendev.org/c/openstack/nova/+/77278021:46
*** rcernin has joined #openstack-nova21:50
melwittthanks gmann22:09
*** gmann is now known as gmann_afk22:13
*** pmannidi has quit IRC22:45
*** pmannidi has joined #openstack-nova22:50
*** spatel has quit IRC22:50
*** spatel has joined #openstack-nova23:02
openstackgerritMerged openstack/nova master: Add requested_networks field to RequestSpec object  https://review.opendev.org/c/openstack/nova/+/74997723:12
*** vishalmanchanda has quit IRC23:15
*** pmannidi_ has joined #openstack-nova23:16
*** pmannid__ has joined #openstack-nova23:18
*** pmannidi has quit IRC23:19
*** pmannidi_ has quit IRC23:20
*** pmannid__ has quit IRC23:22
*** pmannidi has joined #openstack-nova23:24
*** tosky has quit IRC23:48

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