| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-16 | |||
| 21:35:33 | dansmith | wait, no.. the db sync does it for cellsv1 right? | |
| 21:36:01 | mriedem | as far as i can tell, db sync is going to just sync the 'main' db | |
| 21:36:18 | dansmith | no, it does cell0 and the main one | |
| 21:36:31 | dansmith | https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L635 | |
| 21:36:36 | mriedem | oh yeah | |
| 21:36:39 | dansmith | it queries for cell0 and does it, it just doesn't list | |
| 21:37:08 | mriedem | so, | |
| 21:37:13 | mriedem | i think i'm just going to address https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L630 | |
| 21:37:16 | mriedem | and do the cell mapping list | |
| 21:37:19 | mriedem | and sync those mofos | |
| 21:38:02 | mriedem | i mean we could also do this in create_cell easily | |
| 21:38:17 | mriedem | but it looks like we thought about a fanout sync in sync already | |
| 21:42:50 | dansmith | yeah, as long as you continue to honor --local-cell | |
| 21:42:56 | mriedem | i am | |
| 21:43:00 | mriedem | should have a WIP up shortly | |
| 21:43:02 | melwitt | hm, I'm having a problem where I want to do a get of instances across all cells using target_cell and utils.spawn, but it doesn't keep the targeted cell context that way | |
| 21:43:06 | mriedem | and then i'll depends-on that for this devstack patch | |
| 21:43:49 | melwitt | because of the utils.spawn. I wanted to do them all at the same time and wait on all of them to combine the results from each cell | |
| 21:46:42 | dansmith | melwitt: isn't there some magic about storing the current context in eventlet TLS? | |
| 21:46:55 | dansmith | I dunno why or what that is because we pass it everywhere, but... there's something | |
| 21:47:34 | melwitt | dansmith: I dunno but thanks for the hint. I'll see if I can find something related to that, because for now, I'm lost | |
| 21:49:23 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | |
| 21:50:12 | melwitt | it uses the get_current() context from TLS, so that implies my target_cell modified context isn't in get_current(). makes sense because I don't think anything about target_cell saves the new targeted context to TLS | |
| 22:00:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973 | |
| 22:03:32 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 22:03:32 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 22:04:15 | mriedem | dansmith: had to remove the dependencies on that bottom change b/c the stable/newton backports for the devstack/grenade changes were keeping it from merging | |
| 22:12:17 | dansmith | mriedem: ack, thansk | |
| 22:12:39 | mriedem | man the db sync stuff in devstack is getting a bit crazy | |
| 22:12:59 | dansmith | sync, sync, sync again, sure one more why not | |
| 22:13:19 | mriedem | and if we're counting, we'll have 4 dbs in the multinode job: nova_api, nova_cell0, nova (cell1), nova_subnode_cell (cell2) | |
| 22:13:34 | dansmith | well, four dbs for a multi-cell cloud.. not bad :) | |
| 22:13:44 | mriedem | and in pike we'll have the placement db :) | |
| 22:13:55 | mriedem | but at that point it's no longer nova so not our fault | |
| 22:15:48 | dansmith | yeah | |
| 22:17:06 | mriedem | ok here is the devstack change of doom https://review.openstack.org/420976 | |
| 22:17:19 | mriedem | i would be amazed if that worked on the first try | |
| 22:18:03 | dansmith | mriedem: well, we need the late discover_hosts even without trying to create another cell | |
| 22:18:06 | dansmith | don't you want to separate those/ | |
| 22:18:38 | mriedem | i could | |
| 22:18:52 | mriedem | and get less cute with the list_cells awk | |
| 22:19:36 | mriedem | yeah let me separate that quick, basically just if n-cpu; then discover_hosts | |
| 22:19:46 | dansmith | yeah | |
| 22:21:13 | openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 22:28:24 | mriedem | ok done | |
| 22:28:39 | mriedem | dansmith: here is the change you'll need https://review.openstack.org/#/c/420981/1 | |
| 22:29:00 | dansmith | thanks | |
| 22:36:54 | mriedem | jaypipes: +2 on https://review.openstack.org/#/c/398473/ now | |
| 22:36:55 | mriedem | thanks | |
| 22:50:45 | mriedem | alright apparently i'm going to die in an ice storm and should head home, so doing that now, will be back online later | |
| 23:07:53 | openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 23:10:07 | openstackgerrit | Gage Hugo proposed openstack/nova: Fixed warning messages when building docs https://review.openstack.org/420990 | |
| 23:35:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 23:44:59 | openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | |
| 23:45:00 | openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 23:45:00 | openstackgerrit | Dan Smith proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | |
| 23:53:03 | mriedem | hmm, wtf is the difference between gate-tempest-dsvm-multinode-full-ubuntu-xenial-nv and gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv now that both are running neutron | |
| 23:56:14 | mriedem | heh, turns out, not a gd thing | |
| #openstack-nova - 2017-01-17 | |||
| 00:04:16 | mriedem | i farqed up the discover_hosts devstack patch, update coming | |
| 00:05:09 | mikal | mriedem: what does it mean to have an instance mapping in a cell with no hosts? | |
| 00:05:27 | mriedem | mikal: mayhem is what | |
| 00:05:56 | mriedem | mikal: context? | |
| 00:05:59 | mikal | mriedem: see comment on https://review.openstack.org/#/c/420451 | |
| 00:06:13 | mikal | mriedem: you check if there are any hosts, and only if there are none do you check if there are any instances | |
| 00:06:18 | mikal | mriedem: which seems... redundant to me | |
| 00:06:24 | mikal | mriedem: but also not the end of the world | |
| 00:06:43 | mriedem | good question | |
| 00:06:49 | dansmith | they're different things, and if we had a host_map delete command, | |
| 00:06:56 | dansmith | then you'd still need to check that there aren't any instances | |
| 00:07:06 | dansmith | this delete command is to let you delete cells you created and didn't setup | |
| 00:07:25 | dansmith | since you can't really have multiple real cells yet, we don't likely need more of the delete commands just yet | |
| 00:07:31 | mikal | dansmith: but doesn't no hosts imply no instances? | |
| 00:07:36 | dansmith | no | |
| 00:07:45 | dansmith | I mean, I know what you're saying, | |
| 00:07:49 | mikal | dansmith: right, explain that bit in small words for me | |
| 00:07:56 | dansmith | but if you had deleted a host you would still have some instance mappings | |
| 00:08:16 | mikal | So deleting the host which has the instances on it doesn't cascade the delete to the instances? | |
| 00:08:17 | dansmith | there is no foreign key constraint on that stuff, | |
| 00:08:22 | dansmith | so you could have nuked some things | |
| 00:08:33 | dansmith | you can't delete a host yet, so .. no :) | |
| 00:09:03 | mikal | Heh | |
| 00:09:30 | mriedem | like today you can delete a compute node in the db by deleting the nova-compute service for that host | |
| 00:09:44 | mriedem | and i'm pretty sure that doesn't mean you can't still have instnaces in the db pointing at that node | |
| 00:09:48 | dansmith | right | |
| 00:09:52 | mikal | That's a fair point | |
| 00:09:56 | dansmith | just sanity checking | |
| 00:10:01 | mikal | I don't feel we need to hold the new thing to a higher standard than what we have now | |
| 00:14:18 | mriedem | mikal: i'm failing to pick up on sarcasm there | |
| 00:19:00 | mikal | mriedem: Oh, sorry | |
| 00:19:19 | mikal | mriedem: I am glad the new code is consistent in its level of terrible with the old | |
| 00:19:23 | mikal | mgood enough? | |
| 00:19:49 | Kevin_Zheng | Hi cores, any particular reason that we have limited the length of instance tag to be 60 but created a db model with max length of 80? | |
| 00:22:22 | Kevin_Zheng | http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/models.py#n1456 db model | |
| 00:22:56 | mriedem | Kevin_Zheng: hmm, probably a question for jaypipes | |
| 00:23:31 | Kevin_Zheng | mriedm: hmm ok | |
| 00:24:10 | mriedem | my guess is the 60 number in the API comes from the spec https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/tag-instances.html#data-model-impact | |
| 00:24:37 | Kevin_Zheng | http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/tag.py#n18 FYI | |
| 00:24:50 | Kevin_Zheng | yeah, but why 80 in db model :) | |
| 00:25:00 | jaypipes | Kevin_Zheng: no idea... probably just an oversight. | |
| 00:25:22 | mriedem | snikitin: ^ | |