Posted	Nick	Remark
#openstack-nova - 2017-11-30
23:31:00	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
23:46:10	efried	mriedem No, not going to hold it up for that.
#openstack-nova - 2017-12-01
00:24:40	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/523194
00:24:41	openstackgerrit	Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027
00:25:19	mriedem	alex_xu: is this kind of what you were looking for? https://review.openstack.org/#/c/523194/2/nova/api/openstack/compute/rest_api_version_history.rst
00:34:23	tetsuro	jay: are you there?
01:19:24	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
01:39:01	openstackgerrit	Merged openstack/nova master: Hyper-V: fix live migration with CSVs https://review.openstack.org/380448
01:40:02	openstackgerrit	Merged openstack/nova master: Remove unused global variables https://review.openstack.org/524126
01:40:33	openstackgerrit	Merged openstack/nova master: Vzstorage: synchronize volume connect https://review.openstack.org/505708
01:41:04	openstackgerrit	Merged openstack/nova master: Fix incorrect known vcpuset when CPUPinningUnknown raised https://review.openstack.org/517592
01:41:31	openstackgerrit	Merged openstack/nova master: Use NoDBTestCase for powervm driver tests https://review.openstack.org/520118
01:42:03	openstackgerrit	Merged openstack/nova master: [placement] Fix format in placement API ref https://review.openstack.org/521483
01:42:30	openstackgerrit	Merged openstack/nova master: [placement] Fix parameter order in placement API ref https://review.openstack.org/516233
01:42:57	openstackgerrit	Merged openstack/nova master: Fix TypeError of _get_project_id when project_id is None https://review.openstack.org/522100
01:43:23	openstackgerrit	Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505
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

Earlier   Later