Posted	Nick	Remark
#openstack-nova - 2017-01-18
20:53:12	mriedem	make sure you get 1 and not 2
20:53:36	edleafe	mriedem: I wanted a case where only one node was returned. The current are either all or none
20:53:40	mriedem	ok
20:53:40	mriedem	yeah
20:54:01	mriedem	i'll add that quick
20:55:22	edleafe	mriedem: I was going to add it myself, but I didn't want to mess up bauzas's stuff
20:55:44	mriedem	i've got it
20:59:40	openstackgerrit	Matt Riedemann proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134
20:59:41	openstackgerrit	Matt Riedemann proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
21:00:12	mriedem	+2 on that mofo now
21:00:50	edleafe	mriedem: kewl
21:01:09	mriedem	i'm trying to work on my list of 'these bp patches need another +2' by the end of hte day
21:04:46	oomichi	alex_xu: yeah, the first one is good fo me, thanks Kevin_Zheng and alex_xu
21:05:35	oomichi	alex_xu: https://review.openstack.org/#/c/415330/ has johnthetubaguy's -1
21:06:25	openstackgerrit	Ruby Loo proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756
21:13:32	mriedem	mikal: so you love the metadata api,
21:14:18	mriedem	mikal: would changing the content of the network_data.json dict within the metadata api response constitute a bump in the API version, even though we don't filter the results to that level of granularity? i.e. https://review.openstack.org/#/c/388756/18/nova/virt/ironic/driver.py
21:14:27	mriedem	well,
21:14:53	mriedem	this would actually only be config drive since this is modified when the instance is created for the config drive and wouldn't be available via the metadata API, so nvm
21:21:01	openstackgerrit	Francesco Santoro proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061
21:49:55	openstackgerrit	Pushkar Umaranikar proposed openstack/nova: [Do not merge] Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
21:58:16	openstackgerrit	Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401
22:00:45	openstackgerrit	Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509
22:32:28	mriedem	claudiub: should https://review.openstack.org/#/c/397722/ be abandoned?
22:34:02	EmilienM	mriedem: quick update on placement thing I reported this morning. We found a problem in HAproxy config in TripleO, we're working on it now. It's probably related. It's green for Puppet OpenStack CI.
22:34:10	mriedem	ok
22:34:16	EmilienM	bauzas ^ for your morning
23:06:20	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
23:07:23	openstack	Launchpad bug 1655662 in OpenStack Compute (nova) "libvirt xen hvm guests can't hot plug (attach detach) default block devices" [Undecided,New]
23:07:23	mriedem	anthonyper: you might be able to triage this https://bugs.launchpad.net/nova/+bug/1655662
23:16:45	mriedem	melwitt: https://review.openstack.org/#/c/420451/ and down need your watchful eye
23:17:59	melwitt	ack
23:26:49	melwitt	mriedem: I had been thinking, what about deleting erroneously created cells that have e.g. broken urls in them? I guess the thinking is, those won't have hosts or instances mapped to them since their urls never worked?
23:27:36	mriedem	deleting them how?
23:27:41	mriedem	like, automatically?
23:27:59	melwitt	I'm just imagining someone having a problem because they made a mistake creating some cell with a wrong url in it and they can't get rid of it from their list_cells view
23:28:01	melwitt	no
23:28:32	melwitt	oh, wait, I guess those would be empty
23:28:37	mriedem	yeah :)
23:28:41	mriedem	so, just use delete_cell ?
23:29:39	melwitt	yeah. I was thinking if there's any problem where someone could get wedged. I think only in a case where they had a valid cell in the past and now want to get rid of it. but that's probably for later anyway, since it's related to idea of re-mapping cells too
23:34:34	mriedem	this is also pretty easy https://review.openstack.org/#/c/420132/
23:36:27	melwitt	k. I've been chasing a bug in my quota counting patch. that's my excuse for not reviewing these earlier
23:39:12	mriedem	tdurakov: if you have highlights from the live migration meeting this week please post them in https://wiki.openstack.org/wiki/Meetings/Nova#Weekly_Nova_team_meeting
23:48:38	openstackgerrit	Matt Riedemann proposed openstack/nova: hyperv: make sure to plug OVS VIFs after resize/migrate https://review.openstack.org/422314
#openstack-nova - 2017-01-19
00:24:11	openstackgerrit	Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
00:42:50	openstackgerrit	Eric Brown proposed openstack/nova: Remove references to Python 3.4 https://review.openstack.org/417997
01:00:39	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
01:16:37	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
01:52:58	openstackgerrit	Jeremy Liu proposed openstack/nova: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/415080
02:16:10	openstackgerrit	Cao Xuan Hoang proposed openstack/os-vif: Remove support for py33 https://review.openstack.org/422337
02:27:52	alex_xu	oomichi: thanks
02:28:02	openstackgerrit	Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
02:33:21	alex_xu	oomichi: johnthetubaguy have concern on https://review.openstack.org/#/c/415330/, the patch needs to update
02:34:38	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200
02:34:38	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Enable attach traits to ResourceProvider https://review.openstack.org/376201
02:34:39	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API https://review.openstack.org/376202
02:34:40	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
02:36:51	openstackgerrit	Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
02:58:18	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova: VMWare: Move constant power state strings to the constant.py https://review.openstack.org/368383
03:19:29	alex_xu	johnthetubaguy: I just replied https://review.openstack.org/#/c/415330/16, I feel we still have problem.
03:23:45	alex_xu	johnthetubaguy: also replied your question about why those sort key are bad https://review.openstack.org/#/c/415142/23
03:39:17	openstackgerrit	melanie witt proposed openstack/nova: WIP Count resources to check quota for cells https://review.openstack.org/416521
03:50:07	openstackgerrit	Tovin Seven proposed openstack/python-novaclient: Add profiling support to novaclient https://review.openstack.org/254699
04:03:04	tovin07_	hi Vek, I saw that this patch (https://review.openstack.org/#/c/254699/) is not auto-merged while depends-on patch is merged yesterday. I re-check it and wait for hours, nothing happens. I rebase it and lost Workflow +1, could you review that.
04:03:24	tovin07_	Thanks!
05:11:51	openstackgerrit	Sergey Nikitin proposed openstack/nova: DO NOT MERGE https://review.openstack.org/422221
05:17:38	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494
05:55:56	openstackgerrit	Hironori Shiina proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745
06:10:14	gcb	nova cores ,please look at https://review.openstack.org/#/c/421482/. which blocks release new version of oslo.serialization, thanks
06:10:19	openstackgerrit	Hironori Shiina proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745
06:19:12	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568
06:34:35	openstackgerrit	Hironori Shiina proposed openstack/nova: Ironic: Add soft power off support to Ironic driver. https://review.openstack.org/407977
06:49:28	openstackgerrit	Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480
06:49:29	openstackgerrit	Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613
06:49:29	openstackgerrit	Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540
06:55:07	snikitin	alex_xu: Hi, Alex! After your +2 I updated diagnostics object patch and the only change was a changing of copyrights because of Matt's comment. Can you look again, please? https://review.openstack.org/#/c/394480/
06:56:32	openstackgerrit	melanie witt proposed openstack/nova: WIP Count resources to check quota for cells https://review.openstack.org/416521
07:05:37	snikitin	BobBall: Hi, Bob! I fixed you comments about Xen. But I got some problems with generating of device name. Your solution is good but it doesn't work in some cases. I left a comment. All other comments are fixed. Could you look, please? https://review.openstack.org/#/c/399613/
07:20:50	openstackgerrit	Jeremy Liu proposed openstack/nova: Update requirements https://review.openstack.org/422426
07:27:06	openstackgerrit	Derek Higgins proposed openstack/nova: libvirt: fix nova can't delete the instance with nvram https://review.openstack.org/357190
07:27:07	openstackgerrit	Derek Higgins proposed openstack/nova: Remove mox in libvirt destory tests https://review.openstack.org/417227
07:48:48	snikitin	mriedem: Hi, Matt! I fixed copyrights. Can you please look when have time? https://review.openstack.org/#/c/394480/
08:38:13	openstackgerrit	melanie witt proposed openstack/nova: WIP Count resources to check quota for cells https://review.openstack.org/416521
08:38:37	openstackgerrit	Mihail proposed openstack/nova: Bugfix 1646896 : fix code style based on flake8 report https://review.openstack.org/422467
08:59:06	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist https://review.openstack.org/421760
09:24:09	bauzas	p!spy
09:35:05	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist https://review.openstack.org/421760
09:51:06	openstackgerrit	Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512
10:04:47	openstackgerrit	Wang Qing wu proposed openstack/nova: WIP(3): Introduce base PowerVM driver (console) https://review.openstack.org/409402
10:10:11	openstackgerrit	Timofey Durakov proposed openstack/nova: DO NOT MERGE https://review.openstack.org/422221
10:13:22	openstackgerrit	Wang Qing wu proposed openstack/nova: WIP(4): Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404
10:16:53	openstackgerrit	Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512
10:18:06	openstackgerrit	Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441
10:23:51	Kevin_Zheng	johnthetubaguy: so we are good with https://review.openstack.org/#/c/415142/ ?
11:13:40	BobBall	snikitin: Awesome - thanks for your changes. Just to check, did you have some testing with a real XS host?
11:15:52	BobBall	snikitin: I suspect the CI will run the code, but not verify that the answers are right (e.g. does the add_memory_usage work or always just return 0 used)
11:18:01	snikitin	http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/06/413606/11/check/dsvm-tempest-neutron-network/54a6132/
11:18:01	snikitin	BobBall: I'm deploying env at this moment. But I already got bad news from xen CI job :( the message say "ValueError: cannot convert float NaN to integer". That is because of 'utulisation' field.
11:18:44	snikitin	BobBall: yep I will check all fields when my env will be deployed

Earlier   Later