| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-07 | |||
| 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: WIP: vTPM request_filter https://review.opendev.org/678325 | |
| 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 | |
| 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* | |
| 19:29:59 | artom | I mean, the novaclient isn't getting EOL'ed anytime soon, what's wrong with keeping it? | |
| 19:31:04 | mriedem | ok so if you use --block-device-mapping you have to specify one of --image|--image-property|--volume and if you use --image, that's an ephemeral root disk and the bdm is just a non-root volume attachment | |
| 19:32:02 | mriedem | artom: i try as much as possible to use osc for new things so we can stop using nova cli | |
| 19:32:04 | efried | so much for "I can just *" | |
| 19:32:22 | sean-k-mooney | artom: the main blockers to killing the nova client is we dont have a good way to update osc and lack of people working on closing the gaps | |
| 19:32:32 | mriedem | --boot-from-volume preserves the volume by default b/c that's the default in the api for bdm.delete_on_termination (false) | |
| 19:32:50 | artom | sean-k-mooney, well, osc is a moving target as well | |
| 19:33:04 | artom | Now it's moving to openstacksdk, right? So we should be updating that as well? | |
| 19:33:11 | openstackgerrit | Dustin Cowles proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/693414 | |