)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7776,"name":"Paul Carver","email":"pcarver@att.com","username":"pcarver"},"change_message_id":"0b8aa827a44202e90d98015089026b2717c76fa7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"networking-bagpipe: run the networking bgpvpn bagpipe tempest job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Context:"},{"line_number":10,"context_line":"- networking-bagpipe has the fundation code for the reference driver"},{"line_number":11,"context_line":"of the networking-bgpvpn project"},{"line_number":12,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":13,"context_line":"and use with bagpipe driver for bgpvpn"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_94b759f4","line":10,"range":{"start_line":10,"start_character":29,"end_line":10,"end_character":38},"updated":"2016-10-13 11:53:36.000000000","message":"nit: fundation -\u003e foundation","commit_id":"59c4ac1b6107c44be7f9477882187d8113b45c02"},{"author":{"_account_id":7776,"name":"Paul Carver","email":"pcarver@att.com","username":"pcarver"},"change_message_id":"eaf9d380551a24bd515c74a411c750c2783d2cd6","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Additionally, this change adds the networking-bgpvpn tempest job"},{"line_number":14,"context_line":"as a test for the networking-bagpipe project, which is is motivated"},{"line_number":15,"context_line":"by the facts that:"},{"line_number":16,"context_line":"- networking-bagpipe has the fundation code for the reference driver"},{"line_number":17,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":18,"context_line":"use of networking-bagpipe)"},{"line_number":19,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa6399be_3cc8c454","line":16,"range":{"start_line":16,"start_character":29,"end_line":16,"end_character":38},"updated":"2016-10-17 12:31:58.000000000","message":"Looks like you fixed this misspelling in PS2 and then re-misspelled it in PS3.","commit_id":"18c455d610b5a2ea0307bdbb8758af4a9d9ce588"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"84bc0e721d2e208707566ff257e52125fc03a532","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Additionally, this change adds the networking-bgpvpn tempest job"},{"line_number":14,"context_line":"as a test for the networking-bagpipe project, which is is motivated"},{"line_number":15,"context_line":"by the facts that:"},{"line_number":16,"context_line":"- networking-bagpipe has the fundation code for the reference driver"},{"line_number":17,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":18,"context_line":"use of networking-bagpipe)"},{"line_number":19,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa6399be_521c7f69","line":16,"range":{"start_line":16,"start_character":29,"end_line":16,"end_character":38},"in_reply_to":"fa6399be_3cc8c454","updated":"2016-10-17 12:58:25.000000000","message":"Thanks for spotting it.\nI think I\u0027ll stop using the online gerrit editor...","commit_id":"18c455d610b5a2ea0307bdbb8758af4a9d9ce588"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"c65504c459ebf7e480661f227cb20a8409f79c61","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa6399be_97079e1c","line":19,"updated":"2016-10-19 01:00:24.000000000","message":"can you elaborate on what other use cases are out there? Is networking-bagpipe used in any other context and not in conjunction with networking-bgpvpn?","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"9ffd3c849a5840b26560d1349478592e44f2e31f","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_72314c0b","line":19,"in_reply_to":"da6895a0_03e52936","updated":"2016-10-22 07:20:51.000000000","message":"Note that the documentation of networking-bgpvpn does not omit anything related to networking-bgpvpn use case, and that the documentation on networking-bagpipe, which covers both use case, does provide information on both.\n\nI\u0027d argue that when you need to explain something clearly, it can be a reasonable choice to keep focus and not mention each an every more or less remotely related topic. \n\nThat said, our discussion leads me to the idea that we can raise a \"risk of confusion\" warning somewhere and explain that  various solutions rely on IP VPN or E-VPN to address base virtual networking use cases independently of  BGP VPN interconnections; networking-bagpipe being just one among others by the way (I have in mind OpenContrail and Nuage Networks, as examples).","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"f97d4507d7b6fcd2d5cace792b5cfa8e8a8f8071","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_a841ba3b","line":19,"in_reply_to":"da6895a0_3ea6ccc1","updated":"2016-10-21 20:37:36.000000000","message":"Although tempting, we\u0027ve been cautious to not do that in net-bgpvpn doc, to avoid confusion between goals and means, and because it\u0027s not the place to promote a particular ML2 mech driver, even if implemented using the said technology.\n\nWe can perhaps reconsider this in the form of a warning. Somethign like: \"Caution: networking-bagpipe also provides a standalone ML2 mech_driver using E-VPN as a control plane to advertise MAV/VTEP bindings for Neutron L2 networks. This is a different use case than the one addressed in networking-bgpvpn and this mech driver is not required when bagpipe driver for networking-bgpvpn is used.\".","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"5fcc92d5bbd8f502b8fde67922fbe5fa25ed0493","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_03e52936","line":19,"in_reply_to":"da6895a0_a841ba3b","updated":"2016-10-21 21:35:32.000000000","message":"I am frankly baffled by your statement. You omit documentation to avoid confusion? Way to go! (facepalm)","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"10478be58146578fc68630758268c75c3e88a1d8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_3ea6ccc1","line":19,"in_reply_to":"da6895a0_be6821bc","updated":"2016-10-21 16:53:53.000000000","message":"That\u0027s excellent! You should make that available in the docs of the respective projects.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"58828c49f6f6c2c4a041191e43618aebbb151ac1","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_be6821bc","line":19,"in_reply_to":"da6895a0_f8b3a808","updated":"2016-10-21 14:27:21.000000000","message":"I understand the need to draw pictures revealing the whole scheme. Slide 5 of [1] can perhaps help bit.\n\nThe short answer to your question about \"why is there a bagpipe ML2 mech driver?\" is: BGP-based VPNs, in particular E-VPN, is a well suited control plane for L2 networks, so why not implement an approach leveraging it ?  With as a starting point the ML2 ovs/linuxbridge drivers, one way to look at this option is to see it like l2pop, except that the equivalent information as the one in l2pop RPCs is now carried in the BGPVPN control plane (for which we have mature tool to scaling a control plane, including pub-sub).\n\nAn even shorter answer could be: why not ? ;)\nEach ML2 mech driver implements its own way of doing things; this is one among others.\n\nI\u0027ll be happy to discuss this in length next week.\n\n[1] http://www.slideshare.net/ThomasMorin1/neutron-and-bgp-vpns-with-bagpipe","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"feffe6c6ca6175076ce993eff6ba4583c53f8732","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa6399be_a47380b3","line":19,"in_reply_to":"fa6399be_97079e1c","updated":"2016-10-19 09:10:01.000000000","message":"I\u0027ve read through the draft scorecard for networking-bagpipe and acknowledge the fact that we need to explain that more and better.\n\nThe answer to your question above is: networking-bagpipe includes an ML2 mech driver that is enough Neutron-server-side to implement Neutron networks, and relies, on the compute node, on the same code to interface with bagpipe-bgp than the one used in bagpipe driver for networking-bgpvpn driver. This ML2 driver has nothing to do with bagpipe driver for networking-bgpvpn in terms of use case.\n\nSimilarly, we are also working on a bagpipe driver for networking-sfc, which will find its more naturally, it seems, in networking-bagpipe.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"40d060d26e712f4b93163acd2619a3319f9d7700","unresolved":false,"context_lines":[{"line_number":16,"context_line":"by the facts that:"},{"line_number":17,"context_line":"- networking-bagpipe has the foundation code for the reference driver"},{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_f8b3a808","line":19,"in_reply_to":"fa6399be_a47380b3","updated":"2016-10-21 00:23:26.000000000","message":"I appreciate the explaination but without an architecture diagram I am still struggling to understand. \n\nYou say:\n\nThis ML2 driver has nothing to do with bagpipe driver for networking-bgpvpn in terms of use case.\n\nSo why it\u0027s there?\n\nUntil the system and deployment architecture involving the bgpvpn sevice plugin, and the bagppipe ML2 driver and Agent, I fear these questions will keep coming up.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"ad73eb1347ce20abc91e5361f82ea67ce1a7925c","unresolved":false,"context_lines":[{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I4828ed362d6d388c757684df8d81771e7416a890"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_9eb0b88d","line":21,"updated":"2016-10-21 16:56:09.000000000","message":"this is a job to test the integration between bgpvpn and bagpipe, but wouldn\u0027t we need something that test bagpipe in its own right? To me that\u0027s higher priority.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"9ffd3c849a5840b26560d1349478592e44f2e31f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I4828ed362d6d388c757684df8d81771e7416a890"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_dd7309b0","line":21,"in_reply_to":"da6895a0_4363a1b1","updated":"2016-10-22 07:20:51.000000000","message":"Fair enough, indeed there is no urgency for the tempest jobs;\n\nCan we still progress the other jobs ?","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"5fcc92d5bbd8f502b8fde67922fbe5fa25ed0493","unresolved":false,"context_lines":[{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I4828ed362d6d388c757684df8d81771e7416a890"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_4363a1b1","line":21,"in_reply_to":"da6895a0_880eb632","updated":"2016-10-21 21:35:32.000000000","message":"I see no urgency to let this in now that the project has been untested so far (wrt the co-gate). Ocata-1 is 3 weeks from now. You can surely keep the patches separate, but I\u0027d rather see evidence of the workstream I am asking for before letting this merge.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"f97d4507d7b6fcd2d5cace792b5cfa8e8a8f8071","unresolved":false,"context_lines":[{"line_number":18,"context_line":"of the networking-bgpvpn project (it\u0027s currently the most important"},{"line_number":19,"context_line":"use of networking-bagpipe)"},{"line_number":20,"context_line":"- the networking-bgpvpn project has a tempest job to validate deployment"},{"line_number":21,"context_line":"and use with bagpipe driver for bgpvpn"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I4828ed362d6d388c757684df8d81771e7416a890"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_880eb632","line":21,"in_reply_to":"da6895a0_9eb0b88d","updated":"2016-10-21 20:37:36.000000000","message":"Since both are useful in their own right, I would be happier if we could progress this change as is, with a commitment that we work for Ocata-1 on the addition of a tempest API job for the bagpipe ML2 setup.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"892459eaea53065a1822ec7d8b05e39e8c21787e","unresolved":false,"context_lines":[{"line_number":6978,"context_line":"          which: full"},{"line_number":6979,"context_line":"      - \u0027periodic-{name}-{python}-with-neutron-lib-master\u0027:"},{"line_number":6980,"context_line":"          name: networking-bagpipe"},{"line_number":6981,"context_line":"          python: py27"},{"line_number":6982,"context_line":""},{"line_number":6983,"context_line":"- project:"},{"line_number":6984,"context_line":"    name: networking-bgpvpn"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ba5da102_52ab4ad6","line":6981,"updated":"2016-10-29 04:33:58.000000000","message":"you ok with py27 here?","commit_id":"af4b31369214ffb43def55429941ff484775f639"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"65f5c5b65c180a646e50f499cdf41b956853cf00","unresolved":false,"context_lines":[{"line_number":6978,"context_line":"          which: full"},{"line_number":6979,"context_line":"      - \u0027periodic-{name}-{python}-with-neutron-lib-master\u0027:"},{"line_number":6980,"context_line":"          name: networking-bagpipe"},{"line_number":6981,"context_line":"          python: py27"},{"line_number":6982,"context_line":""},{"line_number":6983,"context_line":"- project:"},{"line_number":6984,"context_line":"    name: networking-bgpvpn"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ba5da102_6b7d946d","line":6981,"in_reply_to":"ba5da102_52ab4ad6","updated":"2016-11-01 08:32:31.000000000","message":"the idea is to switch to a py35 job at the same time as we switch the python35 jobs from \u0027experimental\u0027 to \u0027check\u0027.","commit_id":"af4b31369214ffb43def55429941ff484775f639"}],"zuul/layout.yaml":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"c65504c459ebf7e480661f227cb20a8409f79c61","unresolved":false,"context_lines":[{"line_number":11327,"context_line":"    periodic:"},{"line_number":11328,"context_line":"      - periodic-networking-bagpipe-py27-with-neutron-lib-master"},{"line_number":11329,"context_line":"    experimental:"},{"line_number":11330,"context_line":"      - gate-tempest-dsvm-networking-bgpvpn-bagpipe"},{"line_number":11331,"context_line":"      - networking-bagpipe-coverage-db-ubuntu-xenial"},{"line_number":11332,"context_line":"      - gate-networking-bagpipe-python35-db"},{"line_number":11333,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fa6399be_d72af68f","line":11330,"updated":"2016-10-19 01:00:24.000000000","message":"To the untrained eye this job on both projects (bgpvpn and bagpipe) seem like double the effort for no apparent gain. I\u0027d like to understand in more detail why bagpipe and bgpvpn are two separate projects. They are pretty intimate after all, and once I got the chance to spend some time digging into them I can\u0027t see why we\u0027d carry forward the way we have.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"f97d4507d7b6fcd2d5cace792b5cfa8e8a8f8071","unresolved":false,"context_lines":[{"line_number":11327,"context_line":"    periodic:"},{"line_number":11328,"context_line":"      - periodic-networking-bagpipe-py27-with-neutron-lib-master"},{"line_number":11329,"context_line":"    experimental:"},{"line_number":11330,"context_line":"      - gate-tempest-dsvm-networking-bgpvpn-bagpipe"},{"line_number":11331,"context_line":"      - networking-bagpipe-coverage-db-ubuntu-xenial"},{"line_number":11332,"context_line":"      - gate-networking-bagpipe-python35-db"},{"line_number":11333,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"da6895a0_cd09a033","line":11330,"in_reply_to":"da6895a0_fee2949e","updated":"2016-10-21 20:37:36.000000000","message":"Happy to hear that your perspective is clearer now.\n\nI fully agree that we need more test on the ML2 mech driver.\nAs said elsewhere, our plan is to put the focus on fullstack testing to exercise the driver backend.\n\nWe can definitely work on adding tempest API testing for the ML2 driver.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"10478be58146578fc68630758268c75c3e88a1d8","unresolved":false,"context_lines":[{"line_number":11327,"context_line":"    periodic:"},{"line_number":11328,"context_line":"      - periodic-networking-bagpipe-py27-with-neutron-lib-master"},{"line_number":11329,"context_line":"    experimental:"},{"line_number":11330,"context_line":"      - gate-tempest-dsvm-networking-bgpvpn-bagpipe"},{"line_number":11331,"context_line":"      - networking-bagpipe-coverage-db-ubuntu-xenial"},{"line_number":11332,"context_line":"      - gate-networking-bagpipe-python35-db"},{"line_number":11333,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"da6895a0_fee2949e","line":11330,"in_reply_to":"fa6399be_a4ed4090","updated":"2016-10-21 16:53:53.000000000","message":"After these rounds of discussions I think my perspective is a lot more clear and I think some of my confusion stems from how these projects are named and what they actually do.\n\nnetworking-bgpvpn-bagpipe is a CI for use case A, fine. But where is the CI for use case B,  gate-tempest-dsvm-networking-bagpipe, if you will? Namely the standard tempest suite that exercises the Neutron API when bagpipe is used as an ML2 backend? That\u0027s clearly missing.","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"feffe6c6ca6175076ce993eff6ba4583c53f8732","unresolved":false,"context_lines":[{"line_number":11327,"context_line":"    periodic:"},{"line_number":11328,"context_line":"      - periodic-networking-bagpipe-py27-with-neutron-lib-master"},{"line_number":11329,"context_line":"    experimental:"},{"line_number":11330,"context_line":"      - gate-tempest-dsvm-networking-bgpvpn-bagpipe"},{"line_number":11331,"context_line":"      - networking-bagpipe-coverage-db-ubuntu-xenial"},{"line_number":11332,"context_line":"      - gate-networking-bagpipe-python35-db"},{"line_number":11333,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fa6399be_a4ed4090","line":11330,"in_reply_to":"fa6399be_d72af68f","updated":"2016-10-19 09:10:01.000000000","message":"The answer to your question about the non-bgpvpn use cases for bagpipe is the reason why the two are separate projects.\n\nThe gain in having this job is that a regression in networking-bagpipe impacting networking-bgpvpn bagpipe driver will be catched before being merged.\n\nThat said, we know that having the bagpipe driver in two pieces in separate repos is indeed less than ideal. We\u0027ve had this issue in mind some time (e.g. [1] and [2]) but haven\u0027t yet clarified how we can resolve that.\n\nOne possibility would be to move all the code of the networking-bgpvpn bagpipe driver to networking-bagpipe. Perhaps  having the reference driver out-of-tree would be considered a problem ? I\u0027m thinking that we would then want both networking-bgpvpn changes and networking-bagpipe changes to trigger a run of a bagpipe-bgpvpn job ?\n\nI intend to write up a summary of the different things we intend to do (a bagpipe networking-sfc driver is is progress, and we are considering moving bagpipe-bgp itself from github to networking-bagpipe) and the alternatives that I see to make all this better. \n\nIn the meantime having a bgpvpn job in networking-bagpipe will be useful, but not critical, so we will follow whatever you think we should do based on the above information.\n\nI\u0027ll be happy to discuss this next week. \n\n[1] https://bugs.launchpad.net/bgpvpn/+bug/1552603\n[2] https://bugs.launchpad.net/bgpvpn/+bug/1452917\n\nPS: perhaps we could adopt n8g- as a shorthand for networking- in all these discussions..?","commit_id":"231a131762f7e0724fe7104091a4be6a624d2217"}]}