| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-bgpvpn master: Updated from global requirements https://review.openstack.org/535019 | 03:06 |
|---|---|---|
| *** Aju has quit IRC | 03:30 | |
| *** afranc has joined #openstack-net-bgpvpn | 03:30 | |
| *** vthapar has joined #openstack-net-bgpvpn | 05:34 | |
| *** tmorin has joined #openstack-net-bgpvpn | 06:28 | |
| *** tmorin has quit IRC | 06:41 | |
| *** tmorin has joined #openstack-net-bgpvpn | 07:29 | |
| openstackgerrit | Merged openstack/networking-bgpvpn master: Updated from global requirements https://review.openstack.org/535019 | 07:55 |
| openstackgerrit | Łukasz Rajewski proposed openstack/networking-bgpvpn master: Basic tempest tests for port associations https://review.openstack.org/534187 | 08:06 |
| *** matrohon has joined #openstack-net-bgpvpn | 08:07 | |
| openstackgerrit | Łukasz Rajewski proposed openstack/networking-bgpvpn master: Local_pref attr tempest test for port association https://review.openstack.org/534809 | 08:09 |
| doude | Hi tmorin, how are you doing? | 08:16 |
| doude | thanks for your review, I just updated it with your comments | 08:17 |
| *** tmorin has quit IRC | 08:17 | |
| doude | I've another question, about the deprecation/suppression of the old OpenContrail driver | 08:17 |
| *** tmorin has joined #openstack-net-bgpvpn | 08:17 | |
| doude | do you think we can still consider to remove it for Queens and backport a deprecation warning for Pike or is to late? | 08:18 |
| *** matrohon has quit IRC | 08:36 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: Utility functions for port association tests in tempest https://review.openstack.org/534186 | 08:38 |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: Basic tempest tests for port associations https://review.openstack.org/534187 | 08:38 |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: tempest: enable test_bgpvpn_port_association_create_and_update https://review.openstack.org/535225 | 08:38 |
| tmorin | hi doude | 08:38 |
| tmorin | backporting a deprecation warning would be very weird | 08:39 |
| tmorin | I don't think the stable maintainers would like that a lot | 08:39 |
| tmorin | removing it for Queens would be a bit short | 08:39 |
| tmorin | I think it's better to make a deprecation warning with a removal in Rocky | 08:40 |
| doude | ok | 08:40 |
| tmorin | but overall the one important point is to update the doc to point to the v2 driver, I think | 08:40 |
| doude | and do you think it's really necessary to keep a deprecation phase? We could directly remove it | 08:42 |
| doude | tmorin --^ | 08:43 |
| tmorin | in principle it's necessary, whether or not it is important in practice right now: I can't tell | 08:44 |
| *** bfernando has joined #openstack-net-bgpvpn | 08:45 | |
| tmorin | if we purely remove it and this causes an issue for somebody, then restoring it in stable/queens will still be an option | 08:46 |
| tmorin | but that would mean bad karma for us :) | 08:46 |
| doude | yes :) | 08:46 |
| doude | lets do that correctly | 08:46 |
| tmorin | yes, it doesn't cost much to leave this driver around, as long as the documentation points to the up-to-date driver | 08:47 |
| doude | btw, I started a patch to remove it and realized most of methods iin common.utils are used only by bgpvpn driver, should I remove it also? | 08:47 |
| *** tmorin has quit IRC | 09:10 | |
| *** tmorin has joined #openstack-net-bgpvpn | 09:10 | |
| *** tmorin has quit IRC | 09:17 | |
| *** tmorin has joined #openstack-net-bgpvpn | 09:39 | |
| *** tmorin has quit IRC | 09:55 | |
| *** tmorin has joined #openstack-net-bgpvpn | 09:58 | |
| tmorin | doude: it would be great it you could plan to review https://review.openstack.org/507173 before the freeze, next week, any change of doing so ? | 09:59 |
| doude | I'll try in the afternoon tmorin | 10:00 |
| tmorin | great, thx doude! | 10:00 |
| *** tmorin has quit IRC | 10:05 | |
| openstackgerrit | Édouard Thuleau proposed openstack/networking-bgpvpn master: Deprecates old OpenContrail driver https://review.openstack.org/535277 | 10:25 |
| *** bfernando has quit IRC | 11:21 | |
| *** bfernando has joined #openstack-net-bgpvpn | 12:34 | |
| *** tmorin has joined #openstack-net-bgpvpn | 12:52 | |
| *** tmorin has quit IRC | 13:05 | |
| *** bobmel has joined #openstack-net-bgpvpn | 13:19 | |
| doude | tmorin ping | 13:43 |
| *** tmorin has joined #openstack-net-bgpvpn | 13:52 | |
| *** tmorin has quit IRC | 14:09 | |
| *** tmorin has joined #openstack-net-bgpvpn | 14:22 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bgpvpn master: routes-control: add 'local_pref' attribute to BGPVPN resource https://review.openstack.org/535358 | 14:51 |
| *** vthapar has quit IRC | 15:52 | |
| *** tmorin has quit IRC | 16:00 | |
| *** tmorin has joined #openstack-net-bgpvpn | 16:15 | |
| *** tmorin has quit IRC | 16:33 | |
| *** tmorin has joined #openstack-net-bgpvpn | 17:07 | |
| openstackgerrit | Merged openstack/networking-bgpvpn master: Utility functions for port association tests in tempest https://review.openstack.org/534186 | 17:16 |
| openstackgerrit | Merged openstack/networking-bgpvpn master: [bgpvpn_dashboard] Fix bug when a network or router name doesn't exist https://review.openstack.org/533186 | 17:16 |
| *** matrohon has joined #openstack-net-bgpvpn | 17:24 | |
| *** tmorin has quit IRC | 17:33 | |
| *** bfernando has quit IRC | 17:59 | |
| *** tmorin has joined #openstack-net-bgpvpn | 19:49 | |
| *** bobmel has quit IRC | 20:25 | |
| *** bobmel has joined #openstack-net-bgpvpn | 20:27 | |
| *** openstackgerrit has quit IRC | 21:33 | |
| *** tmorin has quit IRC | 21:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!