Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-23
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
01:11:13 dansmith yes, that's how it works
01:12:29 bauzas dansmith: okay, thanks

Earlier   Later