Posted	Nick	Remark
#openstack-nova - 2017-07-22
14:31:11	openstackgerrit	Matt Riedemann proposed openstack/nova master: placement: add retry tight loop claim_resources() https://review.openstack.org/486170
16:34:38	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/486326
16:38:56	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486337
17:44:32	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
22:26:07	catintheroof	hi guys, quick question, does transport_url on nova.conf supports multiple rabbitmq servers for ha ? if so, does anyone has an example ?
#openstack-nova - 2017-07-23
00:55:51	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711
01:05:28	openstackgerrit	Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486337
01:39:13	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/486326
03:06:56	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.53 - services and hypervisors using UUIDs https://review.openstack.org/485435
03:08:23	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711
03:46:24	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711
07:33:22	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
09:59:43	openstackgerrit	Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265
11:56:49	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
12:03:46	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237
13:28:20	mriedem	bauzas: when you're around tomorrow, can you check my comments and concerns in https://review.openstack.org/#/c/408955/64 ? i really dislike what's going on when forcing a cold migration and the flags between the api and conductor or snuck in through the scheduler hints, it just makes the flow for these various cases really really hard to follow and later debug when it breaks
13:28:42	mriedem	i suspect some of this is because of how force was done for live migrate, and if i saw this during that change i would have -1ed for the same reasons
13:29:55	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
13:48:01	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/486396
13:52:25	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486399
14:14:25	openstackgerrit	Merged openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975
15:16:33	openstackgerrit	Alex Xu proposed openstack/nova master: Using plain routes for the microversions test https://review.openstack.org/485061
15:16:33	openstackgerrit	Alex Xu proposed openstack/nova master: Move the note about '/os-volume_boot' to the correct place https://review.openstack.org/486071
15:16:34	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the useless extension block_device_mapping_v1 object https://review.openstack.org/486069
15:16:34	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the code related to extension loading from APIRouterV21 https://review.openstack.org/486414
15:16:35	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the useless FakeExt https://review.openstack.org/486415
15:16:35	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416
16:46:41	openstackgerrit	Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486399
16:52:17	openstackgerrit	Merged openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
17:00:11	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/486396
17:06:14	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
18:19:39	openstackgerrit	Jan Gutter proposed openstack/nova master: Add VIFHostDevice support to libvirt driver https://review.openstack.org/486426
18:30:32	openstackgerrit	Merged openstack/nova master: [placement] cover deleting a custom resource class in use https://review.openstack.org/484154
18:53:30	openstackgerrit	Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
19:04:12	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/486434
19:08:31	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/486435
23:37:21	openstackgerrit	Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/485039
#openstack-nova - 2017-07-24
01:38:58	openstackgerrit	Tristan Cacqueray proposed openstack/nova master: Mask the token used to allow access to consoles https://review.openstack.org/220622
01:39:59	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
03:26:10	openstackgerrit	Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/486475
03:40:52	openstackgerrit	Tristan Cacqueray proposed openstack/nova master: Mask the token used to allow access to consoles https://review.openstack.org/220622
04:56:44	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
05:11:15	openstackgerrit	Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
05:18:57	openstackgerrit	Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
05:44:03	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
07:51:00	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
08:15:02	openstackgerrit	Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265
08:16:53	openstackgerrit	Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943
08:22:36	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove an unnecessary argument in _prep_resize https://review.openstack.org/486521
08:26:41	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
08:47:53	kashyap	lyarwood: mdbooth: When you get a moment, mind taking a gander at this. It's in your wheelhouse: https://review.openstack.org/#/c/485752 -- "libvirt/driver.py: Set cache value for Cinder volume post-migration"
08:55:27	openstackgerrit	Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265
09:09:35	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
09:21:50	openstackgerrit	Hangdong Zhang proposed openstack/os-vif master: Update the documentation link for doc migration https://review.openstack.org/486536
09:27:38	openstackgerrit	Gábor Antal proposed openstack/nova master: Add helper method for waiting migrations in functional tests https://review.openstack.org/486537
09:48:12	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
10:00:44	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644
10:02:32	lyarwood	kashyap: done btw
10:03:50	kashyap	lyarwood: Cool, thank you. Will look. Yes, tests are missing, but wanted to get the core change out of the way first
10:04:44	kashyap	lyarwood: On your point about: "We can remove this now given the _get_volume_config call for each item in devices above."
10:04:56	kashyap	I think we should make such changes a separate one, to make backporting easier?
10:05:09	kashyap	Or does that fall under "related change, and therefore it's OK"
10:05:42	lyarwood	kashyap: erm, it's the same change, you shouldn't introduce duplicate calls to the same method in these flows
10:06:04	kashyap	lyarwood: Oeps, yes agreed. Too much switching between tasks, I'll address it
10:06:10	lyarwood	kashyap: np
10:06:19	kashyap	Thanks for looking!
10:06:43	lyarwood	anytime, I really need to get on top of the stable branch review queue
10:07:21	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
10:32:43	openstackgerrit	Alex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests https://review.openstack.org/483448
10:41:18	openstackgerrit	Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265
10:47:11	sdague	alex_xu: nice catch on - https://review.openstack.org/#/c/485602 I didn't realize we had a conf var for that
10:47:22	sdague	alex_xu: would you want them merged?
10:48:35	openstackgerrit	Balazs Gibizer proposed openstack/nova master: query instance after save to avoid partial update https://review.openstack.org/486561
10:56:31	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237
10:57:00	cdent	bauzas: you might be interested in that ^
11:40:54	claudiub	moshele: hello. around? :)
11:46:24	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561
11:48:10	openstackgerrit	Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943
11:55:32	sdague	cdent: if you want to redo OSAPIFixture to use wsgi intercept, I'd be all pro that
11:56:20	cdent	sdague: cool, I’ll look into that when I have another window of opportunity (which is likely relatively soon)
11:59:36	alex_xu	sdague: i'm ok to merge that. we can add note for that conf later if people want
12:00:43	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
12:01:02	openstackgerrit	sahid proposed openstack/nova master: libvirt: fix memory locking limit https://review.openstack.org/472633
12:02:40	alex_xu	sdague: the patch https://review.openstack.org/#/c/486414/ removed stevedore loading from the nova api. after that patch merged, we finish the goal of pike. The later cleanup can be done later, since we still have some garbage in the unittest.
12:07:40	alex_xu	sdague: oh, I missed understand your words. I think you mean the merge of log_format in the new middelware and config var. I think that sounds good.
12:09:46	alex_xu	sdague: the only strange thing is 'microversion' won't work with eventlet. If the config val includes $(microversion)s, it will generate a KeyError I guess?
12:12:46	sdague	alex_xu: yeh, that's going to be the issue
12:13:10	sdague	maybe wait for mriedem to get up and figure out what approach he wants
12:14:43	moshele	claudiub: hi
12:22:07	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
12:36:24	claudiub	moshele: so, i have a few other issues with sr-iov, if you have some time. :)
12:37:00	moshele	claudiub: I am in a meeting, but I can try to help
12:37:28	claudiub	moshele: so, I see that the vendor_id, and product_id are mandatory, but the issue i'm having is that I cannot extract those from *all* the sr-iov capable nics.
12:38:10	claudiub	moshele: for example, for Intel NICs, it's straight-forward and easy to do, as they are included in the DeviceID, but not for the Mellanox or Chelsio NICs
12:38:53	claudiub	moshele: so, I was wondering - wouldn't reporting the device_id sufficient?
12:39:06	claudiub	*be sufficient
12:39:46	moshele	claudiub: I am not sure is mandatory, can you point me to the place in the code. as I remember it can be use in the pci whitelist
12:40:03	claudiub	sure, just a second.
12:41:30	claudiub	moshele: currently, if i'm not reporting any vendor_id or product_id, it fails here: https://github.com/openstack/nova/blob/master/nova/pci/devspec.py#L277
12:42:18	moshele	claudiub: this for the pci whitelist, you can use also pci address/ pf name instead

Earlier   Later