Friday, 2019-06-21

« Thursday, 2019-06-20 Index Saturday, 2019-06-22 »
*** markvoelker has joined #openstack-nova00:40
openstackgerritDustin Cowles proposed openstack/nova-specs master: WIP: Spec: Use OpenStack SDK in Nova  https://review.opendev.org/66288100:42
*** owalsh has quit IRC00:47
*** owalsh has joined #openstack-nova00:50
*** ricolin has joined #openstack-nova00:55
*** markvoelker has quit IRC00:59
*** bhagyashris has joined #openstack-nova01:08
*** owalsh has quit IRC01:17
*** owalsh has joined #openstack-nova01:22
*** brinzhang has joined #openstack-nova01:33
*** boxiang has joined #openstack-nova01:46
*** brinzhang has quit IRC01:49
*** brinzhang has joined #openstack-nova01:52
*** brinzhang has quit IRC01:55
*** brinzhang has joined #openstack-nova01:55
*** markvoelker has joined #openstack-nova01:56
*** markvoelker has quit IRC02:01
*** alex_xu has joined #openstack-nova02:25
alex_xuls02:25
alex_xuoops02:25
alex_xusean-k-mooney: if we want to put the vpmem on the numa node rp, then it looks like we still need the reshape API, so the upgrade won't be easy with separated rp for vpmem now. But I can't get explict reason we should put the vpmem on numa node rp, or the child rp of numa node rp.02:26
alex_xuif we prefer to the vpmem as the child rp of numa node rp, then we can go to that way, and separate the rp for now02:27
*** kevinluuuuu has quit IRC02:32
sean-k-mooneyya im not sure what the preference is02:33
sean-k-mooneyi really dont mind too much either way but i just tought it might be simpler in the long run if we started with them seperate02:34
sean-k-mooneybut its not a point im really going to rathole on02:34
yonglihemriedem: HI.02:35
sean-k-mooneyyonglihe: while its less of an insane time of mriedem then me its still well after 22:40 for him so he wont be around for about 8-10 hours02:42
alex_xusean-k-mooney: ok, I see02:51
alex_xusean-k-mooney: another concern is people is asking us for a simple version for now, I'm afraid to people feel we add more in the current propsoal. actually, I don't have a preference also, happy to do the thing people like one02:54
sean-k-mooneyalex_xu: ya i know its frustrating that placments lack of support for numa has been holding back feature like this for 3 or 4 releases02:56
sean-k-mooneywe are cripleing the feature to make it simpeler because of the lack fo placmenent freature which created techdebt that need to fix next cycle02:57
alex_xusean-k-mooney: yea, agree with that02:57
sean-k-mooneybut doing it properly this cycle create tech debt when placement finally does support numa02:57
alex_xuI understand your point02:57
*** markvoelker has joined #openstack-nova02:57
*** brinzhang_ has joined #openstack-nova02:58
*** brinzhang has quit IRC03:00
*** markvoelker has quit IRC03:02
openstackgerritsean mooney proposed openstack/nova master: Add transform_image_metadata request filter  https://review.opendev.org/66577503:04
*** psachin has joined #openstack-nova03:06
*** brinzhang_ has quit IRC03:31
*** cfriesen has quit IRC03:45
openstackgerritBoxiang Zhu proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts  https://review.opendev.org/66676703:54
*** markvoelker has joined #openstack-nova03:58
*** markvoelker has quit IRC04:02
*** udesale has joined #openstack-nova04:05
*** dave-mccowan has quit IRC04:09
*** slaweq has joined #openstack-nova04:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix default values in update_cell command  https://review.opendev.org/66238304:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens  https://review.opendev.org/63771604:22
*** slaweq has quit IRC04:25
openstackgerritkarim proposed openstack/nova-specs master: Support VM memory encryption using Intel's MKTME  https://review.opendev.org/66676904:29
*** whoami-rajat has joined #openstack-nova04:35
*** ianw is now known as ianw_pto04:44
*** pcaruana has joined #openstack-nova04:45
*** abhishekk has joined #openstack-nova04:55
*** markvoelker has joined #openstack-nova04:59
*** janki has joined #openstack-nova05:02
*** ratailor has joined #openstack-nova05:02
*** markvoelker has quit IRC05:04
*** Luzi has joined #openstack-nova05:24
*** brinzhang has joined #openstack-nova05:54
*** rcernin has quit IRC05:59
*** markvoelker has joined #openstack-nova06:00
*** abhishekk has quit IRC06:03
*** markvoelker has quit IRC06:04
*** ivve has joined #openstack-nova06:15
*** boxiang_ has joined #openstack-nova06:33
*** dpawlik has joined #openstack-nova06:33
*** boxiang has quit IRC06:34
*** boxiang_ has quit IRC06:35
*** boxiang_ has joined #openstack-nova06:35
*** udesale has quit IRC06:36
*** udesale has joined #openstack-nova06:37
*** damien_r has joined #openstack-nova06:43
*** luksky has joined #openstack-nova06:43
*** damien_r has quit IRC06:52
*** janki has quit IRC06:53
*** maciejjozefczyk has joined #openstack-nova06:59
*** markvoelker has joined #openstack-nova07:01
*** slaweq has joined #openstack-nova07:04
*** damien_r has joined #openstack-nova07:14
*** tesseract has joined #openstack-nova07:19
*** markvoelker has quit IRC07:28
*** tkajinam has quit IRC07:31
*** janki has joined #openstack-nova07:31
*** ccamacho has joined #openstack-nova07:36
*** brinzhang_ has joined #openstack-nova07:38
*** brinzhang__ has joined #openstack-nova07:40
*** ricolin_ has joined #openstack-nova07:40
*** brinzhang has quit IRC07:42
*** brinzhang_ has quit IRC07:43
*** ricolin has quit IRC07:43
*** tssurya has joined #openstack-nova07:44
*** tkajinam has joined #openstack-nova07:44
*** rpittau|afk is now known as rpittau07:47
openstackgerrithuichen proposed openstack/nova stable/queens: Suppot paginaton for server groups by sql  https://review.opendev.org/66678207:48
*** ralonsoh has joined #openstack-nova07:52
*** ttsiouts has joined #openstack-nova07:55
openstackgerritBalazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none'  https://review.opendev.org/65779607:57
*** takashin has left #openstack-nova08:02
kashyapefried: Heya, fully agreed, on jroll's spec for _physical_ TPM passthrough.  Maybe I misphrased...08:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none'  https://review.opendev.org/65779608:11
*** udesale has quit IRC08:11
*** udesale has joined #openstack-nova08:12
*** ttsiouts has quit IRC08:20
*** ttsiouts has joined #openstack-nova08:21
openstackgerritMerged openstack/nova master: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/65701608:22
*** jangutter has joined #openstack-nova08:24
*** ttsiouts has quit IRC08:26
*** janki has quit IRC08:28
*** tkajinam_ has joined #openstack-nova08:30
openstackgerritMerged openstack/nova master: Avoid unnecessary joins in delete_resource_provider  https://review.opendev.org/65702108:32
*** tkajinam has quit IRC08:33
*** lee1 has joined #openstack-nova08:33
*** tkajinam_ has quit IRC08:35
openstackgerritBence Romsics proposed openstack/os-vif master: Insert osprofiler trace info as external_ids to the bridge table  https://review.opendev.org/66571508:35
*** ttsiouts has joined #openstack-nova08:37
*** derekh has joined #openstack-nova08:38
openstackgerritBalazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver  https://review.opendev.org/65670908:39
*** imacdonn has quit IRC08:41
*** imacdonn has joined #openstack-nova08:41
openstackgerritSurya Seetharaman proposed openstack/nova master: API microversion 2.74: Add 'power-update' external event  https://review.opendev.org/64561108:43
*** brinzhang has joined #openstack-nova08:43
*** jiaopengju has joined #openstack-nova08:45
*** ociuhandu has joined #openstack-nova08:46
*** brinzhang__ has quit IRC08:46
gibimelwitt: commented https://review.opendev.org/#/c/631294 I don't know if my suggestion would lead to a simpler test or not. :/08:46
*** ociuhandu has quit IRC08:46
*** ociuhandu has joined #openstack-nova08:46
*** ociuhandu has quit IRC08:50
*** ociuhandu has joined #openstack-nova08:50
openstackgerritSurya Seetharaman proposed openstack/python-novaclient master: API microversion 2.74: Add 'power-update' external event  https://review.opendev.org/66679208:54
*** janki has joined #openstack-nova08:57
*** ttsiouts has quit IRC09:00
*** ttsiouts has joined #openstack-nova09:01
*** ttsiouts has quit IRC09:05
openstackgerritAdam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption  https://review.opendev.org/66442009:06
openstackgerritAdam Spiers proposed openstack/nova master: Use fake flavor instead of empty dict in test  https://review.opendev.org/66255509:06
openstackgerritAdam Spiers proposed openstack/nova master: Pass extra_specs to flavor in vif tests  https://review.opendev.org/66255609:06
openstackgerritAdam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633409:06
openstackgerritAdam Spiers proposed openstack/nova master: Add <launchSecurity> and <driver iommu='on' /> to AMD SEV guest configs  https://review.opendev.org/63631809:06
openstackgerritAdam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required  https://review.opendev.org/64456509:06
openstackgerritAdam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption  https://review.opendev.org/66661609:06
*** owalsh has left #openstack-nova09:06
*** ttsiouts has joined #openstack-nova09:07
*** ricolin_ has quit IRC09:10
*** ricolin has joined #openstack-nova09:11
*** owalsh has joined #openstack-nova09:14
*** owalsh has left #openstack-nova09:19
*** cdent has joined #openstack-nova09:26
*** owalsh has joined #openstack-nova09:27
openstackgerritsean mooney proposed openstack/nova master: Add transform_image_metadata request filter  https://review.opendev.org/66577509:36
*** brinzhang has quit IRC09:39
cdentIt's interesting to read this in the context of OpenStack, especially Nova: https://techbeacon.com/app-dev-testing/forget-monoliths-vs-microservices-cognitive-load-what-matters09:40
*** lee1 is now known as lyarwood09:43
*** lpetrut has joined #openstack-nova09:45
sean-k-mooneycdent: the Germane cognitive load of nova is quite high by its nature09:48
cdentit didn't have to be that way, but yes09:49
sean-k-mooneywell its partly becasue nova is ofthen the theing that end up driving interaction with other services09:50
sean-k-mooneyeven if you precreate teh neuton ports for exampls nova still had sto notify nova when a vm uses them09:50
* cdent nods09:51
*** bhagyashris has quit IRC09:51
sean-k-mooneysome of that is intrisi to being the compute service wich is the main service that consumes other services09:51
sean-k-mooneybut ya its an interesting read09:51
cdentthe nova implementation is very tightly coupled with itself in a weirdly recursive kind of way09:53
sean-k-mooneywell not entirly the api, conductor, compute agent and schduler more or less work as miscroservices09:56
*** ricolin has quit IRC09:57
sean-k-mooneyor well services they are not that small09:57
cdentthat's not really true09:59
sean-k-mooneyhow so09:59
cdentinteractions via rpc are insufficiently hard boundaries09:59
sean-k-mooneythe api does not care what virt dirver you use09:59
sean-k-mooneyam im not sure i would agree10:00
stephenfingibi++ Thanks for hitting https://review.opendev.org/#/c/609460/ again. Much obliged :)10:01
sean-k-mooneythe could be more orbust in some ways but i would not consier it to less stict then something like grpc10:01
sean-k-mooneyits still versioned using ovo10:02
stephenfinDo we have a bug to track those failing MYSQL migration unit tests? I'm used to them intermittently failing on py35 but now py27 is unhappy too http://logs.openstack.org/29/666629/1/check/openstack-tox-py27/93e40ec/testr_results.html.gz10:02
sean-k-mooneystephenfin: if they are teh ones im think of yes10:03
sean-k-mooney ya those fail on all paython version intermiently and there is a bug10:03
stephenfinwith nova or sqlalchemy?10:03
sean-k-mooneynova10:03
sean-k-mooneywell with that error10:04
sean-k-mooneyso in the nova gate10:04
sean-k-mooneyand we have a bug in nova10:04
stephenfinDamn10:04
stephenfinI don't even know where to start with that10:04
stephenfinIt's rather irritating10:04
sean-k-mooneystephenfin: gibi has a partial patch to simplfy the tests10:04
sean-k-mooneyto help with figuring it out10:05
sean-k-mooneynot sure if it has merged yet10:05
sean-k-mooneystephenfin: https://review.opendev.org/#/c/662434/10:05
sean-k-mooneystephenfin: and this is the bug https://bugs.launchpad.net/nova/+bug/182325110:05
openstackLaunchpad bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed]10:05
openstackgerritStephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups  https://review.opendev.org/66663010:05
openstackgerritStephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another  https://review.opendev.org/66663110:05
stephenfinsean-k-mooney++10:06
*** spsurya has joined #openstack-nova10:06
sean-k-mooneystephenfin: instaed of doing an if else chain use a dict of vif_type -> functions10:07
stephenfinI thought about that10:07
stephenfinbut it saved something like 4 lines10:08
sean-k-mooneyits faster and simpler to read10:08
*** ttsiouts has quit IRC10:08
sean-k-mooneythe dict hashse the string and finds the function O(1)10:08
*** ttsiouts has joined #openstack-nova10:09
sean-k-mooneythe if,elif,else tree you built has to do a tone of comparisons10:09
stephenfinI wonder is it significant in the grand scheme of things though?10:09
stephenfinThe additional lookup time10:09
sean-k-mooneywe do this for every interface on every boot/reboot10:10
sean-k-mooneybut it mostly that the code is nicer to read10:10
*** lpetrut has quit IRC10:10
*** lpetrut has joined #openstack-nova10:10
stephenfinHmm, okay. If we're doing it elsewhere in nova I'm happy to switch10:10
*** damien_r has quit IRC10:11
stephenfinThough I must say I do rather like us explicitly highlighting the VIF types we still use the legacy path for https://review.opendev.org/#/c/666630/3/nova/network/os_vif_util.py@51710:11
*** damien_r has joined #openstack-nova10:12
stephenfinIs DVS even a thing still?10:12
sean-k-mooneyyou can still do that10:12
sean-k-mooneyand praobly but didnt we say we were kicking them out of tree10:13
*** ttsiouts has quit IRC10:13
sean-k-mooneywe proably should jsut port these all into os-vif to be honest10:13
jangutterstephenfin: if only python had a "case" statement, right :-p10:16
sean-k-mooneystephenfin: oh dvs is a vmware thing10:17
sean-k-mooneyjangutter: switch statement unless you are channeling sql10:17
janguttersean-k-mooney: eep, was channeling Pascal of all things.10:17
sean-k-mooneyha fair  enough10:18
sean-k-mooneyyou know if python had static type swe could have overload a function on teh argument type too10:18
sean-k-mooneyso many ways to solve this issue in other languages10:19
jangutterstephenfin, sean-k-mooney: I haven't cross-checked, but how many of the VIFs in use by Neutron Stadium things are still legacy?10:20
jangutterstephenfin, sean-k-mooney: I guess I'll have to go through the plugins in https://governance.openstack.org/tc/reference/projects/neutron.html10:20
sean-k-mooneyi dont think that many alot use ovs underneat10:24
janguttersean-k-mooney: yah, and a _lot_ of these are 'noop' plugging.10:25
sean-k-mooneyat this point i would be happy to just add drivers to os vif for the legecay stuff to get it out of nova and then we can drop them form os-vif later10:25
sean-k-mooneyjangutter: we have a driver for that :)10:25
janguttersean-k-mooney: yep.10:25
stephenfinsean-k-mooney: RE: https://review.opendev.org/#/c/666630/3/nova/network/os_vif_util.py@54510:29
stephenfinWe're still going to have a lot of lookups in https://review.opendev.org/#/c/666630/3/nova/network/os_vif_util.py@51310:30
stephenfinand I really don't want to change that as noted above10:30
sean-k-mooney  you dont have too10:30
*** udesale has quit IRC10:31
sean-k-mooneythat shoudl be relitivly quick but you should make it a set10:31
sean-k-mooneynot a tuple10:31
stephenfinokay, let me try10:32
stephenfinIn the interim, want to make sure https://review.opendev.org/#/c/666631/ makes sense?10:32
stephenfin:D10:32
sean-k-mooneystephenfin: if you define the set at module sope as a constant taht woudl also be better10:32
stephenfinEven though it's only used in one place?10:32
sean-k-mooneye.g. if vif_type in LEGACY_VIFS10:32
sean-k-mooneysure why not its cheap10:33
sean-k-mooneylitally put it like  10 lines up on line 49510:33
*** boxiang_ has quit IRC10:33
sean-k-mooneyi dont think our min version is at 3.2.0 yet10:34
sean-k-mooneyfor libvirt10:34
sean-k-mooneyi think its currently at 3.0.010:34
stephenfinYeah, it is. That's why I just updated the docstring for one10:35
sean-k-mooneyoh sorry you fix the todo below that10:36
stephenfinyup10:36
sean-k-mooneydo we set the vlan in the libvirt xml10:36
sean-k-mooneyas libvirt will only do that if we do10:37
sean-k-mooneyok i think we do10:38
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L352-L35410:39
sean-k-mooneyya we should be setting it here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L11610:40
*** ttsiouts has joined #openstack-nova10:41
*** ttsiouts has quit IRC10:46
stephenfinokay, I really don't like using the dict lookup. I know it's slightly faster but it feel out of place in that file, given the amount of if-elses we have10:47
stephenfinI have dragged the legacy VIFs out to their own variable though and used sets instead of tuples for 'in' lookups10:48
sean-k-mooney ok10:48
openstackgerritStephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups  https://review.opendev.org/66663010:48
openstackgerritStephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another  https://review.opendev.org/66663110:48
sean-k-mooneywhen we were using getattr before it was a dict lookup which is why i wanted to keep it that way10:49
stephenfinUnderstandable. I think the additional clarity is worth the small cost though10:50
sean-k-mooneystephenfin: will you get around to rebaseing https://review.opendev.org/#/c/663382/ soon or will i do it?10:56
stephenfinoh, forgot about that. I'll do that today, yeah10:56
sean-k-mooneyok i was talking to a partner about it yesterday so the sooner it lands upstream the sooner we can backport and the sooner they will be happy10:57
sean-k-mooneyok im actuly goint ot go lie down now for a bit ill be back in a few hours10:58
openstackgerritStephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks  https://review.opendev.org/66681411:01
*** bbowen has quit IRC11:04
openstackgerritStephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups  https://review.opendev.org/66663011:05
openstackgerritStephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another  https://review.opendev.org/66663111:05
openstackgerritStephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks  https://review.opendev.org/66681411:05
openstackgerritMerged openstack/os-traits master: Sync Sphinx requirement  https://review.opendev.org/66638611:06
*** Luzi has quit IRC11:07
*** ttsiouts has joined #openstack-nova11:08
jangutterstephenfin: coincidentally, that piece of getattr code is one of the first bits of python code that made me go "wow, you can do that, that's awesome!"11:13
*** jiaopengju has quit IRC11:13
jangutterstephenfin: and also makes the code un-greppable, meaning even though it's cool code, I'm ++ in taking it out.11:13
*** jiaopengju has joined #openstack-nova11:14
*** janki has quit IRC11:14
*** janki has joined #openstack-nova11:19
*** jiaopengju has quit IRC11:29
stephenfinjangutter: Exact same thing here, fwiw :)11:29
*** jiaopengju has joined #openstack-nova11:29
stephenfinthere are some other extraneous uses of getattr that I'll root out over time11:30
openstackgerritAdam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633411:32
openstackgerritAdam Spiers proposed openstack/nova master: Add <launchSecurity> and <driver iommu='on' /> to config.py  https://review.opendev.org/63631811:32
openstackgerritAdam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required  https://review.opendev.org/64456511:32
openstackgerritAdam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption  https://review.opendev.org/66661611:32
*** IvensZambrano has joined #openstack-nova11:32
*** cdent has quit IRC11:35
openstackgerritStephen Finucane proposed openstack/nova master: Unplug VIFs as part of cleanup of networks  https://review.opendev.org/66338211:37
stephenfinsean-k-mooney: I'm going to need a functional test for ^11:37
stephenfinThe initial version of the fix was totally wrong /o\11:37
*** jiaopengju has quit IRC11:39
*** jiaopengju has joined #openstack-nova11:39
*** cdent has joined #openstack-nova11:47
tssuryaThe novaclient CI seems unhappy when running the openstack-tox-docs job (http://logs.openstack.org/92/666792/1/check/openstack-tox-docs/04b52fb/job-output.txt.gz#_2019-06-21_09_04_06_646600)11:59
tssuryais there a bug reported already or should I open one11:59
*** sapd1_x has joined #openstack-nova12:02
*** cdent has quit IRC12:06
*** udesale has joined #openstack-nova12:10
*** jiaopengju has quit IRC12:15
*** jiaopengju has joined #openstack-nova12:15
*** priteau has joined #openstack-nova12:17
*** ab-a has quit IRC12:18
*** ab-a has joined #openstack-nova12:24
*** ratailor has quit IRC12:25
*** priteau has quit IRC12:36
*** priteau has joined #openstack-nova12:40
*** ttsiouts has quit IRC12:55
*** ttsiouts has joined #openstack-nova12:56
*** bbowen has joined #openstack-nova13:00
*** ttsiouts has quit IRC13:01
*** mriedem has joined #openstack-nova13:08
*** jchhatbar has joined #openstack-nova13:18
*** martinkennelly has joined #openstack-nova13:19
*** cdent has joined #openstack-nova13:21
*** janki has quit IRC13:21
*** lbragstad has joined #openstack-nova13:28
*** markvoelker has joined #openstack-nova13:29
*** dave-mccowan has joined #openstack-nova13:29
*** FlorianFa has joined #openstack-nova13:30
*** ttsiouts has joined #openstack-nova13:33
openstackgerritMatt Riedemann proposed openstack/nova master: api-guide: better explain scheduler hints.  https://review.opendev.org/52210613:36
mriedemgibi: easy docs change that you would probably be good for ^13:36
mriedemi'm already +213:36
mriedemnote that change is from nov 2017...13:37
*** eharney has joined #openstack-nova13:38
*** priteau has quit IRC13:41
bauzasmriedem: +Wipped13:42
bauzaseasy peasy enough13:42
bauzasFWIW, still working on providing the placement audit command13:42
openstackgerritMatthew Booth proposed openstack/nova master: Add functional test for bug 1550919  https://review.opendev.org/63129413:43
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)13:43
openstackgerritMatthew Booth proposed openstack/nova master: libvirt: Don't delete disks on shared storage during evacuate  https://review.opendev.org/57884613:43
dansmithbauzas: allow me to provide some english help13:43
dansmithbauzas: something is not "easy peasy" *enough*13:43
bauzas:)13:44
dansmithit's either peasy or not peasy, there is no gray area or gradient of peasiness13:44
bauzasdansmith: but but... https://context.reverso.net/traduction/anglais-francais/peasy13:45
stephenfinmriedem: What do you think I should log here? https://review.opendev.org/#/c/666630/5/nova/network/os_vif_util.py@535 Just "something bad happened" or maybe nothing at all?13:45
stephenfin(Asking you since you've been involved in the earlier patches for both)13:45
*** BjoernT has joined #openstack-nova13:46
mriedembauzas: easy peasy is fine13:46
mriedemeasy enough is fine13:46
dansmithbauzas: exactly.. all of those examples are 100% peasy, right? no fractional peasiness13:46
mriedembut easy peasy enough is something that will cause a shared american/french tree of friendship to whither and die13:47
dansmith^^ this man speaks the truth13:47
dansmithlol13:47
bauzaslol13:47
bauzasgotcha13:47
*** jchhatbar has quit IRC13:47
bauzasso, either "easy peasy" or "peasy enough"13:47
dansmithyes.13:47
dansmithno13:47
* bauzas will pass his TOEIC exam13:47
dansmitheasy peasy, or easy enough13:47
dansmithnot peasy enough13:47
bauzashaha13:48
dansmithgah13:48
* dansmith feels WWIII brewing13:48
mriedemstephenfin: it's the same thing as you'd get before for those vif types so meh13:48
bauzasFWIW https://www.thelocal.fr/20141112/france-has-lowest-english-proficiency-in-eu13:48
bauzasand again now actually https://www.thelocal.fr/20181108/french-ranked-worst-in-western-europe-for-speaking-english13:49
mriedemthere is also a rapper named "young jeezy" if that helps13:49
*** priteau has joined #openstack-nova13:51
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/66684013:52
mnaserdid we get rid of the nova-network api bits?13:53
mnaseri just had a really confused user checking their quota and asking why the security groups listed in osc didn't match horizon (it was getting the nova-network value)13:54
*** mlavalle has joined #openstack-nova13:54
stephenfincool13:55
mnaserhttps://github.com/openstack/nova/blob/c6218428e9b29a2c52808ec7d27b4b21aadc0299/doc/source/admin/quotas.rst13:55
mnaserthis needs to be scrubbed i guess13:55
aspiersstephenfin: thanks, please abandon https://review.opendev.org/#/c/662557/ too13:57
*** markvoelker has quit IRC13:57
stephenfinaspiers: done14:00
aspiersta!14:00
melwittgibi: ack, thanks!14:00
*** mrch_ has quit IRC14:03
*** hemna has quit IRC14:13
*** dpawlik has quit IRC14:20
*** JamesBenson has joined #openstack-nova14:28
mriedemmnaser: i believe there is a microversion that drops the nova-network related quota stuff from that api14:29
mnasermriedem: there is, i confirmed it that using the `nova` command did indeed not show them14:29
mriedemif they are using the openstack cli, that defaults to 2.1 so they'd see those old values14:29
*** rajinir has joined #openstack-nova14:30
mnaserso i guess this is the whole osc uses oldest microversion thing14:30
mnaser++14:30
*** rabel has quit IRC14:32
*** lpetrut has quit IRC14:32
mriedemmnaser: yeah https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/quota.html#quota-show mentions 2.50 for server group quota class but nothing about 2.56, or that in 2.50 the network quota class stuff was removed14:33
mriedemit's all very confusing14:33
mriedemhttps://docs.openstack.org/python-openstackclient/latest/cli/command-objects/quota.html#quota-list could probably be updated to mention that for --compute quotas, certain minimum microversions can be used to filter out no-longer-applicable proxy/network resource quotas14:33
*** BjoernT has quit IRC14:33
mriedemi don't know if that is more or less confusing for users14:34
*** BjoernT has joined #openstack-nova14:34
*** rabel has joined #openstack-nova14:34
*** whoami-rajat has quit IRC14:34
*** jiaopengju has quit IRC14:35
*** jiaopengju has joined #openstack-nova14:35
*** BjoernT has quit IRC14:36
*** BjoernT_ has joined #openstack-nova14:36
*** jiaopengju has quit IRC14:37
*** dklyle has joined #openstack-nova14:37
*** jiaopengju has joined #openstack-nova14:37
*** jiaopengju has quit IRC14:38
mriedemanyway, i added some notes for 2.36 to https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc14:41
*** bnemec is now known as beekneemech14:44
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/66685414:44
*** TheJulia is now known as needssleep14:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Error out interrupted builds  https://review.opendev.org/66685714:54
*** markvoelker has joined #openstack-nova14:54
*** sapd1_x has quit IRC14:58
*** markvoelker has quit IRC14:58
*** hoonetorg has quit IRC15:01
efriedHi folks, I'm going to take the day off today.15:06
*** efried is now known as efried_pto15:06
sean-k-mooneyefried_pto: enjoy o/15:09
*** jangutter has quit IRC15:09
sean-k-mooneystephenfin: ok ill re review https://review.opendev.org/#/c/663382/15:10
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/66686215:11
*** cfriesen has joined #openstack-nova15:17
*** maciejjozefczyk has quit IRC15:21
*** hemna has joined #openstack-nova15:22
*** maciejjozefczyk has joined #openstack-nova15:24
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/66686515:27
*** maciejjozefczyk has quit IRC15:29
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Handle HostMappingNotFound when deleting a compute service  https://review.opendev.org/66687215:39
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Delete resource providers for all nodes when deleting compute service  https://review.opendev.org/66687415:41
*** ttsiouts has quit IRC15:43
*** ttsiouts has joined #openstack-nova15:43
*** gyee has joined #openstack-nova15:46
*** tssurya has quit IRC15:46
*** damien_r has quit IRC15:46
*** ttsiouts has quit IRC15:48
openstackgerritsean mooney proposed openstack/nova master: extend libvirt video model support  https://review.opendev.org/64773315:54
sean-k-mooneymriedem: added the missing test ^15:54
sean-k-mooneyotherwise its the same as the last version15:54
mriedemsean-k-mooney: ack will take a look after lunch15:54
sean-k-mooneysure no rush15:55
*** markvoelker has joined #openstack-nova15:55
*** rpittau is now known as rpittau|afk15:57
*** markvoelker has quit IRC15:59
*** mrch_ has joined #openstack-nova16:01
*** panda is now known as panda-pto16:08
openstackgerritMatt Riedemann proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts  https://review.opendev.org/66676716:11
*** priteau has quit IRC16:17
*** IvensZambrano has quit IRC16:18
*** icarusfactor has joined #openstack-nova16:22
*** factor has quit IRC16:24
*** mgoddard has quit IRC16:24
*** mgoddard has joined #openstack-nova16:24
*** spsurya has quit IRC16:24
*** factor__ has joined #openstack-nova16:26
*** icarusfactor has quit IRC16:26
*** factor__ has quit IRC16:29
*** factor__ has joined #openstack-nova16:29
*** cdent has quit IRC16:30
*** _alastor_ has quit IRC16:32
*** gfhellma has joined #openstack-nova16:35
*** martinkennelly has quit IRC16:37
openstackgerritGhanshyam Mann proposed openstack/nova master: WIP: Multiple API cleanup changes  https://review.opendev.org/66688916:38
*** factor__ has quit IRC16:38
*** factor__ has joined #openstack-nova16:38
*** udesale has quit IRC16:39
*** udesale has joined #openstack-nova16:39
*** jiaopengju has joined #openstack-nova16:44
*** jiaopengju has quit IRC16:45
*** jiaopengju has joined #openstack-nova16:45
*** jiaopengju has quit IRC16:48
*** jiaopengju has joined #openstack-nova16:49
*** jiaopengju has quit IRC16:49
*** udesale has quit IRC16:50
*** jiaopengju has joined #openstack-nova16:51
*** jiaopengju has quit IRC16:51
*** derekh has quit IRC16:51
*** factor__ has quit IRC16:52
*** factor__ has joined #openstack-nova16:52
*** gfhellma_ has joined #openstack-nova16:54
*** markvoelker has joined #openstack-nova16:56
*** gfhellma has quit IRC16:58
*** factor__ has quit IRC17:00
*** factor has joined #openstack-nova17:00
*** markvoelker has quit IRC17:01
*** ralonsoh has quit IRC17:04
*** hoonetorg has joined #openstack-nova17:05
*** lbragstad has quit IRC17:09
*** whoami-rajat has joined #openstack-nova17:09
openstackgerritMichael Bayer proposed openstack/nova master: Don't retry database connections from command line applications  https://review.opendev.org/66689217:10
*** gfhellma_ has quit IRC17:14
*** igordc has joined #openstack-nova17:25
openstackgerritStephen Finucane proposed openstack/nova master: Unplug VIFs as part of cleanup of networks  https://review.opendev.org/66338217:36
*** ociuhandu has quit IRC17:38
*** gfhellma_ has joined #openstack-nova17:46
*** tesseract has quit IRC17:50
openstackgerritStephen Finucane proposed openstack/nova master: hacking: Resolve W503 (line break occurred before a binary operator)  https://review.opendev.org/65155517:54
openstackgerritStephen Finucane proposed openstack/nova master: hacking: Resolve E741 (ambiguous variable name)  https://review.opendev.org/65210317:54
openstackgerritStephen Finucane proposed openstack/nova master: hacking: Resolve W605 (invalid escape sequence)  https://review.opendev.org/65210417:54
openstackgerritStephen Finucane proposed openstack/nova master: tests: Use consistent URL regex substitution  https://review.opendev.org/66594917:54
openstackgerritStephen Finucane proposed openstack/nova master: hacking: Resolve E731 (do not assign a lambda expression, use a def)  https://review.opendev.org/65155417:54
*** hoonetorg has quit IRC17:54
*** markvoelker has joined #openstack-nova17:56
*** rdopiera has quit IRC17:57
*** eharney has quit IRC18:01
*** markvoelker has quit IRC18:01
*** _alastor_ has joined #openstack-nova18:01
*** mvkr has quit IRC18:06
*** hoonetorg has joined #openstack-nova18:07
*** hoonetorg has quit IRC18:10
*** hoonetorg has joined #openstack-nova18:14
mriedemsean-k-mooney: comments in https://review.opendev.org/#/c/647733/ which can be addressed in a follow up18:21
sean-k-mooneyjust looking now18:22
mriedembbl - going to mow before it rains for 5 days18:23
sean-k-mooneycool ill finish up the patch im working on and start working on a followup18:23
*** factor has quit IRC18:28
*** icarusfactor has joined #openstack-nova18:28
*** BjoernT has joined #openstack-nova18:35
*** _alastor_ has quit IRC18:37
*** BjoernT_ has quit IRC18:37
*** eharney has joined #openstack-nova18:40
*** mchlumsky has quit IRC18:47
*** awalende has joined #openstack-nova18:52
*** icarusfactor has quit IRC18:57
*** icarusfactor has joined #openstack-nova18:57
*** markvoelker has joined #openstack-nova18:57
*** psachin has quit IRC18:59
*** Sundar has joined #openstack-nova19:00
*** IvensZambrano has joined #openstack-nova19:01
*** awalende has quit IRC19:11
*** whoami-rajat has quit IRC19:19
*** IvensZambrano has quit IRC19:21
*** lbragstad has joined #openstack-nova19:23
*** markvoelker has quit IRC19:26
openstackgerritsean mooney proposed openstack/nova master: Add transform_image_metadata request filter  https://review.opendev.org/66577520:19
openstackgerritsean mooney proposed openstack/nova master: [WIP] Libvirt: report storage bus traits  https://review.opendev.org/66691420:19
openstackgerritsean mooney proposed openstack/nova master: [WIP] use domain capablites to get supported device models  https://review.opendev.org/66691520:19
*** markvoelker has joined #openstack-nova20:23
redkriegHi all, I'm trying to use AggregateTypeAffinityFilter to ensure that instances of only a certain few flavors can be started on hypervisors in the aggregate group.  I've set the instance_type property to "SKVM" on the aggregate, made sure the hypervisor was added, and ensured that all other hypervisors are in other aggregates.  I then set the property on the flavor of instance_type=SKVM, but when20:27
redkriegI launch it, the filter does not seem to remove things other than SKVM.  Am I misunderstanding how this filter works?20:28
*** markvoelker has quit IRC20:28
sean-k-mooneythe filter will not move existing instances form a host aggratate20:29
redkriegyeah, I was trying to make a new one, I think I did something stupid though.  I just realized AggregateTypeAffinityFilter is not in my scheduler_default_filters.  sorry for the noise, will come back if it's still not working after fixing that20:31
*** BjoernT has quit IRC20:33
openstackgerritMerged openstack/nova master: Add functional recreate test for regression bug 1825537  https://review.opendev.org/65406620:39
openstackbug 1825537 in OpenStack Compute (nova) "finish_resize failures incorrectly revert allocations" [Medium,In progress] https://launchpad.net/bugs/1825537 - Assigned to Matt Riedemann (mriedem)20:39
*** fhalbach has joined #openstack-nova20:39
klindgrenmriedem, looks like you are tackling some live migration stuff right now?  Are you aware of a recently fixed bug, that when a live migration fails, that it leaves the database in a bad state?20:40
klindgrenWe had a live migrate that failed after the copy and mem transfer, during the vif spinup on the destination node, because the destination node didn't have the network segment that the vm needed.  Which left the live migration operation in a completed status.  But the states of the vm totally wrong.  Status was Migration percentage was 100, and the host and node in the DB was pointing at the source HV, but the VM was really running on the destinati20:42
klindgrenon HV.  So all additional actaions agaisnt the VM failed.  UNtil we updated the DB to container the correct HV.20:42
openstackgerritsean mooney proposed openstack/nova master: [FUP] fix backleveling unit test for video models  https://review.opendev.org/66691920:45
sean-k-mooneymriedem: ^ i think that  adresses your outstanding nits. im goint to call it a day. enjoy your weekend o/20:46
*** gfhellma_ has quit IRC20:54
*** gfhellma has joined #openstack-nova21:03
*** pcaruana has quit IRC21:16
openstackgerritMerged openstack/nova master: api-guide: better explain scheduler hints.  https://review.opendev.org/52210621:16
*** lbragstad has quit IRC21:20
*** markvoelker has joined #openstack-nova21:24
*** markvoelker has quit IRC21:29
*** gfhellma has quit IRC21:29
*** openstackgerrit has quit IRC21:33
*** JamesBenson has quit IRC21:51
*** slaweq has quit IRC21:55
*** IvensZambrano has joined #openstack-nova21:56
Nick_Awe are launching instances with a public IPV4 (and v6) on one port/interface and a private IPV4 on another port/interface. Is there a way to force the public addresses to always be on eth0 and private on eth1 for example? I'm trying to tweak how cloud-init handles some things but it's difficult not knowing which interface will end up with with IPs22:03
sean-k-mooneyno openstack does not provide any guarentees on nic ordering22:05
sean-k-mooneywe expose the info via the metadta service so that you can use that info to determing which interface is which22:05
sean-k-mooneyyou should read up on the device role taging feature22:06
Nick_Athank you22:06
sean-k-mooneythe other thing you can do is if you are using dhcp you can disabel the gateway on the private netwrok so it never recives a default route.22:07
sean-k-mooneyassuming you jsut want to ensure that you default traffic goves via the public interface22:07
Nick_Ahmm that might help with part of our problem22:08
*** slaweq has joined #openstack-nova22:16
Nick_Aany idea why metadata would send routes for all /24s in a region? Not just the /24 of the instance's IP.22:20
*** Sundar has quit IRC22:20
*** slaweq has quit IRC22:24
*** markvoelker has joined #openstack-nova22:25
*** gfhellma has joined #openstack-nova22:25
*** markvoelker has quit IRC22:30
*** gfhellma has quit IRC22:32
*** brinzhang has joined #openstack-nova22:35
*** gyee has quit IRC22:40
*** spatel has joined #openstack-nova22:43
*** IvensZambrano has quit IRC22:45
*** spatel has quit IRC22:47
*** dave-mccowan has quit IRC22:49
*** gfhellma has joined #openstack-nova22:58
*** gfhellma_ has joined #openstack-nova22:59
*** igordc has quit IRC23:02
*** gfhellma has quit IRC23:02
*** rajinir has quit IRC23:09
*** slaweq has joined #openstack-nova23:13
*** gfhellma__ has joined #openstack-nova23:23
*** slaweq has quit IRC23:25
*** markvoelker has joined #openstack-nova23:26
*** gfhellma_ has quit IRC23:27
*** markvoelker has quit IRC23:30
*** luksky has quit IRC23:31
*** gfhellma__ has quit IRC23:34
*** openstackgerrit has joined #openstack-nova23:47
openstackgerritDustin Cowles proposed openstack/nova-specs master: Spec: Use OpenStack SDK in Nova  https://review.opendev.org/66288123:47
« Thursday, 2019-06-20 Index Saturday, 2019-06-22 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!