Posted	Nick	Remark
#openstack-nova - 2017-04-11
22:41:39	mriedem	claudiub: yeah good point - mgagne: how do we know which brick connector to get? do we have to get the connection_info dict from cinder for the given volume?
22:42:10	mgagne	mriedem: iirc, extend_volume takes connection_info in parameter so I assume it knows that to do with that
22:42:38	mgagne	mriedem: I can check, just a bit busy with some openstack upgrade atm =)
22:43:02	mriedem	oh, hmm,
22:43:09	mriedem	i guess we have the bdm.connection_info
22:43:16	mriedem	and can lookup the bdm by instance and volume id
22:43:20	mgagne	yes, nova has it afaik in bdm
22:46:32	claudiub	mriedem: anyways. since there's blueprints talk. i was going to propose a sriov + hyperv spec to nova. i finally have some hardware to work / test on. :D it would be a pretty straight-forward implementation IMO, especially since we already support pci passthrough in hyper-v. do I need a spec for that?
22:47:10	mriedem	claudiub: i thought you had a spec for that at one point?
22:47:18	mriedem	maybe it was just a blueprint and a patch that was abandoned?
22:47:37	mriedem	claudiub: i think i'm fine with a specless feature parity blueprint though
22:47:37	claudiub	i don't think I had, never had the hardware to test that feature before.
22:48:11	claudiub	cool, will submit the blueprint and some POC code by the end of the week.
23:03:12	openstackgerrit	Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272
23:24:46	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (1/4) https://review.openstack.org/455563
23:48:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568
23:48:34	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570
23:48:41	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573
23:48:48	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
#openstack-nova - 2017-04-12
00:02:03	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
00:02:31	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
00:02:47	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
00:06:41	mgagne	so... about that nova-specs gate breakage... "sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" is there any change I can rebase on to get it fixed?
00:22:26	openstackgerrit	STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644
00:27:58	openstackgerrit	STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644
00:39:01	clarkb	mgagne: did that fail earlier today? there was an issue breaking docs builds and may hve affected nova-specs?
00:39:09	clarkb	mgagne: you might just try recheckign as it was a test node issue
00:39:17	mgagne	will recheck right now
01:00:03	mgagne	clarkb: no luck after a recheck https://review.openstack.org/#/c/453272/
01:14:57	mriedem	clarkb: was failing all afternoon
01:15:49	mriedem	maybe a new sphinx release and we're legitimately broken
01:28:28	mriedem	hmm, sphinxcontrib-fulltoc 1.2.0 released on 4/10
01:28:59	mriedem	we should be using 1.1 though https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L479
01:30:32	mriedem	maybe pbr 2.1.0?
01:33:37	mriedem	bingo https://github.com/openstack-dev/pbr/commit/2d7c0045f402b31d6223437ba25423eeb32278e1
01:33:39	mriedem	clarkb: ^
01:33:45	mriedem	it's pbr 2.1.0
01:36:03	mriedem	https://bugs.launchpad.net/pbr/+bug/1681983
01:36:03	openstack	Launchpad bug 1681983 in PBR "2.1.0: sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" [Undecided,New]
01:39:42	mriedem	blacklist patch https://review.openstack.org/455865
01:41:07	clarkb	you cant blacklist pbr
01:41:23	clarkb	its installed bfore anyrhing that could blacklist is executed
01:42:05	mriedem	shirt
01:42:07	mriedem	heh
01:42:09	clarkb	it may work in this case because its a post install step though but generally ya
01:48:05	mordred	clarkb, mriedem: I +A'd the u-c blacklist, but also what clarkb said is correct so I don't expect it'll do much - while the pbr issue gets worked, commenting out the man_pages line in conf.py fixes theissue
02:38:21	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Workaround pbr 2.1.0 man_pages bug https://review.openstack.org/455885
02:54:32	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Deprecate multinic, os-virtual-interfaces, and floating IP action APIs https://review.openstack.org/384261
02:54:39	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272
02:55:00	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352
02:58:27	Kevin_Zheng	ping mriedm
02:58:53	Kevin_Zheng	ping mriedem
02:59:11	mriedem	Kevin_Zheng: hi
03:00:04	Kevin_Zheng	mriedem: Hi, about https://review.openstack.org/#/c/455183/2 , I don't quite get your suggestion
03:02:32	mriedem	Kevin_Zheng: i left a reply, see if that makes sense
03:02:42	Kevin_Zheng	mriedem thanks
03:02:49	mriedem	if the for loop does not break, then we raise exception.ConsoleNotAvailable()
03:03:00	mriedem	if the for loop does break, pty is set
03:10:47	Kevin_Zheng	mriedem: thanks, I just searched it, that makes sense :)
03:11:24	mriedem	cool
03:13:10	mriedem	alex_xu: gmann: have you ever noticed that the host power actions on the os-hosts API are all GETs?
03:13:14	mriedem	GET /os-hosts/{host_name}/reboot
03:13:20	mriedem	that's really odd
03:15:44	gmann	mriedem: yea all start etc are GET
03:26:05	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183
03:31:54	mriedem	gmann: i also see there are Tempest negative tests that hit the os-hosts enable/disable/reboot/shutdown/start APIs, those should probably all be moved out of tempest and into nova unit tests, if we don't already have that coverage, but I bet we do
03:32:06	mriedem	i'll mention that in the spec to deprecate os-hosts
03:37:29	gmann	mriedem: yes we can do and i think defcore might not be using those as admin APIs and we have positive cases covered in api sample tests and if any negative cases missing in unit tests we can add easily.
03:39:11	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Deprecate the os-hosts API (spec) https://review.openstack.org/455896
03:39:20	mriedem	gmann: alex_xu: ^
04:17:53	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/455960
04:19:44	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049
04:21:46	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/456002
04:29:36	EmilienM	dansmith: when you have time, any chance to help with this trace? http://logs.openstack.org/15/359215/85/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/0040e01/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-04-12_04_03_47_785
04:29:53	EmilienM	I'm probably wrong but it looks like related to https://review.openstack.org/#/c/442162/
04:33:21	EmilienM	https://bugs.launchpad.net/tripleo/+bug/1682001
04:33:21	openstack	Launchpad bug 1682001 in tripleo "[promotion] nova fails to start a server - host is not mapped to any cell" [Critical,Triaged]
05:13:13	openstackgerrit	赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
05:23:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action https://review.openstack.org/456053
05:28:53	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action https://review.openstack.org/456053
06:49:01	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183
07:06:43	openstackgerrit	YuYang proposed openstack/nova master: api-ref: Nova Update Compute services Link https://review.openstack.org/456085
07:14:49	openstackgerrit	赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
07:36:01	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801
07:41:09	openstackgerrit	Gergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server https://review.openstack.org/451497
08:33:05	openstackgerrit	Gergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server https://review.openstack.org/451497
08:38:43	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
08:38:52	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
08:58:35	Dinesh_Bhor	alex_xu: ping
09:03:02	Dinesh_Bhor	alaski: Hi, Does this mean that If there are some issues then we are not going to fix them? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/scheduler_hints.py#L67-L70
09:03:34	Dinesh_Bhor	alaski: I can see your ML thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067996.html
09:09:46	johnthetubaguy	Dinesh_Bhor: can I help with context?
09:10:06	gibi	mgagne: I left some comment in the spec https://review.openstack.org/#/c/453272/
09:10:24	johnthetubaguy	Dinesh_Bhor: we allow custom scheduler filters and weighers, they can have their own hints, if we blocked those in the API, it would break things. Does that help?
09:11:12	openstackgerrit	赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
09:11:32	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
09:11:33	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
09:11:33	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
09:11:34	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708
09:13:43	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/390654

Earlier   Later