| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-07 | |||
| 16:04:19 | mriedem | efried: dansmith: thanks | |
| 16:04:42 | mriedem | tell me when my asking for reviews on this series gets too annoying | |
| 16:05:34 | mriedem | also, my wife and daughter just left for disney world (meeting up with my mother in law), you know what that means! | |
| 16:05:52 | efried | mriedem: squeaky wheel and all that. Keep spoonfeeding me the easy ones and I'll keep reviewing them. Still haven't made it all the way through the review guide to hit the heavier ones | |
| 16:05:58 | dansmith | mriedem: eating raw brownie batter shirtless over the sink time! | |
| 16:06:07 | mriedem | close! | |
| 16:06:20 | mriedem | let's just say i still haven't showered or put contacts in yet today | |
| 16:06:38 | efried | I can smell you from here | |
| 16:06:39 | dansmith | get some brownie batter and you've got a party! | |
| 16:06:47 | mriedem | efried: yeah i've been trying to front-load as much of the simpler mechanical and test-only things for that reason | |
| 16:06:53 | mriedem | pants of dance off! | |
| 16:06:56 | mriedem | *off | |
| 16:08:14 | efried | dansmith: I'll take my slice of humble pie from you, but you've got some nerve disagreeing with jacky06 on https://review.opendev.org/#/c/688122/ | |
| 16:11:14 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Add spec for VM-scoped SR-IOV NUMA affinity https://review.opendev.org/683174 | |
| 16:11:26 | efried | d'oh, forgot to save my last batch of edits sean-k-mooney ^ | |
| 16:24:08 | dansmith | efried: my finger quivered with nerves as I hit the submit button on that one for sure | |
| 16:54:36 | efried | sean-k-mooney: reviewed, +2 | |
| 16:56:13 | sean-k-mooney | efried: thanks. | |
| 17:07:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert nova-live-migration job to zuulv3 https://review.opendev.org/693364 | |
| 17:45:30 | openstackgerrit | Eric Fried proposed openstack/nova master: Add emulated TPM support to Nova https://review.opendev.org/631363 | |
| 17:45:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934 | |
| 17:45:31 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: vTPM request_filter https://review.opendev.org/678325 | |
| 18:00:16 | sean-k-mooney | efried: by the way are you taking over teh vTPM stuff in general this cycle | |
| 18:02:06 | sean-k-mooney | i have added them to my review list in anycase | |
| 18:03:02 | efried | sean-k-mooney: yes. It's actually the whole reason I'm trying to get a devstack with modern libvirt/qemu. It's a nontrivial delta from the previously approved spec as it has to include the encryption bits. | |
| 18:03:07 | efried | and I still don't know wtf that means yet. | |
| 18:03:11 | efried | so, experimenting. | |
| 18:03:41 | sean-k-mooney | oh so the vTPM will actully encrypt its contence on the hsot | |
| 18:03:45 | sean-k-mooney | *host | |
| 18:04:04 | efried | yes, via a virsh "secret" | |
| 18:04:12 | sean-k-mooney | ya that would be a good improvment. | |
| 18:04:20 | efried | but I don't know how the "secret" gets to nova, and I don't know how the VM unlocks it. | |
| 18:04:35 | efried | the docs are pretty sparse | |
| 18:04:40 | efried | so, experimenting. | |
| 18:05:04 | efried | nice | |
| 18:05:16 | efried_afk | dansmith: local buffering fail --^ | |
| 18:05:42 | dansmith | efried_afk: boo local buffering | |
| 18:35:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use openstack server create --boot-from-volume in test_evacuate.sh https://review.opendev.org/693406 | |
| 18:48:04 | artom | mriedem, maybe a stupid question on ^^ | |
| 18:48:53 | mriedem | sob | |
| 18:51:18 | mriedem | get your downvote in while it's hot | |
| 18:51:52 | artom | OK :) | |
| 18:52:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use openstack server create --boot-from-volume in test_evacuate.sh https://review.opendev.org/693406 | |
| 18:56:03 | artom | mriedem, did we also not change the devstack default disk size to be 2 to help with the gate? | |
| 18:56:47 | artom | I suppose size was 1 before, it can stay 1 | |
| 18:57:28 | mriedem | this? https://review.opendev.org/#/c/690991/ | |
| 18:57:57 | artom | Yeah | |
| 18:58:29 | mriedem | i don't think that's going anywhere anytime soon | |
| 18:58:54 | artom | Fair enough | |
| 18:59:22 | openstackgerrit | Merged openstack/nova master: Use admin neutron client to query ports for binding https://review.opendev.org/690999 | |
| 18:59:30 | openstackgerrit | Merged openstack/nova master: Use admin neutron client to gather port resource requests https://review.opendev.org/691005 | |
| 19:05:29 | artom | \o/ | |
| 19:05:53 | artom | Actually had some of the tests in https://review.opendev.org/#/c/691062/ pass | |
| 19:06:02 | artom | Meaning the basics for a whitebox Zuul job are in place | |
| 19:06:58 | artom | sean-k-mooney's probably the only one who groked that, but hopefully whitebox will become more of a thing with time | |
| 19:08:37 | sean-k-mooney | i should proably add that to my review list | |
| 19:09:10 | sean-k-mooney | also how is that on ps34 already | |
| 19:09:11 | artom | I was going to ask anyways | |
| 19:09:17 | artom | I suck at Zuul | |
| 19:09:22 | artom | And Devstack | |
| 19:09:27 | artom | And... in general :( | |
| 19:09:44 | sean-k-mooney | there is a joke there :) | |
| 19:10:12 | artom | There are jokes everywhere | |
| 19:10:31 | artom | The question is. Who's laughing. | |
| 19:10:41 | artom | *dun dun dun* | |
| 19:11:32 | artom | Wrong channel, but sure | |
| 19:13:59 | sean-k-mooney | im sure i can find upstream ones | |
| 19:15:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove super old br- |
|
| 19:17:39 | efried | mriedem: question on --boot-from-volume | |
| 19:18:25 | mriedem | efried: looks like you answered your own question | |
| 19:18:54 | mriedem | oh wait, | |
| 19:18:55 | mriedem | " The root volume will not be deleted when the server is deleted." | |
| 19:18:57 | efried | no, the docs say "the volume is ... | |
| 19:18:58 | efried | yeah | |
| 19:19:00 | mriedem | i thought it did remove it by default | |
| 19:19:41 | efried | I would say "this is CI which will clean up your mess anyway"... | |
| 19:19:45 | efried | but not sure how CI does cinder | |
| 19:20:23 | efried | I guess it would make sense for it to be a local cinder somehow, but I don't know | |
| 19:20:34 | mriedem | we'd be orphaning the volume. if this were tempest it would make sure to clean up the volume at the end of the test. | |
| 19:20:47 | efried | oh, my comments published backwards, wtf? | |
| 19:20:50 | mriedem | i'd add a volume delete after openstack server delete --wait evacuate-bfv-test | |
| 19:20:54 | efried | they make more sense read the other way around. | |
| 19:21:40 | efried | I thought there was an option to make it delete the volume when the instance goes away. But I guess that's still just in novaclient, not in osc yet? | |
| 19:22:02 | sean-k-mooney | efried: you can set it in the block device mappings | |
| 19:22:08 | mriedem | you can do it in osc if using the --block-device-mapping option, | |
| 19:22:14 | sean-k-mooney | but i dont know if you can do that with osc | |
| 19:22:17 | mriedem | which used to not allow passing id as a source image, | |
| 19:22:21 | sean-k-mooney | oh ok | |
| 19:22:28 | mriedem | but when i added --boot-from-volume i think i fixed that as well | |
| 19:22:46 | mriedem | yeah so i just need to change to use --block-device-mapping | |
| 19:23:09 | mriedem | updating | |
| 19:23:10 | efried | mmkay | |
| 19:24:52 | efried | those docs for --block-device-mapping could stand to be formatted a bit better. Now that we're generating them, you only have to update one place in osc... | |
| 19:24:53 | efried | :P | |
| 19:26:29 | mriedem | god now i have to look at the osc source code to see what it passes for boot_index | |
| 19:26:46 | mriedem | because you can't specify a boot index and if it doesn't pass one it defaults to None (not root) | |
| 19:27:27 | efried | so add that to the docs while you're in there reformatting | |
| 19:27:33 | efried | cause I know you were already doing that | |
| 19:28:02 | mriedem | i'm about to abandon this gd turd | |
| 19:29:45 | artom | *snerk* | |