Posted	Nick	Remark
#openstack-nova - 2017-09-13
21:21:15	bauzas	melwitt: good question, honestly it's about making sure we don't regress performance-wise, so I'd say worth having it a separate target and job
21:21:26	med_	dansmith, you have to pass this flag in ceph to delete a pool... so it's not just nova... "ceph osd pool delete {pool-name} [{pool-name} --yes-i-really-really-mean-it]"
21:21:50	bauzas	for some unknown reason, we added it to the py27 target (probably for a good reason tho )
21:22:22	melwitt	mtreinish: --no-discover is cool, thanks for the hint
21:22:39	bauzas	mtreinish: +2d as well, but I'm tempted to wait for the jobs to finish before +Wing it, thoughts?
21:23:28	melwitt	yeah, I think waiting for the jenkins vote is fine
21:23:40	mtreinish	bauzas: yeah it's find to wait
21:23:57	openstackgerrit	Matthew Treinish proposed openstack/nova master: Cleanup running of osprofiler tests https://review.openstack.org/503831
21:24:18	mtreinish	melwitt, bauzas: ^^^ see if that makes osprofiler a bit less annoying for you
21:24:35	mtreinish	although I'd be totally down with deleting it, not sure what value it actually brings
21:24:53	melwitt	yeah, I don't know the history on that either
21:26:12	cdent	melwitt, mtreinish: the performance group out of mirantis wanted to run in every project, to show that osprofiler was a thing that could be used
21:26:22	cdent	that’s lost traction, I think it may be only nova where the test runs
21:26:48	mtreinish	cdent: yeah I didn't see it in any other projects when I went looking
21:28:32	bauzas	cdent: do you know the reason why it wasn't a separate tox target and a job ?
21:28:58	melwitt	hm, there's still a quite long startup time when running with --no-discover. is it always reinstalling deps in the venv or something?
21:29:13	melwitt	(not related to any of these patches)
21:29:48	mtreinish	melwitt: hmm, with the second patch?
21:29:49	cdent	melwitt: the process of confirming the requirements is pretty big and kind of happens twice
21:30:08	cdent	once for the tox env, and then again for installing nova into the env
21:30:16	cdent	i think that second one can skip
21:30:26	melwitt	mtreinish: yeah. nothing to do with your patches
21:30:38	melwitt	cdent: I see
21:30:58	cdent	there’s room for improvement but I haven’t had a chance to look closely
21:34:33	mtreinish	melwitt: fwiw, I spun up a quick test and the second patch saves ~10secs
21:34:39	mtreinish	so a huge win :p
21:34:49	melwitt	yeah, I just tried it too. sweet. thanks for doing that
21:42:04	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Target context when setting instance to ERROR when over quota https://review.openstack.org/503839
21:49:05	mriedem	melwitt: figured out why that quota test wasn't catching the bug
21:49:10	mriedem	it is the fixture
21:50:21	melwitt	mriedem: yeah, I was just getting to the bottom of it too just now. the compute service defaults to cell1 and non-targeted DB access also defaults to cell1
21:51:30	melwitt	in general when I've wanted to make sure multi cell works, I've written a functional test with two cells and made each instance land per cell and then assert things after that. I should have done it in this case too
21:52:02	melwitt	because the default thing in the unit tests makes it act like single cell I guess
21:52:35	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Target context when setting instance to ERROR when over quota https://review.openstack.org/503839
21:52:41	mriedem	yeah ^
21:52:47	mriedem	if you remove the fix, that shows the failure
21:52:59	mriedem	melwitt: this isn't even multi-cell
21:53:13	mriedem	this is just our cell db fixture lying for us
21:53:31	melwitt	yeah, okay
21:56:30	mtreinish	dansmith: if you were curious (which I know you aren't): http://stestr.readthedocs.io/en/latest/MANUAL.html#combining-test-results and http://stestr.readthedocs.io/en/latest/MANUAL.html#running-tests describe those options I was using
22:06:17	dansmith	mtreinish: cool
22:08:17	melwitt	mriedem: I wonder if we should just make sure default compute cell != default cell when untargeted in nova/test.py
22:08:38	melwitt	because that also makes the test fail with InstanceNotFound. if I make default=True for cell2 instead of cell1
22:10:17	openstackgerrit	Merged openstack/nova master: Fix missed chown call https://review.openstack.org/503079
22:10:36	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
22:10:42	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Removing versioning from resource_provider objects https://review.openstack.org/502164
22:10:43	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the ResourceProvider object https://review.openstack.org/502163
22:10:43	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the ResourceProviderList object https://review.openstack.org/502162
22:10:44	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the Inventory object https://review.openstack.org/502161
22:10:45	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the InventoryList object https://review.openstack.org/502160
22:10:45	openstackgerrit	Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
22:10:46	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the AllocationList object https://review.openstack.org/502158
22:10:47	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the Allocation object https://review.openstack.org/502159
22:10:47	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the UsageList object https://review.openstack.org/502156
22:10:48	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the Usage object https://review.openstack.org/502157
22:10:48	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the ResourceClassList object https://review.openstack.org/502154
22:10:49	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the ResourceClass object https://review.openstack.org/502155
22:10:50	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the TraitList object https://review.openstack.org/502152
22:10:51	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
22:10:51	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Unregister the Trait object https://review.openstack.org/502153
22:45:49	openstackgerrit	Matt Riedemann proposed openstack/nova master: Target context when setting instance to ERROR when over quota https://review.openstack.org/503839
22:45:56	mriedem	melwitt: ok ^ is the best i can do today
22:47:02	melwitt	mriedem: okay, will review
#openstack-nova - 2017-09-14
00:52:58	openstackgerrit	Merged openstack/nova master: Fix test runner config issues with os-testr 1.0.0 https://review.openstack.org/503811
00:53:25	openstackgerrit	Merged openstack/nova master: Cleanup running of osprofiler tests https://review.openstack.org/503831
02:32:17	openstackgerrit	Michael Still proposed openstack/nova master: Add a file like proxy for reading files with privsep https://review.openstack.org/503893
03:35:38	openstackgerrit	Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
03:37:57	openstackgerrit	Lajos Katona proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202
03:40:24	openstackgerrit	Lajos Katona proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202
04:18:23	openstackgerrit	Lajos Katona proposed openstack/nova master: Add functional for live migrate delete https://review.openstack.org/499583
05:01:44	sapd_	Hi everyone, I'm using openstack pike. I create a queue named eventqueues, I bind this queue to nova exchange and configure notify vm state on nova-compute - notifications section. But when consume this queue, I can't get any message.
08:17:36	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386
08:56:53	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add @targets_cell for live_migrate_instance method in conductor https://review.openstack.org/503601
08:59:49	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add @targets_cell for live_migrate_instance method in conductor https://review.openstack.org/503601
09:13:00	Mr_Smurf	Could someone explain to me why it is allowed to attach an interface to an instance for a non admin user when it has shared and external router, but not when the nework is not shared?
09:13:09	Mr_Smurf	https://github.com/openstack/nova/commit/a98aa603550ad4d9f8d16de8c7acd0819680a028
09:14:05	Mr_Smurf	why bypass the policy file in one case but not the other?
09:58:52	openstackgerrit	Elod Illes proposed openstack/nova master: WIP: Add instance.interface_attach notification https://review.openstack.org/503089
12:02:13	rabel	could someone please review https://review.openstack.org/#/c/402632/ ? it is more or less ready since january
13:34:51	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386
14:29:52	pooja	Hi.. I had a question around configuring nova-scheduler in the control plane. For scalability, is it possible to run multiple nova-scheduler processes like api and conductor?
14:30:51	pooja	As I understand, the host manager needs to be aware of all provisioning ops so it may not be possible. Any thoughs? Thanks!
14:54:43	johnthetubaguy	gmann: here we go https://review.openstack.org/#/c/435484
14:54:56	gmann	johnthetubaguy: thanks
15:08:34	openstackgerrit	Merged openstack/os-vif master: Add ``HostPortProfileInfo`` class https://review.openstack.org/441590
15:11:40	bauzas	pooja: you can technically run multiple scheduler services, but since the in-memory state of the objects isn't shared between all schedulers, you can face race conditions at limits, when your cloud capacity is close to be full
15:13:19	bauzas	pooja: that is currently being tackled by the fact the scheduler (since Ocata) now uses a Placement API service that is giving it a shared view of the state of the cloud, but that's only for a couple of resource classes (RAM and CPU, disk as well but still incorrectly reported if disks are shared between computes)
15:16:27	melwitt	bauzas: I thought claims in the scheduler made it so running multiple schedulers won't reschedule because of different in-memory states?
15:17:02	bauzas	melwitt: for CPU, RAM and disk, yes
15:25:49	smcginnis	Video link from John explaining new attach https://www.youtube.com/watch?v=mrgPt0c3cUw
15:26:12	openstackgerrit	Ed Leafe proposed openstack/nova-specs master: Return Selection Objects https://review.openstack.org/498830
15:30:45	mriedem	sdague: replied to your -1 in https://review.openstack.org/#/c/493323/
15:34:41	mriedem	sdague: the way the code series goes, everything is keyed off whether or not the bdm.attachment_id is set, and that's only ever set for a *new* attachment using the new flow, which is the very last patch in the series,
15:34:54	mriedem	and that doesn't turn on until (1) all computes are upgraded and (2) cinder 3.44 is available
15:35:01	mriedem	so people can roll upgrade to this functionality
15:35:16	sdague	https://github.com/openstack/nova/blob/cfdec41eeec5fab220702efefdaafc45559aeb14/nova/compute/api.py#L3741-L3745 that's not called now?
15:35:32	mriedem	no, because if bdm.attachment_id is None: will be True
15:36:18	sdague	ok
15:36:35	mriedem	sdague: https://review.openstack.org/#/c/330285/136/nova/compute/api.py@3581
15:36:49	sdague	that seems dangerous to be implicitly working like this, vs some real const to ensure we don't go down this path

Earlier   Later