Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-03
18:53:46 mriedem *scheduler manager
18:54:17 jaypipes mriedem: back shortly. need to go pick the pugs up from the groomers.
18:54:28 mriedem oolala
19:32:27 jaypipes mriedem: nothin' better than the smell of a sweet-smellin' grumble of freshly-groomed pugs.
19:33:35 mriedem dry or wet?
19:34:57 jaypipes dry of course. :)
19:35:06 mriedem good
19:46:34 jangutter Due to some scary flu, external deadlines and various conjunctions I was out of OpenStack action for about 2 months...
19:46:53 jangutter And boy, did the nova and os-vif bits that I'm interested change!
19:47:54 jaypipes jangutter: glad you are feeling better!
19:47:57 jangutter My question is to do with the new OVS_ACCELERATION vif that moshele contributed... I'm not seeing where the PCI devices get allocated.
19:48:30 jangutter Lemme grab the patch set.
19:48:53 jaypipes jangutter: PCI devices do not get allocated by os-vif. They are allocated by the resource tracker when a launch instance request is received that includes a PCI device request.
19:49:09 jangutter Yep, that I understand.
19:49:19 jaypipes jangutter: the resource tracker has a PCI device manager inside it, and it's that that does the actual "allocation" of a PCI device to a guest
19:49:40 jangutter But does it just pick _any_ available PCI address?
19:50:01 jangutter ( https://review.openstack.org/#/q/topic:ovs_acc )
19:50:17 jaypipes jangutter: it picks any available PCI address that matches the PCI device request that is received from the conductor.
19:50:47 jaypipes jangutter: are you wondering who creates that PCI device request? :)
19:51:32 jangutter Yep, remember, I had the brain-dead idea of putting it into neutron-api bits.
19:51:57 jaypipes jangutter: well, there's still quite a bit of Neutron API that is used to construct the PCI device request.
19:52:08 jaypipes jangutter: and that existed before your idea ;)
19:53:53 jaypipes jangutter: what moshele's patch did on the os-vif side was simply create a specialized VIF type class for the "new" OVS_ACCELERATED vif type. That vif type is something that still needs to be created in Neutron and supplied to the nova boot request by passing in a port_id value that corresponds to the pre-created Neutron port that has the port binding information indicating it's an "OVS_ACCELERATED" port.
19:54:14 jangutter Yep, that I got.
19:54:46 jangutter Very similar to what happened in our patch: but the OVS_ACCELERATED vif is new.
19:55:44 jaypipes jangutter: new in Neutron, yeah?
19:57:00 jangutter Oh, hang on. Does it auto-allocate because of VIFHostDevice ?
19:57:21 jaypipes jangutter: what do you mean by "auto-allocate"?
19:58:47 jangutter I'm looking for where the pci_request is made -> it's still at create_pci_requests_for_sriov_ports?
20:01:38 jaypipes jangutter: yep
20:10:37 jangutter jaypipes: OK, that's weird. It must be right before my eyes, but I can't see any way that https://review.openstack.org/#/c/398265/ triggers a pci_request.
20:14:44 jangutter jaypipes: Aaah, sorry, my addled mind has finally caught up: if vnic_type in network_model.VNIC_TYPES_SRIOV:
20:15:36 jaypipes correct
20:18:03 jangutter OK, now that makes sense. It picks up the PCI device from phynet_name
20:32:55 openstackgerrit Jackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils https://review.openstack.org/479949
20:34:01 openstackgerrit Jackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils https://review.openstack.org/479949
20:39:41 ildikov mriedem: welcome back :)
20:40:17 mriedem thanks - i never really left
20:40:52 ildikov mriedem: I was wondering whether you could review the swap volume patch by any chance, so while you celebrate freedom I can be the slave of fixing review comments? :)
20:41:09 ildikov mriedem: I know, but still :)
20:42:20 mriedem wow
20:42:48 mriedem maybe, i need to wrap something up first and then i'm out for a bit
20:44:28 ildikov I can keep myself busy with attach that's true, but would rather fix swap hoping it'll be just a long and not endless list
20:44:45 ildikov was just an idea :)
20:55:09 mriedem ildikov: did anyone get started on the grenade test changes last week?
20:55:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899
20:55:15 mriedem to attach on the old side and detach on the new side?
20:56:06 ildikov mriedem: stvnoyes1 is looking into Grenade, he had some issues with setting it up, but he was having progress last week
20:56:24 mriedem ok
20:56:32 mriedem i'm going to run out for a bit
21:36:38 andreykurilin mriedem: hi
21:38:38 andreykurilin mriedem: just note: version list is broken in novaclient due to recent changes in devstack. I suppose it is about https://github.com/openstack-dev/devstack/commit/b90bb1a46193b052cd98513213eed688e8e29d23 . I have POC(https://review.openstack.org/#/c/479818/) of fix, tomorrow will add tests and it will be ready.
22:21:38 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
22:29:54 mriedem andreykurilin: ok thanks for the heads up
23:23:05 openstack Launchpad bug 1702194 in python-novaclient "test_version_list fails with "(TypeError): 'NoneType' object has no attribute '__getitem__'"" [Undecided,New]
23:23:05 mriedem andreykurilin: track it with this bug https://bugs.launchpad.net/python-novaclient/+bug/1702194
23:28:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Mask password when logging connection_info during swap volume https://review.openstack.org/466533
23:30:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640
23:34:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640
#openstack-nova - 2017-07-04
00:07:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869
01:04:41 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
01:06:08 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
01:07:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
01:41:39 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227
01:42:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227
01:43:49 alex_xu mriedem: zhenguo talks to me about aggregate with traits. That sounds like he is looking for easy interface to set traits to the RPs in one aggregate. That is something can be done in the client or UI
02:42:03 openstackgerrit Jeffrey Zhang proposed openstack/nova master: Cleanup local files in libvirt when using shared block storage https://review.openstack.org/477780
05:38:56 openstackgerrit Chris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly https://review.openstack.org/461456
05:38:56 openstackgerrit Chris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.openstack.org/468203
05:41:21 cfriesen I'm off on vacation till July 18th, so feel free to tweek these ^ as needed, else I'll do it when I get back.
06:23:58 openstackgerrit suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
07:25:10 lhx__ hi guys, have you ever succeed in debugging nova unit test in local env?
07:26:23 lhx__ something went wrong when using 'tox -e debug ...'
07:42:19 jichen lhx__: https://www.youtube.com/watch?v=lipnzWBVYlk
07:43:53 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
07:43:57 openstackgerrit suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
07:48:02 openstackgerrit suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
07:52:11 lhx__ jichen, thanks, I use the second way, it's also the way in https://wiki.openstack.org/wiki/Testr
07:52:36 openstackgerrit Béla Vancsics proposed openstack/nova master: Reduce (notification) test duplication https://review.openstack.org/391428
07:52:39 lhx__ but it occurs: ERROR nova TypeError: Can't instantiate abstract class SubclassSignatureTestCase with abstract methods _get_base_class
07:53:09 lhx__ this is what I ran into
07:54:21 jichen lhx__: ok, I remember openstack wiki should have this kind of info, I didn't run into your error before ....
07:55:05 lhx__ jichen, I debug in the latest repo
07:55:58 lhx__ is your repo latest?
08:00:40 jichen no, but I don't think it matter, let me try if I can
08:06:47 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
08:16:41 sfinucan lhx__: Off the top of my head, are you sure you're supposed to be running with sudo privileges?
08:25:47 lhx__ sfinucan, sure, it went wrong with the same error no matter what user privilege :(((
08:26:12 sfinucan Ah, that's the only I had, I'm afraid. I also have not seen that
08:26:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add X-Openstack-Request-Id description https://review.openstack.org/474847
08:28:22 lhx__ sfinucan, haven't you seen the debug error in the latest repo?
08:28:39 sfinucan I've just kicked it off now
08:31:20 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473
08:31:20 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/449584
08:31:21 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475
08:31:21 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section https://review.openstack.org/478472
08:31:22 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474
08:31:22 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751
08:31:23 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484
08:31:23 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358

Earlier   Later