| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-07 | |||
| 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 | |
| 19:33:30 | artom | And it was/is missing basic microversion support | |
| 19:33:38 | sean-k-mooney | artom: the intent is we woudl jsut update the sdk for the api changes and then have a thin wrapper in osc | |
| 19:33:55 | sean-k-mooney | but yes its moving slowly to the sdk | |
| 19:34:30 | artom | mriedem, I agree with the sentiment :) It often comes up downstream - clients or whoever will say "I can't do this with osc [which is the recommended client for interaction with our OSP product]" and we'll come back with "use novaclient instead" | |
| 19:35:37 | sean-k-mooney | artom: we only do that if there is no other choice | |
| 19:35:39 | sean-k-mooney | but yes | |
| 19:35:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove the TODO about using OSC for BFV in test_evacuate.sh https://review.opendev.org/693406 | |
| 19:36:01 | efried | hahaha | |
| 19:36:04 | sean-k-mooney | although we really should file a bug downstream to fix the osc gap everytime we do that | |
| 19:37:04 | efried | wasn't there a spec to allow you to flip the 'remove on instance deletion' flag after the instance was created? | |
| 19:37:19 | mriedem | yes | |
| 19:37:30 | sean-k-mooney | efried: i think they wanted to set it as part of the attachment or somthing | |
| 19:37:32 | mriedem | that would be a pretty crappy workaround here | |
| 19:37:43 | efried | https://review.opendev.org/#/c/580336/ | |
| 19:37:55 | artom | *snerk* | |
| 19:37:58 | efried | mriedem: but eventually we'll want osc support for setting that option initially, right? | |
| 19:38:25 | mriedem | i just loudly audibly sighed | |
| 19:38:27 | efried | so like, instead of removing the TODO, you should *expand* it to explain all of that. | |
| 19:39:10 | mriedem | making the --boot-from-volume optional more customizable like the --block-device-mapping option is not what we want, it defeats the purpose of a simple boot from volume option, | |
| 19:39:18 | efried | right | |
| 19:39:22 | mriedem | so that likely means adding bootindex= to --block-device-mapping | |
| 19:39:36 | sean-k-mooney | honestly the default shoudl proably be delete on instnace terminate | |
| 19:39:40 | mriedem | but that has implications for how the command currently requires you to specify --image|--image-property|--volume | |
| 19:39:45 | efried | you need a new option, --remove-volume-on-instance-deletion, which is only allowed with --boot-from-volume | |
| 19:40:01 | mriedem | i don't think that's much better ux | |
| 19:40:07 | efried | I'm not serious. | |
| 19:40:13 | mriedem | heh, ok | |
| 19:40:35 | efried | I know it's tough to tell with my rbf and deadpan humor style. | |
| 19:40:41 | artom | --boot-from-volume size=1,cleanup=plz | |
| 19:40:50 | mriedem | sean-k-mooney: if you have a dumb ass old app which stores data in the root volume, then deleting that by default is likely not what you want | |
| 19:40:57 | mriedem | artom: f and no | |
| 19:41:00 | artom | Conversly, --boot-from-volume size=1,cleanup=nowai | |
| 19:41:00 | mriedem | see what i just said above | |