Posted	Nick	Remark
#openstack-nova - 2017-12-05
02:12:16	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Fix an error message in API validation https://review.openstack.org/524088
02:24:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix wrong argument order in functional test https://review.openstack.org/525363
02:36:31	alex_xu	Kevin_Zheng: should we doc that in the api-ref also?
02:37:00	Kevin_Zheng	alex_xu: which one?
02:37:15	alex_xu	Kevin_Zheng: https://review.openstack.org/#/c/515631/5
02:37:43	alex_xu	Kevin_Zheng: since you said, it is user-noticeable API change...so I guess you want to get attention from the API user, not only the deployer?
02:39:13	Kevin_Zheng	I don't know, maybe it is too much details for normal users?>
02:39:48	alex_xu	Kevin_Zheng: yea, it shouldn't be too much detail, just something like after specific release, if overquota, you will get a error instance...
02:40:25	alex_xu	Kevin_Zheng: I guess that won't effect the API user too much
02:40:44	Kevin_Zheng	But we have only have a Master branch doc
02:40:51	alex_xu	Kevin_Zheng: actually more the end user, since they get an error instance, and they don't what happened.
02:41:09	Kevin_Zheng	it might be different for different versions
02:41:36	Kevin_Zheng	releases
02:41:42	alex_xu	Kevin_Zheng: ah, got it, there is config option
02:42:30	alex_xu	Kevin_Zheng: ok, just note in the release note, I think this is more important for the deployer/end user, I don't think that will break the API user app
02:42:46	alex_xu	Kevin_Zheng: done
02:52:44	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/525006
02:52:52	openstackgerrit	Merged openstack/nova master: Downgrade log for keystone verify client fail https://review.openstack.org/513243
03:13:44	openstackgerrit	Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
03:19:50	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/525386
03:33:41	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/525399
05:28:14	openstackgerrit	Merged openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200
06:12:07	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
06:32:42	openstackgerrit	jichenjc proposed openstack/nova stable/pike: Downgrade log for keystone verify client fail https://review.openstack.org/525475
07:00:47	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
08:54:33	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
08:59:45	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip https://review.openstack.org/525505
09:10:19	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
09:27:39	openstackgerrit	Merged openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631
09:28:38	openstackgerrit	Zhenyu Zheng proposed openstack/nova stable/pike: Mention API behavior change when over quota limit https://review.openstack.org/525511
09:32:12	stephenfin	gmann_afk, mriedem: I left a big comment about the 'tempest-dsvm-neutron-src-oslo.versionedobject' job here https://review.openstack.org/#/c/514310/3/zuul.d/zuul-legacy-jobs.yaml
09:33:00	stephenfin	gmann_afk, mriedem: The tl;dr is that that job should be combined with a number of similar jobs and kept in a central location. There's nothing specific to o.v.o (or nova) about it
09:45:18	openstackgerrit	Merged openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306
10:03:37	openstackgerrit	Merged openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811
10:23:12	openstackgerrit	Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602
10:29:20	openstackgerrit	Merged openstack/nova master: Use ksa adapter for keystone conf & requests https://review.openstack.org/507693
10:50:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: Resolve review comments from I41b03a347 https://review.openstack.org/525255
11:05:33	stephenfin	bauzas: Can you take a second look at this? https://review.openstack.org/#/c/466051/
11:09:38	cdent	stephenfin, bauzas, gibi : interesting grenade bug you might want to be aware of: https://bugs.launchpad.net/nova/+bug/1736385
11:09:39	openstack	Launchpad bug 1736385 in OpenStack Compute (nova) "placement is not being properly restarted in grenade (pike to master)" [Undecided,New]
11:11:54	gibi	cdent: good catch
11:12:37	cdent	I think it is a pretty easy fix in devstack, so gonna to try a test patch
11:13:17	gibi	cdent: I hope it did not hide a lot of undiscovered bugs
11:13:26	cdent	yeah, that’s my fear too
11:25:40	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Test devstack fix for process restart https://review.openstack.org/525534
11:30:59	openstackgerrit	Merged openstack/nova master: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/521391
11:31:05	openstackgerrit	Merged openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502
11:31:12	openstackgerrit	Merged openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880
11:43:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add PowerVM to compute_driver options https://review.openstack.org/523978
11:52:10	openstackgerrit	wanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached https://review.openstack.org/525540
11:55:39	openstackgerrit	Merged openstack/nova master: Make _Provider really private https://review.openstack.org/523932
11:55:45	openstackgerrit	Merged openstack/nova master: ProviderTree.get_provider_uuids() https://review.openstack.org/520243
12:36:10	cdent	stephenfin, bauzas : are you guys able to comment whether specless is okay for this blueprint? https://blueprints.launchpad.net/nova/+spec/vmware-boot-uefi there’s already an implementation and Ken’ichi (on the review) seemed to think so.
12:37:43	openstackgerrit	Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603
12:38:34	openstackgerrit	Hengqing Hu proposed openstack/nova master: Remove build_sphinx section from setup.cfg https://review.openstack.org/525587
12:44:28	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model https://review.openstack.org/525055
12:48:53	openstackgerrit	Alexandru Muresan proposed openstack/nova master: Fix driver tests on Windows https://review.openstack.org/518525
12:52:14	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
12:52:15	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
12:52:15	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
12:52:16	openstackgerrit	Jay Pipes proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
12:52:30	jaypipes	alex_xu: ^^ added additional func test that includes resources filter along with tree filter
12:53:11	jaypipes	alex_xu: I prefer just "tree" to "in_tree", but if everyone else wants to change it, I can
12:59:38	cdent	jaypipes, alex_xu : It is somewhat confusing, but I’m not sure what’s right. There’s a part of me that thinks ‘tree’ actually ought to be a boolean: give me the results and anything else which is a relative in the trees of any results
13:00:28	openstackgerrit	Jay Pipes proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
13:00:39	dmsimard	kashyap: ping ?
13:01:11	openstackgerrit	Merged openstack/nova master: Fix wrong argument order in functional test https://review.openstack.org/525363
13:01:23	dmsimard	Was wondering if you had come across my question yesterday: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-12-04.log.html#t2017-12-04T17:26:46
13:04:01	kashyap	dmsimard: Hey
13:04:35	kashyap	dmsimard: I did have my proxy catch it, I just didn't get around to it.
13:07:07	openstackgerrit	Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599
13:08:15	dmsimard	kashyap: thanks, much appreciated.
13:08:24	kashyap	dmsimard: Reading the bug report, VM building can hang w/ libguestfs & TCG
13:08:38	kashyap	dmsimard: Still catching up on the bug report & libguestfs mailing list thread linked in there
13:09:09	dmsimard	Yeah.. I summarized the issue in the eavesdrop link
13:09:56	kashyap	dmsimard: Ah-ha! So, TCG didn't fully take effect
13:14:35	kashyap	dmsimard: Related aside: Do we have any plans (at least it has been talked about since Paris Summit) to test in a portion of Infra with actual Nested KVM?
13:15:59	kashyap	dmsimard: In the past, the argument was: "It's a bit unstable / buggy". But no bugs were filed. Probably it'd be useful to think about enabling actual nVMX (Intel) / nSVM (AMD) for a portion of the Infra Cloud.
13:16:18	dmsimard	Not sure, I'd love to of course but my opinion doesn't really matter :D
13:16:38	dmsimard	It's about nodepool providers supporting it or not
13:16:40	openstackgerrit	Merged openstack/nova master: Fix some incorrect option references for scheduler filters https://review.openstack.org/521645
13:16:47	openstackgerrit	Merged openstack/nova master: Deprecate the IronicHostManager https://review.openstack.org/521648
13:16:53	openstackgerrit	Merged openstack/nova master: Fix invalid minRam error message https://review.openstack.org/523142
13:17:01	openstackgerrit	Merged openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
13:18:15	kashyap	dmsimard: Yeah, I know it's not in your hands; it's with node providers; Rackspace Infra uses Xen, so that's out of scope.
13:19:26	openstackgerrit	Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167
13:22:02	dmsimard	kashyap: need to brb
13:22:25	kashyap	dmsimard: It's fine; I too need to be briefly AFK to finish some paperwork
13:22:32	kashyap	(Not now, in a few)
13:32:28	openstackgerrit	Merged openstack/nova master: Add debug information to metadata requests https://review.openstack.org/511895
13:32:39	openstackgerrit	Merged openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715
13:32:56	openstackgerrit	Merged openstack/nova master: Fix TypeError in nova-manage db archive_deleted_rows https://review.openstack.org/520765
13:38:24	alex_xu	cdent: so "tree=True&resources=...." means that give me the tree which match those resources?
13:38:51	stephenfin	cdent: I think it best to ask mriedem that
13:39:54	alex_xu	jaypipes: thanks
13:39:56	cdent	alex_xu: sort of: for the resource providers that satisfy the resources request, return those resource providers and their trees
13:40:25	jaypipes	cdent: what do you mean by "and their trees"?
13:41:00	cdent	jaypipes: if rp X has a parent of Y and a sibling of Z, all of X Y and Z would return
13:41:10	cdent	(where X satisfies resources)
13:41:35	jaypipes	cdent: the problem with that is due to GET /resource_providers not returning information about inventory or allocations, you would have no idea *which* of the resource providers met the resources query.

Earlier   Later