Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-23
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
12:43:04 moshele claudiub: did you try to whitelist with pci address?
12:43:34 jangutter claudiub: where do you need to extract the SR-IOV capable PCI nics, is this for capabilities?
12:43:44 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s) https://review.openstack.org/485752
12:44:22 jangutter claudiub: I was confused with device_id for a bit, until I realised it's a synonym for product_id.
12:44:42 claudiub jangutter: nope, product_id, vendor_id, and device_id are distinct
12:45:03 claudiub you can have multiple devices with the same product_id, vendor_id, but not with the same device_id
12:45:22 jangutter claudiub: whoops, you're right! I forgot about that!
12:45:29 claudiub for example, 2 identical devices will have the same product_id and vendor_id, but different device_ids
12:46:15 jangutter claudiub: Is there something in lspci that indicates SR-IOV capability?
12:46:54 claudiub jangutter: i'm on windows / hyper-v. :)
12:46:59 claudiub jangutter: but yeah.
12:47:13 jangutter claudiub: you poor soul.
12:48:01 jangutter claudiub: "Capabilities: [200 v1] Single Root I/O Virtualization (SR-IOV)"
12:52:27 claudiub moshele: hm, I should be able to whitelist it using "devname" too, right?
12:52:45 moshele claudiub: yes
12:52:53 claudiub hm, let's see.
12:53:47 claudiub jangutter: hah. :))
12:54:17 claudiub jangutter: what is that "Capabilities: [200 v1] Single Root I/O Virtualization (SR-IOV)" about?
12:57:16 ralonsoh claudiub: can I ask you what capabilities do you need? rso, tso, txvlan, etc?
13:00:39 jangutter claudiub: sorry, that's what I see with lspci.
13:03:01 claudiub jangutter: ah, i see.
13:03:33 jangutter claudiub: is there perhaps something like that in the Windows Device Manager registry?
13:05:38 claudiub jangutter: you mean for sr-iov? it can easily be checked through this: Get-VMHost (check if the host supports SR-IOV), and Get-NetAdapterSriov (check if the NICs support SR-IOV)
13:05:55 claudiub i'm good in both cases. :)
13:06:36 claudiub ralonsoh: any / all. :) currently looking into hyper-v sr-iov. will have to see what capabilities are available.
13:07:39 ralonsoh claudiub: hmmm, sorry, I didn't read that part (hyper-v). I'll take a look
13:15:06 sdague mriedem: when you get a chance this morning, I'd like to figure out what's needed to merge this - https://review.openstack.org/#/c/485602/ - I think the outstanding question is around using the existing config definition
13:15:29 sdague though that might be a reasonable follow on patch to pull that in for both nova-api and placement
13:17:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
13:17:26 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644
13:23:21 mriedem sdague: so the question is if you're using eventlet, wsgi_log_format config is how you tailor the log format however you want, but with this it's hard-coded so you lose the ability
13:23:37 sdague mriedem: yes
13:24:09 sdague I guess we should determine if that is valuable as config or not

Earlier   Later