Posted	Nick	Remark
#openstack-nova - 2017-04-04
22:39:27	dansmith	as in xen it does actually match
22:39:32	dansmith	or can, or did, or whatever
22:39:44	dansmith	and libvirt with xen may try to make it match, I have no idea
22:40:07	mriedem	the least amount of change will be for me to just abandon the spec :)
22:40:12	dansmith	hah
22:40:51	mriedem	if the only thing we care about is this dumb image bdm override use case, wouldn't it be incrementally better to at least remove device from the attach volume api?
22:40:59	mriedem	or is that not even worth it?
22:41:37	dansmith	it didn't seem clear to me that this wasn't also a problem for ec2 in that api,
22:41:42	dansmith	but I have a hard time understanding his words
22:42:01	dansmith	but yeah, if he doesn't care about that, it'd be worth doing I think
22:42:22	mriedem	yeah i have a hard time following what the ec2api is doing or not
22:42:31	mriedem	i do'nt know if it's merging the bdms before we hit the compute api or what
22:42:41	dansmith	yeah I dunno
22:42:42	mriedem	and i'll be god damned if i'm going to go code spelunking
22:51:12	mriedem	artom: found the answer to my question, we don't include bdm tags in snapshots
22:51:32	mriedem	https://github.com/openstack/nova/blob/master/nova/block_device.py#L290
22:51:46	mriedem	we'd have to add it to copy_from_template
22:52:39	mriedem	would one expect a new server created from the image snapshot with the volume snapshot bdm to have the original bdm tag? i don't know. does anyone care? probably not.
23:06:41	openstackgerrit	Sujitha proposed openstack/nova master: Add sync_instances action in os-cells API https://review.openstack.org/453366
23:28:03	artom	mriedem, always happy to have things work out in my absence;)
23:37:25	openstackgerrit	Sujitha proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779
#openstack-nova - 2017-04-05
00:11:05	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add empty flavor object info in server api-ref https://review.openstack.org/452078
00:11:46	gmann	oomichi: fixed ^^
00:13:12	EmilienM	mriedem: ok so Sem-Ver: api-break is good IIUC
00:15:07	oomichi	gmann: thanks, +2
00:15:20	gmann	oomichi: thanks
00:20:32	mriedem	EmilienM: yeah i'm fine with that
00:20:47	mriedem	if my understanding was correct (my comment in the patch)
00:21:32	EmilienM	mriedem: cool, thanks!
00:21:43	EmilienM	I need to figure if I can send the patch to all projects
00:21:47	EmilienM	I already did nova & ironic so far
00:41:55	Sihan	Hi, I want to continue working on the https://review.openstack.org/#/c/348317/, could someone help to restore the patch?
01:06:00	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731
01:07:02	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731
01:29:11	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Re-propose compute-node-notification for Pike https://review.openstack.org/345714
01:33:57	openstackgerrit	Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731
02:15:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: api-ref: make it clear that os-cells is for cells v1 https://review.openstack.org/453433
02:43:39	openstack	Launchpad bug 1504641 in OpenStack Compute (nova) "Listing volumes respects osapi_max_limit but does not provide a link to the next element" [Wishlist,New]
02:43:39	gmann	mriedem: do you know why i cannot mark this as "Won't Fix" ? - https://bugs.launchpad.net/tempest/+bug/1504641
02:44:10	openstackgerrit	Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230
02:52:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add empty flavor object info in server api-ref https://review.openstack.org/452078
03:02:57	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657
03:07:35	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
04:17:03	openstackgerrit	Zhang Hua proposed openstack/nova master: Try our best to bypass the dirty BDM to make nova more robust https://review.openstack.org/453451
04:42:36	openstackgerrit	LIU Yulong proposed openstack/nova-specs master: Reset the instance keypair while rebuilding (spec) https://review.openstack.org/375221
04:43:37	MarkMielke	Any Block Device Mapping experts online?
04:48:43	MarkMielke	I'm having a bit of trouble understanding how to get the root_gb for bdm with destination_type = 'local', boot_index = 0. It doesn't seem to be recorded any more. volume_size is None.
04:49:22	MarkMielke	I thought no worries - the flavor should work, right? Well, we have instances with local root, even though the flavor specifies root_gb = 0. Probably, it was changed at some point.
05:06:55	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834
06:17:19	openstackgerrit	Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122
06:24:35	openstackgerrit	Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230
06:28:12	openstackgerrit	Shunli Zhou proposed openstack/nova master: Remove mox from cells/test_cells_rpc_driver.py https://review.openstack.org/447412
06:39:29	MarginHu	hi, nova can't get right hypervisor info which hypervisor is a ironic node.
06:42:03	MarginHu	how to clean the cache info ?
06:49:04	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
07:06:21	openstackgerrit	Huan Xie proposed openstack/nova master: WIP: use os-xenapi V2 https://review.openstack.org/453493
07:13:29	openstackgerrit	Béla Vancsics proposed openstack/nova master: Using max api version in notification sample test https://review.openstack.org/443686
07:13:30	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
07:32:11	rabel	hi together!
07:32:38	rabel	i'm still trying to understand default quotas and quota groups. someone in here who knows a little bit about that?
07:44:39	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834
07:49:34	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198
07:58:33	openstackgerrit	Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646
07:59:37	openstackgerrit	zhangyujun proposed openstack/nova master: Query parameter validation for hosts API https://review.openstack.org/418880
08:15:36	openstackgerrit	Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125
08:15:36	openstackgerrit	Lei Zhang proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
08:20:42	openstackgerrit	Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125
08:30:22	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198
08:47:13	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198
08:47:14	openstackgerrit	Sivasathurappan Radhakrishnan proposed openstack/nova master: API changes for live migration of rescued instance https://review.openstack.org/328280
08:56:44	openstackgerrit	Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344
09:10:26	openstackgerrit	Silvan Kaiser proposed openstack/nova master: Removes potentially bad exit value from accepted list in Quobyte volume driver https://review.openstack.org/453537
09:21:55	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
09:34:12	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
09:35:51	openstackgerrit	Bob Ball proposed openstack/nova master: Use physical utilisation for cached images https://review.openstack.org/353862
09:36:19	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
09:36:32	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
09:37:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
09:55:38	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992
09:58:01	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834
10:02:45	markus_z	oomichi: sdague: Please have a look at https://review.openstack.org/#/c/453220/
10:14:06	openstackgerrit	John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872
10:20:45	openstackgerrit	jichenjc proposed openstack/nova master: Add allocation doc for placement https://review.openstack.org/450550
10:23:00	openstackgerrit	jichenjc proposed openstack/nova master: Add allocation doc for placement https://review.openstack.org/450550
10:24:12	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
10:24:24	openstackgerrit	jichenjc proposed openstack/nova master: [placement] Add allocation doc for placement https://review.openstack.org/450550
10:31:20	openstackgerrit	jichenjc proposed openstack/nova master: add usage doc for resource provider https://review.openstack.org/450105
10:39:25	openstackgerrit	jichenjc proposed openstack/nova master: fixed typos and reword stable api doc https://review.openstack.org/442310
10:49:55	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Remove '/servers' extensions info from controllers https://review.openstack.org/453571
10:51:30	openstackgerrit	jichenjc proposed openstack/nova master: Add resource classes description in api-ref https://review.openstack.org/451661
11:19:00	johnthetubaguy	bauzas: cdent: are you folks OK with this one by any chance? https://review.openstack.org/#/c/453030
11:19:44	cdent	johnthetubaguy: i'm not up to date with the changes since my comments since ps1, but in principle, yes
11:23:11	johnthetubaguy	cdent: OK, cool, I just didn't feel I should +W without more placement experts feeling happy with it
11:23:32	cdent	I'm doing a re-review now just to be sure
11:28:13	cdent	johnthetubaguy: had some minor suggestions, but I think they could be followups
11:37:44	openstackgerrit	Emilien Macchi proposed openstack/nova master: https://review.openstack.org/453338
11:39:37	openstackgerrit	Emilien Macchi proposed openstack/nova master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453338
11:42:22	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
11:56:12	yasemin_	hi, i installed stable/ocata , multinode devstack and when i add compute node, it gives an error like Failed to perform requested operation on instance "demo", the instance has an error status: Please try again later [Error: Host 'compute-1' is not mapped to any cell].
11:56:19	yasemin_	can you help me ?

Earlier   Later