Posted	Nick	Remark
#openstack-nova - 2017-01-23
20:06:24	slaweq	oomichi: thx for +2 :)
20:15:56	openstackgerrit	STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644
20:41:10	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
20:41:11	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
20:42:30	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
20:42:31	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
20:52:51	amrith	andreykurilin, please let me know if I can help.
21:17:11	andreykurilin	amrith: I need just several minutes for latest version of patches. they should help
21:18:09	openstackgerrit	Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441
21:26:38	moshele	jaypipes: hi, can you review this simple fix https://review.openstack.org/#/c/423608/
21:51:10	openstackgerrit	Ed Leafe proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807
21:51:10	openstackgerrit	Ed Leafe proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215
21:51:11	openstackgerrit	Ed Leafe proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768
21:51:11	openstackgerrit	Ed Leafe proposed openstack/nova: placement: integrate ProviderTree to report client https://review.openstack.org/415921
21:51:12	openstackgerrit	Ed Leafe proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769
21:51:12	openstackgerrit	Ed Leafe proposed openstack/nova: placement: adds ProviderTree for nested resources https://review.openstack.org/415920
21:51:13	openstackgerrit	Ed Leafe proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693
21:51:14	openstackgerrit	Ed Leafe proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138
21:59:53	openstackgerrit	STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644
22:12:20	amrith	thx andreykurilin
22:13:19	amrith	andreykurilin, this one (412497) isn't looking promising
22:30:19	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
22:37:03	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
22:43:47	openstackgerrit	Ed Leafe proposed openstack/nova: placement: minor refactor _allocate_for_instance() https://review.openstack.org/405705
22:43:47	openstackgerrit	Ed Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472
22:43:48	openstackgerrit	Ed Leafe proposed openstack/nova: placement: report client handle InventoryInUse https://review.openstack.org/404990
22:58:43	openstackgerrit	Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
23:18:20	andreykurilin	amrith: based on novaclient's functional tests, it should work. I do not sure that depends on tag actually works for novaclient's library. Can you recheck it locally?
23:30:05	andreykurilin	amrith: I try to play more with trove code tomorrow, but everything should be ok now
23:39:46	openstackgerrit	Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730
23:42:59	mriedem	dansmith: looks like grenade isn't happy with https://review.openstack.org/#/c/424252/
23:43:42	dansmith	mriedem: isn't that the dep thing?
23:44:31	dansmith	mriedem: yeah, that's the dep thing.. I've been waiting all day to get a run of my patch with this
23:44:38	dansmith	it just merged and I rechecked mine -- just rechecked this one too
23:45:12	mriedem	dansmith: i'm not sure what dep thing you're talking about
23:45:19	mriedem	you changed the d-g patch to not call discover_hosts for grenade
23:46:03	dansmith	mriedem: https://review.openstack.org/#/c/424323/
23:50:44	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/395347
23:51:01	mriedem	dansmith: ok i wasn't aware of that
23:56:16	openstackgerrit	Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
23:57:23	bauzas	Yippee ki-yay mother ^
23:57:44	bauzas	hoping Jenkins to be green after that one
23:57:51	bauzas	now, calling it a day
#openstack-nova - 2017-01-24
00:04:29	takashin	bauzas: Could you review https://review.openstack.org/#/c/408955/ again and check my comments?
00:04:56	bauzas	takashin: hola
00:05:07	bauzas	takashin: sorry, it's 1:05am for me now
00:05:14	bauzas	takashin: I'm just about to visit my bed
00:05:41	bauzas	takashin: but I'll try to look at it by tomorrow in case my above change is not getting a new problme
00:05:59	takashin	bauzas: Thank you.
00:06:24	bauzas	takashin: FWIW, opening a tab to remember it
00:06:39	bauzas	takashin: but ping me later if you wish (I'm UTC+1)
00:07:45	takashin	bauzas: I will ping you later.
00:08:54	openstackgerrit	Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614
00:12:16	openstackgerrit	Ken'ichi Ohmichi proposed openstack/nova: Add unit tests for JSON-Schema' items behavior https://review.openstack.org/415333
00:16:17	takashin	oomichi: Are you around?
00:17:56	oomichi	takashin: yep
00:18:08	takashin	oomichi: Could you review https://review.openstack.org/#/c/408964/ ?
00:18:09	oomichi	may I help
00:18:18	oomichi	takashin: ok, will review it soon
00:18:32	takashin	oomichi: Thank you.
00:19:12	oomichi	takashin: oh, huge one ;)
00:19:30	oomichi	maybe alex_xu also will help
00:22:57	takashin	oomichi: I understand that.
00:31:36	amrith	andreykurilin, I will try
00:47:22	bauzas	mriedem: dansmith: FWIW, I think we probably have a grenade issue for the scheduler upcall change http://logs.openstack.org/61/417961/19/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/b497772/logs/new/screen-n-sch.txt.gz?level=TRACE#_2017-01-24_00_30_31_476
00:47:58	bauzas	I'm getting a MissingAuthPlugin exception from Keystone...
00:56:28	openstackgerrit	Dan Smith proposed openstack/nova: Move instance creation to conductor https://review.openstack.org/319379
00:56:58	dansmith	mriedem: bauzas: melwitt: ^ all three patches are passing now, just updated to remove the WIP from the last one
00:57:28	bauzas	dansmith: I'll look at it tomorrow morning my time
00:57:33	dansmith	bauzas: kthx
00:58:01	bauzas	dansmith: in the meantime, it seems we don't provide the placement opts for grenade
00:58:11	dansmith	bauzas: yeah I saw above, have't looked yet tho
00:58:18	bauzas	dansmith: ie. http://logs.openstack.org/61/417961/19/check/gate-tempest-dsvm-cells-ubuntu-xenial/e99ce7c/logs/etc/nova/nova.conf.txt.gz has a placement section
00:58:29	bauzas	dansmith: vs. http://logs.openstack.org/61/417961/19/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/b497772/logs/etc/nova/nova.conf.txt.gz not having it
00:58:58	bauzas	dansmith: so my WIP change is currently +1 for all the dsvm jobs but not for grenade
00:59:31	bauzas	possibly because we miss https://github.com/openstack-dev/devstack/blob/master/lib/placement#L102 in grenade
01:00:21	dansmith	yeah
01:01:55	bauzas	so the problem is that we have not the above for the Newton cloud
01:02:18	bauzas	which means that it's not really working in grenade
01:02:25	bauzas	I mean, the placement job
01:02:38	bauzas	looks terrible to me
01:04:50	bauzas	dansmith: I need direction here before going to bed
01:05:11	bauzas	dansmith: because tomorrow morning I won't be knowing what to do
01:05:33	dansmith	bauzas: if we have something we have to do in order to upgrade, that has to go into grenade
01:05:45	bauzas	dansmith: when we have a config option missed by the old version but needed by the new version, what to do ?
01:05:56	dansmith	bauzas: however, I think you could just make the grenade job do whatever we do in newton to enable placement, so that you don't make a grenade change
01:06:01	bauzas	just adding it straight to grenade ?
01:06:12	dansmith	which config option?
01:06:29	bauzas	dansmith: I guess those two https://github.com/openstack-dev/devstack/blob/master/lib/placement#L102-L103
01:06:37	dansmith	it's possible to start placement in newton
01:06:42	dansmith	you mean devstack/newton can't?
01:06:58	bauzas	maybe, lemme look at the stable branch
01:07:26	dansmith	bauzas: if not, I think it's going to be kinda ugly.. starting that in grenade, because you'll have to alter the devstack config and because what we *want* is people to start placement in newton
01:07:30	bauzas	dansmith: mmm, weirdo https://github.com/openstack-dev/devstack/blob/stable/newton/lib/placement#L110-L111
01:08:09	bauzas	dansmith: just remembering how grenade works
01:08:46	bauzas	dansmith: so, we first use the devstack stable branch for running the cloud, then applying the master branch
01:08:47	bauzas	right?
01:09:02	bauzas	I'm not very expert of grenade unfortunatelty
01:09:45	bauzas	nevermind, seeing https://github.com/openstack-dev/grenade#basic-flow
01:10:21	bauzas	okay, I'll try to investigate then

Earlier   Later