Wednesday, 2019-08-28

« Tuesday, 2019-08-27 Index Thursday, 2019-08-29 »
*** dpawlik has joined #openstack-nova00:11
*** dpawlik has quit IRC00:15
*** bbowen has quit IRC00:29
*** ozzzo has quit IRC00:39
*** gyee has quit IRC00:43
*** markvoelker has quit IRC00:57
*** larainema has joined #openstack-nova01:00
*** mkrai has joined #openstack-nova01:07
*** tetsuro has joined #openstack-nova01:12
*** tetsuro has quit IRC01:13
openstackgerritzhufl proposed openstack/nova master: [Trivial]Remove unused helper check_temp_folder  https://review.opendev.org/67897301:28
*** threestrands has joined #openstack-nova01:39
*** yikun has joined #openstack-nova01:50
*** markvoelker has joined #openstack-nova01:52
*** markvoelker has quit IRC02:02
*** markvoelker has joined #openstack-nova02:03
*** nicolasbock has quit IRC02:09
*** eharney has quit IRC02:10
*** spsurya has joined #openstack-nova02:10
*** dpawlik has joined #openstack-nova02:11
*** markvoelker has quit IRC02:13
*** markvoelker has joined #openstack-nova02:14
*** dpawlik has quit IRC02:16
*** sapd1 has joined #openstack-nova02:18
*** markvoelker has quit IRC02:24
*** ricolin has joined #openstack-nova02:25
openstackgerritzhufl proposed openstack/nova master: [Trivial]Remove unused helper get_allocated_disk_size  https://review.opendev.org/67897602:41
*** bbowen has joined #openstack-nova02:44
*** itlinux has joined #openstack-nova02:44
*** itlinux has quit IRC02:45
openstackgerritZhengMa proposed openstack/nova master: Check container_format on instance creation  https://review.opendev.org/67340703:06
*** psachin has joined #openstack-nova03:10
*** gbarros has quit IRC03:32
openstackgerritArtom Lifshitz proposed openstack/nova master: Introduce live_migration_claim()  https://review.opendev.org/63566903:43
openstackgerritArtom Lifshitz proposed openstack/nova master: New objects for NUMA live migration  https://review.opendev.org/63482703:43
openstackgerritArtom Lifshitz proposed openstack/nova master: LM: Use Claims to update numa-related XML on the source  https://review.opendev.org/63522903:43
openstackgerritArtom Lifshitz proposed openstack/nova master: NUMA live migration support  https://review.opendev.org/63460603:43
openstackgerritArtom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration  https://review.opendev.org/64002103:43
openstackgerritArtom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration  https://review.opendev.org/67259503:43
openstackgerritArtom Lifshitz proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888703:43
*** brinzhang has quit IRC03:49
*** brinzhang has joined #openstack-nova03:50
*** brinzhang has joined #openstack-nova03:51
*** markvoelker has joined #openstack-nova04:05
*** markvoelker has quit IRC04:10
*** dpawlik has joined #openstack-nova04:13
*** sapd1 has quit IRC04:14
*** dpawlik has quit IRC04:17
*** tkajinam_ has joined #openstack-nova04:37
*** itlinux has joined #openstack-nova04:39
*** tkajinam has quit IRC04:40
*** dave-mccowan has quit IRC04:40
*** brault has joined #openstack-nova04:50
*** itlinux has quit IRC04:52
*** brault has quit IRC04:54
*** itlinux has joined #openstack-nova05:10
openstackgerritmelanie witt proposed openstack/nova master: Remove unused args from archive_deleted_rows calls  https://review.opendev.org/67898805:14
*** udesale has joined #openstack-nova05:14
*** itlinux has quit IRC05:19
*** ratailor has joined #openstack-nova05:19
*** markvoelker has joined #openstack-nova05:20
*** markvoelker has quit IRC05:25
*** jhesketh has quit IRC05:34
*** udesale has quit IRC05:38
*** yaawang has quit IRC05:39
openstackgerritFan Zhang proposed openstack/nova master: Fix exception translation when creating volume  https://review.opendev.org/67899105:40
*** yaawang has joined #openstack-nova05:40
*** jhesketh has joined #openstack-nova05:42
openstackgerritLuyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table  https://review.opendev.org/67844705:45
openstackgerritLuyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs  https://review.opendev.org/67844805:45
openstackgerritLuyao Zhong proposed openstack/nova master: Add resources dict into _Provider  https://review.opendev.org/67844905:45
openstackgerritLuyao Zhong proposed openstack/nova master: Retrive the allocations early  https://review.opendev.org/67845005:45
openstackgerritLuyao Zhong proposed openstack/nova master: Claim resources in resource tracker  https://review.opendev.org/67845205:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces  https://review.opendev.org/67845305:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree  https://review.opendev.org/67845405:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup  https://review.opendev.org/67845505:45
openstackgerritLuyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec  https://review.opendev.org/67845605:45
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847005:45
*** udesale has joined #openstack-nova05:45
*** udesale has quit IRC05:45
*** udesale has joined #openstack-nova05:45
*** Luzi has joined #openstack-nova05:46
openstackgerritDustin Cowles proposed openstack/nova master: Provider config file schema and loader  https://review.opendev.org/67334105:49
openstackgerritDustin Cowles proposed openstack/nova master: Public method to retrieve custom resource providers  https://review.opendev.org/67602905:49
openstackgerritDustin Cowles proposed openstack/nova master: Load the custom resource providers to resource tracker  https://review.opendev.org/67652205:49
openstackgerritzhufl proposed openstack/nova master: [Trivial]Remove unused helper get_allocated_disk_size  https://review.opendev.org/67897605:59
*** lpetrut has joined #openstack-nova06:00
*** dpawlik has joined #openstack-nova06:14
*** shilpasd has joined #openstack-nova06:18
*** udesale has quit IRC06:26
*** udesale has joined #openstack-nova06:27
*** slaweq has joined #openstack-nova06:27
*** yaawang has quit IRC06:31
*** udesale has quit IRC06:31
*** slaweq has quit IRC06:32
*** yaawang has joined #openstack-nova06:33
*** slaweq has joined #openstack-nova06:48
*** macz has joined #openstack-nova06:52
*** macz has quit IRC06:56
dr_gogeta86hi07:07
*** trident has quit IRC07:08
dr_gogeta86nobody know a common pattern between nova-compute dying and rabbitmq connections falls to zero07:09
dr_gogeta86after a while from 10 to 30 minutes07:09
dr_gogeta86nova-compute kills all rabbitmq07:09
dr_gogeta86and going stuck07:09
*** rcernin has quit IRC07:12
*** trident has joined #openstack-nova07:17
*** xek has joined #openstack-nova07:17
openstackgerritZhengMa proposed openstack/nova master: Check container_format on instance creation  https://review.opendev.org/67340707:17
*** ivve has joined #openstack-nova07:28
bauzasgood morning Nova07:32
*** aojea has joined #openstack-nova07:42
*** aojeagarcia has joined #openstack-nova08:00
*** aojea has quit IRC08:01
*** tkajinam_ has quit IRC08:02
*** jangutter has joined #openstack-nova08:03
*** ccamacho has joined #openstack-nova08:04
*** cdent has joined #openstack-nova08:05
*** dougsz has joined #openstack-nova08:08
*** mdbooth_ has joined #openstack-nova08:10
openstackgerritLuyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker  https://review.opendev.org/67845108:11
*** ralonsoh has joined #openstack-nova08:24
*** jaosorior has quit IRC08:28
*** jaosorior has joined #openstack-nova08:29
*** kashyap has joined #openstack-nova08:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth  https://review.opendev.org/67697208:33
openstackgerritBalazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth  https://review.opendev.org/67698008:33
openstackgerritBalazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request  https://review.opendev.org/67149708:35
kashyapaspiers: Thanks for the PS-6 changes here, just got back: https://review.opendev.org/#/c/673790/6/08:38
kashyapaspiers: I _do_ owe a couple of drinks!08:39
openstackgerritBalazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize  https://review.opendev.org/67882708:40
*** derekh has joined #openstack-nova08:43
*** jaosorior has quit IRC08:46
aspierskashyap: yw :)09:00
kashyapaspiers: Investigating the "weird" thing you noticed: https://review.opendev.org/#/c/673790/6/nova/tests/unit/virt/libvirt/test_config.py@21609:01
*** mtreinish has quit IRC09:13
openstackgerritBrin Zhang proposed openstack/nova master: Add user_id and project_id colume to Migration  https://review.opendev.org/67399009:13
*** yedongcan has joined #openstack-nova09:13
*** brault has joined #openstack-nova09:17
openstackgerritya.wang proposed openstack/nova master: vCPU model selection  https://review.opendev.org/67029809:22
openstackgerritya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags  https://review.opendev.org/67029909:22
openstackgerritya.wang proposed openstack/nova master: Support report multi CPU model traits  https://review.opendev.org/67030009:22
*** sapd1 has joined #openstack-nova09:24
*** jaosorior has joined #openstack-nova09:31
*** dougsz has quit IRC09:35
*** yedongcan has quit IRC09:37
aspierskashyap: replied09:45
aspiersstupid question - where's the thing which does linting on our .rst files?09:45
aspiersI can't see it in tox.ini09:45
kashyapaspiers: Thanks; will look09:46
kashyapaspiers: Hmm, I'm unclear why os.loader.features doesn't contain the given keys doesn't make sense.09:47
*** dougsz has joined #openstack-nova09:51
openstackgerritBalazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request  https://review.opendev.org/67149709:57
openstackgerritBalazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize  https://review.opendev.org/67882710:00
sean-k-mooneystephenfin: i noticed another helper function for nomalising tratis i can share between the prefilter and the driver code so im going to pull that out and then respin the first patch to adress your nits10:00
stephenfincool cool10:00
stephenfingot some nits on the following one too if you want to wait 5 mins10:00
sean-k-mooneyalso the verion in the prefilter has a bug10:01
sean-k-mooneysure im just going to make coffee anyway10:01
sean-k-mooneywant me to wait untill you review all tree10:01
sean-k-mooneythen ill adress all your comments10:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request  https://review.opendev.org/67901910:02
*** macz has joined #openstack-nova10:05
*** macz has quit IRC10:09
stephenfinsean-k-mooney: done10:11
sean-k-mooneythanks just looking at the scond one now10:12
openstackgerritStephen Finucane proposed openstack/nova master: Add reshaper for PCPU  https://review.opendev.org/67489510:16
bauzasstephenfin: wow, chilling ^10:22
* bauzas will use his red pen later in the afternoon10:23
sean-k-mooneybauzas: is that like something taht is almost cool10:23
bauzasI should wear a t-shirt saying "I love reshapes"10:23
sean-k-mooneyif you did we would make you write all of them10:24
bauzas\o/10:24
bauzasanyway, lunch now10:24
*** ricolin has quit IRC10:25
*** tbachman has quit IRC10:43
openstackgerritLuyao Zhong proposed openstack/nova master: Claim resources in resource tracker  https://review.opendev.org/67845210:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces  https://review.opendev.org/67845310:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree  https://review.opendev.org/67845410:45
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup  https://review.opendev.org/67845510:45
openstackgerritLuyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec  https://review.opendev.org/67845610:45
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847010:45
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847011:03
*** cdent has quit IRC11:05
*** nicolasbock has joined #openstack-nova11:08
*** tesseract has joined #openstack-nova11:12
*** sapd1 has quit IRC11:14
sean-k-mooneystephenfin: i hope that answers your question. https://review.opendev.org/#/c/665775/14/nova/scheduler/request_filter.py@176 im going to start reworkign the patches now but your comments make sense11:22
*** dtantsur|afk is now known as dtantsur11:26
openstackgerritBrin Zhang proposed openstack/nova master: Add user_id and project_id colume to Migration  https://review.opendev.org/67399011:28
*** markvoelker has joined #openstack-nova11:43
aspierskashyap: I'm getting a parse error on your last sentence :)11:51
*** shilpasd has quit IRC11:54
*** brinzhang has quit IRC11:57
*** brinzhang has joined #openstack-nova11:57
*** jaosorior has quit IRC11:58
*** dpawlik has quit IRC12:02
aspierskashyap: but what I don't understand is why a list of OS bootloader features would be subdivided into two sections called 'type' and 'readonly'12:06
*** ratailor has quit IRC12:06
aspiers'type' makes no sense as a category12:07
*** jaosorior has joined #openstack-nova12:11
*** cdent has joined #openstack-nova12:15
*** shilpasd has joined #openstack-nova12:17
*** nweinber has joined #openstack-nova12:18
kashyapaspiers: Err, sorry for the garbled sentence; I meant, why doesn't os.loader.features with type|readonly make sense.12:20
kashyapaspiers: Okay, you don't think 'type' makes sense as as category...12:20
kashyapaspiers: Anyway, I'll think a bit more on what I can do there.12:24
aspierskashyap: 'type' makes as much sense as a category as 'category' does12:26
aspiersi.e. none12:26
kashyapaspiers: Hmm.  As you'll recall, it's the "type" ("category"? :-)) of loader.  I'll see about reworking12:27
*** tbachman has joined #openstack-nova12:29
*** dpawlik has joined #openstack-nova12:30
*** larainema has quit IRC12:30
*** brinzhang has quit IRC12:39
kashyapaspiers: Also on your "not understanding" why a list of OS bootloader features would be subdivided into the two sections (enums) -- ask libvirt upstream :D12:41
kashyapIt _sounds_ like you temporarily forgot (but it's in the commit message, and you've seen it before :-)) the output spit out by getDomainCapabilities()12:42
* kashyap will now shut up tinker12:43
*** ttsiouts has joined #openstack-nova12:43
*** mgariepy has joined #openstack-nova12:44
*** mdbooth_ has quit IRC12:48
*** ricolin has joined #openstack-nova12:50
*** mdbooth has joined #openstack-nova12:58
alex_xuaspiers: stephenfin in the end, I confused what the descision at here https://review.opendev.org/#/c/664420/42/nova/scheduler/request_filter.py@20613:05
alex_xuin the end, we said use req filter or not...13:06
*** gbarros has joined #openstack-nova13:06
*** mriedem has joined #openstack-nova13:09
*** dave-mccowan has joined #openstack-nova13:11
stephenfinalex_xu: I thought we were going to copy the approach from https://review.opendev.org/#/c/671801/32/nova/scheduler/utils.py13:24
stephenfinso that's why I dragged https://review.opendev.org/#/c/674894/ to the front of the series13:24
stephenfinmriedem, dansmith: You have been summoned by efried https://review.opendev.org/#/c/675338/13:25
stephenfinalex_xu: Yeah, that's what efried's comments are saying too. I guess aspiers is waiting to make sure that gets in first13:26
mriedemstephenfin: why?13:27
stephenfinHaven't a notion13:27
*** mdbooth has quit IRC13:27
mriedemthe extra spec in those api samples could just as well be {'fart': 'sniffer'}13:27
stephenfinMaybe because it's microversion related13:28
stephenfinYup, that's what I thought13:28
mriedemit's not really,13:28
mriedembut i see a good reason to -1 the shit out of your patch13:28
mriedemoh drat https://review.opendev.org/#/c/675338/11/nova/tests/fixtures.py13:28
mriedemyou already got it13:28
stephenfin#winning13:29
mriedem+213:30
*** mdbooth has joined #openstack-nova13:35
*** itlinux has joined #openstack-nova13:37
mriedemmdbooth: for https://review.opendev.org/#/c/677736/ i think we're just waiting for your customer to test it and ack it right?13:38
aspiersstephenfin, alex_xu, efried: my understanding was that we were going to keep it in req filter for now and then change it later13:38
aspierskashyap: I don't have the slightest issue with features being split into two *meaningful* sections. But a section entitled "type" is totally meaningless.13:39
mdboothmriedem: So... they did test it but they didn't provide the same logs as last time. I've asked them for them, but they haven't been provided yet.13:39
mriedemgibi: can i get a re-+W on https://review.opendev.org/#/c/637605/37 ?13:39
mriedemmdbooth: did they notice the slow down?13:40
aspierskashyap: Let me take another tack. Please explain to me what this feature section entitled "type" is supposed to contain.13:40
aspierskashyap: And while you're at it, I'd like to understand what the section entitled "readonly" is supposed to contain.13:40
mdboothmriedem: Previously they were seeing compute hosts flapping, but that was only logged by n-api. We don't have logs for n-api.13:41
mriedemstephenfin: i've re-ordered some of the less complicated changes in the cross-cell resize series to the bottom, could you take a look at this simple one to plumb passing the power_on flag down through driver.spawn()? https://review.opendev.org/#/c/642590/3013:41
stephenfinI sure can13:41
* stephenfin clicks13:41
mriedemwhich incidentally is something i think mdbooth has asked for in the past - the ability to create a stopped server13:41
mdboothmriedem: They *are* experience other environmental issues.13:41
mriedemor maybe i'm dreaming that up13:41
mdboothmriedem: Yep, I have asked for that.13:41
kashyapaspiers: I agree that a generic section is meaningless.  Maybe 'loader_type', which tells _where_ the guest memory should be mapped -- whether it s a 'rom' or 'pflash' ?13:41
mdboothmriedem: So what I wanted to see be sure is: in the presence of their environmental issues, are the compute nodes continuing to flap?13:42
mdboothI'm hoping the answer's no.13:42
mriedemright13:42
aspierskashyap: https://libvirt.org/formatdomaincaps.html#elementsOSBIOS doesn't even make sense here13:42
mdboothHowever, I'm not convinced it's worth waiting, tbh.13:42
aspierskashyap: It says of "type": "Whether loader is a typical BIOS (rom) or an UEFI binary (pflash). This refers to type attribute of the <loader/> element."13:43
mdboothIt's a pretty clear bug, and the fix doesn't break anything we test, or a real environment.13:43
aspierskashyap: so it's portraying it as an either/or choice, but the schema places no restriction on the number of <value> elements, and indeed the example even lists *both* rom and pflash13:43
aspierskashyap: Also, there is no "type attribute of the <loader/> element"13:44
aspierskashyap: the docs for 'readonly' are even worse13:44
kashyapaspiers: The docs _suck_ in this area13:44
aspierskashyap: and the example gives both yes and no as simultaneous values ... I mean, WT actual F13:45
kashyapaspiers: On the either/or choice -- yeah, the docs say that, but the damn 'enum' implies there _could_ be more in future.  Grrr13:45
aspiersIf it is supposed to be a choice between rom and pflash (and maybe others) then it should be <type>rom</type> or <type value="rom"/>13:46
aspiersLikewise <readonly>yes</readonly> or <readonly value="yes"/> etc.13:46
kashyapaspiers: Yeah, the upstream schema as it stands is not really ideal :-(13:46
aspiersbut a mapping from 'type' / 'readonly' to a list of values makes no sense13:46
aspiersIf multiple concurrent types are really allowed then I could get on board with <types>...</types>, or maybe even just about tolerate <enum name='types'>...</enum>13:47
aspiersbut 'type' (singular) rather than 'types' (plural) ... no13:48
*** ttsiouts has quit IRC13:48
*** ttsiouts has joined #openstack-nova13:48
kashyapRight, so the "reality" is: it is a binary choice as it stands, not a multiple types at the _same_ time.13:49
aspierskashyap: Do you know whether rom and pflash are mutually exclusive values? Or can it actually be both at once?13:49
aspiersWhat is actually valid here?13:49
kashyapaspiers: Let me check with Michal Privoznik13:49
kashyap(libvirt dev)13:49
*** itlinux has quit IRC13:49
aspierskashyap: Thanks. Maybe it's more that the example XML given is wrong13:49
aspierskashyap: I can imagine that they might have written that to show all the <value>s which are allowed, without realising that putting them all together at once is deeply confusing.13:50
*** eharney has joined #openstack-nova13:50
stephenfinmriedem: We don't need to post to the mailing list when we add a parameter to the virt driver interface like that, do we?13:51
stephenfin(the 'power_on' parameter)13:51
kashyapaspiers: I completely agree on the confusion there, libvirt upstream definitely needs to address this.13:51
aspierskashyap: thanks for the moral support ;-)13:52
aspierskashyap: I've just pinged Jim Fehlig about it too13:52
mriedemstephenfin: "need" to? no. but i can.13:53
*** ttsiouts has quit IRC13:53
artomstephenfin, depends on who you ask, but it won't hurt13:53
kashyapaspiers: Yeah, it is much needed frustration.  As you can see, if we're duking around this damn schema so much, it's clearly not us :D13:53
kashyaps/much needed/understandable/13:53
stephenfinmriedem: Yeah, I think it would be helpful at least, if you have time13:53
*** priteau has joined #openstack-nova13:55
mriedemstephenfin: done13:56
stephenfinta13:56
*** aojeagarcia has quit IRC13:58
aspierskashyap: Actually this is not just a libvirt issue. The confusion is significantly compounded by placing 'type' and 'readonly' under os.loader.features.13:58
aspierskashyap: It should be os.loader.type and os.loader.readonly instead.13:58
kashyapaspiers: In the Nova config class?13:58
aspierskashyap: yes13:58
aspiers'type' is not a feature13:59
aspierskashyap: Imagine future code like: if 'rom' in os.loader.features['type']: ...14:00
aspierskashyap: vs. if os.loader.type == 'rom':14:00
*** eharney has quit IRC14:00
aspiersor: if 'rom' in os.loader.types: ...14:00
aspiersThe latter two are instantly understandable; the former is a #wtf moment14:01
kashyapaspiers: Nod.  Okay, first step is to untangle this by moving type/readonly under os.loader, rather than os.loader.features14:02
aspiersPublic safety warning: I'm probably using overly antagonistic language right now, due to the mood which the ongoing clusterfuck in British politics has just put me into. Sorry in advance for any offense caused.14:02
aspierskashyap: Yes that would be a very good step I think14:03
aspierskashyap: and then if multiple types are allowed, we can ignore <type> in the libvirt schema and use os.loader.types (plural) instead14:03
kashyapaspiers: Okay, Michal confirms - rom|pflash _are_ mutually exclusive.  (As we guessed)14:04
aspiersAhah :-)14:04
aspiersSo the example XML is wrong14:04
aspiersand hence our test fixtures too14:04
aspierskashyap: What about 'readonly' and 'secure'?14:05
kashyapmprivozn: Yeah, you can't have secure "yes" and "no" at the same time -- that'll be "Schrödinger's XML"...14:06
kashyapaspiers: They are also mutually exclusive.14:06
kashyapaspiers: Okay, so on the example, this is what Michal says on _why_ both of them are listed:14:06
aspierskashyap: mprivozn isn't here ;-) wrong window?14:06
kashyapaspiers: Here:14:07
kashyap16:04 < kashyap> mprivozn: Then, I'll submit a docs change to show just _one_ value under type|readonly|secure: https://libvirt.org/formatdomaincaps.html#elementsOSBIOS14:07
kashyap16:05 < mprivozn> kashyap: please don't14:07
kashyap16:05 < kashyap> (Because accepted values are shown in the below text)14:07
kashyap16:05 < kashyap> mprivozn: Hmm, why not?  You wanted them there to show what are the possible values?14:07
kashyap16:05 < mprivozn> kashyap: the idea for domaincaps is to enumerate all accepted values; listing just one variant goes against that14:07
kashyap16:06 < mprivozn> kashyap: forget OSBIOS for a second and take a look at some other examples, diskDevice for instance14:07
kashyapaspiers: So following the above example, diskDevice lists 4 different values (disk, cdrom, floppy, lun).14:08
aspiersOK, so all the docs are wrong not just this?14:09
kashyapaspiers: I.e. it can be _one_ of those values.14:09
aspiersI'm sorry, but docs giving example XML which doesn't even validate against the schema (notional or otherwise) does not make sense14:09
kashyapaspiers: No, the dev says docs are not wrong, and they intentionally enumerated as such14:09
aspiersThe allowed values can be listed *below* the example XML, or as comments in it14:09
kashyapaspiers: See this point above: "the idea for domaincaps is to enumerate all accepted values; listing just one variant goes against that"14:09
aspiersby "domaincaps" he means https://libvirt.org/formatdomaincaps.html ?14:10
aspiersor something else?14:10
kashyapaspiers: Yes, indeed14:10
aspiersshould we move this to #libvirt?14:10
kashyap#virt, on OFTC, yes14:10
aspiersoh OK14:10
sean-k-mooneykashyap: isnt #virt internal?14:11
kashyapsean-k-mooney: No.  #virt is the community channel for libvirt on OFTC14:11
aspiersweird, why not #libvirt?14:11
kashyap(I mean, yes, too.  But external exists, too)14:11
kashyapaspiers: Because it includes adjacent projects like virt-manager, virt-tools, libosinfo, etc, in addition to libvirt14:12
*** cdent has quit IRC14:16
*** cdent has joined #openstack-nova14:16
*** cdent has quit IRC14:22
mriedemstephenfin: replied on https://review.opendev.org/#/c/637605/3714:22
*** openstackgerrit has quit IRC14:22
mriedemand up, thanks for hitting those14:22
dansmithmriedem: you wanna look at artom's latest changes on this per our discussion yesterday and opine on whether or not you can tolerate the hackery? https://review.opendev.org/#/c/635669/4514:23
stephenfinAh, I meant would it make sense to add a 'should_raise' parameter to the 'delete' function (i.e. make it optional). It was a nit though14:24
stephenfinshould_raise or something along those lines14:24
mriedemdansmith: sure14:26
*** markvoelker has quit IRC14:26
dansmithartom: left some clarifying questions in there, btw14:29
*** eharney has joined #openstack-nova14:29
*** markvoelker has joined #openstack-nova14:29
*** dpawlik has quit IRC14:29
dansmithartom: have you looked at why that DNM tempest test failed the live migration job?14:32
dansmithlooks like during setup, so maybe just something in the script14:33
dansmither, no14:33
*** amotoki_ has quit IRC14:34
*** amotoki has joined #openstack-nova14:35
*** markvoelker has quit IRC14:37
*** markvoelker has joined #openstack-nova14:38
*** Luzi has quit IRC14:40
mriedemhttps://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/logs_87/678887/6/check/nova-live-migration/d3d4357/ara-report/result/092050bb-201e-4657-ac3f-16aebd4578ff/ failed but not sure why yet14:40
mriedemoh the job timed out14:40
mriedem/home/zuul/workspace/devstack-gate/functions.sh: line 1018:  5091 Killed                  timeout -s 9 ${REMAINING_TIME}m bash -c "source $WORKSPACE/devstack-gate/functions.sh && $cmd"14:40
dansmithyeah, but I figure it might be because it was polling for that to go active or something14:41
dansmithlooks like it started building an instance right around the on the subnode14:43
mriedemthe funny thing is,14:44
mriedemit doesn't even seem to run the live migration stuff14:44
mriedemit goes right to the test_evacuate.sh script14:44
*** gbarros has quit IRC14:44
dansmithyeah14:45
dansmithI didn't look to see the ordering14:45
mriedem2019-08-28 04:37:56.230 | 1. test with all local storage (use default for volumes) 2019-08-28 04:37:56.232 | NOTE: test_volume_backed_live_migration is skipped due to https://bugs.launchpad.net/nova/+bug/1524898 2019-08-28 04:37:57.817 | Missing value auth-url required for auth plugin password 2019-08-28 04:37:57.909 | + environment:post_test_hook:L1:   /opt/stack/new/nova/gate/test_evacuate.sh14:46
openstackLaunchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,In progress] - Assigned to Lee Yarwood (lyarwood)14:46
mriedemmaybe the flavor creation is blowing up?14:46
dansmithah14:46
mriedemwe're losing the tracing too14:47
dansmithI still wanna know why this instance didn't build in case it is related to tweaking the claim stuff,14:47
mriedemfrom one of the sourced scripts14:47
dansmithbut it seems like it's waiting for the neutron event14:47
dansmithit's still in building state at the end of the run14:48
dansmithah, artom said something about that early this morning..14:48
mriedeminstance c2b876e6-b616-4259-b03b-f0b2bd3d8908 right?14:49
dansmithyup14:49
mriedemyup start waiting for the event here https://zuul.opendev.org/t/openstack/build/d3d4357c13be452fa8e64a6b612c256d/log/logs/subnode-2/screen-n-cpu.txt.gz#190514:50
mriedemAug 28 04:38:17.325822 ubuntu-bionic-vexxhost-vexxhost-sjc1-0010477321 nova-compute[24765]: DEBUG nova.compute.manager [None req-4d7771be-f2e2-429d-87c7-1cd4469cede5 demo admin] [instance: c2b876e6-b616-4259-b03b-f0b2bd3d8908] Preparing to wait for external event network-vif-plugged-afd502d7-a21d-48f2-8607-881a86f0781f {{(pid=24765) prepare_for_instance_event /opt/stack/new/nova/nova/compute/manager.py:329}}14:50
mriedemgot the event here14:51
mriedemhttps://zuul.opendev.org/t/openstack/build/d3d4357c13be452fa8e64a6b612c256d/log/logs/subnode-2/screen-n-cpu.txt.gz#194114:51
mriedemAug 28 04:38:21.643105 ubuntu-bionic-vexxhost-vexxhost-sjc1-0010477321 nova-compute[24765]: DEBUG nova.compute.manager [req-61851260-be42-4748-9d79-4b0ec5c08ed6 req-3c63a866-a638-4ec7-9f56-3a7aeb2e9958 service nova] [instance: c2b876e6-b616-4259-b03b-f0b2bd3d8908] Received event network-vif-plugged-afd502d7-a21d-48f2-8607-881a86f0781f {{(pid=24765) external_instance_event /opt/stack/new/nova/nova/compute/manager.py:8819}}14:51
mriedem4 seconds later14:52
*** gbarros has joined #openstack-nova14:52
*** lpetrut has quit IRC14:52
dansmithpresumably unrelated to his change though if we've made it to the neutron event stage14:52
mriedemi think during artom's revert resize ovs hybrid plug race thing that evacuate could have the same kind of race b/c of the time that we trigger the event by updating the port binding and when the driver's spawn method start waiting for it14:54
mriedemalthough, if we didn't have a handler registered for the event callback we'd see "Received unexpected event" in the logs14:55
mriedemwhich we don't14:55
mriedemso idk what's going on there14:55
*** mlavalle has joined #openstack-nova14:55
mriedemthe guest power state goes from paused > running > paused agin14:56
mriedem*agian14:56
mriedemgah14:56
mriedembut still, i'm wondering why the run_tests script didn't actually run any live migration tests14:57
*** igordc has joined #openstack-nova14:57
kashyapaspiers: When you get a sec, summarized our discussion from #virt (OFTC) here, for other Stackers: https://review.opendev.org/#/c/673790/14:58
*** mkrai_ has joined #openstack-nova14:59
*** openstackgerrit has joined #openstack-nova15:01
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888715:01
*** mkrai has quit IRC15:01
*** shilpasd has quit IRC15:02
mriedemdansmith: artom: let's see if ^ helps15:02
*** dpawlik has joined #openstack-nova15:02
dansmithcool15:02
melwittmriedem: follow up patch is here https://review.opendev.org/67898815:05
*** dpawlik has quit IRC15:07
melwittdansmith: the multi-cell archive_deleted_rows set (this patch and the one on top of it) have a +2 from mriedem, if you would fancy taking a look please15:07
melwitthttps://review.opendev.org/50748615:07
mriedemgot it15:08
melwittdanke15:09
dansmithmelwitt: I know, I've had it open just have to get to it15:14
melwittok, thanks. sorry didn't know you already knew15:15
*** lpetrut has joined #openstack-nova15:15
dansmithyeah, I opened the tab when mriedem pinged us yesterday15:15
melwittk15:18
*** markvoelker has quit IRC15:18
*** donnyd has quit IRC15:19
*** donnyd has joined #openstack-nova15:20
*** donnyd has quit IRC15:20
*** markvoelker has joined #openstack-nova15:21
*** donnyd has joined #openstack-nova15:21
*** hemna has joined #openstack-nova15:22
mriedemartom: did you ever follow up with getting something on the starlingx weekly meeting about testing the numa LM series?15:22
*** ivve has quit IRC15:24
*** gyee has joined #openstack-nova15:26
*** ash2307 has joined #openstack-nova15:28
*** ash2307 has left #openstack-nova15:28
mriedemjust realized that we have 41 approved blueprints for train with only 12 complete and 2 weeks to FF15:30
mriedemprobably time to start deferring things that we know aren't going to land15:31
*** mkrai_ has quit IRC15:32
donnydso yesterdays test of mounting _base on a shared location didn't go well15:34
*** dpawlik has joined #openstack-nova15:34
donnydfor some unknown reason qemu-img info reports back that the cached image could not be found15:35
*** dpawlik has quit IRC15:39
*** niceplace_ has joined #openstack-nova15:39
*** hemna has quit IRC15:40
*** hemna has joined #openstack-nova15:41
*** cdent has joined #openstack-nova15:46
*** panda is now known as panda|rover15:47
mriedemstephenfin: can we effectively mark this complete? https://blueprints.launchpad.net/nova/+spec/remove-cells-v116:00
mriedemthere are only 2 open changes https://review.opendev.org/#/q/topic:bp/remove-cells-v1+(status:open)16:00
openstackgerritMerged openstack/nova master: Specify availability_zone to unshelve  https://review.opendev.org/66385116:00
stephenfinYup, it's complete. Those two are very tricky post-removal cleanup patches16:00
stephenfinI'll come back to them when I can16:01
cdentcells v1 is gone? wooooooo!16:03
mriedemmelwitt: would be sweet if we can get https://review.opendev.org/#/c/640898/ approved by the end of the week - it's 2 weeks to FF and it'd be nice to land that series in time for train16:03
*** ricolin has quit IRC16:04
*** ricolin has joined #openstack-nova16:04
melwittmriedem: indeed, I'm gonna update it after this last call I'm on. thanks for reminding16:04
*** tesseract has quit IRC16:08
dansmithmriedem: are you pretty confident on this archive multi cells thing? I've skimmed the logic and tests and just can't really get myself to go deep on it16:09
dansmithseems pretty low risk even if there is something wrong in here, so I'm tempted to just +W based on my skimming16:09
dansmiths/seems/should be/16:09
*** ricolin has quit IRC16:12
openstackgerritMerged openstack/nova master: scheduler: Flatten 'ResourceRequest.from_extra_specs', 'from_image_props'  https://review.opendev.org/67489416:12
openstackgerritMerged openstack/nova master: Remove 'hw:cpu_policy', 'hw:mem_page_size' extra specs from API samples  https://review.opendev.org/67533816:12
mriedemdansmith: bombs away16:12
mriedemand yeah i've gone over it several times in detail now16:13
mriedemprobably annoying detail for melwitt16:13
mriedemthe nova-next change on top also shows it working as expected16:13
dansmithoh sweet I didn't even notice16:15
*** efried is now known as efried_rollin16:16
dansmithgot those as well16:16
*** itlinux has joined #openstack-nova16:17
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add online migration for legacy NUMA objects  https://review.opendev.org/53741416:19
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Rename 'nova.tests.unit.test_nova_manage'  https://review.opendev.org/67909516:19
openstackgerritStephen Finucane proposed openstack/nova master: Ensure online migrations have a unique name  https://review.opendev.org/67909616:19
stephenfinmriedem: You'll probably be interested in the last of those ^16:19
*** itlinux is now known as itlinux-away16:20
dansmithmriedem: 2019-08-28 16:12:53.222764 | primary | 2019-08-28 16:12:53.222 | Missing value auth-url required for auth plugin password16:21
dansmithmriedem: on the grenade lm job16:21
dansmithmriedem: the regular one failed with node_failure16:21
*** itlinux-away has quit IRC16:25
*** brault has quit IRC16:27
melwittthanks dansmith++16:29
*** N3l1x has joined #openstack-nova16:32
artommriedem, dansmith, was on calls in the car all morning, catching up on scrollback16:33
artommriedem, crap, forgot about that starlingx meeting thing :(16:33
*** macz has joined #openstack-nova16:33
artomThanks for reminding me16:33
artom(And right now I need to drive daughter to school, so I need to run again)16:35
artomThough I'll try to respond to dansmith on that initial claims patch before I do so16:35
dansmithartom: we should try to get that test patch running again first16:36
dansmithI'm guessing we16:36
dansmithjust need to source openrc first somewhere16:36
artomdansmith, yeah, I need to look at that too - sean-k-mooney seemed to also have ideas16:38
dansmithgI think I got it16:38
openstackgerritDan Smith proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888716:39
dansmithpulled a couple lines from the ceph hook which use osc16:39
dansmithartom: I think his ideas were about making it suitable for all time, and that's not the point of this :)16:39
artomdansmith, heh, indeed, quick and dirty all the way :D16:40
sean-k-mooneywell i was just suggestin using the way i know works16:40
sean-k-mooneybut also yes they are the less hacky way of doing it16:40
sean-k-mooneyif you way works thats good enough for this16:40
dansmithI wish we could control which job runs on a patch.. like if the commit message starts with "DNM" then zuul can -2 it and only run tests the commit message asks for or something16:41
artomdansmith, https://review.opendev.org/#/c/635669/45 there, hope that answers is16:46
artom*it16:46
dansmithartom: yeah, although I'm not sure why we need the call out if the current ordering will work16:49
dansmithespecially if we plan to remove it from the sriov path and have it in here in the long term16:49
mriedemdansmith: we could also make that DNM change run faster but commenting out all other jobs except the ones we want to run16:49
* mriedem goes to get a haircut16:49
dansmithmriedem said he would go through the hackery in there and see if he could stomach it16:49
*** mriedem is now known as mriedem_away16:49
dansmithpost-haircut Iguess16:49
mriedem_awayi got distracted with the DNM debugging earlier16:50
dansmiththat's fine,16:50
dansmithI want to see that test run anyway16:50
artomdansmith, I don't like depending on an ordering that's so far away, but maybe we could call it out in a comment in the compute manager, and lest _test_pci() do its thing16:51
artom*let16:51
dansmithartom: can you just remove the other accounting and rely on this one?16:52
dansmithI mean, I don't want to go all into refactoring it, just curious16:52
artomdansmith, that would be a larger effort to make SRIOV live migration use the move claim16:52
dansmithand yes, not relying on ordering is good, but..16:52
artomWhich should get done, but probably not as part of this series16:52
dansmithI'm not sure I understand why it's any different16:53
dansmithbecause SRIOV migration is not a different operation, so I would expect you'd still hit all this code and be fine16:53
dansmithbut mkay16:53
artomI guess I'm just being overly cautious, and don't want to start a potential rabbit hole at this stage?16:53
dansmithyeah for sure16:53
dansmithjust... you said it would work with the current ordering16:54
artomOh, you mean that? yeah, we could do that16:54
dansmithno,16:54
dansmithI'm taking that to its logical (to me) conclusion16:54
dansmithanyway,16:54
dansmiththis is why we really should have been working on this set at the beginning of a cycle and not the end16:54
dansmithso it's frustrating to have to bake in debt like this because of the timeline,16:55
artomYeah16:55
dansmithespecially if you're going to be busy again early next cycle such that it never gets cleaned up16:55
artomI mean, there's no guarantee that we would have stumbled on this earlier, but it's a true thing as a matter of principle, so I can't disagree16:55
artomAnyways, I gotta run for first school day16:56
artomWell, half-day16:56
*** artom has quit IRC16:56
*** derekh has quit IRC16:57
openstackgerritMerged openstack/nova master: Add nova.compute.utils.delete_image  https://review.opendev.org/63760516:57
openstackgerritMerged openstack/nova master: Refactor ComputeManager.remove_volume_connection  https://review.opendev.org/64218316:58
*** dougsz has quit IRC17:01
*** lbragstad has joined #openstack-nova17:01
*** itlinux has joined #openstack-nova17:02
*** spatel has joined #openstack-nova17:06
spatelsean-k-mooney: hey!! afternoon17:06
*** itlinux has quit IRC17:07
*** dtantsur is now known as dtantsur|afk17:08
*** gbarros has quit IRC17:14
donnydDo people connect cdrom devices for reasons other than config-drive in most cases?17:15
dansmithmriedem_away: numa flavor got created after my change17:21
*** panda|rover is now known as panda|rover|off17:23
*** gbarros has joined #openstack-nova17:24
jrollso I'm doing GPU passthrough with a few different devices. one of them apparently exposes some SRIOV PFs, so it's presenting as `type-PF` rather than `type-PCI`, and so passthrough isn't working with a `pci_passthrough` property on the flavor. do I need to like, figure out how to do SRIOV and go that route, or is there a way to tell nova to maek it just a regular PCI device?17:25
dansmithmriedem_away: and...failed to boot an instance with it17:25
*** brault has joined #openstack-nova17:30
*** gbarros has quit IRC17:33
*** brault has quit IRC17:34
*** dpawlik has joined #openstack-nova17:35
*** dpawlik has quit IRC17:40
*** cdent has quit IRC17:42
*** eharney_ has joined #openstack-nova17:43
*** eharney has quit IRC17:44
*** brault has joined #openstack-nova17:44
*** brault has quit IRC17:45
*** cdent has joined #openstack-nova17:46
*** eharney_ is now known as eharney17:46
*** jmlowe has quit IRC17:51
*** mmethot has quit IRC17:58
*** mmethot has joined #openstack-nova17:58
*** jmlowe has joined #openstack-nova18:06
sean-k-mooneydansmith:  Failed to start libvirt guest: libvirtError: Requested operation is not valid: cpu affinity is not supported18:12
*** priteau has quit IRC18:12
*** ociuhandu has joined #openstack-nova18:12
sean-k-mooney <domain type='qemu'>18:12
sean-k-mooneysetting the virt_type to kvm did not work18:13
*** psachin has quit IRC18:13
sean-k-mooneyyep its set to qemu18:13
sean-k-mooneyhttps://d8076f0c34babfa2d8fa-c3c714f53a28264bde7e35c9b01061ab.ssl.cf2.rackcdn.com/678887/8/check/nova-live-migration/21801ca/logs/etc/nova/nova-cpu.conf.txt.gz18:13
sean-k-mooneywhich is why the job failed18:14
sean-k-mooneythe local.conf has LIBVIRT_TYPE=kvm https://d8076f0c34babfa2d8fa-c3c714f53a28264bde7e35c9b01061ab.ssl.cf2.rackcdn.com/678887/8/check/nova-live-migration/21801ca/logs/local.conf.txt.gz18:14
dansmithsean-k-mooney: cool, gonna push up a change?18:14
*** cdent has quit IRC18:14
sean-k-mooneythe issue it seams is its set twice18:15
sean-k-mooneyyes ill fix it18:15
sean-k-mooneywe just need to set the virt type in the zuul.yaml instead18:15
dansmithmaybe we need to set it in the hook and restart compute?18:15
dansmithokay18:15
dansmither, no the hook won't affect the subnode I guess18:16
sean-k-mooneythe base gate jobs hardcode it to qemu but we can just override it18:16
dansmithack18:16
mriedem_awayoverriding it is what i did in the run.yaml18:21
*** mriedem_away is now known as mriedem18:21
sean-k-mooneyya but it didnt work18:22
lbragstadjohnthetubaguy any chance you'd be willing to take a peek at https://review.opendev.org/#/c/644615/ ?18:22
sean-k-mooneyhttps://github.com/openstack/devstack/blob/master/.zuul.yaml#L22318:22
sean-k-mooneyim just going to set this18:22
mriedemi removed it from the run_test script b/c it was only setting it on the primary and not the subnode compute as well, but we shouldn't need to do that since it's in local.conf18:22
mriedemsean-k-mooney: yeah, that's what i did in the run.yaml18:22
mriedemsean-k-mooney: https://review.opendev.org/#/c/678887/8/playbooks/legacy/nova-live-migration/run.yaml18:22
sean-k-mooneyi didnt think you coudl do it there18:22
sean-k-mooney or rather i hav enever tried too18:23
sean-k-mooneyoh ya18:23
mriedemit's the legacy way18:23
sean-k-mooneyso it ended up being there twice18:23
sean-k-mooneyboth with qemu and then with kvm18:23
*** igordc has quit IRC18:23
sean-k-mooneyi dont know why qemu18:23
sean-k-mooneywon18:23
sean-k-mooneybut it did18:23
mriedemyeah, so i guess just overriding in zuul.yaml might work18:24
sean-k-mooneyill set it in all the places18:24
sean-k-mooneyone of them will work :)18:24
mriedemi'd say while doing that, also comment out the jobs we don't care about running18:24
sean-k-mooneyya no need to was gate time18:25
mriedemcould also just do post-config|$NOVA_CPU_CONF18:25
sean-k-mooneyyep18:25
mriedembut i don't really trust that LIBVIRT_TYPE variable18:25
sean-k-mooneyi can do that too18:25
mriedemdansmith: so is it worth me looking at https://review.opendev.org/#/c/635669/45 right now?18:26
dansmithyes18:26
*** tbachman has quit IRC18:27
* mriedem puts on diaper18:28
*** prometheanfire has quit IRC18:32
*** tbachman has joined #openstack-nova18:32
*** prometheanfire has joined #openstack-nova18:32
openstackgerritsean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888718:32
sean-k-mooneythat is proably over kill but it should for the config to what we want it to have18:33
*** itlinux has joined #openstack-nova18:44
*** itlinux has quit IRC18:48
openstackgerritsean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888718:50
*** bbowen has quit IRC18:51
openstackgerritsean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888718:52
*** gbarros has joined #openstack-nova18:59
*** lpetrut has quit IRC19:02
mriedemdansmith: i've disposed of my soiled diaper19:11
dansmithmriedem: I was looking for confirmation that the hacks to avoid disrupting the sriov stuff would fly with you, which I assume from the summary is ... yes?19:12
mriedemit's definitely better than it was19:13
mriedembeing explicit i mean19:13
dansmithyes, but... is that a yes? :)19:13
sean-k-mooneyill redeploy with this version either later today or tomorow and report what i find.19:14
*** ociuhandu has quit IRC19:14
*** tbachman has quit IRC19:15
dansmithsean-k-mooney: thanks19:16
*** Sundar has joined #openstack-nova19:19
*** ivve has joined #openstack-nova19:20
mriedemdansmith: well, compared to what? the alternative of making numa live migration miss the release to retrofit the sriov pci claim stuff into the actual MoveClaim? if so, then yes this dirty hackery is acceptable i guess19:22
mriedemi have a feeling it's a hack that we'll have forever19:22
mriedembut such is nova19:22
dansmithyes, that's what I mean, and yes that's the shame I cast on artom earlier this morning about not cleaning it up after the release19:22
dansmithwe'll see if the shame works or not19:22
*** efried_rollin is now known as efried19:22
melwittcast shame 319:23
*** itlinux has joined #openstack-nova19:23
melwitthighest level spell19:23
sean-k-mooneyone way or another we will converge them in U19:23
mriedemartom is pretty good at saving throws though19:23
melwittlol19:23
sean-k-mooneyif artom does not work on it i will.19:23
dansmithI'm going to pretend I understand all this dorkery and chuckle19:24
melwittI know, you're just too cool19:24
mriedemhttps://en.wikipedia.org/wiki/Saving_throw god19:24
sean-k-mooneywhat the DC on this charisma check19:24
mriedemif it weren't for artom's high charisma he'd have been dead long ago19:25
dansmithI just never played any of that stuff so I don't know19:25
melwittsean-k-mooney: I had to google that xD19:25
melwittlol mriedem19:25
melwittthat was a good one19:26
sean-k-mooneyon of my friends runs a game and got me to sit in at teh weekend so that make 3 times i have played a dnd game now... not sure if that is a good or a bad thing19:26
melwittI'm sure it's good. I have never played, I'm video game only19:27
mriedemi've never actually played a sit down board game but my brother had all that stuff and i rocked the ibm pc floppies for https://en.wikipedia.org/wiki/Pool_of_Radiance19:27
sean-k-mooneymost of my iteraction with D&D was baldur's gate on the pc19:28
melwittsimilar for me. I played https://en.wikipedia.org/wiki/Eye_of_the_Beholder_III:_Assault_on_Myth_Drannor19:28
*** itlinux_ has joined #openstack-nova19:28
*** itlinux has quit IRC19:31
dansmithman ya'll'er dorks19:33
sean-k-mooneysay the ham radio nerd :)19:33
melwittshots fired19:34
dansmithi, uh19:34
dansmithwell,19:34
*** nicolasbock has quit IRC19:34
dansmithI guess I'll be proud of being a radio nerd and you all can be, uh, proud of your games :D19:34
sean-k-mooney:)19:35
* dansmith quietly puts sean-k-mooney on "the list"19:35
* sean-k-mooney worth it19:36
melwittlol19:36
*** eharney has quit IRC19:36
*** dpawlik has joined #openstack-nova19:36
dansmithsean-k-mooney: do we need to recheck that LM test19:36
dansmith?19:36
dansmithah, you removed a bunch of other jobs19:36
dansmithI thought the zuul report looked weird19:37
sean-k-mooneyit did but i still have an issue in v1119:37
sean-k-mooneyim just goint to remove the sub group stuff19:37
sean-k-mooneywhich is what i did in v1019:37
dansmithokay19:37
openstackgerritsean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888719:38
*** ash2307 has joined #openstack-nova19:39
*** ash2307 has left #openstack-nova19:39
*** dpawlik has quit IRC19:41
*** ralonsoh has quit IRC19:42
sean-k-mooneyok its actully running this time19:43
sean-k-mooneyhttp://zuul.openstack.org/stream/951e3e39a4014163aeb5c238c84a2840?logfile=console.log19:43
sean-k-mooneyill keep an eye on it until dinner is done cooking19:44
melwittI'm gonna be out tomorrow and friday, fyi19:45
*** spatel has quit IRC20:03
*** gyee has quit IRC20:05
*** itlinux has joined #openstack-nova20:06
*** atmark has quit IRC20:07
*** mloza has joined #openstack-nova20:08
*** itlinux_ has quit IRC20:09
*** slaweq has quit IRC20:09
*** s10 has joined #openstack-nova20:12
*** itlinux has quit IRC20:18
*** itlinux has joined #openstack-nova20:20
*** tbachman has joined #openstack-nova20:21
*** gyee has joined #openstack-nova20:23
*** brault has joined #openstack-nova20:32
*** itlinux has quit IRC20:33
*** itlinux has joined #openstack-nova20:33
*** jmlowe has quit IRC20:36
*** brault has quit IRC20:37
openstackgerritsean mooney proposed openstack/nova master: Libvirt: report storage bus traits  https://review.opendev.org/66691420:38
openstackgerritsean mooney proposed openstack/nova master: libvirt: use domain capabilities to get supported device models  https://review.opendev.org/66691520:38
openstackgerritsean mooney proposed openstack/nova master: Add transform_image_metadata request filter  https://review.opendev.org/66577520:38
sean-k-mooneyi have artoms series stacking. going for dinner backin 30 mins or so20:38
openstackgerritMerged openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method  https://review.opendev.org/64259020:39
openstackgerritMerged openstack/nova master: Add Destination.allow_cross_cell_move field  https://review.opendev.org/61403520:39
*** xek has quit IRC20:39
sean-k-mooneyalso it looks like the legacy jobs just entrily ignore the stuff in the zuul.yaml20:41
sean-k-mooneyit failed again because it used qemu20:41
*** nweinber has quit IRC20:43
*** bbowen has joined #openstack-nova20:53
*** eharney has joined #openstack-nova20:55
*** markvoelker has quit IRC20:58
*** markvoelker has joined #openstack-nova20:58
openstackgerritMerged openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.opendev.org/50748620:59
openstackgerritMerged openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook  https://review.opendev.org/67284020:59
*** belmoreira has joined #openstack-nova20:59
melwitt\o/20:59
*** itlinux has quit IRC21:00
mriedemsean-k-mooney: yeah i wasn't sure if that was going to work b/c of the legacy job format, that's why i threw it into the run.yaml21:03
mriedemsean-k-mooney: we could....push a dnm devstack patch that removes that LIBVIRT_TYPE=qemu default and add a depends-on from the nova patch21:03
mriedemor simply changes the default in devstack to kvm and depends-on that21:03
*** itlinux has joined #openstack-nova21:04
sean-k-mooneyor you know hard code it in nova21:10
*** markvoelker has quit IRC21:10
*** markvoelker has joined #openstack-nova21:10
sean-k-mooneyit finished stacking while i was having dinner so ill test it now21:11
*** slaweq has joined #openstack-nova21:11
*** itlinux has quit IRC21:12
*** itlinux has joined #openstack-nova21:12
*** slaweq has quit IRC21:16
*** belmoreira has quit IRC21:16
*** jmlowe has joined #openstack-nova21:18
mriedemyeah i guess we could hardcode it in nova in the dnm patch, that's the easiest21:20
*** artom has joined #openstack-nova21:20
*** itlinux has quit IRC21:22
*** ircuser-1 has quit IRC21:22
*** belmoreira has joined #openstack-nova21:25
fungiany nova-stable-core (besides mriedem, thanks!!!) care to have a look at https://review.opendev.org/678098 ?21:27
mriedemmelwitt: ^21:27
fungii've got an advisory staged and just want to set the date on it once i know that's approved, or likely to get approved21:27
*** nicolasbock has joined #openstack-nova21:28
fungii wouldn't normally bug folks, but it's been sitting there for a bit21:28
*** tbachman has quit IRC21:32
*** dpawlik has joined #openstack-nova21:37
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for validating instance and node  https://review.opendev.org/65602821:37
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for setting instance id  https://review.opendev.org/65969021:37
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node  https://review.opendev.org/65969121:37
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node  https://review.opendev.org/67021321:37
*** markvoelker has quit IRC21:39
*** markvoelker has joined #openstack-nova21:40
*** dpawlik has quit IRC21:41
*** s10 has quit IRC21:42
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Microversion 2.77: Support Specifying AZ to unshelve  https://review.opendev.org/66513621:43
*** markvoelker has quit IRC21:45
*** eharney has quit IRC21:50
sean-k-mooneyok i need to do it again but i think the latest patch works.21:51
*** deepak_mourya_ has quit IRC21:51
mriedemtakashin: alex_xu: i think the 2.77 novaclient change is ready https://review.opendev.org/66513621:54
*** mriedem has quit IRC21:54
*** eharney has joined #openstack-nova21:55
*** Sundar has quit IRC21:56
sean-k-mooneyartom: dansmith: mriedem: http://paste.openstack.org/show/766650/ that is the vm xml before and after + the pci request before and after and the migration_context object22:01
sean-k-mooneythe flavor had cpu pinning enabeld. the hosts had non overlapping vcpu_pin_sets so the pinning had to be recalulated and the vm had an sriov-macvtap port22:02
*** ivve has quit IRC22:03
sean-k-mooneyit look like cold migration back to the first host also works22:04
*** markvoelker has joined #openstack-nova22:05
sean-k-mooneythe pci device in teh pci_device table are also updated correctly http://paste.openstack.org/show/766651/22:10
*** markvoelker has quit IRC22:11
*** slaweq has joined #openstack-nova22:11
*** belmoreira has quit IRC22:11
*** ccamacho has quit IRC22:14
*** ircuser-1 has joined #openstack-nova22:15
*** gbarros has quit IRC22:15
*** mlavalle has quit IRC22:16
*** slaweq has quit IRC22:17
*** N3l1x has quit IRC22:18
melwittfungi: sorry about that. I see smcginnis has got it, thanks22:42
fungiyep, thanks for following up!22:43
fungii'm planning to circulate the advisory tomorrow22:43
sean-k-mooneymelwitt: do we have that backported downstream yet22:43
sean-k-mooneyif not i can try and do that tomorow22:44
melwittsean-k-mooney: not that I'm aware of22:44
sean-k-mooneywe havent relesat with stien yet but it would be good to do before we go to rc22:44
melwittfungi: great, thank you for putting it together!22:44
sean-k-mooneyit doesnt actuly affect our product since we dont support linuxbridge22:44
sean-k-mooneyso its not as much of an issue downstream22:45
melwittsean-k-mooney: yeah. maybe could bring it up on the team call tomorrow, I'm not sure what would be the guidance about it22:45
sean-k-mooneyit was discussed downstream i just dont know if we did the backport yet22:46
melwittoh ok22:46
melwittI didn't do it so unless someone else did22:46
sean-k-mooneyit shoud be trivial enough to do.22:47
sean-k-mooneyanyway with that ill call it a night22:47
sean-k-mooneyo/22:47
*** tkajinam has joined #openstack-nova22:51
melwitto/22:53
*** macz has quit IRC22:57
*** slaweq has joined #openstack-nova23:11
*** slaweq has quit IRC23:15
*** adriant has quit IRC23:17
artomsean-k-mooney, so that's a woot?23:17
artom(If you're not gone)23:18
*** adriant has joined #openstack-nova23:18
*** adriant3 has joined #openstack-nova23:19
*** rcernin has joined #openstack-nova23:21
*** adriant3 has quit IRC23:21
*** adriant4 has joined #openstack-nova23:22
*** adriant has quit IRC23:23
*** adriant49 has joined #openstack-nova23:24
*** adriant49 has quit IRC23:24
openstackgerritMerged openstack/nova master: [Trivial]Remove unused helper check_temp_folder  https://review.opendev.org/67897323:27
*** adriant4 has quit IRC23:28
*** dpawlik has joined #openstack-nova23:38
*** dpawlik has quit IRC23:43
*** tetsuro has joined #openstack-nova23:44
*** adriant has joined #openstack-nova23:48
openstackgerritMerged openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug  https://review.opendev.org/67809823:51
« Tuesday, 2019-08-27 Index Thursday, 2019-08-29 »

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