| *** vthapar has joined #openstack-net-bgpvpn | 03:36 | |
| *** matrohon has quit IRC | 08:33 | |
| *** bfernando has joined #openstack-net-bgpvpn | 08:47 | |
| *** tmorin has joined #openstack-net-bgpvpn | 08:59 | |
| *** openstackgerrit has joined #openstack-net-bgpvpn | 09:40 | |
| openstackgerrit | Édouard Thuleau proposed openstack/networking-bgpvpn master: Deprecates old OpenContrail driver https://review.openstack.org/535277 | 09:40 |
|---|---|---|
| openstackgerrit | Łukasz Rajewski proposed openstack/networking-bgpvpn master: Local_pref attr tempest test for port association https://review.openstack.org/534809 | 09:53 |
| *** vthapar has quit IRC | 11:10 | |
| openstackgerrit | Merged openstack/networking-bgpvpn master: routes-control: add advertise_extra_routes to router_association https://review.openstack.org/507173 | 11:24 |
| openstackgerrit | Merged openstack/networking-bgpvpn master: Basic tempest tests for port associations https://review.openstack.org/534187 | 11:24 |
| *** bfernando has quit IRC | 11:39 | |
| *** bobmel has quit IRC | 12:45 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: routes-control: add 'local_pref' attribute to BGPVPN resource https://review.openstack.org/535358 | 13:25 |
| *** bfernando has joined #openstack-net-bgpvpn | 13:32 | |
| *** bobmel has joined #openstack-net-bgpvpn | 13:40 | |
| doude | tmorin yo | 13:50 |
| tmorin | hi doude :) | 13:51 |
| doude | I just show your last comment on CLI RC patch | 13:51 |
| *** vthapar has joined #openstack-net-bgpvpn | 13:51 | |
| tmorin | yep | 13:51 |
| doude | I think you don't use last version of the patch | 13:51 |
| tmorin | I really don't get what can be happening, but I haven't digged | 13:51 |
| doude | and the project ID is not shown if you are not admin | 13:51 |
| tmorin | errm... yes, you are correct | 13:51 |
| doude | I mean is not return by the neutron APi | 13:51 |
| doude | so I think the first 2 output you past was done as an admin | 13:52 |
| doude | and last as user | 13:52 |
| doude | and with the last version of the patch, you will see the project ID two times when you are admin | 13:52 |
| tmorin | yes | 13:54 |
| tmorin | (except both of my CLIs were done as non-admin) | 13:55 |
| tmorin | I just gave my +1 again | 13:55 |
| doude | ok | 13:55 |
| doude | thx | 13:55 |
| tmorin | sorry for the confusion... | 13:55 |
| doude | no problem | 13:56 |
| doude | you prefer I create a new patch/bug for the type in thedoc script conf.py? | 13:56 |
| doude | tmorin --^ | 13:57 |
| tmorin | really as you prefer | 13:58 |
| tmorin | this is "only documentation" | 13:58 |
| tmorin | it's unlikely I would say that we would need to backport the fix alone ... | 13:58 |
| doude | ok I'll separate it | 13:58 |
| doude | sure understood | 13:58 |
| tmorin | I was going to say that I hadn't a strong opinion | 13:59 |
| tmorin | but doing a separate change is always better in terms of practices... | 13:59 |
| doude | I have to comply with the good practice | 13:59 |
| doude | :) | 13:59 |
| doude | tmorin ping | 14:59 |
| tmorin | yep ? | 14:59 |
| doude | question about requirements | 14:59 |
| doude | If I added a dep in bgpvpn but which already in neutron, should I add it in bgpvpn requirements? | 15:00 |
| doude | I ask for debtcollector | 15:00 |
| tmorin | yes, "good practice" | 15:00 |
| doude | again :) | 15:00 |
| tmorin | "what if neutron stops using it" | 15:00 |
| doude | I add it with the same version restriction as Neutron? | 15:01 |
| tmorin | very theoretical... | 15:01 |
| doude | debtcollector>=1.2.0 # Apache-2.0 | 15:01 |
| tmorin | all projects have to use the same dependency line for dependencies declared in global-requirements.txt | 15:02 |
| doude | k | 15:02 |
| tmorin | which is the case for debtcollector (I just checked) | 15:02 |
| tmorin | so yes | 15:02 |
| doude | thx | 15:02 |
| openstackgerrit | Édouard Thuleau proposed openstack/networking-bgpvpn master: Fix small typo in docs configuration file https://review.openstack.org/535794 | 15:09 |
| openstackgerrit | Édouard Thuleau proposed openstack/networking-bgpvpn master: Deprecates old OpenContrail driver https://review.openstack.org/535277 | 15:13 |
| *** matrohon has joined #openstack-net-bgpvpn | 15:37 | |
| tmorin | doude: this one is small and ready for a review: https://review.openstack.org/#/c/535358 | 16:35 |
| openstackgerrit | Cedric Savignan proposed openstack/networking-bgpvpn master: Add Heat support for Port Associations https://review.openstack.org/535833 | 16:36 |
| *** bfernando has quit IRC | 16:42 | |
| doude | tmorin BigInteger ;) | 16:56 |
| *** tmorin has quit IRC | 17:00 | |
| *** matrohon has quit IRC | 17:14 | |
| openstackgerrit | Merged openstack/networking-bgpvpn master: Fix small typo in docs configuration file https://review.openstack.org/535794 | 17:46 |
| *** vthapar has quit IRC | 18:03 | |
| *** bobmel_ has joined #openstack-net-bgpvpn | 18:49 | |
| *** bobmel has quit IRC | 18:51 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: tempest: enable test_bgpvpn_port_association_create_and_update https://review.openstack.org/535225 | 19:45 |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: db: refresh port association db object after route update https://review.openstack.org/535913 | 19:45 |
| *** tmorin has joined #openstack-net-bgpvpn | 19:48 | |
| *** tmorin has quit IRC | 20:04 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/networking-bgpvpn master: DNM testing whether lib/neutron switch breaks this repo https://review.openstack.org/535930 | 20:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!