Earlier  
Posted Nick Remark
#openstack-nova - 2018-10-11
15:57:21 mriedem i just wouldn't want someone new to spin up devstack and look and think "oh nova-compute needs db access"
15:57:44 dansmith well, we also want to make sure we're not cheating on this somehow
15:57:52 mriedem right
16:08:06 mriedem you know, the service catalog clearly needs more cinder entries
16:22:43 melwitt
16:25:13 aspiers melwitt: I should have the spec submitted in the next few hours
16:25:26 melwitt ok, cool
16:32:44 mriedem heh oops http://paste.openstack.org/show/731907/
16:33:06 mriedem installed new versioned object code, restarted computes to new code but forgot to restart conductor
16:33:42 openstackgerrit Georg Hoesch proposed openstack/nova master: handle logfiles for tcp-based consoles https://review.openstack.org/575735
16:38:01 mriedem welp, bfv with volume type works http://paste.openstack.org/show/731908/
16:43:04 mriedem others might want to start reviewing that series,
16:43:09 mriedem it ends on the 14th
16:43:31 melwitt oof ok
16:45:41 sean-k-mooney mriedem_afk: that is booting with a precreated voluem or you are asking for a new volume to be created? i never do boot from volume via cli just horizon so just interested in understanding how it works
16:46:35 sean-k-mooney im guessin --block-device id=b9cdd407-14fc-42fb-beb9-d53988d866e3,source=image,dest=volume,size=1,bootindex=0,shutdown=remove,volume_type=lvmdriver-1 mean create an volume of 1GB form this image and boot with it
16:47:04 melwitt yes
16:48:39 sean-k-mooney cool the shutdown=remove is sligtly odd. that is so we can swap the root disk when the insance is shutdown im guessing?
16:49:08 dansmith shutdown=remove means delete the volume when you delete the instance, and that's been around a long time
16:49:29 melwitt yeah. it maps to delete_on_termination=True in the code
16:50:01 sean-k-mooney i see delete_on_termination=True convayes that much more cleanly
16:58:36 openstackgerrit melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
16:59:10 melwitt mriedem_afk: I added the code comment ^
17:26:48 openstackgerrit Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517
17:35:59 dansmith mriedem_afk: I feel like you could probably look at this and judge it quickly: https://review.openstack.org/#/c/609517/4
17:36:14 dansmith error handling in volume detach during live migration.. two of your favorite things :)
17:48:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517
18:03:40 aspiers following up from the whitespace discussion earlier, are typo fixes to documentation more welcomed than hated?
18:05:11 melwitt typo fixes to user-facing documentation are helpful
18:05:19 jaypipes aspiers: I think it depends on the reviewer. Personally, I'm fine with such fixes and encourage them. Increasing the quality of our docs is important to me.
18:05:41 aspiers thanks, submitting :)
18:06:21 openstackgerrit Adam Spiers proposed openstack/nova master: fix "you" typo https://review.openstack.org/609777
18:08:03 jaypipes aspiers: -3.
18:08:11 jaypipes aspiers: j/k :)
18:08:14 aspiers lol
18:08:21 imacdonn wow .. -3 ... yikes!
18:08:34 jaypipes :)
18:08:34 aspiers thanks, now I have to change my underwear
18:08:38 jaypipes +Wallaby'd
18:08:47 aspiers X-D
18:09:49 aspiers that'll push me up from being # 97,381 in the stackalytics rankings to # 97,380 \o/
18:10:05 imacdonn congrats ;)
18:10:16 aspiers and we all know success in life is defined by stackalytics
18:10:59 mriedem dansmith: i'll be the judge of how quickly i can judge something
18:11:11 dansmith aspiers: yeah my last loan officer said "aand, just one more thing, what's your stackalytics ranking?"
18:11:19 aspiers hahaha
18:11:29 dansmith mriedem: I should hope so
18:12:10 aspiers I barely squeaked through CBP at Denver airpor because my stackalytics were so poor
18:13:55 imacdonn dansmith sean-k-mooney mriedem: please rereview ... note that I had to tweak the logic a bit (see comment in review) ... https://review.openstack.org/#/c/608091/
18:14:45 aspiers dansmith: luckily my UK passport will let me into Berlin for the next few months at least :-/
18:14:55 dansmith heh
18:15:05 imacdonn you non-european european, you
18:15:24 aspiers actually I'm technically also an Irish citizen, but haven't got around to applying for that passport yet
18:15:34 aspiers (thanks mum)
18:15:40 imacdonn I have both UK and Irish passports .. it's been handy in the past
18:15:46 aspiers oh nice
18:16:07 imacdonn and a green card, so I have a few bases covered ;)
18:16:33 aspiers wow, good work
18:16:39 aspiers I have Global Entry at least
18:20:48 aspiers hmm, I guess I should submit the blueprint first, otherwise the spec will point to a non-existing bp
18:23:13 openstackgerrit melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
18:23:35 melwitt attempt number 29413059 ^
18:23:41 dansmith aspiers: yeah, which is auto-minus-2-auto-abandon
18:24:09 aspiers uh-oh ... well, that's still better than -3, so I'm improving ;-)
18:24:29 artom As long as it's not -3.14, no one should be without pie
18:24:29 openstackgerrit Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779
18:24:46 aspiers artom: I'm with you on that
18:25:04 aspiers haha
18:25:18 artom dansmith, I mean, I'll take the "giant" as a compliment
18:25:35 aspiers melwitt: here it is finally ... https://blueprints.launchpad.net/nova/+spec/amd-sev-libvirt-support
18:26:03 aspiers my very first nova spec, so I'm prepared for the -3s to roll in
18:26:12 dansmith artom: heh
18:26:24 aspiers artom: ROFL
18:27:05 aspiers melwitt: one of the key questions is whether the built-in trait:HW_CPU_x86_SEV support should be included in this spec, or split off into a separate spec
18:27:13 aspiers melwitt: I'll gratefully take guidance on that
18:27:25 melwitt that's a jaypipes question
18:27:30 aspiers OK thanks
18:27:45 melwitt but I think in this spec would be fine
18:27:51 aspiers also, as a distinct nova newb, I've probably got the wrong end of the stick on a whole bunch of other stuff
18:28:04 jaypipes aspiers: I'll have a gander later today.
18:28:08 aspiers a lot of it is based on reading source code over the last few days
18:28:13 aspiers jaypipes: awesome, thanks!
18:29:09 aspiers intention is for me (and/or maybe colleagues) to do the heavy lifting, so we're not expecting anyone else to come in and do the hard work for us
18:29:12 mriedem dansmith: i left some replies in https://review.openstack.org/#/c/605573/15
18:29:20 aspiers ... although of course we wouldn't object to that ;-)
18:30:40 aspiers oh crap, I forgot to finish the work items and dependencies sections
18:31:47 imacdonn -4 ?
18:32:13 mriedem ah traits, the new flavor extra spec
18:32:14 aspiers I'm OK with that if I get in the Guinness World Records for lowest Gerrit score
18:32:45 mriedem aspiers: have you read http://lists.openstack.org/pipermail/openstack-dev/2018-October/135446.html ?
18:32:52 aspiers almost certainly not
18:33:01 mriedem probably answers your trait questoin
18:33:03 mriedem *question
18:33:21 aspiers cool, thanks
18:34:40 mriedem tl;dr is we'll translate a flavor extra spec / image property to a trait for scheduling and the virt driver would use the extra spec/image property
18:34:41 aspiers mriedem: hooray, it seems my proposal accidentally aligns with upstream consensus on this
18:35:28 aspiers wait - doesn't that translation correspond to the first approach listed in that mail, rather than the second?
18:35:40 openstackgerrit Jim Rollenhagen proposed openstack/nova-specs master: Use conductor groups to partition nova-compute services for Ironic https://review.openstack.org/609709
18:37:08 mriedem i guess
18:37:20 mriedem i don't remember being around for the decision

Earlier   Later