| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-09 | |||
| 18:04:06 | johnthetubaguy | mriedem: thats a good idea, get a list of sticking point | |
| 18:05:56 | mriedem | sdague: i can use your local_conf builder in project-config now right? it just won't do anything until the d-g changes are in | |
| 18:09:42 | cfriesen | can someone explain why we need to check "if 'id' in self" in objects/flavor.py? Do we not populate the "id" field when retrieving from the api DB? | |
| 18:10:35 | mriedem | has to do with the api db lookup doesn't it? | |
| 18:12:43 | melwitt | id gets generated only after a create(). so if a flavor object wasn't looked up and hasn't yet been create()'d, it won't have 'id' | |
| 18:13:10 | dansmith | all of this needs to diaf now | |
| 18:13:24 | dansmith | it's long since unused | |
| 18:13:37 | cfriesen | dansmith: I'm in newton | |
| 18:13:47 | mriedem | but...when i upgrade directly from mitaka to pike...i might need this to work dansmith | |
| 18:14:02 | dansmith | mriedem: oh well in that case, I'll kill it right now | |
| 18:14:13 | mriedem | cfriesen: melwitt answered your question | |
| 18:14:21 | mriedem | cfriesen: and dansmith is talking about killing it in pike | |
| 18:14:41 | cfriesen | melwitt: thanks. yes, killing in pike is fine. | |
| 18:15:24 | mriedem | well i know what dan is going to be doing later today | |
| 18:16:30 | dansmith | s/later today/right fucking now/ | |
| 18:30:53 | openstackgerrit | Maciej Szankin proposed openstack/nova master: A demo script for updating resource inventory https://review.openstack.org/382613 | |
| 18:51:36 | mriedem | raj_singh: johnthetubaguy: fyi https://review.openstack.org/#/c/431713/ | |
| 18:53:22 | openstackgerrit | Diana Clarke proposed openstack/nova master: Fix nova-manage cell_v2 metavar strings https://review.openstack.org/431714 | |
| 18:58:51 | raj_singh | mriedem: nice, thx | |
| 18:59:20 | dansmith | have I mentioned that I hate flavors? | |
| 18:59:28 | EmilienM | I hate discover_hosts command | |
| 18:59:37 | EmilienM | and the fact you need to run it manually for big clouds | |
| 19:00:17 | EmilienM | we are having hard time in tripleo to understand when to run it (at the end of deployment I guess) but it's racy because we can't control the nova-compute registration | |
| 19:03:26 | EmilienM | dansmith: do you know how many time (roughly) would take a discover_hosts with ~100 hosts? | |
| 19:03:32 | mriedem | EmilienM: then enable the auto discovery in the scheduler | |
| 19:03:45 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Disassociate floating ip when fixed ip is removed https://review.openstack.org/350299 | |
| 19:03:54 | dansmith | EmilienM: loading 100 rows from the database is pretty fast | |
| 19:04:14 | EmilienM | mriedem: it's not suggested by nova if I read the code | |
| 19:04:26 | mriedem | EmilienM: i think that means like CERN | |
| 19:04:40 | mriedem | 10's of thousands of hosts | |
| 19:04:43 | mriedem | but dansmith can correct me | |
| 19:05:02 | dansmith | mriedem: nah, fewer per cell | |
| 19:05:09 | dansmith | mriedem: a few hundred at most I think | |
| 19:05:12 | mriedem | oh right | |
| 19:05:18 | EmilienM | mriedem: https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L138-L143 | |
| 19:05:19 | dansmith | which at host add time is.. not a problem | |
| 19:05:52 | mriedem | i'd think 100 hosts per cell is small enough | |
| 19:06:00 | mriedem | but i assume there is also some hedging in there | |
| 19:06:03 | mriedem | given it came in late | |
| 19:06:09 | EmilienM | dansmith: would it be acceptable in TripleO / Director to use discover_hosts_in_cells_interval ? | |
| 19:06:24 | EmilienM | you can say no, but I need to find another way to run this command after all computes are run then | |
| 19:06:36 | dansmith | EmilienM: I think you'll just race with it, and I don't understand why you can't run this after everything is setup | |
| 19:07:06 | mriedem | tripleo doesn't have some kind of post-deploy hook? | |
| 19:07:13 | EmilienM | dansmith: because nova-compute start at step 5 (last step) | |
| 19:07:20 | mriedem | so add a step 6? | |
| 19:07:34 | EmilienM | that's an option but it adds runtime in deployment | |
| 19:08:28 | EmilienM | mriedem: we have post-deploy | |
| 19:08:40 | EmilienM | mriedem: we're currently investigating if it fits in there | |
| 19:16:14 | cfriesen | does nova handle modifying extra-specs on flavors that are currently in-use by instances? If so, is there any way to retrieve the flavor extra specs that were actually used to boot a given instance? | |
| 19:25:14 | mriedem | cfriesen: the flavor, along with it's extra specs, are stored with the instance after it's created | |
| 19:25:19 | mriedem | in the instance_extra table | |
| 19:25:28 | mriedem | so you can delete the original flavor and still resize/migrate the instance | |
| 19:26:06 | mriedem | cfriesen: as for the 2nd part of your question, i think you can pull deleted flavors if you're an admin | |
| 19:26:31 | mriedem | i.e. i think you get the flavor id via the server GET, and then can GET /flavors/id | |
| 19:26:36 | mriedem | which will read deleted flavors | |
| 19:26:55 | mriedem | but you have to be admin to read deleted flavors | |
| 19:28:15 | cfriesen | mriedem: if I boot an instance with a flavor, then admin changes extra specs on that flavor, is there any way for me to dump the extra-specs that were used when I booted the instance? | |
| 19:29:10 | mriedem | cfriesen: not via the REST API i don't think so | |
| 19:29:16 | mriedem | we had a spec proposed in mitaka for this... | |
| 19:29:18 | mriedem | i'll dig it up | |
| 19:29:21 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove main_db flavor compatibility https://review.openstack.org/431722 | |
| 19:29:51 | mriedem | cfriesen: https://review.openstack.org/#/c/265282/ | |
| 19:30:05 | mriedem | basically i think the idea was, add GET /servers/<server_id>/flavor | |
| 19:30:37 | mriedem | cfriesen: mayhap you want to dust that off for pike | |
| 19:31:56 | cfriesen | mriedem: sweet, thanks. | |
| 19:32:46 | mriedem | let me know if you want me to restore that | |
| 19:33:43 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove unused validation code from block_device https://review.openstack.org/410006 | |
| 19:34:13 | cfriesen | mriedem: I'll have to talk to some folks here. looks useful though. | |
| 19:34:33 | mriedem | cfriesen: it might be easier than whatever fork you're cooking up :) | |
| 19:35:15 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Use unique volume id in fake_volume https://review.openstack.org/431723 | |
| 19:35:15 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Run api sample tests against 2.latest https://review.openstack.org/430352 | |
| 19:36:12 | cfriesen | mriedem: we've got a longstanding private patch to prevent deleting or modifying flavors that are in-use, was just wondering if it was worth porting it. I like that spec better long-term. | |
| 19:36:39 | mriedem | cfriesen: why would deleting or modifying them if they are in use matter since nova stores the original? | |
| 19:37:00 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Update hacking version https://review.openstack.org/389338 | |
| 19:37:20 | cfriesen | mriedem: the patch was originally introduced before nova stored the original, so stuff actually broke. currently it's mostly there to allow end users to see what the instance actually booted with since something like that spec doesn't exist yet. | |
| 19:37:35 | mriedem | ok | |
| 19:37:45 | mriedem | cfriesen: did you guys just get to newton? | |
| 19:37:54 | dansmith | cfriesen: nova never allowed you to update flavors | |
| 19:37:56 | mriedem | because you know we dropped api extensions in newton right? | |
| 19:38:02 | cfriesen | mriedem: yep, and yep | |
| 19:38:03 | mriedem | dansmith: i think he means extra specs | |
| 19:38:04 | dansmith | cfriesen: you could only soft-delete and re-create | |
| 19:38:07 | dansmith | ah | |
| 19:38:11 | mriedem | cfriesen: so you guys reverted the extension disabled patch? | |
| 19:38:38 | mriedem | you don't have to answer, but i'm interested | |
| 19:38:44 | mriedem | because i figured people would do that | |
| 19:38:49 | dansmith | hah | |
| 19:38:50 | dansmith | I can rephrase if you want | |
| 19:39:03 | dansmith | cfriesen: so you guys are officially committing original openstack sin now right? | |
| 19:39:15 | cfriesen | mriedem: It's actually been other guys doing the port, I'm just starting to look at it. Not actually sure what they did. | |
| 19:39:26 | mriedem | it's always the other guys | |
| 19:39:27 | mriedem | psh | |
| 19:39:29 | mriedem | :P | |
| 19:39:31 | cfriesen | dansmith: private patches? | |
| 19:39:38 | dansmith | cfriesen: against the api? yeah :) | |
| 19:39:52 | mriedem | changing the api is interop sin | |
| 19:40:00 | mriedem | if you don't care about interop, then sin away | |
| 19:40:13 | mriedem | we all know canada is rife with sinners | |
| 19:40:19 | dansmith | and if you don't care about killing the rest of the world's interop bunnies | |
| 19:40:26 | dansmith | mriedem: true story | |