Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-15
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
00:15:59 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
00:17:03 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193
01:10:38 openstackgerrit Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198
01:55:16 openstackgerrit Alex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification https://review.openstack.org/420453

Earlier   Later