Posted	Nick	Remark
#openstack-nova - 2017-06-12
22:31:10	cfriesen	if anyone's on....do we create a migration record for a "rebuild", and if so where is that in the code?
22:31:47	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/472814
22:54:11	openstackgerrit	Merged openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662
22:59:03	mriedem	efried: does ^ close the service token bp?
22:59:15	mriedem	the rest looked like dnm testing patches
23:07:41	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
23:12:32	mriedem	good god has anyone read the "Strawman Queens PTG week slicing" thread yet and can summarize?
23:12:37	mriedem	because i don't want to start it
23:37:49	openstackgerrit	Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
23:41:59	mriedem	dansmith: i think i just realized something we broke with the os-services API being cellsified
23:42:13	mriedem	broke/made more obviously not dumb
23:42:33	mriedem	for things like enable/disable, today you pass in the host/binary to lookup the service
23:42:45	mriedem	with multi-cells, we now lookup the host mapping by the host
23:42:51	mriedem	host mapping will only exist for compute services
23:42:54	openstackgerrit	John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
23:42:59	mriedem	but i think before you could technically disable nova-scheduler services
23:43:03	mriedem	which doesn't mean anything, but you could do it
23:50:00	openstackgerrit	Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
23:53:16	mriedem	Kevin_Zheng: pretty easy bug to fix https://bugs.launchpad.net/nova/+bug/1697563
23:53:16	openstack	Launchpad bug 1697563 in OpenStack Compute (nova) "api-ref: GET /os-services does not describe host/binary request query filter parameters" [Medium,Triaged]
23:58:43	dansmith	mriedem: yeah, so someone was recently asking me why disable on nova-scheduler didn't actually stop those services at the OS level
23:59:05	dansmith	mriedem: we should probably return "40x: You can't do that thing" if they try
23:59:08	mriedem	the api-ref docs all say "compute" service too
23:59:10	mriedem	which is how i found this
23:59:13	dansmith	okay
23:59:30	mriedem	so that's the question i have in https://review.openstack.org/#/c/464280/9/nova/api/openstack/compute/services.py@281
23:59:35	mriedem	which is, do we now make this a 400
23:59:44	mriedem	before that new microversion, it's going to be a 404 now for non-compute services
#openstack-nova - 2017-06-13
00:00:01	mriedem	with the microversion, we could get non-compute things again, or just close that weird gap
00:00:40	mriedem	probably better to restrict to nova-compute services for things like enable/disable/forced_down
00:08:04	dansmith	mriedem: you mean after we made the change it's 404 now instead of 200 right?
00:12:23	mriedem	yes
00:12:41	mriedem	we could get it back to a 200, but it seems better to just fix the weird behavior we used to allow
00:13:08	mriedem	this would be good to remember for when someone shows up in a year asking why their weird script no longer disables nova-scheduler services...
00:13:15	dansmith	agreed
00:13:19	dansmith	not sure 404 is the right thing though
00:13:47	dansmith	since we expose the service, they did a PUT on it.. probably needs to be 499: Dumbass or something
00:14:22	mriedem	or a 400
00:14:23	mriedem	but yeah
00:14:27	mriedem	anyway have to run for a bit
01:22:11	Kevin_Zheng	mriedem: assigned
01:46:54	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
03:18:50	openstackgerrit	falseuser proposed openstack/nova master: Delete the instance files after revert a resize task https://review.openstack.org/469814
03:55:18	melwitt	git status
03:55:27	melwitt	dangit.
03:58:01	openstackgerrit	melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246
03:58:01	openstackgerrit	melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247
03:58:02	openstackgerrit	melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
03:58:02	openstackgerrit	melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245
03:58:03	openstackgerrit	melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
03:58:03	openstackgerrit	melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
03:58:04	openstackgerrit	melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241
03:58:06	openstackgerrit	melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
03:58:11	openstackgerrit	melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859
03:58:14	openstackgerrit	melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860
03:58:24	openstackgerrit	melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
03:58:25	openstackgerrit	melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861
03:58:36	openstackgerrit	melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
03:58:39	openstackgerrit	melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862
04:04:38	openstackgerrit	falseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.openstack.org/464991
04:14:35	openstackgerrit	falseuser proposed openstack/nova master: Add img_linked_clone support in libvirt https://review.openstack.org/473362
04:25:46	openstackgerrit	melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634
04:25:47	openstackgerrit	melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635
04:46:42	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/467948
04:49:13	openstackgerrit	falseuser proposed openstack/nova master: [BugFix] Include the video ram when comparing the difference of two flavors https://review.openstack.org/460388
04:56:40	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/462017
05:03:50	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
05:10:03	openstackgerrit	falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466
05:41:37	melwitt	alex_xu: hey, just saw your review comment about the status code 406. I thought because of the trace you showed earlier, it would be 400? http://logs.openstack.org/34/469634/12/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/56d8abe/logs/screen-n-cpu.txt.gz#_Jun_12_22_00_17_707953
05:41:39	openstackgerrit	Naichuan Sun proposed openstack/nova master: XenAPI: device tagging https://review.openstack.org/333781
05:42:48	alex_xu	melwitt: that 400 is for user_id and project_id doesn't know by the old API. If we request a microversion, the api server doesn't know, the 406 will returned
05:43:00	melwitt	alex_xu: ah, gotcha. thanks
05:43:07	alex_xu	melwitt: np
05:44:59	alex_xu	melwitt: an example https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L718
05:45:16	melwitt	cool, thank you
05:45:29	alex_xu	yw
06:01:03	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466637
06:13:36	openstackgerrit	melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635
06:13:36	openstackgerrit	melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634
06:57:01	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
07:01:55	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
07:02:12	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add host/binary request query filter parameters for GET /os-services API https://review.openstack.org/473708
07:25:49	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add host/binary request query filter parameters for GET /os-services API https://review.openstack.org/473708
07:52:39	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
07:57:42	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
08:03:35	openstackgerrit	falseuser proposed openstack/nova master: Ensure the root device will be replaced in rebuild operation https://review.openstack.org/467588
08:19:39	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add missing query filter params for GET /os-services API https://review.openstack.org/473708
08:27:26	f10d4	hi
08:27:28	f10d4	gibi: szia
08:27:39	bauzas	p!spy
08:48:11	gibi	f10d4: szia
09:15:37	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
10:31:00	openstackgerrit	Pooja Jadhav proposed openstack/nova master: Nova flavor-list swap should be 0 when no value is specified https://review.openstack.org/471715
10:37:00	openstackgerrit	falseuser proposed openstack/nova master: Ensure the root device will be replaced in rebuild operation https://review.openstack.org/467588
10:47:41	johnthetubaguy	bauzas: did you ever take a look at this cold migrate patches? https://review.openstack.org/#/c/408955
10:47:46	johnthetubaguy	bauzas: super related to the live-migrate work you did
11:01:09	mdbooth	Very quickly, before I knock up a quick script to create some default flavors, network, and cirros glance image, does this already exist in a re-usable form?
11:21:38	kashyap	mdbooth: In a DevStack env?
11:21:53	mdbooth	kashyap: In an openstack cloud
11:22:13	mdbooth	Might be devstack, but isn't in my particular case

Earlier   Later