Posted	Nick	Remark
#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] Enable attach traits to ResourceProvider https://review.openstack.org/376201
02:34:38	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200
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: Standardization of VM diagnostics info API. https://review.openstack.org/355540
06:49:29	openstackgerrit	Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613
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	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:01	snikitin	http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/06/413606/11/check/dsvm-tempest-neutron-network/54a6132/
11:18:44	snikitin	BobBall: yep I will check all fields when my env will be deployed
11:19:06	BobBall	snikitin: ahhh - this is the updated tempest test - awesome.
11:19:11	BobBall	snikitin: I hadn't been tracking that.
11:20:39	snikitin	BobBall: also I have a dirty trick in my mind :) I can create a fake tempest test with self.assertEqual(1, diagnostics). When it will fail on Cen Ci job we will get whole diagnostics responce
11:20:52	BobBall	snikitin: I like that trick :)
11:37:12	dims	jaypipes : bauzas : this one is good to go. tiny fixes. helps fix up several tempest scenarios - https://review.openstack.org/#/c/421911/
11:45:05	openstackgerrit	Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614
12:03:51	blinky_ghost	hi all can anybody tell me what this error means: "error: Index 'allocations_resource_provider_class_used_idx' already exists!" when I migrate nova db schema from liberty to mitaka. Thanks
12:07:22	openstackgerrit	Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441
12:30:30	openstackgerrit	Gábor Antal proposed openstack/nova: Transform instance.live_migration_rollback notification https://review.openstack.org/402124
12:40:11	bhagyashris	Hi all, I am facing issue after hitting any nova commands eg. nova list and got error at n-api as DBNonExistentDatabase: (pymysql.err.InternalError) (1049, u"Unknown database 'nova_cell0'"). Can anyone have any idea?
12:40:36	openstackgerrit	Béla Vancsics proposed openstack/nova: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
12:44:08	sambetts	johnthetubaguy, jroll: ping can we go through your comment here please, we're struggling to understand why that would cause the races https://review.openstack.org/#/c/364413/24/nova/virt/ironic/driver.py@1242
12:44:40	jroll	(and/or how said race would cause a port to be stolen)
12:49:40	_gryf	EmilienM, around?
12:51:21	openstackgerrit	Wang Qing wu proposed openstack/nova: WIP(6): Introduce base PowerVM driver (ssp drv) https://review.openstack.org/422594
12:58:43	openstackgerrit	Abhishek Kekane proposed openstack/python-novaclient: x-openstack-request-id logged twice in logs https://review.openstack.org/422602
12:58:46	EmilienM	_gryf: what's up?
13:00:05	openstackgerrit	sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023
13:00:08	openstackgerrit	sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364
13:00:09	openstackgerrit	sahid proposed openstack/nova: objects: add new field cpus_reserved in NUMACell https://review.openstack.org/400874
13:00:09	openstackgerrit	sahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875
13:00:12	openstackgerrit	sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024
13:00:17	openstackgerrit	sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878
13:00:18	openstackgerrit	sahid proposed openstack/nova: numa: take into account cpus reserved https://review.openstack.org/400876
13:00:18	openstackgerrit	sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877
13:10:30	openstackgerrit	Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614
13:25:08	johnthetubaguy	sambetts: jroll: sorry, was having lunch
13:26:01	johnthetubaguy	sambetts: jroll: I think its the using two ports for the same instance, and we would reset the bind host, but its very theoretical
13:26:31	johnthetubaguy	sambetts: jroll: do we need to set the bind_host to None, or could we just skip that?
13:29:11	johnthetubaguy	sambetts: jroll: I am basically good with this as it is, probably just leaning towards a quick follow up patch to for that one line change I suggested, if that would be helpful, also just making sure I am not missing something
13:37:57	archies	JOIN #openstack-meeting
13:38:12	archies	JOIN /#openstack-meeting
13:40:21	sambetts	johnthetubaguy: I'm still not sure I get what passing host_id=None through would do, we want to force it to None if it is already set, because otherwise it would prevent Ironic binding its provisoning network port
13:40:46	johnthetubaguy	sambetts: but if its already set, someone else owns the port
13:41:17	johnthetubaguy	sambetts: we should fail earlier on if thats already set
13:41:33	johnthetubaguy	(hence the race is a really tiny window, if at all)
13:41:50	sambetts	johnthetubaguy: +1 surly you can't pass a port that has host_id set into --nic port-id right??
13:42:24	sambetts	johnthetubaguy: that would be dangerous for normal cases
13:42:26	johnthetubaguy	right, but you can pass the same port into two instances, and they race each other to apply the binding, the last one "wins"
13:42:57	johnthetubaguy	I would just rather we didn't send bind_host_id=None, we just don't update bind_host_id
13:43:10	johnthetubaguy	its a niggle though, not a blocker
13:43:28	sambetts	johnthetubaguy: but you are OK writing host_id=<the last one> over the one already set?
13:43:53	sambetts	johnthetubaguy: surely we should fail over if at that point the host_id is already set on that port
13:43:57	johnthetubaguy	sambetts: no, there is a fix something in the neutron API soon, so we don't do that any more
13:44:17	johnthetubaguy	basically, the neutron API isn't atomic, and they are adding a generation id we can use to make sure the update is atomic

Earlier   Later