| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-15 | |||
| 14:41:19 | sean-k-mooney | sfinucan: hi o/, back from boston? | |
| 14:41:58 | sdague | lyarwood: yeh, that seems likely, I wonder why you are hitting it consistently but it's not showing up other places | |
| 14:46:34 | lyarwood | sdague: no idea, we have eventlet==0.21.0 in the gate at the moment | |
| 14:47:12 | sdague | lyarwood: I guess it's a race, I wonder if your env is on the slower or faster side of things | |
| 14:47:20 | sdague | just to figure out when things expose | |
| 14:49:07 | lyarwood | sdague: hopefully on the faster side vs the gate given it's the only heavy workload on my current pretty beefy host | |
| 14:50:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make discover_hosts only query for unmapped ComputeNode records https://review.openstack.org/427903 | |
| 14:58:37 | mriedem | jaypipes: https://review.openstack.org/#/c/427903/ is the last piece in the 'make auto-discover_hosts faster' series | |
| 14:59:01 | jaypipes | mriedem: k, will review. currently on a plane out to China... | |
| 14:59:08 | jaypipes | mriedem: spotty internet.. | |
| 14:59:10 | mriedem | oh fun | |
| 15:01:14 | alex_xu | jaypipes: $20 for more than 10 hours flight, it is totally worth :) | |
| 15:03:13 | openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Add description of server addresses objects https://review.openstack.org/464331 | |
| 15:05:07 | jaypipes | mriedem: done. | |
| 15:05:23 | mriedem | thanks | |
| 15:05:34 | jaypipes | alex_xu: still on way to LAX first, then the biggie. :) | |
| 15:06:59 | jaypipes | mriedem, sdague, anyone: seen this? from monotonic import monotonic as now # noqa | |
| 15:07:00 | jaypipes | RuntimeError: no suitable implementation for this system | |
| 15:07:00 | jaypipes | ||
| 15:07:00 | jaypipes | File "/home/jaypipes/src/git.openstack.org/openstack/nova/.tox/functional/local/lib/python2.7/site-packages/monotonic.py", line 169, in <module> | |
| 15:07:00 | jaypipes | raise RuntimeError('no suitable implementation for this system') | |
| 15:07:26 | jaypipes | just installed a fresh venv for tox/functional... | |
| 15:07:38 | mriedem | i have no | |
| 15:07:40 | mriedem | *not | |
| 15:07:51 | artom | jaypipes, there's been some chatter about it, yeah | |
| 15:07:57 | artom | jaypipes, see lyarwood's link above | |
| 15:08:10 | artom | https://github.com/eventlet/eventlet/issues/401 | |
| 15:09:25 | lyarwood | & https://bugs.launchpad.net/nova/+bug/1690819 | |
| 15:09:25 | openstack | Launchpad bug 1690819 in OpenStack Compute (nova) "nova-manage [..] api_db sync on F25 results in RuntimeError: no suitable implementation for this system" [Undecided,New] | |
| 15:09:45 | lyarwood | jaypipes: going back to eventlet 0.20.1 works around the issue for now btw | |
| 15:09:51 | lyarwood | for me at least | |
| 15:11:19 | sdague | that looks like it landed today | |
| 15:11:47 | jaypipes | lyarwood, artom: thx much for the help! | |
| 15:12:01 | jaypipes | yet another buggy eventlet release... | |
| 15:12:19 | artom | Well, if you monkey-patch all evarything... | |
| 15:12:46 | jaypipes | indeed. | |
| 15:13:48 | sdague | I proposed a revert on it - https://review.openstack.org/#/c/464664/ | |
| 15:17:09 | jaypipes | sdague: cool, cheers | |
| 15:27:17 | artom | mriedem, mind checking out https://review.openstack.org/#/c/415520/33 again? There's a link down there to the passing Tempest test. Thanks :) | |
| 15:27:27 | mriedem | yeah | |
| 15:27:50 | cfriesen | has anyone looked at the "fairness" behaviour of the oslo_concurrency.Lock() and Semaphore() and RWLock()? We're seeing some indications that if one eventlet drops the lock and immediately reaquires it then it can "starve" other eventlets waiting on the same lock. | |
| 15:29:17 | mriedem | dansmith: see alex_xu's question here https://review.openstack.org/#/c/461532/4/nova/compute/api.py - was there a reason we didn't do a 409 on HostAPI.compute_node_get when there are duplicates? apparently i pointed that out in the original review but never published my comments until just now. | |
| 15:31:07 | dansmith | mriedem: will looks when I'm off this cal | |
| 15:43:31 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 15:46:04 | ralonsoh | Hi, I have bug related to SR-IOV. When a PF is attached again to the OS (detached from the VM), the VF config (sriov_numvfs) is unset | |
| 15:46:09 | ralonsoh | #link https://bugs.launchpad.net/nova/+bug/1679703 | |
| 15:46:09 | openstack | Launchpad bug 1679703 in OpenStack Compute (nova) "Unable to boot instance with VF( direct-port ) because the PF is not online" [Undecided,New] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez) | |
| 15:46:30 | ralonsoh | I propose two ideas in the bug. Do you have any feedback? | |
| 15:47:02 | ralonsoh | IMO, the second one is cheaper and easier | |
| 15:56:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: tests for non-shared with shared https://review.openstack.org/461899 | |
| 15:56:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test for agg association not sharing https://review.openstack.org/461901 | |
| 15:56:51 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test non-shared out of inventory https://review.openstack.org/461900 | |
| 15:56:51 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: shared resources when finding providers https://review.openstack.org/461574 | |
| 16:01:20 | mriedem | ralonsoh: i'd ask moshele and/or cfriesen | |
| 16:01:36 | ralonsoh | mriedem: thanks! | |
| 16:01:53 | ralonsoh | moshele, cfriesen: ping | |
| 16:02:25 | moshele | ralonsoh: hi | |
| 16:02:48 | ralonsoh | moshele: can you take a look at my comment in #link https://bugs.launchpad.net/nova/+bug/1679703? | |
| 16:02:48 | openstack | Launchpad bug 1679703 in OpenStack Compute (nova) "Unable to boot instance with VF( direct-port ) because the PF is not online" [Undecided,New] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez) | |
| 16:02:55 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 16:02:56 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 16:02:58 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 16:03:01 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 16:03:05 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 16:03:06 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 16:03:11 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 16:03:30 | ralonsoh | moshele: it's about attaching a PF to a VM and then binding it again to the OS | |
| 16:03:40 | ralonsoh | moshele: the VF information is lost | |
| 16:04:28 | moshele | ralonsoh: I would expect libvirt to restore the num of vfs | |
| 16:04:53 | moshele | ralonsoh: did you send mail in libvrit community? | |
| 16:04:56 | ralonsoh | moshele: so you think the first option is the valid one | |
| 16:05:10 | ralonsoh | moshele: ahhh, so libvirt itself | |
| 16:05:17 | ralonsoh | moshele: ok, I'll do it | |
| 16:07:42 | moshele | ralonsoh: let me know what they say | |
| 16:18:33 | lyarwood | mdbooth: https://review.openstack.org/#/c/463992/3/nova/tests/unit/virt/libvirt/test_config.py - I'm not missing a volume driver where we are using netfs am I? | |
| 16:19:39 | mdbooth | lyarwood: Bizarrely, I'm currently looking at exactly the same thing | |
| 16:20:52 | lyarwood | haha great | |
| 16:20:55 | mdbooth | lyarwood: We don't appear to have a volume driver which sets netfs | |
| 16:21:15 | mdbooth | Although I'm just looking at all uses of source_protocol | |
| 16:21:26 | mdbooth | Possible it could be imported from somewhere | |
| 16:21:54 | mriedem | mdbooth: yeah, that was added when gluster was added, | |
| 16:22:07 | mriedem | i didn't remove it because i didn't know what might rely on it and break | |
| 16:22:26 | mriedem | in https://review.openstack.org/#/c/463992/ | |
| 16:22:27 | mdbooth | mriedem: I saw that, just looking for myself. | |
| 16:22:30 | lyarwood | yeah, I assumed we were setting it via connection_info in LibvirtNetVolumeDriver but I can't find anything on the cinder side | |
| 16:22:30 | mriedem | ok | |
| 16:22:43 | openstackgerrit | Dan Smith proposed openstack/nova master: Make compute auto-disable itself if builds are failing https://review.openstack.org/463597 | |
| 16:22:57 | mriedem | lyarwood: the source_protocol is set in nova | |
| 16:23:03 | mriedem | in the volume driver's get_config method | |
| 16:23:25 | lyarwood | mriedem: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/net.py#L46 | |
| 16:23:26 | mriedem | we could potentially remove the netfs stuff after this since it appears to be dead code now | |
| 16:23:27 | mdbooth | nova/virt/libvirt/volume/net.py: source_protocol = connection_info['driver_volume_type'] | |
| 16:24:00 | lyarwood | argh wrong line number | |
| 16:24:08 | mriedem | mdbooth: ok, i think that's just probably wrong | |
| 16:24:12 | mriedem | i.e. bad var name | |
| 16:24:43 | mdbooth | mriedem: It seems to be used in get_config there | |
| 16:24:48 | lyarwood | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/net.py#L96 | |
| 16:24:55 | mdbooth | Yeah, that one | |
| 16:25:10 | mdbooth | So that means it could come from cinder and end up in the libvirt xml | |
| 16:26:12 | mdbooth | cinder doesn't appear to set driver_volume_type to netfs anywhere, though | |