| *** armax has quit IRC | 05:34 | |
| *** gsagie has quit IRC | 05:42 | |
| *** gsagie has joined #openstack-neutron-ovn | 05:51 | |
| *** openstack has joined #openstack-neutron-ovn | 07:53 | |
| *** mestery has joined #openstack-neutron-ovn | 12:42 | |
| otherwiseguy | russellb: finally a workflow+1 on ovs in global-requirements.txt | 13:40 |
|---|---|---|
| otherwiseguy | that took...a while. | 13:40 |
| russellb | otherwiseguy: yeah saw that | 13:42 |
| otherwiseguy | I'm still not 100% on whether the minimum version req is a good thing, since distro packages aren't going to have 2.4.0 for a while, but meh. | 13:42 |
| russellb | otherwiseguy: meh indeed | 13:42 |
| russellb | otherwiseguy: i'd document it if I knew a good place to do so | 13:43 |
| gsagie | i am writing some basic unit tests, does that mean we wont be able to add them? | 13:43 |
| russellb | otherwiseguy: I guess neutron and networking-ovn README | 13:43 |
| russellb | gsagie: no, it means we can add them :) | 13:43 |
| russellb | tox will install ovs from PyPI | 13:44 |
| russellb | and it has the version we need, so it's fine | 13:44 |
| gsagie | russellb : ok great | 13:44 |
| gsagie | russellb : need any help from me to prepare something to the summit? | 13:46 |
| russellb | gsagie: the reviews and testing you're doing are perfect | 13:46 |
| russellb | i'm slowly working on a blog post as i work through the multi-node testing | 13:46 |
| gsagie | cool, thats usefull information | 13:47 |
| russellb | gsagie: i don't have any presentations or anything | 13:47 |
| russellb | we have a design summit session, but there's not much prep needed ... | 13:47 |
| *** mestery has quit IRC | 13:53 | |
| *** armax has joined #openstack-neutron-ovn | 14:06 | |
| *** shettyg has joined #openstack-neutron-ovn | 14:06 | |
| *** shettyg has quit IRC | 14:07 | |
| russellb | gsagie: multi-node OVN isn't working for me, but shettyg left a message on here confirming that it seems broken, so i don't think it's our setup | 14:10 |
| *** shettyg has joined #openstack-neutron-ovn | 14:13 | |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Document adding a second compute node https://review.openstack.org/181602 | 14:17 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: Install neutron if not done automatically. https://review.openstack.org/181695 | 14:17 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Add OVN_ROLE, OVN_REMOTE, and OVN_UUID https://review.openstack.org/181502 | 14:17 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Only run ./configure when needed https://review.openstack.org/182038 | 14:17 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Ensure openvswitch modules are loaded https://review.openstack.org/182063 | 14:17 |
| gsagie | russellb : ok, thanks for the update, also blocked on that | 14:21 |
| russellb | <shettyg> russellb: Just checked that ovn-controller from tip of ovn master does let me ping across hosts. As soon as I revert to a older version of ovn-controller, I can | 14:21 |
| russellb | <shettyg> *does not let me | 14:21 |
| russellb | <shettyg> If you have access to commit 8b59c345e0612a2c0844a2149ac9a2320b4fa556 from Ben's ovs-reviews repo, then that would work. | 14:21 |
| russellb | gsagie: ^ in case you want to try it | 14:21 |
| russellb | i'm about to set that up | 14:21 |
| gsagie | ok, thanks! | 14:22 |
| russellb | and then see if i can work out the OVN patch to fix it | 14:22 |
| gsagie | russellb : do you think we should also add the geneve kernel module loading to devstack plugin? because i didnt see it happens in my setup (unless i missed it) | 14:22 |
| russellb | gsagie: i added it | 14:23 |
| russellb | in that last update | 14:23 |
| gsagie | cool :) | 14:23 |
| gsagie | thanks! | 14:23 |
| russellb | https://review.openstack.org/#/c/182063/2/devstack/plugin.sh,cm | 14:23 |
| russellb | shettyg: seems 8b59c345e0612a2c0844a2149ac9a2320b4fa556 isn't in blp's repo anymore ... but i'm guessing it's his ovn branch + the physical_init() fix | 14:26 |
| shettyg | Let me push my local copy to a github repo, if that helps. | 14:27 |
| russellb | shettyg: sure | 14:27 |
| russellb | once i have things working i guess i can try to git-bisect where it broke | 14:28 |
| shettyg | russellb: https://github.com/shettyg/ovn | 14:31 |
| shettyg | russellb: There were a couple of other bugs introduced during the review process, so it looks like this is one of them. | 14:32 |
| russellb | shettyg: *nods* ... you all have an internal conference going on? | 14:33 |
| shettyg | russellb: Yes. It is called RADIO | 14:33 |
| russellb | OK, was wondering if blp would be on IRC today so I could bug him about the breakage :) | 14:33 |
| russellb | we'll figure it out though :) | 14:33 |
| shettyg | If he gets bored, he will likely login. But he is a co-author of a bunch of papers, so I guess he will be busy | 14:34 |
| russellb | shettyg: you won't be in Vancouver for the OpenStack summit next week by chance, will you? | 14:37 |
| shettyg | russellb: No. The repo that I linked to needs an additional fix (which I missed adding). I will push it too. | 14:41 |
| russellb | shettyg: ok, yeah, the missing init | 14:41 |
| shettyg | russellb: A little different one. I pushed it. You can have a look. | 14:44 |
| russellb | ok | 14:44 |
| russellb | oh | 14:45 |
| russellb | otherwiseguy: i think the networking-ovn requirements.txt may need to be updated as well, to reflect the min version | 14:50 |
| openstackgerrit | Gal Sagie proposed stackforge/networking-ovn: Remove incorrect comment https://review.openstack.org/182335 | 14:52 |
| gsagie | russellb: https://review.openstack.org/#/c/177893/ | 14:52 |
| gsagie | you added it.. :) | 14:52 |
| gsagie | ohh | 14:52 |
| gsagie | yeah.. got it | 14:52 |
| gsagie | want me to do it? | 14:54 |
| russellb | sure, whoever :) | 14:55 |
| russellb | just said otherwiseguy since he had just updated the one in neutron, made me think of it | 14:55 |
| gsagie | but i am still wondering, if thats the version and OVS isnt there yet, does that work because thats the version for the python-binding? | 14:56 |
| russellb | another random thought ... we need to make it easier to restart just the OVN daemons in a devstack env ... could convert it to run each in a screen tab just like everything else | 14:56 |
| gsagie | "<otherwiseguy> I'm still not 100% on whether the minimum version req is a good thing, since distro packages aren't going to have 2.4.0 for a while, but meh." | 14:56 |
| russellb | this file is only used with pip | 14:57 |
| russellb | and that version is what we have in PyPI | 14:57 |
| otherwiseguy | russellb: Just did a rebase on the neutron one and everything looked very unhappy (even pep8). Went ahead and updated it to have min version as well just to see if that fixed it. | 14:57 |
| gsagie | ahh ok | 14:57 |
| russellb | the downside is it might be confusing to packagers trying to understand what they should be packaging | 14:57 |
| russellb | but we don't have a good place to document that an older version is fine for neutron | 14:58 |
| gsagie | yeah, thats why i was confused, maybe the name "ovs" is too general (dont know if you could have made a different name) | 14:58 |
| russellb | and it's just this newer version required for networking-ovn | 14:58 |
| gsagie | because this is basically just the ovs python bindings | 14:58 |
| russellb | gsagie: that's what the python module is though | 14:58 |
| otherwiseguy | russellb: yeah, if I was a nice person I'd just go ahead and split out the RH python-openvswitch package and update the spec to just pull specific versions with pip... | 14:58 |
| otherwiseguy | since they don't really require any specific OVS version. | 14:59 |
| russellb | otherwiseguy: it seems odd to package from PyPI, though | 15:20 |
| russellb | the upstream releases of python-openvswitch are still the main ovs releases | 15:21 |
| otherwiseguy | russellb: since the OVS and python-openvswitch codebases aren't at all related to each other, there's no real reason to sync the distro releases. Can't think of why we'd want to require openvswitch 2.4.0 just to use notifications in the python lib. | 15:46 |
| openstackgerrit | Merged stackforge/networking-ovn: Remove incorrect comment https://review.openstack.org/182335 | 15:46 |
| otherwiseguy | I generally look at it like they just kind of accidentally share a repository. :p | 15:47 |
| russellb | otherwiseguy: but until upstream starts making python-ovs releases separate from ovs itself ... | 15:56 |
| russellb | for the particular issue at hand. ovs 2.4 will be out soon anyway | 15:56 |
| openstackgerrit | Gal Sagie proposed stackforge/networking-ovn: requirements.txt: Add ovs. https://review.openstack.org/177893 | 15:59 |
| *** openstackgerrit_ has quit IRC | 16:11 | |
| *** shettyg has quit IRC | 16:33 | |
| *** mestery has joined #openstack-neutron-ovn | 16:37 | |
| *** mestery has quit IRC | 16:38 | |
| *** shettyg has joined #openstack-neutron-ovn | 16:46 | |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Document adding a second compute node https://review.openstack.org/181602 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: Install neutron if not done automatically. https://review.openstack.org/181695 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: drop ovn.filters https://review.openstack.org/181480 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Add multi-node support https://review.openstack.org/181502 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Only run ./configure when needed https://review.openstack.org/182038 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Ensure openvswitch modules are loaded https://review.openstack.org/182063 | 17:32 |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: devstack: Don't force enable key,mysql,rabbit https://review.openstack.org/181561 | 17:32 |
| russellb | gsagie: reworked things so that ovn-controller and ovn-northd run in screen, now | 17:32 |
| russellb | jenkins seems really pissed | 18:10 |
| russellb | our neutron requirements.txt dependency seems broken ... git+git://git.openstack.org/openstack/neutron.git#egg=neutron | 18:14 |
| * russellb talks to himself ...... | 18:19 | |
| *** shettyg has quit IRC | 18:55 | |
| openstackgerrit | Russell Bryant proposed stackforge/networking-ovn: DO NOT MERGE: test revert of neutron patch https://review.openstack.org/182411 | 18:57 |
| otherwiseguy | russellb: don't go crazy. | 19:02 |
| otherwiseguy | russellb, great hero...or greatest hero? | 19:09 |
| *** SunnyRainbow has joined #openstack-neutron-ovn | 19:09 | |
| russellb | otherwiseguy: wat | 19:09 |
| otherwiseguy | fixing https://review.openstack.org/#/c/182413/ | 19:09 |
| russellb | oh :-) | 19:11 |
| russellb | otherwiseguy: i was raging for a little bit | 19:11 |
| *** openstackgerrit_ has joined #openstack-neutron-ovn | 19:16 | |
| * russellb stares at openstackgerrit and openstackgerrit_ | 19:17 | |
| *** shettyg has joined #openstack-neutron-ovn | 19:25 | |
| *** openstackgerrit_ has quit IRC | 19:25 | |
| *** armax has quit IRC | 20:03 | |
| *** shettyg has quit IRC | 20:15 | |
| *** shettyg has joined #openstack-neutron-ovn | 20:19 | |
| *** openstackgerrit has quit IRC | 20:37 | |
| *** openstackgerrit has joined #openstack-neutron-ovn | 20:37 | |
| *** armax has joined #openstack-neutron-ovn | 21:01 | |
| *** SunnyRainbow has quit IRC | 21:30 | |
| *** armax has quit IRC | 21:56 | |
| *** mestery has joined #openstack-neutron-ovn | 23:09 | |
| *** SunnyRainbow has joined #openstack-neutron-ovn | 23:12 | |
| *** SunnyRainbow has quit IRC | 23:38 | |
| *** armax has joined #openstack-neutron-ovn | 23:48 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!