| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-15 | |||
| 17:22:45 | dansmith | no, that's the thing.. we call it "nova_api_cell0" but put main db schema into it | |
| 17:22:55 | dansmith | that's why we don't explode and that's why it's super confusing | |
| 17:26:07 | mriedem | oh i see http://git.openstack.org/cgit/openstack/nova/tree/nova/cmd/manage.py#n1104 | |
| 17:26:45 | mriedem | that would have to pass database='api' to make it the api db schema | |
| 17:26:53 | mriedem | ok yeah so that's an easier fix | |
| 17:27:18 | mriedem | i think we can just rename the db in grenade/devstack since cell0 isn't used in newton anyway, and then fix the conf option we use in map_cell0 | |
| 17:27:22 | mriedem | i'll open a bug | |
| 17:28:02 | dansmith | mriedem: the easiest thing to do would be to merge the create_cell change, then I can make devstack and grenade use the same exact code path, and change the db name at the same time | |
| 17:28:36 | dansmith | sorry, | |
| 17:28:48 | dansmith | I meant make devstack and grenade take the same code path for cellsv1 and cellsv2 | |
| 17:30:19 | openstack | Launchpad bug 1656673 in OpenStack Compute (nova) "map_cell0 should use the main DB connection, not the API DB" [High,Confirmed] | |
| 17:30:19 | mriedem | https://bugs.launchpad.net/nova/+bug/1656673 | |
| 17:31:17 | mriedem | sounds like i should also work on creating a list_cells CLI today | |
| 17:32:10 | dansmith | well, it's not required for any of this, but I agree we should have that for clarity | |
| 17:33:30 | mriedem | not for any of *this* right, but several commands take a cell uuid, like i discover_hosts | |
| 17:34:01 | dansmith | well, discover hosts doesn't need it, but yes, agreed | |
| 17:34:49 | mriedem | ok, i'm still unfed and unshowered and therefore unusable, so i'll do those in awhile | |
| 17:39:15 | dansmith | mriedem: those? or just list_cells? | |
| 17:39:28 | dansmith | I think I can do the devstack one without too much drama | |
| 17:39:36 | dansmith | and I am fed, caffeinated | |
| 17:57:29 | mriedem | sorry i meant i was going to work on list_cells | |
| 17:57:33 | mriedem | leaving you all of the hard work | |
| 17:57:44 | dansmith | okay, all good | |
| 17:57:46 | mriedem | oh i'm caffeinated, | |
| 17:57:49 | mriedem | that's top priority | |
| 17:58:02 | dansmith | heh | |
| 18:46:59 | openstackgerrit | Dan Smith proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 18:46:59 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 19:02:34 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 19:02:35 | openstackgerrit | Dan Smith proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 19:04:58 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 19:05:16 | dansmith | I think the devstack fail is related to the failure to talk to neutron, as I don't see any evidence that it's not finding the cell0 map, but we'll see on recheck | |
| 19:08:06 | mriedem | i'm surprised the devstack change doesn't require the grenade change | |
| 19:08:13 | mriedem | or vice-versa, | |
| 19:08:22 | mriedem | but that's just b/c i'm usually used to this being complicated | |
| 19:11:40 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-manage cell_v2 list_cells command https://review.openstack.org/420440 | |
| 19:11:50 | dansmith | I dunno, I thought the point of the changes in grenade was to insulate them from devstack | |
| 19:12:09 | dansmith | remember the first version of one of those, I used the method from devstack and sdague said I should just put the commands themselves in there | |
| 19:12:17 | dansmith | I have to run off in a little bit, but I will check on these when I get back | |
| 19:18:01 | openstackgerrit | Matt Riedemann proposed openstack/nova: Remove nova-manage image from man pages https://review.openstack.org/420442 | |
| 19:40:10 | mriedem | melwitt: dansmith: it occurs to me that we're going to need a HostMappingList object to get host mappings by cell mapping, to be used in a nova-manage cell_v2 delete_cell command to see if the cell mapping is empty or not | |
| 19:42:12 | mriedem | but i'm not sure if we're going to need to backport the delete_cell command to newton or not, if not then i can make it use a new HostMappingList object, otherwise i have to write it to query the DB API directly from the CLI | |
| 19:52:52 | dansmith | mriedem: I think it's fine to not have a full set of commands in newton | |
| 19:53:14 | dansmith | and like I said, I think it's okay if the simple command serves as the "upgrade from newton" command | |
| 19:53:48 | mriedem | ok | |
| 19:56:11 | mriedem | we might want to fix https://review.openstack.org/#/c/420439/ outside of your series | |
| 19:57:12 | openstackgerrit | Dan Smith proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 19:57:12 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 19:57:13 | mriedem | since i'm assuming getting instance creation moved to conductor working is going to be a bit of a beast | |
| 19:57:23 | dansmith | mriedem: yeah, that one isn't actually dependent on anything | |
| 19:57:31 | dansmith | other than the devstack change but that's just for testing | |
| 19:57:39 | dansmith | once we see that work with and without this, we can remove that | |
| 19:57:46 | dansmith | I dunno why gerrit is showing it that way though, | |
| 19:57:51 | dansmith | oh wait | |
| 19:58:40 | dansmith | yeah, that was unintentional | |
| 19:58:44 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 19:58:45 | dansmith | fixed ^ | |
| 19:58:45 | openstackgerrit | Dan Smith proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 19:58:53 | dansmith | gotta run now, back in a bit | |
| 20:04:13 | mriedem | ok i'm going to start on delete_cell, hopefully crank that out by 3:30 at which point i have to stop for football | |
| 21:13:32 | mriedem | something blowing up in grenade http://logs.openstack.org/39/420439/4/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/89fb1cf/logs/new/screen-n-api.txt.gz?level=TRACE#_2017-01-15_20_32_29_818 | |
| 21:13:40 | mriedem | DBNonExistentDatabase: (pymysql.err.InternalError) (1049, u"Unknown database 'nova_cell0'") | |
| 21:14:01 | mriedem | which is why i think the devstack change will need to depend on the grenade change probably | |
| 21:36:20 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add InstanceMappingList.get_by_cell_id https://review.openstack.org/420450 | |
| 21:36:20 | openstackgerrit | Matt Riedemann proposed openstack/nova: Create HostMappingList object https://review.openstack.org/420449 | |
| 21:36:21 | mriedem | dansmith: melwitt: delete cell command and friends ^ | |
| 21:36:21 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-manage cell_v2 delete_cell command https://review.openstack.org/420451 | |
| 21:47:07 | dansmith | "have to stop for football" | |
| 21:48:23 | mriedem | dansmith: dvr | |
| 21:48:31 | mriedem | dansmith: so i'm updating your nova cell0 rename thing right now | |
| 21:48:39 | mriedem | and adding a release note and depends-on the grenade change | |
| 21:49:01 | mriedem | dansmith: we also unfortunately have some existing renos that say the cell0 db name has to be the <api db name>_cell0 | |
| 21:49:04 | mriedem | oops | |
| 21:49:48 | dansmith | okay just got back and reading stuff | |
| 21:49:51 | dansmith | so I will wait for that | |
| 21:49:59 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 21:50:00 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | |
| 21:51:04 | mriedem | i'm going to link the nova patch into the grenade patch too for clarity | |
| 21:52:14 | dansmith | I can do it, go football | |
| 21:54:25 | dansmith | mriedem: | |
| 21:54:33 | dansmith | oh right, grenade patch heh | |
| 21:55:15 | mriedem | updated | |
| 21:57:02 | openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 21:57:10 | mriedem | i'm back here now! | |
| 21:57:18 | dansmith | fine, then: | |
| 21:57:20 | dansmith | go football | |
| 21:57:21 | mriedem | ok, so grenade patch is updated, nova change depends on it with renos | |
| 21:57:29 | mriedem | and my list and delete cells patches are all up | |
| 21:57:34 | dansmith | cool, thanks | |
| 22:00:03 | mriedem | and https://wiki.openstack.org/wiki/Nova-Cells-v2 should be all updated | |
| 22:00:12 | mriedem | ok, f'ing football time | |
| 22:04:39 | mriedem | btw, ftr, i hate aaron rodgers' stupid hair marying face | |
| 22:04:47 | mriedem | *hail | |
| #openstack-nova - 2017-01-16 | |||
| 00:12:21 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and response in os-quota-sets.inc https://review.openstack.org/416461 | |
| 00:12:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | |
| 00:13:18 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 00:14:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc https://review.openstack.org/417233 | |
| 00:15:20 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 00:15:31 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | |
| 00:15:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | |