Posted	Nick	Remark
#openstack-nova - 2017-03-12
01:12:59	dansmith	ah
01:13:06	dansmith	yeah that was a major problem
01:13:32	mriedem	i also finally figured out how to turn on the backlit keyboard, heh - even though the old one had it, but i never used it
01:13:39	mriedem	finally i can work in the dark
01:13:43	mriedem	like a real troll
01:14:27	dansmith	speaking of staying in the dark, we're just about to start "gremlins" and eat
01:14:48	mriedem	ok. just eat before midnight.
01:14:50	mriedem	ttyl
04:54:31	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
04:54:32	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
04:54:32	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
05:30:05	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add resource provider traits API https://review.openstack.org/376202
06:19:22	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
06:19:23	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add resource provider traits API https://review.openstack.org/376202
14:22:44	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.rebuild notification https://review.openstack.org/384922
14:27:31	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.rebuild notification https://review.openstack.org/384922
14:34:01	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.rebuild.error notification https://review.openstack.org/396621
14:41:16	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.rebuild.error notification https://review.openstack.org/396621
14:58:45	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
14:58:52	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
15:01:54	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
15:06:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Transform instance.rebuild notification https://review.openstack.org/384922
15:45:42	openstackgerrit	Matt Riedemann proposed openstack/nova master: Transform instance.rebuild notification https://review.openstack.org/384922
15:45:43	openstackgerrit	Matt Riedemann proposed openstack/nova master: Transform instance.rebuild.error notification https://review.openstack.org/396621
15:56:40	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
15:57:04	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
15:58:01	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
15:59:46	openstackgerrit	Alex Szarka proposed openstack/nova master: Reduce code complexity in vmops._spawn method https://review.openstack.org/438502
16:18:41	openstackgerrit	Alex Szarka proposed openstack/nova master: Reduce code complexity in vmops._spawn method https://review.openstack.org/438502
17:34:43	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
17:41:17	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357
19:04:09	openstackgerrit	Alex Szarka proposed openstack/nova master: Reduce code complexity in vmops._spawn method https://review.openstack.org/438502
#openstack-nova - 2017-03-13
00:31:43	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
00:31:43	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
00:31:44	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
00:31:44	openstackgerrit	Alex Xu proposed openstack/nova master: placement: Add resource provider traits API https://review.openstack.org/376202
01:45:08	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for support set vgamem for video https://review.openstack.org/438787
01:56:19	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for support set vgamem for video https://review.openstack.org/438787
02:38:31	openstackgerrit	Maho Koshiya proposed openstack/nova master: Add interfaces functional negative tests https://review.openstack.org/442892
04:04:42	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512
04:04:54	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
04:05:08	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
04:05:12	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
06:47:11	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731
06:56:42	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731
08:27:48	Qiming	alex_xu, there?
08:28:47	robcresswell	Quick question; someone's added a patch to Horizon that hides the Soft Reboot button if the Instance state is anything other than Active; is this correct? Had a look at the API docs but they only seem to show how to form the request, not any of the conditions around its usage.
08:31:01	openstackgerrit	Sergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects https://review.openstack.org/394480
08:31:01	openstackgerrit	Sergey Nikitin proposed openstack/nova master: Added nova objects for intance diagnostics https://review.openstack.org/434745
08:31:02	openstackgerrit	Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540
08:31:03	openstackgerrit	Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613
09:00:08	bauzas	good morning Nova
09:02:31	alex_xu	Qiming: hi, i'm here
09:08:19	Qiming	alex_xu, I was checking admin password
09:11:51	Qiming	alex_xu, it turns out that I will need to use "/servers/{server_id}/os-server-password" for password GET, and DELETE, use POST /servers/{server_id}/action for password update, and the password can be created only via POST /servers
09:24:57	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Typo fix in releasenotes: deprecate network options https://review.openstack.org/444778
09:25:41	gmann	sfinucan: alex_xu oomichi simple one - https://review.openstack.org/#/c/444778/
09:26:44	sfinucan	gmann: Done
09:27:03	gmann	sfinucan: :) thanks
09:27:55	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Use detach_volume during default rebuild https://review.openstack.org/443932
09:27:55	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520
09:27:56	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Only destroy BDMs after successful detach call https://review.openstack.org/440693
09:27:56	openstackgerrit	Lee Yarwood proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665
09:27:57	openstackgerrit	Lee Yarwood proposed openstack/nova master: db: Add attachment_id to block_device_mapping https://review.openstack.org/437597
09:28:13	alex_xu	gmann: done
09:28:26	alex_xu	Qiming: i'm thinking whether all of those password are same thing
09:28:59	Qiming	alex_xu, they are, IMO
09:29:02	gmann	alex_xu: thanks
09:30:38	gmann	alex_xu: Qiming admin one is different(updated using changePassword) and os-server-password is metadata one
09:31:51	gmann	because delete call just remvoe from metadata which is encrypted one, it does not change in server
09:32:31	Qiming	gmann, I'm no nova expert, should I care if the password is stored in metadata or somewhere else?
09:33:09	Qiming	gmann, the doc tells me I can specify password for admin, I can change it, I can get it, and I can wipe it out
09:33:28	alex_xu	I remember the admin one is for xen driver
09:34:48	Qiming	gmann, the strange thing is that they are located in different locations, probably fine for command line tools, but not so good for SDK modeling
09:37:12	lyarwood	mdbooth: morning, https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/cinder-new-attach-apis should be ready for reviews once you have time today, the focus is really on the db and object changes first if you wouldn't mind focusing on them.
09:38:16	openstackgerrit	Thomas Bechtold proposed openstack/nova master: Log exception when hypervisor version check fails https://review.openstack.org/434927
09:45:47	dakhmetov	Greetings, guys!
09:45:57	dakhmetov	Could you please review this patch: https://review.openstack.org/#/c/399679/
09:50:47	dakhmetov	I know a couple of people who are really affected by the inconsistency in instance's libvirt.xml. It will be great if you help me reviewing it.
09:54:06	johnthetubaguy	dakhmetov: so I love half of your change, and worry about the other half
09:54:34	johnthetubaguy	dakhmetov: using instance.user_id and instance.project_id totally makes sense, we should have done that for the beginning
09:55:06	johnthetubaguy	username and projectname... thats feels wrong, what about we don't add that any more?
09:55:07	dakhmetov	2 johnthetubaguy please share your concern
09:55:16	johnthetubaguy	I will add a note on the patch
09:55:23	gmann	Qiming: admin pass can be injected while creating server and change password using action API is for xen and libvirt as alex_xu mentioned
09:55:48	johnthetubaguy	dakhmetov: a related comment came up on friday at a HPC meetup I went to, so agreed it needs fixing
09:55:55	openstackgerrit	Michael Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support https://review.openstack.org/441183
09:56:39	gmann	@Qiming not sure about exact SDK requirement on that and if any issue on nova side?
09:58:48	johnthetubaguy	dakhmetov: I am worried about how usernames and project names change over time in keystone, but we keep those names just matching what they were went the instance booted
09:58:58	johnthetubaguy	dakhmetov: maybe thats OK for your use case I guess?
10:00:22	dakhmetov	2 johnthetubaguy: yes, I also thought about it. The only option to avoid this issue is open Keystone session there each time
10:01:48	johnthetubaguy	dakhmetov: my idea was to only have the ids in there, from the instance object, and leave the others blank for now?
10:02:31	johnthetubaguy	that would ensure they are "correct" in some sense, but I feel bad breaking the folks that want the names there
10:04:59	dakhmetov	johnthetubaguy: sure, there are a lot of folks who needs names there, as it described in nova xmlns 1.0
10:07:53	dakhmetov	so, what we have in current patch seems to be best solution, despite the fact that the stored user and project names can change
10:08:01	mdbooth	dakhmetov: I've reviewed a change related to this recently. Do you have a link?
10:08:49	dakhmetov	mdbooth: exactly! https://review.openstack.org/#/c/399679
10:09:09	mdbooth	Ah, ok. You fishing for a second +2?
10:09:33	dakhmetov	mdbooth: yes, a kind of :)
10:11:12	mdbooth	johnthetubaguy: My take on this is that we break things by rebuilding the libvirt xml. We shouldn't be doing that, and this is a necessary workaround.

Earlier   Later