Friday, 2019-04-26

*** tetsuro has joined #openstack-nova00:03
*** slaweq has joined #openstack-nova00:11
*** markvoelker has quit IRC00:19
*** lbragstad has quit IRC00:20
*** slaweq has quit IRC00:24
*** slaweq has joined #openstack-nova01:11
*** cooper6581 has quit IRC01:14
*** tetsuro has quit IRC01:18
*** itlinux has joined #openstack-nova01:19
*** mlavalle has quit IRC01:23
mriedemdims: didn't you used to run nova unit tests in osx? ^01:23
*** slaweq has quit IRC01:24
*** dave-mccowan has quit IRC01:28
*** whoami-rajat has joined #openstack-nova01:31
*** ileixe has quit IRC01:40
*** ileixe has joined #openstack-nova01:41
*** gmann_afk is now known as gmann01:42
*** hongbin has joined #openstack-nova01:44
mnaserI usually run them.. I haven't had troubles01:50
mnaseri.. don't usually run the whole suite tho01:51
mriedemtsk01:54
mriedemtsk01:54
* mriedem doesn't either01:54
*** itlinux has quit IRC02:08
*** ileixe has quit IRC02:10
*** itlinux has joined #openstack-nova02:10
dimsmriedem : right as mnaser said, not the full suite ... a while ago02:10
*** ileixe has joined #openstack-nova02:11
*** slaweq has joined #openstack-nova02:15
*** boxiang has quit IRC02:20
*** itlinux has quit IRC02:20
*** boxiang has joined #openstack-nova02:21
*** slaweq has quit IRC02:24
openstackgerritMerged openstack/python-novaclient master: Drop py35 tests  https://review.opendev.org/64387302:28
*** mriedem has quit IRC02:32
openstackgerritBrin Zhang proposed openstack/nova-specs master: Change the default value of swap in the show flavor details API  https://review.opendev.org/64891902:32
*** ileixe has quit IRC02:47
*** samueldmq has quit IRC03:04
*** slaweq has joined #openstack-nova03:16
*** nicolasbock has quit IRC03:21
*** slaweq has quit IRC03:24
*** ileixe has joined #openstack-nova03:27
*** psachin has joined #openstack-nova03:33
*** psachin has quit IRC03:33
*** psachin has joined #openstack-nova03:36
*** hongbin has quit IRC03:39
*** udesale has joined #openstack-nova04:00
*** slaweq has joined #openstack-nova04:15
*** pcaruana has joined #openstack-nova04:23
*** slaweq has quit IRC04:24
*** ileixe has quit IRC04:33
*** ileixe has joined #openstack-nova04:36
*** threestrands has joined #openstack-nova04:38
openstackgerritmelanie witt proposed openstack/nova master: Add 'path' query parameter to console access url  https://review.opendev.org/64937204:44
*** ratailor has joined #openstack-nova05:06
*** ratailor_ has joined #openstack-nova05:09
*** ratailor has quit IRC05:11
*** slaweq has joined #openstack-nova05:11
openstackgerritMerged openstack/nova master: Don't run tempest/devstack jobs on nova/test.py only changes  https://review.opendev.org/65512105:13
*** boxiang has quit IRC05:20
*** boxiang has joined #openstack-nova05:20
*** boxiang has quit IRC05:21
*** boxiang has joined #openstack-nova05:22
*** slaweq has quit IRC05:22
*** boxiang has quit IRC05:23
*** boxiang has joined #openstack-nova05:23
*** mmethot has quit IRC05:24
*** Luzi has joined #openstack-nova05:39
*** ivve has joined #openstack-nova05:43
*** igordc has joined #openstack-nova05:49
*** igordc has quit IRC05:54
*** igordc has joined #openstack-nova05:54
*** ccamacho has quit IRC05:57
*** igordc has quit IRC06:02
*** rtjure has joined #openstack-nova06:08
*** dpawlik has joined #openstack-nova06:10
*** slaweq has joined #openstack-nova06:16
*** mmethot has joined #openstack-nova06:17
*** sapd1_x has joined #openstack-nova06:18
*** luksky has joined #openstack-nova06:20
*** markvoelker has joined #openstack-nova06:26
openstackgerritBrin Zhang proposed openstack/nova-specs master: Change the default value of swap in the show flavor details API  https://review.opendev.org/64891906:42
*** pcaruana has quit IRC06:48
*** pcaruana has joined #openstack-nova06:55
*** brinzhang has joined #openstack-nova07:00
*** rpittau|afk is now known as rpittau07:02
*** rcernin has quit IRC07:05
*** slaweq has quit IRC07:15
*** slaweq has joined #openstack-nova07:17
*** lpetrut has joined #openstack-nova07:19
*** rtjure has quit IRC07:19
*** Luzi has quit IRC07:20
*** udesale has quit IRC07:20
*** tosky has joined #openstack-nova07:20
*** sapd1_x has quit IRC07:21
*** udesale has joined #openstack-nova07:23
*** Luzi has joined #openstack-nova07:27
*** sapd1_x has joined #openstack-nova07:28
*** rtjure has joined #openstack-nova07:28
*** brinzhang has quit IRC07:32
*** brinzhang has joined #openstack-nova07:33
*** ccamacho has joined #openstack-nova07:42
*** threestrands has quit IRC07:50
*** brault has joined #openstack-nova07:54
*** brault has quit IRC07:57
*** tesseract__ has joined #openstack-nova07:58
*** tesseract__ has quit IRC07:58
*** tesseract__ has joined #openstack-nova08:00
*** tesseract__ has quit IRC08:01
*** lpetrut has quit IRC08:02
*** takashin has left #openstack-nova08:05
*** sapd1_x has quit IRC08:10
*** tesseract has joined #openstack-nova08:23
*** tkajinam has quit IRC08:25
*** dtantsur|afk is now known as dtantsur08:26
openstackgerritMerged openstack/nova master: Delete require_instance_exists_using_uuid  https://review.opendev.org/65099408:28
*** ttsiouts has joined #openstack-nova08:30
*** luksky has quit IRC08:31
*** rcernin has joined #openstack-nova08:32
*** derekh has joined #openstack-nova08:32
slaweqhi nova developers :)08:33
slaweqcan someone familiar with nova-metadata take a look at my comment in https://bugs.launchpad.net/neutron/+bug/1821912/comments/12 ?08:34
openstackLaunchpad bug 1821912 in neutron "intermittent ssh failures in various scenario tests" [High,Confirmed]08:34
gibiefried: thanks for the onboarding slide update08:34
slaweqI'm not sure what's exectly happend (or not happend) on nova's side there so any help would be useful08:35
slaweqgibi: efried: if You would have some time, maybe You can take a look at ^ :)08:35
slaweqthx in advance08:35
*** ralonsoh has joined #openstack-nova08:35
*** tesseract has quit IRC08:37
*** tesseract has joined #openstack-nova08:37
openstackgerritya.wang proposed openstack/nova-specs master: Add spec for expose-auto-converge-post-copy  https://review.opendev.org/65168108:47
gibislaweq: hi! looking... (but not super familiar with the metadata part of nova)08:48
slaweqgibi: thx a lot08:48
*** yedongcan has joined #openstack-nova08:48
openstackgerritStephen Finucane proposed openstack/nova master: test_rpc: "deepcopy" global options  https://review.opendev.org/65584309:08
*** ttsiouts has quit IRC09:11
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Prevent "qbr" Linux Bridge from replying to ARP messages  https://review.opendev.org/65533209:20
*** ttsiouts has joined #openstack-nova09:23
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Always disconnect volumes after libvirtError exceptions  https://review.opendev.org/65571209:26
lyarwoodmdbooth: ^ and the one below it might interest you btw, simple bugfixes.09:27
kashyapaspiers: Hi, when you're about, some more design comments upcoming for the getDomCapabilities() patch09:29
kashyapaspiers: Talking to libvirt folks, they strongly suggest to not cache the capabilities at all.  I will write some details and possible suggestions in the patch.09:30
aspierskashyap: ok09:33
kashyapaspiers: In short, if you want to start thinking about it: we're suggested to query domCapabilities once at the beginning of building domain XML process, but then forget them right after09:34
*** panda|off is now known as panda09:34
kashyapI asked the libvirt dev to also comment in this bug: https://bugzilla.redhat.com/show_bug.cgi?id=168347109:34
openstackbugzilla.redhat.com bug 1683471 in libvirt "getDomainCapabilities claims SEV is supported for pc-i440fx-1.4 machine type" [Unspecified,Closed: notabug] - Assigned to libvirt-maint09:34
aspierskashyap: but this is the point I already made in the comment - that approach totally fails in the OpenStack world09:34
kashyapIn the doc string, right?09:35
aspierskashyap: because we need to know stuff before placement time, not just at launch time09:35
aspierskashyap: https://review.opendev.org/#/c/655268/2/nova/virt/libvirt/host.py@69009:35
aspiersI suspect the libvirt folks were only thinking of the single hypervisor paradigm when they designed this09:36
kashyapaspiers: Right, I'm still thinking, how to resolve that09:36
*** ltyrex has joined #openstack-nova09:36
aspiersbut anyway, what is the problem with memoizing? it's not caching to disk09:37
aspierswhich caps would actually change?09:37
kashyapaspiers: For example, when QEMU is updated, you get new features09:39
kashyap(And as you know domCapabilities are machine type dependent.)09:39
aspiersDidn't we already talk about this? When qemu is updated, nova-compute would be restarted, so the memoization vanishes09:40
kashyapaspiers: Meanwhile: https://bugzilla.redhat.com/show_bug.cgi?id=1683471#c1109:40
openstackbugzilla.redhat.com bug 1683471 in libvirt "getDomainCapabilities claims SEV is supported for pc-i440fx-1.4 machine type" [Unspecified,Closed: notabug] - Assigned to libvirt-maint09:40
kashyapaspiers: Oh, right.  Forgive me if I forget something, I'm still somewhat ill, and not fully functional.09:41
aspierskashyap: np! :)09:41
kashyapaspiers: Mind mentioning that in the docstring?  The point when libvirt/QEMU gets updated, the Compute agent needs restarting, and the memoization vanishes09:43
aspierskashyap: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-04-23.log.html#t2019-04-23T12:42:1109:43
ltyrexdoes someone remember which table has the CPU pinning mapping in Openstack?09:43
aspierskashyap: sure, happy to09:43
kashyapThanks, I was revisiting the IRC log.09:44
gibislaweq: replied in the bug https://bugs.launchpad.net/neutron/+bug/1821912/comments/1309:45
openstackLaunchpad bug 1821912 in neutron "intermittent ssh failures in various scenario tests" [High,Confirmed]09:45
gibislaweq: it feels like something was wrong with the VM hosting the devstack during the time period09:45
slaweqgibi: thx a lot09:46
slaweqgibi: I saw it but wasn't sure if that rabbitmq issue can be related somehow09:46
*** luksky has joined #openstack-nova09:48
openstackgerritEric Fried proposed openstack/nova master: Correct signature on fake_get_availability_zones  https://review.opendev.org/65580609:49
gibislaweq: there is gap in the log for the same time in the dstat log as well http://logs.openstack.org/47/653947/3/gate/tempest-slow-py3/629abee/controller/logs/screen-dstat.txt.gz#_Apr_25_14_56_44_34984409:51
gibislaweq: seem like the VM did not got execution time so every service was slowed to halt09:52
*** sean-k-mooney has joined #openstack-nova09:55
kashyapaspiers: Added some notes in the change; ruthlessly correct me if something is wrong :-)09:58
aspiersI am doing ;-)09:59
kashyap(Just a heads-up: won't be very responsive this afternoon due to prior committments; will read the change once my queue is free.)10:00
aspiersnp10:00
*** threestrands_ has joined #openstack-nova10:03
openstackgerritcaoyuan proposed openstack/os-traits master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65506610:05
aspierskashyap: very nice notes, thanks. I'm incorporating the missing points into the docstring10:06
aspierskashyap: believe it or not the C API *does* allow omitting some parameters, whereas they are mandatory in Python10:07
aspierskashyap: again replying in Gerrit10:07
aspiersgotta dash to an appointment now though, back a bit later10:07
openstackgerritsean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration  https://review.opendev.org/62958910:16
*** threestrands_ has quit IRC10:16
*** nicolasbock has joined #openstack-nova10:24
slaweqgibi: yes, that may be this10:25
*** gnuoy has joined #openstack-nova10:35
*** ttsiouts has quit IRC10:36
*** ttsiouts has joined #openstack-nova10:42
*** yedongcan has left #openstack-nova10:43
*** tbachman has quit IRC10:43
*** ttsiouts_ has joined #openstack-nova10:47
*** ttsiouts has quit IRC10:50
*** ttsiouts has joined #openstack-nova10:55
*** ttsiouts_ has quit IRC10:57
openstackgerritsean mooney proposed openstack/nova master: [WIP] Libvirt: add nfv job  https://review.opendev.org/65219711:01
*** panda is now known as panda|lunch11:10
*** sean-k-mooney has quit IRC11:11
*** sean-k-mooney has joined #openstack-nova11:15
*** derekh has quit IRC11:15
*** derekh has joined #openstack-nova11:15
*** rcernin has quit IRC11:16
*** tesseract has quit IRC11:21
*** tesseract has joined #openstack-nova11:21
openstackgerritStephen Finucane proposed openstack/nova master: test_rpc: Stop f****** with global state  https://review.opendev.org/65584311:21
*** sean-k-mooney has quit IRC11:22
*** udesale has quit IRC11:22
kashyapaspiers: I should've read the C API closer (instead of skimming) -- I think it incorporates some sane defaults.  I'll believe you, of course11:30
*** ttsiouts has quit IRC11:35
kashyapaspiers: BTW, it's a bug -- the virConnectGetDomainCapabilities() allowing skipping params, while its Python binding not :-)11:38
kashyapaspiers: A libvirt dev is pushing a patch upstream11:38
*** ttsiouts has joined #openstack-nova11:38
kashyaps/skipping parms/skipping _some_ params/11:39
aspierskashyap: you mean a bug in the Python binding? that's what I suspected11:52
kashyapaspiers: Bug in the C API, from the sounds of it.11:54
aspiers?11:55
kashyapIf you run `virsh domcapabilities`, the C API is run with with NULL, NULL, NULL, 011:55
aspiersright11:55
aspiersand then it obtains defaults https://libvirt.org/git/?p=libvirt.git;a=blob;f=src/qemu/qemu_driver.c;h=f48d9256e4cc0f6282dcd7aa13bb7a1b53508eb3;hb=HEAD#l1989811:56
aspiersI have all this in a draft comment on the Gerrit review11:56
aspiersI only just got back to my desk11:56
kashyapNo rush11:56
*** panda|lunch is now known as panda11:57
*** tesseract has quit IRC11:59
kashyapaspiers: Yeah, it's indeed a bug in Python binding12:00
kashyap    def getDomainCapabilities(self, emulatorbin, arch, machine, virttype, flags=0):12:00
kashyap        """Prior creating a domain (for instance via virDomainCreateXML12:00
*** tesseract has joined #openstack-nova12:00
kashyap        or virDomainDefineXML) it may be suitable to know what the12:00
kashyap        underlying emulator and/or libvirt is capable of. For12:00
kashyap        instance, if host, libvirt and qemu is capable of VFIO12:00
kashyap        passthrough and so on. """12:00
kashyap        ret = libvirtmod.virConnectGetDomainCapabilities(self._o, emulatorbin, arch, machine, virttype, flags)12:00
kashyap        if ret is None: raise libvirtError ('virConnectGetDomainCapabilities() failed', conn=self)12:00
kashyap        return ret12:00
kashyapaspiers: The "flags=0"12:00
*** arne_wiebalck has quit IRC12:01
*** yonglihe has quit IRC12:01
aspierskashyap: I think flags is *always* 012:02
aspierskashyap: the problem is with omitting any of the other 412:02
*** rajinir has quit IRC12:02
aspierskashyap: maybe it works if I pass None12:02
*** arne_wiebalck has joined #openstack-nova12:03
*** yonglihe has joined #openstack-nova12:03
*** rajinir has joined #openstack-nova12:03
aspierskashyap: but in any case we don't want to take the defaults in nova, but instead iterate over all relevant (arch, machine) tuples12:03
* kashyap in a meeting; bbiab12:03
kashyapaspiers: Let's see what the upstream fix is12:03
*** tbachman has joined #openstack-nova12:04
aspierskashyap: sure12:07
aspierskashyap: expect a new patch set shortly12:07
*** tesseract has quit IRC12:09
*** tbachman has quit IRC12:10
*** tesseract has joined #openstack-nova12:10
*** tesseract has quit IRC12:12
*** tbachman has joined #openstack-nova12:12
*** tesseract has joined #openstack-nova12:13
*** brinzhang has quit IRC12:24
*** ttsiouts has quit IRC12:26
*** ttsiouts has joined #openstack-nova12:26
*** ttsiouts has quit IRC12:28
*** ttsiouts_ has joined #openstack-nova12:28
*** lbragstad has joined #openstack-nova12:31
*** mriedem has joined #openstack-nova12:33
kashyapaspiers: Just back; indeed, we can pass 'None' to the API, meanwhile.12:35
kashyap(I commented on the change.)12:35
kashyapFun, fun.12:35
*** markvoelker has quit IRC12:40
mriedemgibi: looks like this introduces its own problem https://review.opendev.org/#/c/654466/4..5/oslo_config/cfg.py12:40
mriedembnemec: any ideas on how we can make that counter work in ConfigOpts? ^12:41
*** ttsiouts_ has quit IRC12:41
mriedemoh i think i see12:44
mriedemself._get_counter isn't set until after register_opt which calls __clear_cache12:45
mriedemfixing12:45
*** ttsiouts has joined #openstack-nova12:46
*** ratailor_ has quit IRC13:00
*** eharney has quit IRC13:00
*** ttsiouts has quit IRC13:04
*** ttsiouts has joined #openstack-nova13:05
*** dpawlik has quit IRC13:07
*** Luzi has quit IRC13:09
gibimriedem: good point13:09
*** ttsiouts has quit IRC13:09
gibimriedem: have you seen stephenfin's patch trying to solve this in a different way https://review.opendev.org/#/c/655843/ ?13:12
mriedemi glanced at it...13:13
*** boxiang has quit IRC13:17
*** boxiang has joined #openstack-nova13:17
kashyapaspiers: There it is: https://www.redhat.com/archives/libvir-list/2019-April/msg01418.html -- [python PATCH] Allow virConnect.getDomainCapabilities() to have no arguments13:18
*** ttsiouts has joined #openstack-nova13:20
kashyapaspiers: I'm not "-1"ing it, because you already know what to do there ('None' for @emulatorbin)13:22
*** openstackgerrit has quit IRC13:27
*** ttsiouts has quit IRC13:30
*** ttsiouts has joined #openstack-nova13:31
*** ivve has quit IRC13:33
*** ttsiouts has quit IRC13:35
*** openstackgerrit has joined #openstack-nova13:36
openstackgerritDan Smith proposed openstack/nova master: Add ironic driver image type capabilities  https://review.opendev.org/65572913:36
openstackgerritDan Smith proposed openstack/nova master: Add vmware driver image type capabilities  https://review.opendev.org/65573013:36
openstackgerritDan Smith proposed openstack/nova master: Add xenapi driver image type capabilities  https://review.opendev.org/65573113:36
openstackgerritDan Smith proposed openstack/nova master: Add zvm driver image type capabilities  https://review.opendev.org/65573213:36
*** ttsiouts has joined #openstack-nova13:38
aspierskashyap: that would make one of my previous patches wasted effort ...13:43
*** ttsiouts has quit IRC13:49
*** ttsiouts has joined #openstack-nova13:50
*** mchlumsky has joined #openstack-nova13:52
*** eharney has joined #openstack-nova13:54
*** ttsiouts has quit IRC13:54
*** ttsiouts has joined #openstack-nova13:56
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add heal_instance_mappings command  https://review.opendev.org/65590813:57
mriedemmnaser: ^ got around to porting your script from the bug into a nova-manage command13:57
mnaseropenstackgerrit: oooo that's awesoem14:01
mnasererr14:02
mnasermriedem: ^ oops14:02
*** bnemec is now known as beekneemech14:04
mriedemhttps://review.opendev.org/#/c/586742/ got me thinking about it again14:06
kashyapaspiers: Sorry, what is "that"?14:07
aspierskashyap: https://review.opendev.org/#/c/640483/14:07
*** JamesBenson has joined #openstack-nova14:09
*** JamesBenson has quit IRC14:09
*** JamesBenson has joined #openstack-nova14:09
*** Sundar has joined #openstack-nova14:09
*** ccamacho has quit IRC14:13
*** dpawlik has joined #openstack-nova14:19
*** dpawlik has quit IRC14:23
*** mlavalle has joined #openstack-nova14:25
*** artom has quit IRC14:26
*** luksky has quit IRC14:29
*** ttsiouts has quit IRC14:57
openstackgerritAdam Spiers proposed openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API  https://review.opendev.org/65526814:57
*** ttsiouts has joined #openstack-nova14:57
*** boxiang has quit IRC15:01
*** boxiang has joined #openstack-nova15:01
*** ttsiouts has quit IRC15:02
*** gibi is now known as gibi_off15:02
*** tesseract has quit IRC15:06
mriedemjaypipes: dansmith: i'm having a brain fart on what i thought was a simple db query https://review.opendev.org/#/c/655908/1/nova/objects/instance_mapping.py@406 - i've got a test with an instance mapping with no cell_id but my query is not returning it and i'm at a loss to why15:07
mriedemdo you see anything obvious?15:07
mriedemi have a feeling the joinedload is screwing it up15:08
mriedemsince there is no related cell mapping to join15:08
dansmithI would think that would matter15:09
mriedemoh actually it's just the join() i htink15:09
*** boxiang has quit IRC15:09
mriedem.options(joinedload('cell_mapping')) will load up the null attribute15:09
*** boxiang has joined #openstack-nova15:09
dansmithI meant "wouldn't"15:09
dansmithI would think you could still query by that column directly, not sure why you would need to join the other15:10
dansmithbut no, nothing really stands out to me15:10
aspiershrm, the first of my SEV patches has bizarrely just started failing openstack-tox-lower-constraints due to having an old os-traits which misses the SEV trait15:11
mriedemi run it through _from_db_object which loads the attribute on the ovo15:11
mriedemget a lazy-load session error otherwise15:11
mriedembut yeah it was the join() that was filtering it out, fixed15:11
mriedemthanks15:11
aspierscan I just up the version in lower-constraints? has something recently changed with the way lower-constraints is tested?15:11
mriedemaspiers: if you require a new min of os-traits then yes you need to bump the version in lower-constraints.txt15:12
aspiersmriedem: in the same commit?15:12
kashyapmriedem: If you get a minute, when backporting this fix of yours to Rocky (admittedly, downstream), any other patch is required? -- https://review.opendev.org/#/c/591607/ ("Force refresh instance info_cache during heal")15:12
mriedemjesus15:12
mriedemi got this same questoin from sean-k-mooney earlier about some other patch going to newton15:12
openstackgerritMerged openstack/nova master: Create request spec, build request and mappings in one transaction  https://review.opendev.org/58674215:13
mriedemkashyap: yes, there is an online data migration below that in the series,15:13
openstackgerritEric Fried proposed openstack/nova master: Hacking N363: Don't use spec[_set]='string'  https://review.opendev.org/65037015:13
aspierskashyap: not sure if it helps here, but in general https://github.com/aspiers/git-deps can help answer that question15:13
mriedemand that online data migration has resulted in a couple of regressions in stein still being worked15:13
aspierskashyap: https://github.com/aspiers/git-deps/blob/master/USE-CASES.md#use-case-1-porting-between-branches15:14
kashyapaspiers: Ah, thanks.  I was doing a lame `git blame -C -M -L [...]`, and nothing turned up besides Matt's commit15:14
mriedemkashyap: http://lists.openstack.org/pipermail/openstack-discuss/2019-April/005242.html15:14
kashyapmriedem: /me clicks; thank you, sir15:15
mriedemaspiers: yes same commit15:15
kashyapAnd actually, thank _you_ for the work.  Someone downstream is bugging me to backport it15:15
aspiersmriedem: cool thanks15:15
kashyapmriedem: Whatever beverage (you didn't take up my Berlin "root beer" offer) you like in Denver.   (Maybe they have a "user beer" :D)15:16
mriedemkashyap: you can thank Maciej Jozefczyk since he did most of the work and the online data migration15:16
kashyapAh, thanks to him, too, from OVH15:16
mriedemwhat is that now, like a 6 pack you owe me?15:16
*** zbr|rover is now known as zbr|over15:16
kashyapGuitly as charged.  I'll shell it out in Denver15:16
mriedemdtroyer has also said he owes me a few, i'll have rot gut15:16
kashyapmriedem: You got an Amazon Wishlist (that doesn't contain things like Hubble Telescope or something like that) :D15:17
mriedemheh, yeah but don't really want to get into that game15:17
mriedemnor share my dirty wishlist secrets15:18
kashyapHehe15:18
kashyapBut I hate how much I depend on Amazon.  But no one can beat their prices.  Damn them15:18
kashyap(I still go and support the local stores as much as I can.)15:18
aspiers+1 on that15:18
*** hongbin has joined #openstack-nova15:18
kashyapAlthough I live in Gent, I sometimes order from the Netherlands-based "bol.com" (a mini Amazon, as it were), but at least 5-6 euros more expensive15:19
*** ltyrex has quit IRC15:19
mordredkashyap: I have very conflicting feelings about my retail reliance on amazon ... because on the one side, amazon. but on the other side, I'm not sure that their competitors in the space are *better*15:20
kashyapaspiers: Thanks for the interesting 'git-deps' URL.  (Bookmarked, I'll check it out "soon".)15:20
aspierskashyap: the video is only a few mins :)15:20
kashyapmordred: Yeah.  But "the competitor is not better" is not a very satisfying argument15:20
mordredno, it's not15:21
*** lbragstad has quit IRC15:21
kashyapIt's just obscene the way they under-cut & squish everything else.  Like a 9000-pound blue whale15:21
aspierskashyap: I guess you missed the L&L I gave on it in Denver last year https://www.youtube.com/watch?v=f6anrSKCIgI15:21
mordredyup15:21
kashyap(Okay, okay, I'll get off my high horse)15:21
mordredkashyap: to be fair - a 9000-pound blue whale would actually be pretty awesome15:22
* mordred very much wants to go diving with humpbacks ...15:22
kashyapaspiers: I didn't go to any other Denver event.  This is the first time for me.  (I make it to at most one OpenSt^H^H^OpenInfra event a year)15:22
aspierskashyap: ah ok15:22
openstackgerritAdam Spiers proposed openstack/nova master: Add new "supports_amd_sev" capability to libvirt driver  https://review.opendev.org/63868015:23
kashyapmdbooth: Heh.  (I'm actually liking my "pivot" to "buy as much locally as you can, even if it's a tad more expensive".  Although, I'm in a very convenient "city town"  of 250K people, where I can get most things done via biking.  I know that my fellow folks in the US (given its huge surface area) don't have the same convenience15:25
efriedstephenfin: What's your schedule for the PTG? Are you around all three days? Which bits are consumed with non-nova?15:26
*** rpittau is now known as rpittau|afk15:27
kashyapefried: For mine, you asked the other day -- if it's still possible, still got a slot on Saturday morning to move the Secure Boot?  Sorry for the re-shuffle15:28
kashyap(I won't cry into my pillow if you can't move it.)15:29
efriedlooking15:29
efriedkashyap: 1045 Saturday morning?15:29
kashyapSure15:29
kashyapThank you!15:29
efriedkashyap: 15 minutes?15:30
gibi_offmriedem: problem reproduced with the lastest patch set but the stack trace is still too long http://logs.openstack.org/68/654468/7/check/openstack-tox-py36/1c4087e/job-output.txt.gz#_2019-04-26_15_22_42_14348615:31
kashyapefried: Yeah, should suffice.15:32
efriedk15:32
kashyapI normally don't digress :D15:32
mdboothefried: I believe stephenfin is travelling today.15:32
kashyap(If I may say so myself.)15:32
efriedkashyap: It's hardly you I'm worried about15:32
efriedmdbooth: ack, thanks.15:33
kashyapefried: Guess you're sharpening your "no-ratholing" knife :D15:35
*** gyee has joined #openstack-nova15:36
efriedkashyap: I'm bringing special choke holds15:36
kashyapHaha15:36
*** shilpasd has quit IRC15:36
kashyapSounds ominous.15:36
kashyap"Shime-waza", I see.15:37
mriedemgibi_off: hmm, wth15:38
efriedkashyap: ah, judo. The fact that they can enumerate their chokes...15:38
mriedemi'm unable to set the recursion depth limit below 55, so bumped it to 10015:38
mriedembut i wonder why the _get_counter in oslo.config isn't raising an error, unless the cache is getting reset along with our counter and that's why we continue to recurse15:39
*** lbragstad has joined #openstack-nova15:39
gibi_offmriedem: based on the stack trace it is now recursing on the code we added for trobleshooting https://review.opendev.org/#/c/654466/6/oslo_config/cfg.py@262715:40
mriedemmethinks the cache is always getting reset15:40
mriedemha15:41
mriedemsob15:41
gibi_offaccessing  self._get_counter calls __getattr__ that calls back to _get15:42
mriedemyup15:43
gibi_offsh*t15:43
mriedemso __getattr__ just needs to check for '_get_counter' and return it directly15:43
mriedemrather than recurse15:43
mriedemyeah?15:44
gibi_offmriedem: yeah15:44
mriedemupdating15:44
gibi_offmriedem: but then I don't get why the self.__cache cause the same thing15:44
mriedemdoesn't cause the same thing?15:44
mriedemmaybe it is...15:44
mriedemand maybe that's the bug15:45
gibi_offmriedem: doesn't call __getattr__15:45
gibi_offahh15:45
gibi_offmaybe that is the bgu15:45
gibi_offas you said15:45
* gibi_off has to go and pack for Denver. See you there!15:46
mriedemo/15:46
gibi_offo/15:47
mriedemhmm, __getattr__ shouldn't even be called on an attribute that's already on self15:47
mriedemhttps://docs.python.org/3.6/reference/datamodel.html#object.__getattr__15:47
*** psachin has quit IRC15:54
*** psachin has joined #openstack-nova16:00
*** wwriverrat has quit IRC16:13
*** igordc has joined #openstack-nova16:18
*** artom has joined #openstack-nova16:22
*** bbowen has joined #openstack-nova16:25
*** Sundar has quit IRC16:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add heal_instance_mappings command  https://review.opendev.org/65590816:45
mriedemmnaser: now with a buttload of tests and docs and a reno ^16:45
*** sapd1_x has joined #openstack-nova16:45
*** ccamacho has joined #openstack-nova16:45
*** ccamacho has quit IRC16:46
*** mdbooth has quit IRC16:48
*** artom has quit IRC16:51
openstackgerritEric Fried proposed openstack/nova-specs master: Abandon flavor-classes backlog spec  https://review.opendev.org/64909516:55
openstackgerritEric Fried proposed openstack/nova-specs master: Abandon instance-tasks backlog spec  https://review.opendev.org/64910016:56
openstackgerritEric Fried proposed openstack/nova-specs master: Abandon parallel-scheduler backlog spec  https://review.opendev.org/64910116:56
openstackgerritEric Fried proposed openstack/nova-specs master: Abandon same-instances-scheduling backlog spec  https://review.opendev.org/64910216:56
*** panda is now known as panda|off16:56
openstackgerritEric Fried proposed openstack/nova master: Remove [ironic]api_endpoint option  https://review.opendev.org/64348317:02
efriedeandersson: rebased that on your fix ^17:03
efriedmriedem: since this is fresh in your head, would you please? ^17:03
*** ttsiouts has joined #openstack-nova17:03
*** derekh has quit IRC17:05
mnasermriedem: awesome, thank you for picking that up.17:06
*** eharney has quit IRC17:09
efriedmriedem: I have an idea for the cinder ksa opts.17:09
efriedThe important thing - especially if we plan to ditch cinderclient - is that operators be able to use consistent opts. It matters less that we actually construct a ksa adapter (we just throw it away anyway).17:10
efriedSo I propose to skip the common get_ksa_adapter util and instead explicitly use the ksa opts when available in place of the legacy cinder ones to do the same thing the legacy cinder ones do.17:11
efriedShould be a totally seamless transition for now. And then we have ksa opt support.17:12
*** artom has joined #openstack-nova17:14
*** artom has quit IRC17:14
*** tjgresha has left #openstack-nova17:14
*** artom has joined #openstack-nova17:14
*** tjgresha has joined #openstack-nova17:15
*** sapd1_x has quit IRC17:16
openstackgerritEric Fried proposed openstack/nova master: WIP: Use ksa adapter for cinder client  https://review.opendev.org/50834517:17
efried(not this ^)17:17
*** tjgresha has quit IRC17:19
*** tjgresha has joined #openstack-nova17:19
*** tjgresha has quit IRC17:19
*** tjgresha has joined #openstack-nova17:20
*** ttsiouts has quit IRC17:21
*** ttsiouts has joined #openstack-nova17:21
*** bbowen has quit IRC17:22
*** bbowen has joined #openstack-nova17:22
*** ralonsoh has quit IRC17:22
*** ttsiouts has quit IRC17:26
*** tjgresha has quit IRC17:31
*** tjgresha has joined #openstack-nova17:31
mriedem+2 on https://review.opendev.org/64348317:34
mriedemidk about all that other sassafrass17:34
*** tjgresha has quit IRC17:36
*** ccamacho has joined #openstack-nova17:38
*** ivve has joined #openstack-nova17:50
*** mchlumsky has quit IRC17:51
*** mchlumsky has joined #openstack-nova17:52
mriedemfinal releases for pike are out17:56
*** ccamacho has quit IRC17:58
openstackgerritEric Fried proposed openstack/nova master: WIP: Support ksa config opts for cinder  https://review.opendev.org/65598518:00
openstackgerritEric Fried proposed openstack/nova master: WIP: Support ksa config opts for cinder  https://review.opendev.org/65598518:01
efriedmriedem: it look like ^18:01
mriedemartom: has there been any progress on the numa live migration series since stein?18:05
artommriedem, no :( New downstream fires keep popping up18:05
mriedemnot enough asbestos18:06
artommriedem, need to relocate RH head office here: https://goo.gl/maps/barnrHHvjL3B1kr5718:08
mriedemyou could work in that quarry18:11
mriedemplenty of natural sunlight18:11
mriedemfresh air18:11
*** jaosorior has quit IRC18:11
artom"fresh"18:11
*** hemna has quit IRC18:28
*** eharney has joined #openstack-nova18:29
*** hemna has joined #openstack-nova18:41
openstackgerritMerged openstack/nova master: SR-IOV Live migration indirect port support  https://review.opendev.org/62011518:57
openstackgerritMerged openstack/nova master: Fix mock specs set to strings  https://review.opendev.org/65545518:57
*** dtantsur is now known as dtantsur|afk18:57
jaypipesaspiers, efried, kashyap: hmm, so if these things like ssbd, virt-ssbd, idpb, etc are actually *not* x86 CPU instruction set extensions, then perhaps we should carve out a completely different are in os-traits for them.19:03
jaypipesaspiers, efried, kashyap: so, what exactly ARE these things then? :)19:03
aspiersare they cpu flags detected by the kernel?19:04
aspiersI have no idea, not looked at them19:04
jaypipesI don't know :) I'm asking you guys...19:04
efriedkashyap is the best one to answer that, but I think aspiers got it right.19:04
aspiersIIRC the kernel does expose cpu flags for certain things it spots in a chipset19:04
aspiersvulnerabilities are the obvious example here19:05
jaypipesor perhaps the actual answer is "they are some random bitmask for some ioctl() call that is entirely unstructured and hidden in some C header file in the kernel somewhere"19:05
efriedThe docs describe them as "CPU features"19:05
efriedtalk about turning them on and off19:06
efriedhttps://qemu.weilnetz.de/doc/qemu-doc.html#important_005fcpu_005ffeatures_005fintel_005fx8619:06
efried(sometimes)19:06
aspiersoh, there's a separate "bugs" line in /proc/cpuinfo19:06
aspiersprobably safer to ignore me ;-)19:06
*** luksky has joined #openstack-nova19:09
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Fix nova-grenade-live-migration run book for opendev migration  https://review.opendev.org/65580019:17
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree  https://review.opendev.org/64020719:22
*** dims has quit IRC19:25
*** psachin has quit IRC19:25
*** dims has joined #openstack-nova19:30
openstackgerritMatt Riedemann proposed openstack/nova master: Always pass HostAPI to get_availability_zones  https://review.opendev.org/65558319:30
*** dims has quit IRC19:31
*** dims has joined #openstack-nova19:31
*** hemna has quit IRC19:36
*** dims has quit IRC19:38
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-multi-cell job  https://review.opendev.org/65522219:42
openstackgerritMatt Riedemann proposed openstack/nova master: Enable n-novnc in nova-multi-cell job  https://review.opendev.org/65571119:42
openstackgerritAdam Spiers proposed openstack/nova master: Add new "supports_amd_sev" capability to libvirt driver  https://review.opendev.org/63868019:42
*** mmethot has quit IRC19:43
*** brault has joined #openstack-nova19:47
*** dims has joined #openstack-nova19:48
*** brault has quit IRC19:52
*** dims has quit IRC19:52
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-multi-cell job  https://review.opendev.org/65522219:53
openstackgerritMatt Riedemann proposed openstack/nova master: Enable n-novnc in nova-multi-cell job  https://review.opendev.org/65571119:53
*** dims has joined #openstack-nova19:59
*** dims has quit IRC20:04
*** dims has joined #openstack-nova20:05
openstackgerritMatt Riedemann proposed openstack/nova master: Remove MIN_COMPUTE_MULTIATTACH conditions in API  https://review.opendev.org/64975720:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-status upgrade check for minimum required cinder API version  https://review.opendev.org/64975920:11
openstackgerritMatt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility  https://review.opendev.org/62106120:11
openstackgerritArtom Lifshitz proposed openstack/nova master: Run revert resize tests in nova-live-migration  https://review.opendev.org/65349820:14
openstackgerritArtom Lifshitz proposed openstack/nova master: Revert "Wait for network-vif-plugged on resize revert"  https://review.opendev.org/63939620:14
openstackgerritArtom Lifshitz proposed openstack/nova master: Revert resize: wait for external events in compute manager  https://review.opendev.org/64488120:14
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate  https://review.opendev.org/57126520:16
*** tbachman has quit IRC20:19
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914  https://review.opendev.org/64152120:34
openstackbug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,In progress] https://launchpad.net/bugs/1818914 - Assigned to Matt Riedemann (mriedem)20:34
openstackgerritMatt Riedemann proposed openstack/nova master: Remove unused context parameter from RT._get_instance_type  https://review.opendev.org/64179220:34
openstackgerritMatt Riedemann proposed openstack/nova master: Update usage in RT.drop_move_claim during confirm resize  https://review.opendev.org/64180620:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid  https://review.opendev.org/61401220:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method  https://review.opendev.org/61403620:34
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Add instance hard delete  https://review.opendev.org/65098420:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add Instance.hidden field  https://review.opendev.org/63112320:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add TargetDBSetupTask  https://review.opendev.org/62789220:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask  https://review.opendev.org/63158120:34
openstackgerritMatt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask  https://review.opendev.org/63385320:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method  https://review.opendev.org/62131320:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_dest compute method  https://review.opendev.org/63329320:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add PrepResizeAtDestTask  https://review.opendev.org/62789020:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method  https://review.opendev.org/63483220:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image  https://review.opendev.org/63760520:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask  https://review.opendev.org/62789120:34
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor ComputeManager.remove_volume_connection  https://review.opendev.org/64218320:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method  https://review.opendev.org/64259020:34
openstackgerritMatt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API  https://review.opendev.org/63804820:35
openstackgerritMatt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server  https://review.opendev.org/63826820:35
openstackgerritMatt Riedemann proposed openstack/nova master: Add archive_deleted_rows wrinkle to cross-cell functional test  https://review.opendev.org/65165020:35
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellWeigher  https://review.opendev.org/61435320:35
openstackgerritMatt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API  https://review.opendev.org/63826920:35
*** tbachman has joined #openstack-nova20:37
openstackgerritMerged openstack/nova master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65439820:38
*** mriedem has quit IRC20:52
*** tjgresha has joined #openstack-nova20:53
*** tbachman has quit IRC21:02
*** hongbin has quit IRC21:04
*** mchlumsky has quit IRC21:04
*** beekneemech has quit IRC21:06
*** whoami-rajat has quit IRC21:10
*** pcaruana has quit IRC21:16
openstackgerritmelanie witt proposed openstack/nova stable/stein: Create request spec, build request and mappings in one transaction  https://review.opendev.org/65601821:22
*** mmethot has joined #openstack-nova21:37
openstackgerritEric Fried proposed openstack/nova master: WIP: Use OpenStack SDK for placement  https://review.opendev.org/65602321:38
*** luksky has quit IRC21:38
*** eharney has quit IRC21:42
*** JamesBenson has quit IRC21:48
openstackgerritDustin Cowles proposed openstack/nova master: WIP/PoC: Use SDK instead of ironicclient for node.list  https://review.opendev.org/65602721:49
openstackgerritDustin Cowles proposed openstack/nova master: WIP/PoC: Use SDK instead of ironicclient for checking instance exists  https://review.opendev.org/65602821:49
*** francoisp has quit IRC21:51
openstackgerritmelanie witt proposed openstack/nova stable/rocky: Create request spec, build request and mappings in one transaction  https://review.opendev.org/65603021:52
openstackgerritmelanie witt proposed openstack/nova stable/queens: Create request spec, build request and mappings in one transaction  https://review.opendev.org/65603121:53
openstackgerritAdam Spiers proposed openstack/nova master: Add Venn diagram showing taxonomy of traits and capabilities  https://review.opendev.org/65603221:54
*** igordc has quit IRC21:55
openstackgerritAdam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633421:59
*** artom has quit IRC21:59
*** imacdonn has quit IRC22:02
*** imacdonn has joined #openstack-nova22:03
*** KH-Jared has quit IRC22:06
efrieddustinc: Why are you labeling these WIP/PoC? It looks like you're including tests for them, yes?22:08
dustincthere are failures that I haven't figure out yet22:09
efriedokay.22:10
dustincstarting with 64289922:10
dustincI just haven't been able to figure out but wanted to push them before I travel to summit22:10
efrieddustinc: this? keystoneauth1.exceptions.auth_plugins.MissingAuthPlugin: An auth plugin is required to determine endpoint URL22:11
dustincyeah22:13
efrieddustinc: Okay, I just fixed that via https://review.opendev.org/#/c/656023/22:15
efriedAs noted in the commit, we need to move the fixture stuff into the first patch.22:15
efrieddustinc: You'll still get a failure in one test because your mock is raising the wrong exception, but easy to fix from there.22:15
openstackgerritMerged openstack/nova master: Add post-release checklist items to the PTL guide  https://review.opendev.org/65100922:17
*** slaweq has quit IRC22:24
*** ivve has quit IRC22:33
*** artom has joined #openstack-nova22:48
eanderssonWhere in the database is scheduler hints stored for a vm?22:48
melwitteandersson: request_specs table in the api database22:54
eanderssonYep just found it22:55
eanderssonThanks melwitt22:55
eanderssonI assume there is no way to update scheduler hints using cli right? :D22:56
melwittI don't think so23:01
eanderssonScheduler hints are great, but just a odd scenario where we don't want them to be respected during a cold migration.23:08
*** tosky has quit IRC23:08
*** dakshina-ilangov has joined #openstack-nova23:12
*** JamesBenson has joined #openstack-nova23:15
*** JamesBenson has quit IRC23:19
eanderssonApparently you can override the target host in cold migration, but it isn't an option in the openstack client.23:24
eanderssonOnly in the nova client.23:24
*** nicolasbock has quit IRC23:40
*** slaweq has joined #openstack-nova23:42
artomeandersson, yeah, it's a whole thing: https://etherpad.openstack.org/p/DEN-osc-compute-api-gaps23:42
*** slaweq has quit IRC23:46
*** gyee has quit IRC23:49
*** tetsuro has joined #openstack-nova23:50

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