*** vthapar has joined #openstack-net-bgpvpn | 04:32 | |
*** tmorin has joined #openstack-net-bgpvpn | 07:59 | |
*** enikher has joined #openstack-net-bgpvpn | 08:10 | |
*** matrohon has joined #openstack-net-bgpvpn | 08:14 | |
*** enikher1 has joined #openstack-net-bgpvpn | 08:44 | |
*** enikher has quit IRC | 08:45 | |
vthapar | tmorin: thanks for the catch. that is the error I was expecting and will take care of it. | 08:56 |
---|---|---|
matrohon | vthapar, you're mocking the create_client that doesn't exist in liberty | 09:29 |
matrohon | tmorin vthapar : it seems the gate get the net-odl liberty code even if it is not specified in the test-requirements of bgpvpn | 09:31 |
tmorin | matrohon, vthapar: this is surprising, the devstack vm jobs would indeed follow the branch of the change and take the same one for dependencies, but here we are *not* talking about devstack vm jobs | 09:38 |
tmorin | I don't think the python unit test jobs would do that | 09:38 |
tmorin | I just checked | 09:39 |
tmorin | http://logs.openstack.org/52/274552/1/check/gate-networking-bgpvpn-python27/ffbc66b/console.html#_2016-02-01_08_56_50_180 | 09:39 |
tmorin | --> networking-odl HEAD in the test is a460580 | 09:39 |
tmorin | this is the current master's HEAD of networking-odl | 09:39 |
matrohon | tmorin, so the mock should work... | 09:39 |
tmorin | https://github.com/openstack/networking-odl/commits/master | 09:40 |
* matrohon looking at the logs of the gate job | 09:40 | |
matrohon | tmorin, vthapar : the mock error I was talking about happens in my config, with the net-odl on the stable/liberty branch | 09:42 |
matrohon | not in the gate | 09:42 |
tmorin | matrohon: ok, I thought you were looking at the gate tests | 09:43 |
matrohon | vthapar, tmorin, I think the gate test is not relevant since it uses the master branch of net-odl | 09:52 |
tmorin | true | 09:52 |
tmorin | vthapar: can you update test-requirements.txt in change 274552 ? | 09:55 |
vthapar | tmorin, matrohon: sorry guys, was away in a meeting. will update the test-requirements | 09:55 |
*** enikher has joined #openstack-net-bgpvpn | 10:01 | |
*** enikher1 has quit IRC | 10:03 | |
openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn: Follow Neutron master https://review.openstack.org/274722 | 10:10 |
*** enikher1 has joined #openstack-net-bgpvpn | 10:13 | |
*** enikher has quit IRC | 10:15 | |
*** openstackgerrit has quit IRC | 10:17 | |
*** openstackgerrit has joined #openstack-net-bgpvpn | 10:17 | |
tmorin | matrohon: can you look at the change above ? ( it's ready and passing tests ) | 10:18 |
matrohon | tmorin, done | 10:34 |
tmorin | thanks! | 10:34 |
openstackgerrit | Merged openstack/networking-bgpvpn: Follow Neutron master https://review.openstack.org/274722 | 10:38 |
*** vthapar has quit IRC | 10:57 | |
*** enikher has joined #openstack-net-bgpvpn | 13:27 | |
*** enikher1 has quit IRC | 13:29 | |
openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn: bagpipe! agent, update RPC setup code https://review.openstack.org/275213 | 13:43 |
openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn: bagpipe: agent, update RPC setup code https://review.openstack.org/275213 | 13:44 |
*** enikher1 has joined #openstack-net-bgpvpn | 14:27 | |
*** enikher has quit IRC | 14:29 | |
*** enikher has joined #openstack-net-bgpvpn | 15:34 | |
*** enikher1 has quit IRC | 15:36 | |
*** matrohon has quit IRC | 16:57 | |
*** enikher has quit IRC | 17:03 | |
*** tmorin has quit IRC | 17:13 | |
*** openstackgerrit has quit IRC | 17:32 | |
*** openstackgerrit has joined #openstack-net-bgpvpn | 17:32 | |
*** matrohon has joined #openstack-net-bgpvpn | 19:04 | |
*** matrohon has quit IRC | 19:13 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!