Posted	Nick	Remark
#openstack-nova - 2017-12-01
01:43:24	openstackgerrit	Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
01:43:30	openstackgerrit	Merged openstack/nova master: VMware: Handle concurrent registrations of the VC extension https://review.openstack.org/484667
01:44:03	openstackgerrit	Merged openstack/nova master: [placement] Add 'CUSTOM_' prefix description in API ref https://review.openstack.org/521495
01:44:32	openstackgerrit	Merged openstack/nova master: Update description of Rebuild in server_concepts.rst https://review.openstack.org/522086
02:08:46	gmann	gibi: thanks
02:10:18	gmann	mriedem: on query schema BP - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-query-param
02:11:06	gmann	mriedem: only API left is cell (v1), which anyway going to be deprecated and removed so i am thinking to skip the schema for this
02:11:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007
02:14:05	gmann	mriedem: else this BP is completed.
02:19:13	gmann	mriedem: we did for deprecated APIs but cell one is not just deprecation case but going to be removed that why we can skip this ? or its easy schema for cell i can implement if you want
02:28:07	openstackgerrit	Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443
02:53:19	openstackgerrit	Merged openstack/nova master: Remove old-style quotas code https://review.openstack.org/511689
03:01:32	alex_xu	jaypipes: sir, good evening, do you want to unlock this patch https://review.openstack.org/#/c/517119/?
03:01:47	alex_xu	jaypipes: or anything I can help to unlock that patch
03:58:48	efried	alex_xu I believe jaypipes said he was working on it locally.
04:00:09	efried	alex_xu Never mind - he rebased it, but I think he thought he was done at that point :)
04:02:41	openstackgerrit	jichenjc proposed openstack/nova master: Refactor placement version check https://review.openstack.org/512497
04:07:56	openstackgerrit	jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215
04:33:53	openstackgerrit	Merged openstack/nova stable/pike: doc: fix flavor notes https://review.openstack.org/509438
04:43:13	openstackgerrit	Nguyen Van Trung proposed openstack/nova-specs master: Add doc8 to pep8 check for nova-specs project https://review.openstack.org/519561
04:44:00	alex_xu	efried: cool, then just let us waiting for the update
04:50:38	openstackgerrit	Merged openstack/nova master: Do not use “-y” for package install https://review.openstack.org/518238
05:02:42	openstackgerrit	jichenjc proposed openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
05:21:01	openstackgerrit	jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392
05:21:02	openstackgerrit	jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397
06:25:36	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
06:31:29	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
06:45:05	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653
07:11:07	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
07:44:55	nsingh	efried: To create keystoneauth session we are using some parameters like insecure, verify, cacert, cert, and key. And to creating sessionClient we are passing keystoneauth session for nova/glance/cinder. but these conf options from session are not used anywhere while creating session client. so is it only necessary to create session?
08:13:51	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506
08:17:50	openstackgerrit	Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443
08:39:40	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
08:39:41	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
08:49:46	openstackgerrit	sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457
09:00:17	openstackgerrit	Erik Berg proposed openstack/nova stable/pike: Fix binary name https://review.openstack.org/524531
09:00:19	openstackgerrit	Merged openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871
09:16:51	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
10:11:30	openstackgerrit	Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309
10:15:01	stephenfin	Man, Gerrit is slooowww this morning
10:19:44	bauzas	stephenfin: probably because of the close week-end ? :p
10:20:00	bauzas	or because it's snowing here
10:20:16	bauzas	FWIW, resorts are opening this week-end \o/
10:20:30	openstackgerrit	Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309
11:10:47	openstackgerrit	Merged openstack/nova stable/pike: Fix race in delete allocation in ServerMovingTests https://review.openstack.org/508872
12:08:54	efried	nsingh I have noticed that too - In Nova we go through all this trouble to create a session from conf parameters, and then we create the client for the service which creates its own session.
12:09:18	efried	nsingh Eventually the vision (per mordred) is to get rid of the clients altogether and just use the ksa adapter for the API communication.
12:32:51	nsingh	efried: thank you for the info. :)
12:38:12	ildikov	johnthetubaguy: hi, daily pretty please: https://review.openstack.org/#/c/330285/
12:43:52	openstackgerrit	Merged openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655
13:22:22	fried_rice	Friday!
13:24:16	openstackgerrit	Chen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command https://review.openstack.org/524588
13:26:08	fried_rice	stephenfin You around?
13:27:34	fried_rice	or bauzas or gibi or alex_xu ? Easy review, which will merge another patch for a backportable bug: https://review.openstack.org/#/c/524245/
13:40:46	stephenfin	fried_rice: I am. Looking now
13:40:53	fried_rice	stephenfin Thanks!
13:42:14	finucannot	fried_rice: Are you finding Gerrit slow today too?
13:42:28	fried_rice	finucannot Yeah, totally. I asked in -infra, but nobody bit yet.
13:42:43	finucannot	So not my network. Good to know.
13:42:52	fried_rice	That was an hour ago - more people are awake now. I'll ask again...
13:43:05	jaypipes	fried_rice: mornin
13:43:10	fried_rice	jaypipes Howdy!
13:43:19	cdent	I’ve been out of a bed for a long time, but I make no claims to being awake.
13:43:28	leakypipes	fried_rice: what was the thing with alex from earlier?
13:43:46	fried_rice	leakypipes He was asking about https://review.openstack.org/#/c/517119/
13:44:04	leakypipes	fried_rice: ah, k
13:44:06	fried_rice	I had thought you were making local changes to it, but looking back it seemed you had only rebased it.
13:44:13	leakypipes	fried_rice: yeah, I'll work on that this mroning.
13:44:17	fried_rice	Cool beans.
13:45:30	fried_rice	finucannot FYI apparently we're "waiting for an infra-root to speak some healing words" (per AJaeger)
13:45:48	fried_rice	Oh, you're in -infra too. I'll quit relaying.
13:48:46	AJaeger	fried_rice: I'm not here in general, continue relaying, please
13:48:52	mriedem	gmann_afk: no we don't need to add query param schema for the cells APIs
13:48:56	mriedem	those are likely gone in rocky
13:49:13	fried_rice	AJaeger But finucannot is in -infra, so he can follow the conversation there.
13:49:20	AJaeger	;)
14:04:28	openstackgerrit	Eric Fried proposed openstack/nova stable/pike: Proper error handling by _ensure_resource_provider https://review.openstack.org/524618
14:04:55	fried_rice	mriedem ^ pike cherry-pick, with conflicts. Think I'll wait for zuul before doing the ocata one.
14:06:15	mriedem	ok
14:07:19	finucannot	lyarwood: I assume you could review this, seeing as you only cherry-picked it (vs. writing it)? https://review.openstack.org/#/c/524157/
14:09:05	mriedem	finucannot: not really how backport reviews work
14:09:43	finucannot	mriedem: ok, I was curious about that :) Can you review because you wrote it? Can he review because he backported?
14:09:44	mriedem	i must say, however, that whoever did write the original must be very handsome indeed
14:10:39	mriedem	done
14:14:53	Guest78_	I'm having issues unshelving an instance that has PCI passthrough aliases associated with it......the wrong hypervisor host is selected. Is there a setting that I'm missing to include PciPassthroughFilter when scheduling for unshelve?
14:16:56	mriedem	unshelve should go through the same scheduling as the initial create
14:17:07	mriedem	so if you have the filters setup for the initial create properly, unshelve should work the same
14:17:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027
14:18:05	leakypipes	Guest78_: note that this channel is for development questions, so generally that would be a question for the openstack@ mailing list. However, since it's Friday... if you just want a quick and dirty fix, you can probably muck with the data in the instance_extra table for that shelved instance and change the flavor's PCI requirements manually.
14:18:06	Guest78_	80',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed: PciDeviceRequestFailed: PCI device request [InstancePCIRequest(alias_name='GeForce1080',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed
14:18:06	Guest78_	That's what I thought but that doesn't seem to be the case: 2017-12-01 13:55:52.843 2450 WARNING nova.scheduler.host_manager [req-b7876409-1149-4f8a-837d-46f0023fed10 6136e80cc67e4759843a5f95f7a9a98b 7618d5a893f04349b9feb2ca541a6fce - default default] Selected host: overcloud-novacompute-0.localdomain failed to consume from instance. Error: PCI device request [InstancePCIRequest(alias_name='GeForce10\
14:19:11	ildikov	mriedem: hi
14:19:33	ildikov	mriedem: rebased the new attach patch and ran your test on it too, it looks good so far
14:19:57	leakypipes	giblet: ++
14:20:04	leakypipes	giblet: on both the beer and the learning.
14:20:25	coffee_cat	giblet: I think it's late enough in the afternoon to get a beer now :)
14:20:43	cdent	there’s no beer in the hosue
14:24:50	giblet	ahh I'm still in the tea phase for this afternoon
14:25:49	openstack	Launchpad bug 1735719 in OpenStack Compute (nova) "Cell_v2 discover_hosts Not Working, after deleted a host from a cell " [Medium,Triaged]
14:25:49	mriedem	superdan: just came across this in triage https://bugs.launchpad.net/nova/+bug/1735719 - seems the delete_host command should mark the compute node as mapped=0 so it can be discovered again

Earlier   Later