Monday, 2019-06-03

*** frankwang has joined #openstack-nova00:07
*** slaweq has joined #openstack-nova00:11
*** _hemna has joined #openstack-nova00:23
*** ivve has joined #openstack-nova00:24
*** slaweq has quit IRC00:24
*** markvoelker has quit IRC00:28
*** brinzhang has joined #openstack-nova00:44
*** _hemna has quit IRC00:56
*** tkajinam has quit IRC01:00
*** tkajinam has joined #openstack-nova01:02
*** frankwang has quit IRC01:03
*** rcernin has joined #openstack-nova01:09
*** frankwang has joined #openstack-nova01:10
*** slaweq has joined #openstack-nova01:11
*** guozijn has joined #openstack-nova01:16
*** markvoelker has joined #openstack-nova01:24
*** slaweq has quit IRC01:24
*** tkajinam_ has joined #openstack-nova01:45
*** tkajinam has quit IRC01:48
*** owalsh has quit IRC01:49
*** whoami-rajat has quit IRC01:54
*** owalsh has joined #openstack-nova01:57
*** markvoelker has quit IRC01:58
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.opendev.org/56615302:11
*** minmin has joined #openstack-nova02:11
*** slaweq has joined #openstack-nova02:11
*** owalsh_ has joined #openstack-nova02:18
*** owalsh has quit IRC02:20
*** slaweq has quit IRC02:25
*** jhesketh has joined #openstack-nova02:52
*** _hemna has joined #openstack-nova02:53
*** markvoelker has joined #openstack-nova02:55
*** owalsh_ is now known as owalsh02:56
*** Dinesh_Bhor has quit IRC03:08
*** Dinesh_Bhor has joined #openstack-nova03:08
openstackgerritMerged openstack/nova-specs master: Fix a typo of 'keystoneauth1'  https://review.opendev.org/61418703:10
*** slaweq has joined #openstack-nova03:12
*** slaweq has quit IRC03:25
*** _hemna has quit IRC03:28
*** markvoelker has quit IRC03:29
*** BjoernT has joined #openstack-nova03:36
*** cfriesen has joined #openstack-nova03:50
*** BjoernT has quit IRC03:55
*** frankwang has quit IRC04:05
*** guozijn has quit IRC04:13
*** whoami-rajat has joined #openstack-nova04:19
*** cfriesen has quit IRC04:24
*** altlogbot_0 has quit IRC04:36
*** altlogbot_0 has joined #openstack-nova04:37
*** janki has joined #openstack-nova04:38
*** pcaruana has joined #openstack-nova05:08
*** abhishekk has joined #openstack-nova05:10
*** slaweq has joined #openstack-nova05:11
*** slaweq has quit IRC05:16
*** _hemna has joined #openstack-nova05:24
*** markvoelker has joined #openstack-nova05:25
*** udesale has joined #openstack-nova05:26
*** threestrands has joined #openstack-nova05:28
*** frankwang has joined #openstack-nova05:35
*** dtantsur|afk is now known as dtantsur05:44
*** efried has quit IRC05:46
*** guozijn has joined #openstack-nova05:48
*** _hemna has quit IRC05:57
*** markvoelker has quit IRC05:58
*** lpetrut has joined #openstack-nova06:06
*** slaweq has joined #openstack-nova06:11
*** dpawlik has joined #openstack-nova06:16
*** sapd1_x has joined #openstack-nova06:21
*** slaweq has quit IRC06:24
*** efried has joined #openstack-nova06:24
*** _hemna has joined #openstack-nova06:26
*** ivve has quit IRC06:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Avoid lazy-loading instance.flavor in cold migration  https://review.opendev.org/66268306:29
*** maciejjozefczyk has joined #openstack-nova06:29
*** ccamacho has joined #openstack-nova06:29
*** mandre has joined #openstack-nova06:38
*** slaweq has joined #openstack-nova06:39
mandrehi folks, what would be the recommended way to provide a relatively big file (hundreds of kb) to an instance at boot time?06:41
mandrethe --file option to openstack server create looks promising but it seems I'm hitting the size limit...06:42
*** markvoelker has joined #openstack-nova06:55
*** threestrands has quit IRC06:58
*** damien_r has joined #openstack-nova07:00
*** _hemna has quit IRC07:00
*** threestrands has joined #openstack-nova07:03
openstackgerritYongli He proposed openstack/nova master: Add server sub-resource topology API  https://review.opendev.org/62147607:09
*** lpetrut has quit IRC07:11
*** _hemna has joined #openstack-nova07:11
*** luksky has joined #openstack-nova07:12
*** aarents__ has joined #openstack-nova07:12
*** guozijn has quit IRC07:13
*** _hemna has quit IRC07:16
*** rpittau|afk is now known as rpittau07:19
*** luksky has quit IRC07:23
*** frank_wang has joined #openstack-nova07:23
*** brinzh has joined #openstack-nova07:23
*** frankwang has quit IRC07:26
*** brinzhang has quit IRC07:26
*** tstrul has joined #openstack-nova07:27
*** minmin has quit IRC07:28
*** minmin has joined #openstack-nova07:29
*** markvoelker has quit IRC07:29
openstackgerritYongli He proposed openstack/nova master: Clean up orphan instances virt driver  https://review.opendev.org/64891207:31
openstackgerritYongli He proposed openstack/nova master: clean up orphan instances  https://review.opendev.org/62776507:31
*** lpetrut has joined #openstack-nova07:31
openstackgerritSylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing  https://review.opendev.org/58908507:33
*** sridharg has joined #openstack-nova07:35
*** ivve has joined #openstack-nova07:36
*** luksky has joined #openstack-nova07:39
*** xek has joined #openstack-nova07:40
*** BlackDex has quit IRC07:41
*** jbryce has quit IRC07:41
*** mgariepy has quit IRC07:41
*** hamzy has quit IRC07:41
*** yikun has quit IRC07:41
*** rm_work has quit IRC07:41
*** icey has quit IRC07:41
*** mnaser has quit IRC07:41
*** bauzas has quit IRC07:41
*** mugsie has quit IRC07:41
*** evrardjp has quit IRC07:41
*** canori01 has quit IRC07:41
openstackgerritSilvan Kaiser proposed openstack/nova stable/rocky: Fixes multi-registry config in Quobyte driver  https://review.opendev.org/66269507:42
*** jcosmao has joined #openstack-nova07:43
*** irclogbot_3 has quit IRC07:44
*** jaosorior has joined #openstack-nova07:44
*** dansmith has quit IRC07:45
*** irclogbot_3 has joined #openstack-nova07:45
openstackgerritLuyao Zhong proposed openstack/nova master: db: Add vpmems to instance_extra  https://review.opendev.org/66269707:46
*** dansmith has joined #openstack-nova07:47
*** abhishekk has quit IRC07:47
*** dasp has quit IRC07:47
*** Kevin_Zheng has quit IRC07:47
*** amorin has quit IRC07:47
*** lxkong has quit IRC07:47
*** sorrison has quit IRC07:47
*** alex_xu has quit IRC07:47
*** tinwood has quit IRC07:47
*** ab-a has quit IRC07:47
*** dtantsur has quit IRC07:47
*** egonzalez has quit IRC07:47
openstackgerritLuyao Zhong proposed openstack/nova master: object: Add VirtualPMEM and VirtualPMEMList objs  https://review.opendev.org/63454807:49
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace  https://review.opendev.org/63454907:50
*** sapd1_x has quit IRC07:51
*** abhishekk has joined #openstack-nova07:52
*** dasp has joined #openstack-nova07:52
*** Kevin_Zheng has joined #openstack-nova07:52
*** ab-a has joined #openstack-nova07:52
*** sorrison has joined #openstack-nova07:52
*** lxkong has joined #openstack-nova07:52
*** alex_xu has joined #openstack-nova07:52
*** tinwood has joined #openstack-nova07:52
*** dtantsur has joined #openstack-nova07:52
*** egonzalez has joined #openstack-nova07:52
*** BlackDex has joined #openstack-nova07:53
*** jbryce has joined #openstack-nova07:53
*** mgariepy has joined #openstack-nova07:53
*** hamzy has joined #openstack-nova07:53
*** yikun has joined #openstack-nova07:53
*** icey has joined #openstack-nova07:53
*** rm_work has joined #openstack-nova07:53
*** mnaser has joined #openstack-nova07:53
*** bauzas has joined #openstack-nova07:53
*** canori01 has joined #openstack-nova07:53
*** mugsie has joined #openstack-nova07:53
*** evrardjp has joined #openstack-nova07:53
kashyapmnaser: Yeah, the Secure Boot spec, I'll resume work on it this week.  It is very close to getting merged.07:54
*** tssurya has joined #openstack-nova07:56
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report pmem namespaces resources by provider tree  https://review.opendev.org/63455107:57
openstackgerritLuyao Zhong proposed openstack/nova master: API: parse pmem related flavor extra spec  https://review.opendev.org/63455207:57
openstackgerritLuyao Zhong proposed openstack/nova master: scheduler: translate vpmems request to placement request group  https://review.opendev.org/63455307:57
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: boot VM with vpmems and cleanup vpmems when destroying  https://review.opendev.org/63455507:57
openstackgerritLuyao Zhong proposed openstack/nova master: compute: update pmem allocation to instance  https://review.opendev.org/66270207:57
*** threestrands has quit IRC07:58
*** ociuhandu has joined #openstack-nova08:13
*** ociuhandu has quit IRC08:13
*** ociuhandu has joined #openstack-nova08:13
*** tstrul has quit IRC08:14
*** priteau has joined #openstack-nova08:15
openstackgerritLuyao Zhong proposed openstack/nova master: update migrations_context to support migration with vpmems  https://review.opendev.org/66270908:20
*** jchhatbar has joined #openstack-nova08:20
*** janki has quit IRC08:20
openstackgerritLuyao Zhong proposed openstack/nova master: support VM resizing with vpmem data migration  https://review.opendev.org/63455608:23
*** chason has joined #openstack-nova08:23
*** jangutter has joined #openstack-nova08:25
*** markvoelker has joined #openstack-nova08:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix no propagation of nova context request_id  https://review.opendev.org/66271508:33
*** ttsiouts has joined #openstack-nova08:35
*** rcernin has quit IRC08:36
*** jchhatbar has quit IRC08:40
*** janki has joined #openstack-nova08:40
*** cdent has joined #openstack-nova08:40
*** tkajinam_ has quit IRC08:46
gibistephenfin: I still wondering about what we should do with https://review.opendev.org/#/c/660774/3/nova/compute/manager.py@447108:52
*** mdbooth has quit IRC08:53
*** tinwood has quit IRC08:55
*** tinwood has joined #openstack-nova08:57
*** kaisers has quit IRC08:58
*** markvoelker has quit IRC08:59
*** kaisers has joined #openstack-nova08:59
*** minmin has quit IRC09:00
*** tinwood has quit IRC09:01
openstackgerritMerged openstack/os-traits master: hw: cpu: Rework the directory layout; add missing traits  https://review.opendev.org/65519309:02
kashyapaspiers: --^ :-)09:03
kashyap(Morning, BTW)09:03
kashyapcdent: Thanks for putting the above through.09:04
cdentpleasure09:04
*** tinwood has joined #openstack-nova09:04
kashyapcdent: Now, I think aspiers needs a new release of 'os_traits' with the above patch.09:05
cdentyeah, doing that now09:05
kashyapAh, you're already 3 steps ahead; thanks!09:05
*** cdent has quit IRC09:11
*** _hemna has joined #openstack-nova09:12
*** abhishekk has quit IRC09:15
*** ttsiouts has quit IRC09:18
*** dtantsur has quit IRC09:20
*** dtantsur has joined #openstack-nova09:25
*** luksky has quit IRC09:27
openstackgerritMerged openstack/nova stable/rocky: Block swap volume on volumes with >1 rw attachment  https://review.opendev.org/66233309:28
*** cdent has joined #openstack-nova09:31
*** takashin has left #openstack-nova09:32
*** guozijn has joined #openstack-nova09:34
*** mdbooth has joined #openstack-nova09:39
kashyapalex_xu: Hi, when you're about: hope I answered your question here, on the MDS doc: https://review.opendev.org/#/c/661574/309:44
*** ttsiouts has joined #openstack-nova09:46
*** _hemna has quit IRC09:46
*** brinzh has quit IRC09:51
*** markvoelker has joined #openstack-nova09:55
*** bbowen_ has joined #openstack-nova09:57
*** bbowen has quit IRC10:00
*** aspiers has joined #openstack-nova10:10
aspiersbah, my znc crashed, no idea why10:10
* aspiers reads from eavesdrop10:10
aspierscdent: thanks for taking care of 0.13.0!10:10
cdentyou're welcome10:11
aspierscdent, kashyap: I guess we should have been using reno in os-traits, or is the policy not to bother?10:12
cdentwe've not bothered for the addition of traits10:13
kashyapaspiers: I welcome using 'reno' where possible.10:13
aspierscdent: any particular reason why not?10:13
aspiersmight be useful for distro packages at least10:13
cdentbecause it would be duplicating the contents of the files, almost verbatim10:13
aspierswell, the contents of the commit messages maybe10:14
cdentif there were something other than a list of strings in the package, it would make sense10:14
aspiersno great loss I guess10:14
*** ttsiouts has quit IRC10:14
aspiersalthough it doesn't sound like much additional effort either10:14
cdentthe reason we encourage the comments linking to docs is so that the point of the trait is obvious, with the trait10:14
cdentI'd really rather not add that much redundancy10:15
aspiersI'm thinking from the distro packager and operator perspective, not for developers10:15
aspiersthe former two shouldn't have to read code10:15
* kashyap bbiab10:15
cdentI think reno makes sense for the services, but for libraries that are merely symbols the distros should just do the normal thing: read the commit histroy10:15
aspiersyeah, commit history is probably good enough10:16
*** guozijn has quit IRC10:16
cdentI'm not trying to be a jerk about it or anything, it's just that there's limited capacity and things like that level of white glove service is why some people choose disros10:17
aspierssure10:18
aspiersyou're succeeding, not just trying :)10:18
cdentsome other day I'll successfully be a jerk10:19
aspiersI'm sure; I can promise you it's pretty easy - for me anyway ;-)10:19
cdentI've been having quite a lot of thoughts (mostly vague) lately on how to economize upstream development10:20
cdentmostly with regard to making gaps more visible10:23
aspiersoh?10:26
aspierswhat kinds of gaps?10:26
*** markvoelker has quit IRC10:28
cdentaspiers: some of the concepts are sort of in this https://anticdent.org/openstack-denver-summit-reflection.html but basically: making it more obvious where we don't have enough resources by simply choosing to not do things10:30
aspierscdent: yikes! 60-80 hours?10:33
cdentat least10:33
aspiersI do 30 (well, theoretically - probably do more due to same self-imposed perverse sense of loyalty)10:34
aspiersall good thoughts there10:35
cdentIs that 30 of upstream work plus some amount of time of other work, or 30 is your work week, or something else?10:36
aspierswork week10:36
cdentI'm very happy to hear that you're able to do that, there really needs to be more of that.10:37
aspiersI don't understand why any developer outside a startup would need to work long weeks10:38
cdentinteresting. you're the only one I speak to regularly who has said they do not10:38
aspiers40 I can understand, but why more?10:39
aspiersI mean, if you get paid for the overtime then fair enough, otherwise it seems crazy10:39
cdentobsessiveness10:39
aspiersIf I'm gonna hack in my spare time, it'll be on other FOSS10:39
cdentthe aforementioned perverse loyalty10:39
aspiersHah, I'm totally obsessive too10:39
cdentthere's also a twisted sense of "must keep the dream (a vague thing) alive"10:40
aspiersIf opendev Gerrit had an activity infogram like GitHub does, you'd see me doing plenty at weekends. but then I take it easier in the week to balance things out10:40
aspiersAs in, "OpenStack is dying, gotta save it" ?10:40
aspiersI think it's a long way from that10:40
cdentsomething a bit like that, but not quite the same. not quite openstack is dying, but some of the economic models associated with it10:41
aspiersyeah maybe10:41
cdentfor me, personally, a lot of it was the goal of getting placement extracted10:42
aspiersWell that's a nice thing to be proud of, looking back10:42
cdentsince it was supposed to be extracted from the start, it was, to me, 3 years late10:42
cdentso each year it was more late, was more uphill that needed to be travelled10:42
aspiersknow the feeling10:42
*** dpawlik has quit IRC10:54
*** janki has quit IRC10:57
*** janki has joined #openstack-nova10:58
kashyapcdent: Do write it up.  (Yikes, indeed!  60-80 hours)11:00
cdentkashyap: the blog post written above is a start, but there will probably be some followups if the feedback loop is sufficiently dense11:00
*** guozijn has joined #openstack-nova11:01
kashyapI also try not to do more than 40 (focused) hours, effectively they translate to 38 1/2 (official).11:01
*** ttsiouts has joined #openstack-nova11:01
cdentI think the "focused" thing is a bit of a trap. Presumably we should charge our employers for the time it takes to get warmed up, and the time it cost for them to interrupt us?11:02
kashyapFewer hours, restful sleep, comfortably paced days, realistic deadlines, time to consider --- these are things I value for a long, sustaible stint in this industry.11:02
kashyapcdent: Yeah, I know what you mean; when we "turn off", our minds are still on the problem often times.  There's a lot of "residue" that needs cleaned up11:02
kashyapI feel, that's one reason why most people those who do intense knowledge work don't feel restored after a weekend.11:03
*** trident has quit IRC11:04
cdentor to go the other way why some people who do intense knowledge work prefer not to come down off the intensity: it takes too much effort to get back up there11:05
*** trident has joined #openstack-nova11:06
kashyapcdent: Ha: "I've limited my sphere of concern" -- sounds like Stoicism :-)11:08
aspierskashyap: I've done the same11:08
cdentkashyap: yes it's exactly that11:09
cdentaccept the things I cannot change, and such11:09
kashyap[Look up "Epictetus and sphere of choice"]11:09
aspiersI've weaned myself away from being the bottleneck for various downstream stuff11:09
kashyapaspiers: Excellent; effective > productive.11:09
aspiersPeople were always like "oh, it's an HA problem, ask aspiers"11:09
aspiersbut I'm managing to get away from that11:10
kashyapaspiers: Yeah, I hear you.  Spreading expertise is always a challenge, and is a deliberate choice11:11
aspiersI made some training slides, gave some training sessions, recorded them, made the videos available, documented as much as I could11:12
aspiersthen just point people at that11:12
kashyapYeah, you did more than you can.11:12
johnthetubaguysome of the most productive folks I know are always very good at FAQ management like that, I wish I was better at it!11:13
kashyapjohnthetubaguy: Yes!  "Document _anything_ that moves" is the motto I try to drill down to whoever is willing to listen :D11:14
kashyap(Triply important when everyone is remote)11:14
kashyapcdent: On another point from your post: "it probably also means limiting required attendance to events like PTGs and summits"11:15
*** dpawlik has joined #openstack-nova11:15
kashyapI've said it before, I'll say it again: it is absolute baloney for people to _insist_ to join 2 PTGs.  (Let alone the maddening times when people were having *4* in-person meet-ups at a time, running around like head-less chickens.)11:16
cdentI don't mind them existing. I mind them being exclusive11:17
kashyapcdent: IOW, I'm with your view there; and people should *learn* to write better, and engage in written discussion.  Of course, _do_ meet in person, by all means; it's vital.  But limit it to once (and make it optional).  People have various reasons.11:17
cdentIf, for example, red hat sent all openstack engineers to the ptg, then cool11:17
cdentbut if only special ones can go, that's bs11:17
cdentkashyap: yes on the writing11:19
cdentbut you and I have covered that many times11:19
kashyap(Nod)11:20
kashyapcdent: More generally, do get your rest back, and feel restored.  There lies more power to you.11:20
cdentyes, thank you.11:20
*** jaypipes has joined #openstack-nova11:23
*** luksky has joined #openstack-nova11:24
*** markvoelker has joined #openstack-nova11:25
openstackgerritMerged openstack/nova stable/queens: Block swap volume on volumes with >1 rw attachment  https://review.opendev.org/66234011:29
*** janki has quit IRC11:33
*** frank_wang has quit IRC11:38
*** udesale has quit IRC11:40
*** udesale has joined #openstack-nova11:40
*** _hemna has joined #openstack-nova11:43
openstackgerritBoris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633411:43
*** ttsiouts has quit IRC11:43
*** ttsiouts has joined #openstack-nova11:44
*** ttsiouts_ has joined #openstack-nova11:47
*** dave-mccowan has joined #openstack-nova11:48
*** ttsiouts has quit IRC11:48
*** dave-mccowan has quit IRC11:55
*** tesseract has joined #openstack-nova11:57
*** markvoelker has quit IRC11:58
*** janki has joined #openstack-nova11:59
*** guozijn has quit IRC12:00
openstackgerritBalazs Gibizer proposed openstack/nova master: Simplfy test setup for TestNovaMigrations* tests  https://review.opendev.org/66243412:03
*** janki has quit IRC12:13
*** priteau has quit IRC12:15
*** guozijn has joined #openstack-nova12:15
*** _hemna has quit IRC12:16
*** eharney has joined #openstack-nova12:25
aspiersefried: after I implement the request filter thing for hw:mem_encryption and bbobrov addresses my feedback on his reviews, I think we'll be pretty close to being ready to hit the runway12:41
*** icey has quit IRC12:42
*** alex_xu has quit IRC12:42
*** lxkong has quit IRC12:43
*** rm_work has quit IRC12:43
*** lxkong has joined #openstack-nova12:43
*** jbryce has quit IRC12:43
*** icey has joined #openstack-nova12:43
*** alex_xu has joined #openstack-nova12:44
*** Kevin_Zheng has quit IRC12:44
*** cdent_ has joined #openstack-nova12:44
*** priteau has joined #openstack-nova12:45
*** canori01 has quit IRC12:45
*** jbryce has joined #openstack-nova12:46
*** edleafe has joined #openstack-nova12:46
*** rm_work has joined #openstack-nova12:47
*** cdent has quit IRC12:48
*** yikun has quit IRC12:50
*** cdent_ has quit IRC12:50
*** cdent has joined #openstack-nova12:54
*** yikun has joined #openstack-nova12:54
alex_xukashyap: after the instance live-migrated to _patched_ compute node, the guest won't get new cpu flag I thought, is it right?12:58
mnaseralex_xu: afaik cpu flags will never change in a running host12:59
mnasers/host/vm/12:59
alex_xumnaser: yea, that is what I'm thought also13:00
kashyapalex_xu: If you live-migrate the guest to a patched node, it is as effective as a cold-reboot, as migration serves the same role as guest cold reboot of re-exec()ing QEMU.13:01
alex_xukashyap: but we copy same domain xml in the destination node?13:01
kashyapalex_xu: Yeah; and libvirt tries very hard to not enable any newer QEMU's features on migration *without* explicit request.13:02
*** priteau has quit IRC13:03
alex_xukashyap: that means you need to restart your guest to get the new feature13:03
alex_xus/new feature/new cpu feature/13:03
alex_xuon the _patched_ node13:03
kashyapalex_xu: Yeah, the guest needs to be restarted at some point.13:04
kashyapThe point of live migration is that, it allows them to _start_ upgrading Compute nodes.13:04
kashyapBut we should actually determine all of this with a solid test:13:04
*** owalsh_ has joined #openstack-nova13:04
kashyap(1) Two Compute nodes: Compute-Old and Compute-Patched13:05
kashyap(2) Start a guest on Compute-Old, live-migrate it to Compute-Patched13:05
*** owalsh has quit IRC13:06
kashyap(3) "See what happens" (the guest on -Patched should have same flags as on -Old) — until the guest is cold-rebooted13:06
alex_xui see your point13:06
*** ttsiouts_ has quit IRC13:06
kashyapBut in general, FWIW, I'd suggest live migrating when upgrading compute nodes13:07
*** zul has joined #openstack-nova13:07
*** ttsiouts has joined #openstack-nova13:07
alex_xui' ok with that13:07
kashyapThat's the reason why I took the live migration approach, and _then_ cold-start the guest.13:07
*** kaisers has quit IRC13:07
kashyapThat way, it reduces the impact13:07
alex_xuanother thought is I'm think we shouldn't suggest the way of upgrading compute nodes in that doc, the ugprading of compute nodes should be in the upgrade doc.13:08
alex_xubut it should be fine13:08
kashyapYeah, I mentioned it for completeness' sake, as it made sense in that context.13:09
*** lbragstad has joined #openstack-nova13:09
*** _hemna has joined #openstack-nova13:09
alex_xukashyap: I change to +213:10
*** kaisers has joined #openstack-nova13:10
*** ttsiouts has quit IRC13:11
*** ttsiouts has joined #openstack-nova13:11
kashyapalex_xu: Thank you13:13
*** jaosorior has quit IRC13:15
*** cmart has joined #openstack-nova13:20
*** guozijn has quit IRC13:21
*** BjoernT has joined #openstack-nova13:23
*** _hemna has quit IRC13:23
efriedaspiers: ack. jfyi, there's gonna be a mktme spec coming down the pipe here pretty soon. You'll be pleased to see all the "make this generic" prep will be justified.13:23
aspiersefried: oh, interesting! who's working on that?13:24
aspiersyou?13:24
efriednah, I'll be reviewing it closely, but it'll be coming from someone else, not completely sure who.13:24
efriedI think it'll be in U though.13:24
aspiersgotcha13:25
aspiersefried: I'll only add to you to reviews from bbobrov which I think are ready. No point you burning time reviewing stuff where I have already identified some deficiencies which need addressing.13:26
efriedaspiers: I've been keeping my eye on those with the same philosophy :)13:26
aspiersefried: Having said that, I did write a couple of comments addressed to you13:27
aspiersbut they can wait until the -1s are dealt with13:28
kashyapefried: When you get time, the other day you said you might have some comments: https://review.opendev.org/#/c/661574/ ["Document mitigation for Intel MDS security flaws"]13:29
kashyapefried: I'll wait to respin it until I get your comments; FWIW, it has a +2 from Alex13:29
efriedkashyap: ack, I'll look at that rn...13:30
*** BjoernT_ has joined #openstack-nova13:34
*** mnaser has quit IRC13:34
*** BjoernT has quit IRC13:35
*** mnaser has joined #openstack-nova13:37
*** panda has quit IRC13:39
*** panda has joined #openstack-nova13:42
*** tbachman has joined #openstack-nova13:42
*** mriedem has joined #openstack-nova13:44
lyarwoodjohnthetubaguy: https://review.opendev.org/#/c/652800/ - thanks for the review, as this is already failing in the gate could you remove your +W so I could add a bug link in?13:48
*** cmart has quit IRC13:48
efriedkashyap: passive aggressive +0 with lots of notes https://review.opendev.org/#/c/66157413:49
efriedlyarwood: You don't need johnthetubaguy to remove his +W; it'll clear if you just submit a new patch set.13:49
efried...I'm pretty sure? Now you got me doubting.13:49
lyarwoodefried: ah cool thanks, I'll try that now13:50
*** amodi has joined #openstack-nova13:56
*** burt has joined #openstack-nova13:58
*** BjoernT has joined #openstack-nova14:02
kashyapefried: Heh, just looking.  On the "SMT vulnerable" thing, yes, that didn't sit well with me either.  Allow me to expand:14:04
*** BjoernT_ has quit IRC14:05
efriedkashyap: Expand in the document14:05
efriedI believe you14:05
kashyapefried: You will see "SMT vulnerable" when you have not disabled hyperthreading in the BIOS14:05
kashyap(Yes, will certainly expand there, too.)14:05
efriedYeah, so I don't know that that means. All I know is I see 'vulnerable' and that worries me.14:05
efriedbut I don't know if "Mitigation: ... vulnerable" means I've mitigated the vulnerability14:06
efriedor what14:06
efriedmaybe it'll be obvious to anybody who's actually qualified to be following this guide14:06
efriedif so, tell me that.14:06
efriedor maybe it would be clear by reading, from top to bottom, the link you referenced14:07
efriedwhich I did not do as part of my review :)14:07
efriedif so, you can tell me that too.14:07
cdentfg14:09
cdentoops ww14:09
cdentI temporarily have more screen real estate than I can see in one glance14:09
efriedthat sounds like a problem I want to have14:15
edleafeHeh, I type into the wrong window at least once a day. Fortuantely, I don't type my passwords too often14:18
kashyapedleafe: Hope you use a password manager :D14:23
*** lpetrut has quit IRC14:23
*** lpetrut has joined #openstack-nova14:23
* kashyap does, to avoid the bane of remembering passwords across 10s of sites. KeePassXC for the win. (Or if you're more hardcore UNIX-y types, want only command-line tools, then there is pass(1).)14:24
edleafekashyap: Been using LastPass for 8 years now, and love it. But some VMs I use require passwords instead of SSH keys. LastPass doesn't work at a terminal :)14:24
kashyapNod; whatever works (and is not buggy).14:25
kashyapefried: Whoops, missed the rest of the chat here.  I'll respond to them all on the review.14:26
*** mlavalle has joined #openstack-nova14:28
*** lpetrut has quit IRC14:35
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Add regression recreate test for bug 1830747  https://review.opendev.org/66277414:39
openstackbug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem)14:39
*** mgariepy has quit IRC14:40
*** igordc has joined #openstack-nova14:50
efriedkashyap, aspiers, mdbooth: Who's good with libvirt's use of lvm/cinder for swap devices?14:51
efried...or has the background to understand that fairly quickly14:51
mdboothefried: Me, probably14:51
*** owalsh has joined #openstack-nova14:52
mdbooth*cinder* for swap?14:52
efriedmdbooth: Cool. Ima throw you on a review if you don't mind.14:52
* mdbooth didn't think that was a thing14:52
efriedI'm not sure, really.14:52
efriedhttps://review.opendev.org/#/c/618621/14:52
efriedIt's pretty raw at this point - I get the impression the contributor is very new to coding.14:52
efriedbut perhaps you can provide guidance as to whether the fix is sane, going in the right direction.14:52
kashyapefried: mdbooth indeed14:53
efriedmdbooth: Note that this came from ML thread http://lists.openstack.org/pipermail/openstack-discuss/2019-June/006807.html which I just responded to - but my response was only in the context of "here's how to get tests to pass".14:54
*** owalsh_ has quit IRC14:54
efriedI don't have the background to know whether the fix is any good.14:54
mdboothAck. I have a short meeting in a couple of minutes, but I'll try to look at this properly within the next few hours.14:55
efriedthank you mdbooth14:55
*** belmoreira has joined #openstack-nova15:00
*** ccamacho has quit IRC15:01
*** cfriesen has joined #openstack-nova15:02
*** ttsiouts has quit IRC15:02
*** ttsiouts has joined #openstack-nova15:03
*** ccamacho has joined #openstack-nova15:04
*** macza has joined #openstack-nova15:06
*** _alastor1 has joined #openstack-nova15:06
*** eharney has quit IRC15:06
*** dklyle has joined #openstack-nova15:07
*** yaawang has quit IRC15:07
*** ttsiouts has quit IRC15:07
*** yaawang has joined #openstack-nova15:07
*** Sundar has joined #openstack-nova15:08
*** igordc has quit IRC15:08
*** igordc has joined #openstack-nova15:08
*** sridharg has quit IRC15:10
*** irclogbot_3 has quit IRC15:10
*** altlogbot_0 has quit IRC15:11
*** irclogbot_3 has joined #openstack-nova15:11
*** altlogbot_2 has joined #openstack-nova15:12
*** ttsiouts has joined #openstack-nova15:12
*** belmoreira has quit IRC15:14
openstackgerritMerged openstack/nova master: Add extra logging to request filters  https://review.opendev.org/65762915:15
*** belmoreira has joined #openstack-nova15:15
*** priteau has joined #openstack-nova15:17
*** eharney has joined #openstack-nova15:19
*** whoami-rajat has quit IRC15:20
*** _alastor1 is now known as _alastor_15:22
*** tbachman has quit IRC15:23
*** xek has quit IRC15:23
mriedemjust fyi, dfw nodes are having issues this morning so you'll be rechecking if you hit one of those nodes15:25
mriedemerror message of "('Connection broken: IncompleteRead(0 bytes read)', IncompleteRead(0 bytes read))"15:26
*** READ10 has joined #openstack-nova15:26
*** damien_r has quit IRC15:29
*** ivve has quit IRC15:30
gibimriedem: left a comment in https://bugs.launchpad.net/nova/+bug/182947915:31
openstackLaunchpad bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Undecided,Incomplete]15:31
*** priteau has quit IRC15:32
*** abhishekk has joined #openstack-nova15:33
*** dasp has quit IRC15:35
*** abhishekk has quit IRC15:37
openstackgerritMerged openstack/nova master: Ensure that metadata proxy raises correct exception  https://review.opendev.org/66148015:38
*** abhishekk has joined #openstack-nova15:39
*** tbachman has joined #openstack-nova15:44
efriedthanks mriedem, seen a few of those :(15:47
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix no propagation of nova context request_id  https://review.opendev.org/66271515:47
*** zbr has quit IRC15:47
*** dasp has joined #openstack-nova15:53
*** ircuser-1 has joined #openstack-nova15:53
*** jaypipes has quit IRC15:54
efriedmriedem: Loading an instance from the db, setting expected_attrs makes it load those fields right away; if I don't do that, I can still access the fields, but they'll get loaded later, which is another db hit and therefore slow. Do I have this right?15:55
*** zbr has joined #openstack-nova15:56
mriedemcorrect15:56
mriedeme.g. https://review.opendev.org/#/c/657021/15:56
efriedCool, so https://review.opendev.org/#/c/662683/ is sane15:56
mriedemi reported that bug but it should be hopefully what i said to do in the bug15:57
efriedoh15:58
efriedheh, yup, just so.15:58
*** ttsiouts has quit IRC15:59
*** ttsiouts has joined #openstack-nova16:00
*** amodi has quit IRC16:01
*** belmoreira has quit IRC16:01
*** tbachman has quit IRC16:03
*** ttsiouts has quit IRC16:04
*** jdillaman has joined #openstack-nova16:10
*** gyee has joined #openstack-nova16:15
*** dtantsur is now known as dtantsur|afk16:15
efriedstephenfin: You around to send https://review.opendev.org/#/c/644793/ to its maker?16:18
*** abhishekk has quit IRC16:18
*** tssurya has quit IRC16:20
*** manjeets has joined #openstack-nova16:25
*** cdent has quit IRC16:27
*** tesseract has quit IRC16:27
kashyapefried: It's a holiday in Ireland today.16:28
kashyap(And a holiday in Belgium on Monday :D)16:28
kashyap(Next one; so is in Germany and France, too.)16:28
*** rpittau is now known as rpittau|afk16:28
kashyapefried: Responded here: https://review.opendev.org/#/c/661574/3.  I'd like to one thing: is my "short and unsatisfying statement" at the end amenable to you, given the complexity?16:29
kashyaps/to/"to know"/16:29
openstackgerritMerged openstack/nova stable/stein: [ironic] Don't remove instance info twice in destroy  https://review.opendev.org/66247916:34
*** panda has quit IRC16:35
mdboothArgh, gerrit--16:35
kashyapmdbooth: Call it what it is: "Grrrrrrit!" :D16:36
mdboothhaha16:36
*** dosaboy has joined #openstack-nova16:36
*** ivve has joined #openstack-nova16:37
*** panda has joined #openstack-nova16:37
*** tbachman has joined #openstack-nova16:38
openstackgerritJohn Garbutt proposed openstack/nova master: Move default policy target  https://review.opendev.org/65769616:41
openstackgerritJohn Garbutt proposed openstack/nova master: Better policy unit tests  https://review.opendev.org/65769716:41
openstackgerritJohn Garbutt proposed openstack/nova master: Add functional test for admin_actions  https://review.opendev.org/65769816:41
openstackgerritJohn Garbutt proposed openstack/nova master: WIP: add scope check, see tests catch the change  https://review.opendev.org/65782316:41
*** luksky has quit IRC16:50
*** Sundar has quit IRC16:52
*** jdillaman has quit IRC16:58
*** whoami-rajat has joined #openstack-nova17:03
openstackgerritAdam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported  https://review.opendev.org/63868017:15
efriedkashyap: responded17:23
openstackgerritAdam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported  https://review.opendev.org/63868017:24
aspiersefried: if Zuul likes this ^^^ then it should be ready to go17:25
aspiersgotta dash, cya tomorrow17:25
efriedaspiers: Okay, did we already get the g-r patch through?17:25
efriednope17:26
efriedso it will fail constraints job until https://review.opendev.org/#/c/662805/17:26
aspiersg-r?17:26
aspiersoh17:27
efriedsokay, just leave it as is and recheck once ^ lands17:27
efriedI'm not sure if depends17:27
efrieddepends-on works to a requirements patch17:27
aspiersYeah OK17:27
aspiersgotta dash, l8r!17:27
efriedo/17:28
openstackgerritBoris Bobrov proposed openstack/nova master: Use <launchSecurity> element when SEV was required  https://review.opendev.org/66255717:32
openstackgerritEric Fried proposed openstack/os-traits master: Create trait for NUMA subtree affinity  https://review.opendev.org/65789817:34
openstackgerritEric Fried proposed openstack/os-traits master: Create trait for device subtree affinity  https://review.opendev.org/65789817:37
openstackgerritEric Fried proposed openstack/os-traits master: Create trait for device subtree affinity  https://review.opendev.org/66281117:38
openstackgerritEric Fried proposed openstack/os-traits master: Create trait for NUMA subtree affinity  https://review.opendev.org/65789817:41
*** udesale has quit IRC17:41
openstackgerritBoris Bobrov proposed openstack/nova master: Add <launchSecurity> element to guest config for AMD SEV  https://review.opendev.org/63631817:44
*** maciejjozefczyk has quit IRC17:45
*** amodi has joined #openstack-nova17:46
openstackgerritBoris Bobrov proposed openstack/nova master: Add iommu driver to devices  https://review.opendev.org/64456417:51
*** macza has quit IRC17:55
*** ociuhandu_ has joined #openstack-nova17:58
openstackgerritBoris Bobrov proposed openstack/nova master: Allow guest devices to include <driver iommu='on' />  https://review.opendev.org/64456418:00
*** ociuhandu_ has quit IRC18:01
*** cmart has joined #openstack-nova18:01
*** ociuhandu has quit IRC18:01
openstackgerritBoris Bobrov proposed openstack/nova master: Detect that SEV is required and enable iommu for devices  https://review.opendev.org/64456518:01
openstackgerritBoris Bobrov proposed openstack/nova master: Use <launchSecurity> element when SEV is required  https://review.opendev.org/66255718:02
openstackgerritBoris Bobrov proposed openstack/nova master: Use <launchSecurity> element when SEV is required  https://review.opendev.org/66255718:02
openstackgerritBoris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested  https://review.opendev.org/66255818:02
openstackgerritBoris Bobrov proposed openstack/nova master: Add <launchSecurity> element to guest config for AMD SEV  https://review.opendev.org/63631818:03
openstackgerritBoris Bobrov proposed openstack/nova master: Allow guest devices to include <driver iommu='on' />  https://review.opendev.org/64456418:05
openstackgerritBoris Bobrov proposed openstack/nova master: Detect that SEV is required and enable iommu for devices  https://review.opendev.org/64456518:06
*** ociuhandu has joined #openstack-nova18:10
*** markvoelker has joined #openstack-nova18:13
openstackgerritBoris Bobrov proposed openstack/nova master: Use <launchSecurity> element when SEV is required  https://review.opendev.org/66255718:15
*** ociuhandu has quit IRC18:15
openstackgerritBoris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested  https://review.opendev.org/66255818:15
openstackgerritBoris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested  https://review.opendev.org/66255818:16
*** damien_r has joined #openstack-nova18:19
efriedkashyap: Did you have a spec at some point about "live resize"?18:21
efriedah, it was claudiub18:22
*** damien_r has quit IRC18:23
openstackgerritMerged openstack/nova master: Avoid lazy-loading instance.flavor in cold migration  https://review.opendev.org/66268318:24
openstackgerritSylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing  https://review.opendev.org/58908518:38
*** luksky has joined #openstack-nova18:43
*** altlogbot_2 has quit IRC18:43
*** altlogbot_1 has joined #openstack-nova18:44
*** _erlon_ has joined #openstack-nova18:46
*** ttsiouts has joined #openstack-nova18:49
*** bbowen_ has quit IRC18:52
*** jdillaman has joined #openstack-nova19:05
openstackgerritMatt Riedemann proposed openstack/nova master: Add heal_instance_mappings command  https://review.opendev.org/65590819:08
*** cmart has quit IRC19:09
*** READ10 has quit IRC19:15
*** imacdonn has quit IRC19:18
*** imacdonn has joined #openstack-nova19:19
openstackgerritMatt Riedemann proposed openstack/nova master: Handle lazy-load of Migration.cross_cell_move  https://review.opendev.org/66283319:19
*** eharney has quit IRC19:19
*** altlogbot_1 has quit IRC19:21
openstackgerritMerged openstack/nova master: Reset the stored logs at each notification test steps  https://review.opendev.org/65684419:25
*** ttsiouts has quit IRC19:25
openstackgerritMerged openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests.  https://review.opendev.org/64389619:25
*** ttsiouts has joined #openstack-nova19:26
*** altlogbot_3 has joined #openstack-nova19:26
*** ttsiouts has quit IRC19:27
*** ttsiouts has joined #openstack-nova19:28
*** dklyle has quit IRC19:29
*** dklyle has joined #openstack-nova19:29
*** mgariepy has joined #openstack-nova19:33
*** cmart has joined #openstack-nova19:35
*** ttsiouts has quit IRC19:38
*** ttsiouts has joined #openstack-nova19:38
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Add regression recreate test for bug 1830747  https://review.opendev.org/66277419:39
openstackbug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem)19:39
openstackgerritMerged openstack/nova master: Bump openstackdocstheme to 1.30.0  https://review.opendev.org/66235319:42
*** ttsiouts has quit IRC19:43
openstackgerritMerged openstack/nova master: Exclude broken ironicclient versions 2.7.1  https://review.opendev.org/66240419:44
mriedemcross-cell-resize is out of the runway slot and https://blueprints.launchpad.net/nova/+spec/add-host-and-hypervisor-hostname-flag-to-create-server takes its place (2nd time in i think)19:53
*** ttsiouts has joined #openstack-nova20:04
*** bbowen_ has joined #openstack-nova20:04
*** hamzy_ has joined #openstack-nova20:06
*** mugsie_ has joined #openstack-nova20:09
*** evrardjp_ has joined #openstack-nova20:09
*** BlackDex_ has joined #openstack-nova20:10
*** bauzas_ has joined #openstack-nova20:12
*** BlackDex has quit IRC20:15
*** hamzy has quit IRC20:15
*** bauzas has quit IRC20:15
*** mugsie has quit IRC20:15
*** evrardjp has quit IRC20:15
*** bauzas_ is now known as bauzas20:15
*** imacdonn has quit IRC20:17
*** trident has quit IRC20:18
*** trident has joined #openstack-nova20:20
*** dpawlik has quit IRC20:21
*** ttsiouts has quit IRC20:29
*** ttsiouts has joined #openstack-nova20:29
*** ttsiouts has quit IRC20:34
*** manjeets has quit IRC20:40
*** altlogbot_3 has quit IRC20:45
mriedemmelwitt: efried: what do you want to do about a reno for https://review.opendev.org/#/c/579897/620:48
*** Sundar has joined #openstack-nova20:50
efriedmriedem: oh, yeah, a reno is a good idea. /me goes and pulls up the reno template...20:50
efriedsheesh, that's not very helpful.20:51
efriedmriedem: 'fixes' is fine I guess.20:51
*** slaweq has quit IRC20:52
mriedemok might as well get my nits while in there if you're updating20:53
*** nicolasbock has joined #openstack-nova20:53
efriedack20:54
efriedgimme a few20:54
*** pcaruana has quit IRC21:07
*** cfriesen has quit IRC21:19
*** slaweq has joined #openstack-nova21:20
*** _hemna has joined #openstack-nova21:20
*** cfriesen has joined #openstack-nova21:21
*** whoami-rajat has quit IRC21:22
*** _hemna has quit IRC21:25
*** slaweq has quit IRC21:25
*** eharney has joined #openstack-nova21:29
openstackgerritMerged openstack/nova master: Noop CantStartEngineError in targets_cell if API DB not configured  https://review.opendev.org/58191221:31
*** cfriesen has quit IRC21:33
*** eharney has quit IRC21:52
*** rcernin has joined #openstack-nova21:58
*** BjoernT has quit IRC22:10
openstackgerritMerged openstack/nova master: Workaround missing RequestSpec.instance_group.uuid  https://review.opendev.org/66178622:13
openstackgerritMerged openstack/nova master: Set/get group uuid when transforming RequestSpec to/from filter_properties  https://review.opendev.org/66255022:13
openstackgerritMerged openstack/nova master: Cleanup migrate flags  https://review.opendev.org/65105622:13
*** mlavalle has quit IRC22:15
openstackgerritEric Fried proposed openstack/nova master: Hide hypervisor id on windows guests  https://review.opendev.org/57989722:20
efriedmriedem, melwitt: ^22:20
efriedhopefully I did that right.22:20
mriedemSundar: efried: sean-k-mooney: gibi: my first attempt at the nova/cyborg spec https://review.opendev.org/#/c/603955/22:21
efriedthanks mriedem22:21
openstackgerritMichael Still proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing.  https://review.opendev.org/64479322:21
Sundarmriedem: Thank you very much. Will respond soon.22:25
*** luksky has quit IRC22:28
*** imacdonn has joined #openstack-nova22:34
*** BjoernT has joined #openstack-nova22:38
*** BjoernT has quit IRC22:39
*** imacdonn has quit IRC22:47
*** tkajinam has joined #openstack-nova22:52
*** zbr has quit IRC23:00
*** imacdonn has joined #openstack-nova23:08
openstackgerritDustin Cowles proposed openstack/nova-specs master: Spec: Use OpenStack SDK in Nova  https://review.opendev.org/66288123:41
*** rcernin has quit IRC23:43
*** rcernin has joined #openstack-nova23:44
*** frank_wang has joined #openstack-nova23:57
openstackgerritArtom Lifshitz proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type  https://review.opendev.org/66288723:59

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