Tuesday, 2020-07-21

*** openstack has joined #openstack-nova07:30
*** ChanServ sets mode: +o openstack07:30
*** rcernin_ has quit IRC07:34
*** tosky has joined #openstack-nova07:37
*** dtantsur|afk is now known as dtantsur07:55
*** xek_ has joined #openstack-nova07:55
bauzasgood morning Nova08:00
gibibauzas: good morning08:02
*** brinzhang0 has joined #openstack-nova08:05
alex_xustephenfin: do you remember why we return cpuset string instead of json list in metadata API?https://review.opendev.org/#/c/688936/30/nova/api/metadata/base.py@453, feel like json list is more natual there08:07
*** ociuhandu has joined #openstack-nova08:08
*** brinzhang_ has quit IRC08:08
*** rcernin_ has joined #openstack-nova08:16
openstackgerritRico Lin proposed openstack/nova master: [TEST][DNM] Validate ARM64 Unittests  https://review.opendev.org/74209408:18
*** aj_mailing has quit IRC08:24
*** priteau has joined #openstack-nova08:25
*** derekh has joined #openstack-nova08:25
*** rcernin_ has quit IRC08:26
*** jangutter has quit IRC08:30
*** jangutter has joined #openstack-nova08:31
*** yankcrime has joined #openstack-nova08:32
*** rcernin_ has joined #openstack-nova08:47
*** maciejjozefczyk_ has joined #openstack-nova08:51
*** maciejjozefczyk has quit IRC08:54
*** tetsuro has joined #openstack-nova08:58
*** tetsuro has quit IRC09:00
*** tetsuro has joined #openstack-nova09:03
bauzasgibi: +1 on https://review.opendev.org/#/c/741601/2 for a nit09:06
bauzasgibi: just tell me you'll provide a FUP for unsucessful claims and then I'll +2 it09:06
*** rcernin_ has quit IRC09:07
*** martinkennelly has joined #openstack-nova09:15
*** markvoelker has joined #openstack-nova09:16
openstackgerritMerged openstack/nova stable/queens: Check cherry-pick hashes in pep8 tox target  https://review.opendev.org/74188309:20
*** markvoelker has quit IRC09:21
*** k_mouza has joined #openstack-nova09:21
*** tetsuro has quit IRC09:22
*** tetsuro has joined #openstack-nova09:25
*** xek_ has quit IRC09:28
*** tetsuro has quit IRC09:29
openstackgerritLuigi Toscano proposed openstack/nova stable/stein: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full  https://review.opendev.org/74211309:34
*** rcernin_ has joined #openstack-nova09:41
*** xiaolin has joined #openstack-nova09:47
*** k_mouza has quit IRC09:53
*** rcernin_ has quit IRC09:55
*** zhanglong has quit IRC09:59
*** k_mouza has joined #openstack-nova10:01
stephenfinalex_xu: Actually, I don't. That didn't even cross my mine10:04
stephenfin*mind10:04
*** ratailor_ has quit IRC10:10
*** ratailor_ has joined #openstack-nova10:10
*** psachin has joined #openstack-nova10:22
openstackgerritLee Yarwood proposed openstack/nova master: Ensure source compute is up when confirming a resize  https://review.opendev.org/69929110:26
*** Liang__ has quit IRC10:26
*** markvoelker has joined #openstack-nova10:27
*** rcernin_ has joined #openstack-nova10:28
*** markvoelker has quit IRC10:31
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Don't age swap files in the image cache  https://review.opendev.org/64109310:41
*** udesale_ has joined #openstack-nova10:56
*** udesale has quit IRC10:58
openstackgerritLee Yarwood proposed openstack/nova master: Removes the delta file once image is extracted  https://review.opendev.org/73442111:02
*** markvoelker has joined #openstack-nova11:18
*** markvoelker has quit IRC11:22
alex_xustephenfin: this one need your +w again https://review.opendev.org/#/c/740933/7, huaqiang fixes the unittest, and add exception catching at api layer11:34
lyarwoodelod: https://review.opendev.org/#/c/742051/ would you mind having a look at this from melwitt please?11:40
*** ratailor_ has quit IRC11:41
lyarwoodelod: https://review.opendev.org/741760 also if you have time11:47
*** sean-k-mooney has quit IRC11:48
gibigibizer@riverbed:~                                                                                                                                                    [000] [13:49:41]11:49
*** iurygregory has quit IRC11:49
gibi$ javac11:49
gibisorry wrong window11:49
gibibash: javac: command not found11:49
lyarwood^_^11:51
*** tkajinam has quit IRC11:56
gibibauzas: thanks, I have to get back to that during the afternoon11:59
*** iurygregory has joined #openstack-nova11:59
*** raildo has joined #openstack-nova12:01
elodlyarwood: yepp, added them to my TODO list!12:01
devfazhi, we having some strange issue with one of our instances. it seems there is some kind of address-translation done in the "qbr-bridge". Our pakets go correct into the bridge (qvb), but somehow get src-ip-changed while in the bridge, because the pakets on the tap-device are "broken" (src-ip is replaced with phy. IP of hypervisor, which is - of course - not part of the tenant network at all)12:04
devfazany ideas what may cause an ip-addr-change in qbr?12:04
*** ociuhandu_ has joined #openstack-nova12:04
devfaztried to check the iptables and ipset rules, but greping for the network/ip doesnt help here :(12:04
*** ociuhandu has quit IRC12:07
gibidevfaz: I have no idea. Have you tried asking the networking guys about it on #openstack-neutron  ?12:14
*** ociuhandu_ has quit IRC12:15
*** ociuhandu has joined #openstack-nova12:16
*** k_mouza has quit IRC12:18
devfazgibi: thx a lot for your quick answer. I think I found a global masq-rule.. having to take a look.12:18
devfazjust someone else runs into this - if im true, there is an issue with docker-default-network and the masq-rule it creates. I will analyse and report back.12:19
*** dave-mccowan has joined #openstack-nova12:27
openstackgerritStephen Finucane proposed openstack/nova master: rpc: Rework 'get_notifier', 'wrap_exception'  https://review.opendev.org/74166312:35
openstackgerritStephen Finucane proposed openstack/nova master: mypy: Add type annotations to top-level modules  https://review.opendev.org/70565812:35
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Expand type hints for nova.block_device  https://review.opendev.org/74217012:35
stephenfinalex_xu: done12:36
*** rcernin_ has quit IRC12:54
alex_xustephenfin: thanks12:56
openstackgerritSergey Galas' proposed openstack/nova master: fixes broken shared live migration of a vm with a vif aka Bug #1888395  https://review.opendev.org/74218013:00
openstackbug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [Undecided,New] https://launchpad.net/bugs/1888395 - Assigned to Sergey Galas' (shrike742)13:00
openstackgerritSasha Andonov proposed openstack/nova stable/pike: libvirt: Do not reraise DiskNotFound exceptions during resize  https://review.opendev.org/74218113:01
lyarwoodstephenfin: https://review.opendev.org/#/c/710861/ - so reading https://review.opendev.org/#/c/715406/ back you suggested we DON'T want to backport this correct?13:02
openstackgerritSergey Galas' proposed openstack/nova master: fixes broken shared live migration of a vm with a vif aka Bug #1888395  https://review.opendev.org/74218013:11
openstackbug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [Undecided,New] https://launchpad.net/bugs/1888395 - Assigned to Sergey Galas' (shrike742)13:11
*** nhicher has left #openstack-nova13:28
*** nweinber has joined #openstack-nova13:29
*** bnemec has joined #openstack-nova13:38
openstackgerritHuaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs  https://review.opendev.org/68893613:38
openstackgerritHuaqiang Wang proposed openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance  https://review.opendev.org/73699713:38
openstackgerritHuaqiang Wang proposed openstack/nova master: test: add some notes and new tests for mixed instance  https://review.opendev.org/74139513:38
*** psachin has quit IRC13:49
openstackgerritMerged openstack/nova stable/queens: fix scsi disk unit number of the attaching volume when cdrom bus is scsi  https://review.opendev.org/73031313:57
openstackgerritMerged openstack/nova master: Switch from unittest2 compat methods to Python 3.x methods  https://review.opendev.org/73751113:57
stephenfinlyarwood: Oh, I think it was more a case of I didn't think it was backportable that shouldn't be backported13:58
*** yaawang has quit IRC14:05
*** yaawang has joined #openstack-nova14:05
openstackgerritMerged openstack/nova master: Remove unnecessary retrieval of Migration object  https://review.opendev.org/73035114:07
openstackgerritMerged openstack/nova master: Remove unused function parameters  https://review.opendev.org/73035214:07
*** xek_ has joined #openstack-nova14:17
*** xek_ is now known as xek14:17
openstackgerritMerged openstack/nova master: Move image verification and writing out of download method  https://review.opendev.org/73873814:22
openstackgerritMerged openstack/nova master: Repro gen conflict in COMPUTE_STATUS_DISABLED handling  https://review.opendev.org/70486514:22
*** jeevanpuchakay has joined #openstack-nova14:22
*** markmcclain has quit IRC14:23
*** markmcclain has joined #openstack-nova14:24
*** nweinber_ has joined #openstack-nova14:31
*** nweinber_ has quit IRC14:31
*** artom has joined #openstack-nova14:34
*** Luzi has quit IRC14:34
bauzasstephenfin, other cores : you could be interested in https://review.opendev.org/#/c/733703/14:38
bauzastl:dr: routed networks spec14:38
*** mlavalle has joined #openstack-nova14:38
nightmare_unrealhello I am trying to implement 'nova migration-list' cmd for osc cli. I have implemented most part but I am not able to understand from where the values are coming for certain fields like UUID, project_ID and user_id14:39
stephenfinnightmare_unreal: can you give us an example?14:41
openstackgerritElod Illes proposed openstack/nova stable/pike: Check cherry-pick hashes in pep8 tox target  https://review.opendev.org/74221214:43
openstackgerritStephen Finucane proposed openstack/nova master: Remove support for Intel CMT events  https://review.opendev.org/74092214:44
*** k_mouza has joined #openstack-nova14:45
openstackgerritBalazs Gibizer proposed openstack/nova-specs master: Support SRIOV interface attach and detach  https://review.opendev.org/74160114:47
gibibauzas: fixed up the spec ^^14:47
*** Luzi has joined #openstack-nova14:48
bauzasgibi: +2/+w14:49
nightmare_unrealsure stephenfin  . if you run 'nova migration-list' you get this output. I am not able to get where it gets UUID, project_id and user_id . https://usercontent.irccloud-cdn.com/file/Ypg9cDNc/migration-list.png14:49
nightmare_unrealLooking at the source code here : https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/migrations.py , I can't see those fields14:50
*** Luzi has quit IRC14:50
*** nweinber_ has joined #openstack-nova14:50
*** nweinber_ has quit IRC14:52
*** rcernin_ has joined #openstack-nova14:52
stephenfinnightmare_unreal: https://docs.openstack.org/api-ref/compute/?expanded=list-migrations-detail14:52
stephenfinthat's the API it's using, and the docs describe the various fields in the response14:53
gibibauzas: thansk14:53
gibithanks even14:53
stephenfinnightmare_unreal: looks like they were introduced in microversions, which might be why you're not seeing them in your own response14:53
stephenfin*s14:53
nightmare_unrealthanks stephenfin  . I will go through it. Thing that was bothering me was migration object has only these attributes : ['__add__', '__class__', '__contains__', '__delattr__', '__delitem__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__getitem__', '__gt__', '__hash__', '__iadd__', '__imul__', '__init__', '__init_subclass__', '__iter__', '__le__', '__len__',14:54
nightmare_unreal'__lt__', '__module__', '__mul__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__reversed__', '__rmul__', '__setattr__', '__setitem__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', '_append_request_id', 'append', 'append_request_ids', 'clear', 'copy', 'count', 'extend', 'index', 'insert', 'pop', 'remove', 'request_ids', 'request_ids_setup', 'reverse', 'sort',14:54
nightmare_unreal'x_openstack_request_ids']14:54
nightmare_unrealand it does not contain the fields I mentioned14:54
nightmare_unrealstephenfin:  yeah microversions can be the thing to look at too14:55
stephenfinThat doesn't look like a migration object14:55
stephenfinThough that depends on what you mean by object14:55
*** rcernin_ has quit IRC14:57
nightmare_unrealhmm15:01
nightmare_unreali was running compute_client.migrations.list() command15:01
openstackgerritElod Illes proposed openstack/nova stable/stein: compute: Allow snapshots to be created from PAUSED volume backed instances  https://review.opendev.org/72917615:01
stephenfinah, you mean Migration from novaclient15:07
stephenfinYeah, I don't know how that's supposed to work tbh15:07
stephenfinbauzas: looking15:07
openstackgerritBalazs Gibizer proposed openstack/nova master: DNM: Test the state of VMware NSX 3pp CI  https://review.opendev.org/73411415:08
*** jeevanpuchakay has quit IRC15:12
*** brinzhang0 has quit IRC15:16
*** brinzhang0 has joined #openstack-nova15:16
openstackgerritMerged openstack/nova-specs master: template: consider openstack client besides novaclient  https://review.opendev.org/71772215:18
openstackgerritMerged openstack/nova-specs master: Support SRIOV interface attach and detach  https://review.opendev.org/74160115:20
stephenfinbauzas: replied on https://review.opendev.org/#/c/733703/15:20
stephenfinI think I'm lacking the background knowledge to review this properly at the moment, but I hope my questions aren't too dumb15:21
bauzasstephenfin: okay, will write a new rev15:23
artommelwitt, could you revisit https://review.opendev.org/#/c/729763/4 when you get a chance?15:28
*** gyee has joined #openstack-nova15:41
elodlyarwood: btw, I don't have +2+W right @ openstackclient repo o:)15:45
openstackgerritTakashi Natsume proposed openstack/nova master: objects: Update keypairs when saving an instance  https://review.opendev.org/68304315:46
elodlyarwood: also, I had another type of fix for the same issue that melwitt's patch solves, which I guess now can be abandoned - https://review.opendev.org/#/c/713865/15:46
*** brinzhang_ has joined #openstack-nova15:47
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Routed network scheduling spec  https://review.opendev.org/73370315:48
bauzasstephenfin: fer your plezure ^15:49
* bauzas needs to go off for today15:49
bauzas++15:49
*** brinzhang0 has quit IRC15:50
artombauzas, grah, left comments on PS4 just as you uploaded PS515:55
* gibi also leaves the building for today15:58
gibio/15:58
*** udesale_ has quit IRC15:59
*** k_mouza has quit IRC16:04
*** k_mouza has joined #openstack-nova16:05
*** ociuhandu_ has joined #openstack-nova16:17
*** k_mouza has quit IRC16:20
*** ociuhandu has quit IRC16:20
*** ociuhandu_ has quit IRC16:21
*** xek has quit IRC16:24
*** hamalq has joined #openstack-nova16:27
*** maciejjozefczyk_ has quit IRC16:27
*** hamalq has quit IRC16:28
*** priteau has quit IRC16:28
*** k_mouza has joined #openstack-nova16:28
*** hamalq has joined #openstack-nova16:29
*** sapd1 has joined #openstack-nova16:35
*** links has quit IRC16:37
*** ralonsoh_ has joined #openstack-nova16:41
*** ralonsoh has quit IRC16:43
*** k_mouza has quit IRC16:45
*** JamesBenson has joined #openstack-nova16:47
*** tesseract has quit IRC16:49
*** aj_mailing has joined #openstack-nova16:52
melwittartom: sure, will do16:57
melwittlyarwood: I lost +Ws on this patch and the one above it bc of rebase if you could please revisit later https://review.opendev.org/69145916:58
*** derekh has quit IRC17:01
*** priteau has joined #openstack-nova17:01
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Re-enable live snapshot for paused instances  https://review.opendev.org/74033417:06
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Remove workaround for really old QEMU  https://review.opendev.org/74033517:06
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Use better variable names, types for '_create_guest'  https://review.opendev.org/74094517:06
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Attempt to restore some sanity to snapshot  https://review.opendev.org/74033617:06
*** dtantsur is now known as dtantsur|afk17:15
*** hamalq has quit IRC17:16
*** hamalq has joined #openstack-nova17:16
*** priteau has quit IRC17:21
openstackgerritMerged openstack/nova stable/stein: Remove stale nested backport from InstancePCIRequests  https://review.opendev.org/73819917:31
*** ralonsoh_ has quit IRC17:33
openstackgerritMerged openstack/nova stable/stein: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full  https://review.opendev.org/74211317:34
openstackgerritMerged openstack/nova master: libvirt: apply mixed instance CPU policy  https://review.opendev.org/72848117:34
openstackgerritLuigi Toscano proposed openstack/nova stable/rocky: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full  https://review.opendev.org/74225817:39
*** ralonsoh has joined #openstack-nova17:41
*** brinzhang0 has joined #openstack-nova17:48
*** nightmare_unreal has quit IRC17:51
*** brinzhang_ has quit IRC17:52
*** ralonsoh has quit IRC18:08
*** vishalmanchanda has quit IRC18:26
*** haleyb has quit IRC18:39
*** haleyb has joined #openstack-nova18:41
*** sapd1 has quit IRC18:44
*** ociuhandu has joined #openstack-nova18:48
*** ociuhandu has quit IRC18:53
*** ociuhandu has joined #openstack-nova18:56
*** brinzhang0 has quit IRC19:20
*** xek has joined #openstack-nova20:07
*** adriant has quit IRC20:39
*** adriant has joined #openstack-nova20:40
*** ircuser-1 has joined #openstack-nova20:40
*** nweinber has quit IRC20:55
*** nweinber has joined #openstack-nova20:56
*** nweinber has quit IRC20:56
openstackgerritJiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD  https://review.opendev.org/57430120:59
openstackgerritmelanie witt proposed openstack/nova master: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices  https://review.opendev.org/74156121:04
lyarwoodmelwitt: thanks for that review and update, really appreciated and apologies for not spelling this out in the change itself.21:06
melwittlyarwood: np, sorry for all the grief on this one. I was just not understanding the second test scenario fully, it wasn't you, it was me21:07
*** raildo has quit IRC21:08
melwittsome of the original comments in the test were written by me the first time I learned that code, so I had a preexisting idea of the test in my head and had trouble processing the new changes 😝21:08
*** JamesBenson has quit IRC21:12
lyarwoodmelwitt: yeah this flow is really really awkward at the moment, I'd really like to break it down and simplify things in master after this if I can21:22
lyarwoodmelwitt: I'm thinking of looking up which config the device is actually listed in before calling for the detach21:23
lyarwoodmelwitt: so we always use the correct flags and hopefully avoid getting this error code21:23
*** jsuchome has quit IRC21:26
melwittlyarwood: ack21:31
melwittlyarwood: also I dunno if you saw earlier but I need +W back on these two patches pls https://review.opendev.org/691459 after the rebases21:32
lyarwoodmelwitt: ah sorry I missed that ./me looks21:32
melwittthanks. hopefully it is just easy21:32
lyarwoodyup ACK'd21:33
lyarwoodthanks for fixing the func tests :)21:33
melwittthank you! and np, took me a comically long time to find the already-done change on stable/rocky21:33
melwittglad it worked on queens21:34
lyarwoodah the joys of stable maint21:37
lyarwoodzomg this thing is broken! ./me spends 3 days fixing it ; oh look Matt fixed and backported this to the branch above this a year ago21:38
lyarwood^ that happened all the damn time when I started working on Nova, thankfully it's not that often now.21:39
melwittlol, thanks for making me feel better 😆21:40
lyarwoodhaha sorry yeah that does come across wrong21:42
lyarwoodit has definietly happened to me on the older branches recently FWIW21:42
melwittno, it's ok :) I know what you meant21:42
lyarwoodthe nova-live-migration job stuff was super fun >.<21:43
melwittyeah that looked impossible. kudos for getting through it21:44
*** xek has quit IRC21:46
*** rcernin_ has joined #openstack-nova22:22
*** rcernin_ has quit IRC22:22
*** rcernin has joined #openstack-nova22:34
*** efried has quit IRC22:43
*** efried has joined #openstack-nova22:43
*** bnemec has quit IRC22:47
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in FIP policies  https://review.opendev.org/74203022:50
*** tkajinam has joined #openstack-nova22:55
openstackgerritAlex Deiter proposed openstack/nova master: Detach is broken for multi-attached fs-based volumes  https://review.opendev.org/74171223:00
*** martinkennelly has quit IRC23:04
*** mlavalle has quit IRC23:08
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of multinic policies  https://review.opendev.org/74231523:36
*** iurygregory has quit IRC23:42
openstackgerritMerged openstack/nova master: Validate CPU pinning configuration for mixed instance  https://review.opendev.org/71626823:46
openstackgerritMerged openstack/nova master: Calculate the CPU usage for mixed instance  https://review.opendev.org/71626923:46
*** tosky has quit IRC23:47
openstackgerritMerged openstack/nova master: scheduler: mixed instance CPU resource translating  https://review.opendev.org/72848223:50
openstackgerritMerged openstack/nova master: compute: bump nova-compute version and check in API  https://review.opendev.org/74093323:50

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