Posted	Nick	Remark
#openstack-nova - 2017-03-13
16:41:00	mriedem	raghav: that's going to cause you all kinds of confusion
16:41:06	mriedem	so i'd start over w/o n-cell enabled
16:41:13	raghav	mriedem: so what should i do now
16:41:38	mriedem	raghav: restack w/o n-cell for one
16:41:41	raghav	mriedem: in local.conf i have not enabled n-cell
16:42:19	mriedem	raghav: well, it's not enabled by default, so something is running it in your setup
16:42:40	mriedem	raghav: also, be aware that cells v2, which is required in ocata, is not the same as the nova-cells service,
16:42:44	mriedem	or n-cell service in devstack
16:42:47	mriedem	i know it's confusing
16:43:58	raghav	mriedem: i have n-cells-region and n-cells-child running
16:45:06	raghav	mriedem: if i re-stack again those will get enabled
16:45:58	sdague	raghav: why do you have those running?
16:46:35	raghav	mriedem: i dont know how they are running, i just stacked without enabling them
16:46:36	melwitt	raghav: what's in your stack-screenrc? are there any n-cell* in there?
16:47:01	raghav	melwitt: yes n-cell-region and n-cell-chile
16:47:08	sdague	raghav: so... step one, please get back to a base clean environment
16:47:08	raghav	melwitt: yes n-cell-region and n-cell-child
16:48:07	melwitt	raghav: okay. so it looks like you definitely have remnants of an earlier install where n-cell was part of ENABLED_SERVICES
16:48:16	mriedem	dansmith: did you still want a specific unit test for https://review.openstack.org/#/c/443403/11/nova/context.py ?
16:49:09	openstackgerrit	Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936
16:49:15	dansmith	mriedem: yes, I'm not sure why we wouldn't
16:49:20	raghav	melwitt: how can i remove it
16:49:32	dansmith	mriedem: are you doing both of those in a follow-up?
16:50:49	mriedem	dansmith: i can do them in this change
16:51:16	mriedem	raghav: maybe posting your local.conf in a gist would help us
16:51:29	mriedem	raghav: personally i'd re-clone devstack from stable/ocata to make sure it's completely clean
16:51:32	mriedem	in a new vm
16:53:13	openstackgerrit	John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809
16:53:25	melwitt	yeah, I'm not 100% an unstack.sh/stack.sh after killing processes would result in a fresh read of ENABLED_SERVICES
16:53:44	sdague	that should, but a ./clean.sh is even safer
16:53:44	mriedem	i would completely start over
16:53:54	sdague	yeh, completely fresh not a bad idea either
16:53:56	openstackgerrit	John Garbutt proposed openstack/nova-specs master: WIP: Replace live migration steps and delays with memory iteration logic https://review.openstack.org/443339
16:54:04	sdague	there are clearly bits left over from some past config
16:56:35	raghav	ok
16:56:37	raghav	thanks
17:15:08	openstackgerrit	Juan Antonio Osorio Robles proposed openstack/nova master: Make versioned notifications topics configurable https://review.openstack.org/444947
17:30:04	openstackgerrit	melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
17:30:04	openstackgerrit	melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521
17:30:05	openstackgerrit	melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
17:35:04	mriedem	dansmith: i'm really confused about this test https://github.com/openstack/nova/blob/master/nova/tests/unit/test_context.py#L307-L310
17:35:21	mriedem	shouldn't the assertion be such that the context db/mq connections within the target_cell context match what's in the mapping?
17:36:11	dansmith	mriedem: it does
17:36:52	dansmith	mriedem: maybe it should assert that get_context is called with the database url, and same for rpc, but otherwise it's checking that they get switched and restored
17:39:01	mriedem	oh nvm
17:39:09	mriedem	i was expecting to see it have the fake urls
17:46:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Decrement quota usage when deleting an instance in cell0 https://review.openstack.org/443395
17:46:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: Temporarily untarget context when deleting from cell0 https://review.openstack.org/443403
18:01:05	openstackgerrit	Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
18:01:05	openstackgerrit	Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
18:01:06	openstackgerrit	Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
18:01:06	openstackgerrit	Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163
18:01:07	openstackgerrit	Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292
18:01:07	openstackgerrit	Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174
18:01:08	openstackgerrit	Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293
18:01:08	openstackgerrit	Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487
18:01:09	openstackgerrit	Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
18:01:09	openstackgerrit	Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
18:01:10	openstackgerrit	Dan Smith proposed openstack/nova master: Add SameCellFilter https://review.openstack.org/445035
18:01:10	openstackgerrit	Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
18:05:09	dansmith	johnthetubaguy: have you gone away for the day yet?
18:05:51	melwitt	hah, my counting quota patch failed mriedem's recent quota regression functional test (the expected fail part) because I forgot it also fixes the problem
18:06:31	johnthetubaguy	dansmith: about to go
18:06:42	johnthetubaguy	dansmith: whats up?
18:06:48	dansmith	johnthetubaguy: okay well, just heads up on https://review.openstack.org/#/c/445035/
18:08:03	johnthetubaguy	dansmith: ah, cool, I will try go through that chain again tomorrow
18:08:44	johnthetubaguy	dansmith: I forgot we don't get the cell from the host object already, which kinda sucks, but that cache seems like a good stop gap
18:09:57	dansmith	johnthetubaguy: it should fill up pretty fast, not require too much memory, and only change when we add a host, so not too bad I think
18:16:36	siva_krishnan	johnthetubaguy: can you unabandon thhese patches https://review.openstack.org/#/c/308198/ https://review.openstack.org/#/c/328280/ ?
18:52:02	mriedem	rfolco: do you know who juergen leopold at ibm is? running ci on nova. i'm guessing a z person.
18:52:46	rfolco	mriedem, hmm let me check, I knew Arne from zKVM CI from Germany these days...
18:54:39	rfolco	mriedem, I guess his on the same team. They were having problems with their CI last week.
18:54:56	mriedem	ok, i just emailed him and copied markus
18:55:05	mriedem	asking to start using a system id rather than his email
18:55:12	mriedem	and just asking what he's running
19:01:43	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
19:13:06	openstackgerrit	Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228
19:13:51	openstackgerrit	Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298
19:14:01	openstackgerrit	Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185
19:14:11	openstackgerrit	Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146
19:24:24	cfriesen	someone pointed out to me that InstanceActionEvent.pack_action_event_finish() uses a "message" field, but the InstanceActionEvent model in the DB uses "details". Is this a bug?
19:26:08	cfriesen	and given that it's been in there since 2014, is anyone actually looking at it?
19:28:50	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Remove config_drive attribute from build_requests table https://review.openstack.org/432511
19:37:44	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357
19:39:22	sean-k-mooney	sdague: o/
19:39:36	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/445120
19:39:43	sean-k-mooney	sdague: not sure if your about currently but could you take a look at https://review.openstack.org/#/c/444788/ if you get a chance
19:40:39	sean-k-mooney	sdague: i asked sc68cal earlier and he said he would like you to take a look at it first.
19:40:56	openstackgerrit	Matthew Booth proposed openstack/nova master: Add RequestContext.for_cell() https://review.openstack.org/445140
19:40:56	openstackgerrit	Matthew Booth proposed openstack/nova master: Add InstanceMappingList.get_by_instance_uuids https://review.openstack.org/445139
19:40:57	openstackgerrit	Matthew Booth proposed openstack/nova master: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142
19:40:57	openstackgerrit	Matthew Booth proposed openstack/nova master: Add InstanceList.get_by_instance_mapping_list() https://review.openstack.org/445141
19:42:09	mdbooth	dansmith: I was playing with the cell context thing and came up with this series ^^^
19:42:12	sean-k-mooney	sdague: it addresses a sidefect of https://review.openstack.org/#/c/435575/ which due to the fact that devstack installes test-requirements globally effect the test cases of ovs in our ci and any other pep8 tests that are not run in a virtual envirionment
19:43:03	mdbooth	dansmith: It only affects server_external_events, but the idea's there.
19:46:33	openstackgerrit	Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730
20:04:19	sdague	sean-k-mooney: wait, this is all to include flake8-import-order?
20:04:28	sdague	can we just revert that instead?
20:05:16	sdague	sean-k-mooney: or more importantly, please explain to me the actual issue that's currently happening, how we got there, and why this is the fix?
20:06:17	sean-k-mooney	sdague: well the flake8-import-order change is what broke use but if your are running tempest test only there is noe reasong to install all the test requirements

Earlier   Later