| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-24 | |||
| 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 | |
| 13:24:22 | sdague | placement goes the hard coded route here as well | |
| 13:24:28 | sdague | so this is consistent with placement | |
| 13:24:30 | mriedem | %(request_line)s must come from eventlet? | |
| 13:24:49 | mriedem | and we're hard-coding: %(REQUEST_METHOD)s %(REQUEST_URI)s | |
| 13:25:34 | mriedem | log_format='%(client_ip)s - - [%(date_time)s] "%(request_line)s" %(status_code)s %(body_length)s %(wall_seconds).6f' | |
| 13:25:34 | mriedem | this is the eventlet wsgi default | |
| 13:26:24 | mriedem | seems odd that we define a default for our config option that's different from the eventlet wsgi default | |
| 13:26:32 | sdague | mriedem: https://github.com/openstack/oslo.service/blob/5a8a712c26cb053cbabf49044e6831ed98a2c2d5/oslo_service/wsgi.py#L184 | |
| 13:26:56 | sdague | mriedem: yeh, well, it's all leaky abstraction | |
| 13:27:02 | sdague | those are eventlet native stanzas | |
| 13:27:12 | mriedem | nova uses it here https://github.com/openstack/nova/blob/master/nova/wsgi.py#L184 | |
| 13:30:27 | sdague | mriedem: ah, I guess we're duping more of oslo.service than I realized, I thought that had been fully factored back out | |
| 13:30:41 | sdague | but, regardless, the current keying are eventlet server keywords | |
| 13:31:02 | mriedem | right, what i'm thinking is if we used the config, | |
| 13:31:14 | mriedem | it would be nice to have it default to None so we could tell if the operator actually cares about overriding it, | |
| 13:31:27 | mriedem | if it's None, and you're using eventlet wsgi, then you get eventlet wsgi defaults for the log format | |
| 13:31:30 | bauzas | mdbooth: do you remember some problems with volume migrations for their type ? https://bugs.launchpad.net/nova/+bug/1705700 | |