Tuesday, 2015-05-12

*** armax has quit IRC05:34
*** gsagie has quit IRC05:42
*** gsagie has joined #openstack-neutron-ovn05:51
*** openstack has joined #openstack-neutron-ovn07:53
*** mestery has joined #openstack-neutron-ovn12:42
otherwiseguyrussellb: finally a workflow+1 on ovs in global-requirements.txt13:40
otherwiseguythat took...a while.13:40
russellbotherwiseguy: yeah saw that13:42
otherwiseguyI'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
russellbotherwiseguy: meh indeed13:42
russellbotherwiseguy: i'd document it if I knew a good place to do so13:43
gsagiei am writing some basic unit tests, does that mean we wont be able to add them?13:43
russellbotherwiseguy: I guess neutron and networking-ovn README13:43
russellbgsagie: no, it means we can add them :)13:43
russellbtox will install ovs from PyPI13:44
russellband it has the version we need, so it's fine13:44
gsagierussellb : ok great13:44
gsagierussellb : need any help from me to prepare something to the summit?13:46
russellbgsagie: the reviews and testing you're doing are perfect13:46
russellbi'm slowly working on a blog post as i work through the multi-node testing13:46
gsagiecool, thats usefull information13:47
russellbgsagie: i don't have any presentations or anything13:47
russellbwe have a design summit session, but there's not much prep needed ...13:47
*** mestery has quit IRC13:53
*** armax has joined #openstack-neutron-ovn14:06
*** shettyg has joined #openstack-neutron-ovn14:06
*** shettyg has quit IRC14:07
russellbgsagie: 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 setup14:10
*** shettyg has joined #openstack-neutron-ovn14:13
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Document adding a second compute node  https://review.openstack.org/18160214:17
openstackgerritRussell Bryant proposed stackforge/networking-ovn: Install neutron if not done automatically.  https://review.openstack.org/18169514:17
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Add OVN_ROLE, OVN_REMOTE, and OVN_UUID  https://review.openstack.org/18150214:17
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Only run ./configure when needed  https://review.openstack.org/18203814:17
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Ensure openvswitch modules are loaded  https://review.openstack.org/18206314:17
gsagierussellb : ok, thanks for the update, also blocked on that14: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 can14:21
russellb<shettyg> *does not let me14:21
russellb<shettyg> If you have access to commit 8b59c345e0612a2c0844a2149ac9a2320b4fa556 from Ben's ovs-reviews repo, then that would work.14:21
russellbgsagie: ^ in case you want to try it14:21
russellbi'm about to set that up14:21
gsagieok, thanks!14:22
russellband then see if i can work out the OVN patch to fix it14:22
gsagierussellb : 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
russellbgsagie: i added it14:23
russellbin that last update14:23
gsagiecool :)14:23
russellbshettyg: seems 8b59c345e0612a2c0844a2149ac9a2320b4fa556 isn't in blp's repo anymore ... but i'm guessing it's his ovn branch + the physical_init() fix14:26
shettygLet me push my local copy to a github repo, if that helps.14:27
russellbshettyg: sure14:27
russellbonce i have things working i guess i can try to git-bisect where it broke14:28
shettygrussellb: https://github.com/shettyg/ovn14:31
shettygrussellb: There were a couple of other bugs introduced during the review process, so it looks like this is one of them.14:32
russellbshettyg: *nods* ... you all have an internal conference going on?14:33
shettygrussellb: Yes. It is called RADIO14:33
russellbOK, was wondering if blp would be on IRC today so I could bug him about the breakage :)14:33
russellbwe'll figure it out though :)14:33
shettygIf he gets bored, he will likely login. But he is a co-author of a bunch of papers, so I guess he will be busy14:34
russellbshettyg: you won't be in Vancouver for the OpenStack summit next week by chance, will you?14:37
shettygrussellb: No. The repo that I linked to needs an additional fix (which I missed adding). I will push it too.14:41
russellbshettyg: ok, yeah, the missing init14:41
shettygrussellb: A little different one. I pushed it. You can have a look.14:44
russellbotherwiseguy: i think the networking-ovn requirements.txt may need to be updated as well, to reflect the min version14:50
openstackgerritGal Sagie proposed stackforge/networking-ovn: Remove incorrect comment  https://review.openstack.org/18233514:52
gsagierussellb: https://review.openstack.org/#/c/177893/14:52
gsagieyou added it.. :)14:52
gsagieyeah.. got it14:52
gsagiewant me to do it?14:54
russellbsure, whoever :)14:55
russellbjust said otherwiseguy since he had just updated the one in neutron, made me think of it14:55
gsagiebut 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
russellbanother 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 else14: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
russellbthis file is only used with pip14:57
russellband that version is what we have in PyPI14:57
otherwiseguyrussellb: 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
gsagieahh ok14:57
russellbthe downside is it might be confusing to packagers trying to understand what they should be packaging14:57
russellbbut we don't have a good place to document that an older version is fine for neutron14:58
gsagieyeah, thats why i was confused, maybe the name "ovs" is too general (dont know if you could have made a different name)14:58
russellband it's just this newer version required for networking-ovn14:58
gsagiebecause this is basically just the ovs python bindings14:58
russellbgsagie: that's what the python module is though14:58
otherwiseguyrussellb: 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
otherwiseguysince they don't really require any specific OVS version.14:59
russellbotherwiseguy: it seems odd to package from PyPI, though15:20
russellbthe upstream releases of python-openvswitch are still the main ovs releases15:21
otherwiseguyrussellb: 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
openstackgerritMerged stackforge/networking-ovn: Remove incorrect comment  https://review.openstack.org/18233515:46
otherwiseguyI generally look at it like they just kind of accidentally share a repository. :p15:47
russellbotherwiseguy: but until upstream starts making python-ovs releases separate from ovs itself ...15:56
russellbfor the particular issue at hand. ovs 2.4 will be out soon anyway15:56
openstackgerritGal Sagie proposed stackforge/networking-ovn: requirements.txt: Add ovs.  https://review.openstack.org/17789315:59
*** openstackgerrit_ has quit IRC16:11
*** shettyg has quit IRC16:33
*** mestery has joined #openstack-neutron-ovn16:37
*** mestery has quit IRC16:38
*** shettyg has joined #openstack-neutron-ovn16:46
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Document adding a second compute node  https://review.openstack.org/18160217:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: Install neutron if not done automatically.  https://review.openstack.org/18169517:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: drop ovn.filters  https://review.openstack.org/18148017:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Add multi-node support  https://review.openstack.org/18150217:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Only run ./configure when needed  https://review.openstack.org/18203817:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Ensure openvswitch modules are loaded  https://review.openstack.org/18206317:32
openstackgerritRussell Bryant proposed stackforge/networking-ovn: devstack: Don't force enable key,mysql,rabbit  https://review.openstack.org/18156117:32
russellbgsagie: reworked things so that ovn-controller and ovn-northd run in screen, now17:32
russellbjenkins seems really pissed18:10
russellbour neutron requirements.txt dependency seems broken ... git+git://git.openstack.org/openstack/neutron.git#egg=neutron18:14
* russellb talks to himself ......18:19
*** shettyg has quit IRC18:55
openstackgerritRussell Bryant proposed stackforge/networking-ovn: DO NOT MERGE: test revert of neutron patch  https://review.openstack.org/18241118:57
otherwiseguyrussellb: don't go crazy.19:02
otherwiseguyrussellb, great hero...or greatest hero?19:09
*** SunnyRainbow has joined #openstack-neutron-ovn19:09
russellbotherwiseguy: wat19:09
otherwiseguyfixing https://review.openstack.org/#/c/182413/19:09
russellboh :-)19:11
russellbotherwiseguy: i was raging for a little bit19:11
*** openstackgerrit_ has joined #openstack-neutron-ovn19:16
* russellb stares at openstackgerrit and openstackgerrit_ 19:17
*** shettyg has joined #openstack-neutron-ovn19:25
*** openstackgerrit_ has quit IRC19:25
*** armax has quit IRC20:03
*** shettyg has quit IRC20:15
*** shettyg has joined #openstack-neutron-ovn20:19
*** openstackgerrit has quit IRC20:37
*** openstackgerrit has joined #openstack-neutron-ovn20:37
*** armax has joined #openstack-neutron-ovn21:01
*** SunnyRainbow has quit IRC21:30
*** armax has quit IRC21:56
*** mestery has joined #openstack-neutron-ovn23:09
*** SunnyRainbow has joined #openstack-neutron-ovn23:12
*** SunnyRainbow has quit IRC23:38
*** armax has joined #openstack-neutron-ovn23:48

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!