| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-11 | |||
| 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 | aspiers | thanks, now I have to change my underwear | |
| 18:08:34 | jaypipes | :) | |
| 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 | 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:29 | artom | As long as it's not -3.14, no one should be without pie | |
| 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 | |
| 18:37:51 | aspiers | that mail says "Ultimately, the decision was made to go with the second approach." | |