[RFE] Creating a new stadium project for BGP Dynamic Routing effort

[vikram.choudhary]

This bug is logged for getting driver's team opinion on creating a new stadium project for BGP dynamic routing project.

[Armando Migliaccio (armando-migliaccio)]

I think we all agreed that this was the natural step forward.

I wonder if we should call the repo neutron-l3 rather than neutron-bgp, in case we expand the scope of the repo beyond bgp based dynamic routing?

[Ihar Hrachyshka (ihar-hrachyshka)]

I don't think we should be afraid of having more repos in case of wider scope. Neither we should be afraid of renaming repos in case new naming fits better.

[Armando Migliaccio (armando-migliaccio)]

renaming is a nightmare...I am not sure if you were born when we went from Quantum to Neutron, but that was a tad disruptive :)

[Armando Migliaccio (armando-migliaccio)]

Status update:

http://eavesdrop.openstack.org/meetings/neutron_drivers/2016/neutron_drivers.2016-03-24-22.01.log.html#l-215

[Armando Migliaccio (armando-migliaccio)]

Let's make sure we're ok with the scope/name of the new repo, but other than that, you're blessed.

[Armando Migliaccio (armando-migliaccio)]

Just to be clear: creating a repo is simply the first step. For this to be marked complete you'd have to arrange the project to tackle all aspects of an OpenStack project: docs, testing, release, upgrades etc.

[vikram.choudhary (vikschw)]

@armax: Thanks for the approval. Sure we will try our best in making this complete from all the angles.

[Steve Ruan (ruansx)]

@armax: About the repo name, I suggest to use networking-l3 or networking-routing. Because:
1. current most stadium project named with a prefix "networking-"

This repo is planed to run bgp, for future, I hope it can also includes other feature like static routing, policy based routing. So, this repo will help neutron:
1. choose the best route, and validate of nexthop
2. advertise/learn routes to/from its peer.

[Steve Ruan (ruansx)]

@armax: About the repo name, I suggest to use networking-l3 or networking-routing. Because current most stadium project named with a prefix "networking-"

This repo is planed to run bgp, for future, I hope it can also includes other feature like static routing, policy based routing. So, this repo will help neutron:
1. choose the best route, and validate of nexthop
2. advertise/learn routes to/from its peer.

[vikram.choudhary (vikschw)]

Since the functionality is much dependent on the neutron, I would suggest to use neutron-xxx instead of networking-xxx (Just like other X-aaS services). Moreover, I feel we can go with neutron-routing / neutron-dynamic-routing / neutron-bgp / RoutingaaS. This one we can discuss over the proposed review link "https://review.openstack.org/#/c/268726".

For the time being I will go with 'neutron-bgp' as the scope of this defect is specific to BGP.

[Armando Migliaccio (armando-migliaccio)]

@vikram: going with neutron-bgp because the scope is specific to BGP right now defeats the whole point of having the discussion before creating the repo in case the scope extends over time!

[Mickey Spiegel (emspiege)]

To address armax's comment and align with vikram's proposal, my vote is for neutron-dynamic-routing. This name also has the benefit of alignment with the current name of this feature.

I don't think neutron-l3 or neutron-routing sufficiently differentiates which aspect of L3 is being covered in this project, dynamic routing protocols. I don't think we want to pull any of the other L3 functionality out of neutron, so we should be careful to choose a name that implies the appropriate scope of the project.

[Na Zhu (nazhu)]

I think neutron-dynamic-routing is reasonable, it is suit for all the dynamic routing protocols.
If ospf or other dynamic routing protocols are added in future, we can also use this name.

[vikram.choudhary (vikschw)]

@armax: Thanks for the clarification. I understand your point. We have updated our proposal with 'neutron-dynamic-routing'. Plz let us know if it's fine.

[vikram.choudhary (vikschw)]

New repository: https://github.com/openstack/neutron-dynamic-routing

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/309326

[OpenStack Infra (hudson-openstack)]

Change abandoned by vikram.choudhary (<email address hidden>) on branch: master
Review: https://review.openstack.org/309343
Reason: Will combine the change with https://review.openstack.org/#/c/309326, cannot resolve the jerkin failures here ;(

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/309326
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=4ba80f3f1c21669f391118be120f6a41f1186e22
Submitter: Jenkins
Branch: master

commit 4ba80f3f1c21669f391118be120f6a41f1186e22
Author: vikram.choudhary <email address hidden>
Date: Fri Apr 22 20:26:18 2016 +0530

    Prepare initial sandbox for neutron-dynamic-routing

    This patch-set prepares the basic code structure with all
    the tools required for running static and unit tests.

    Note: All the imported code from the seed repo is being removed
          temporarily and will be added after the required re-factoring
          needed as per the new repo in the subsequent patch-sets.

    Co-Authored-By: Ryan Tidwell <email address hidden>
    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

    Change-Id: I9bff3d916279c4f335b309e7a2c2e943ac6f6cde

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/310747

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/310749

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/311047

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/312038

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/312324

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/310735
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=cf6cf44e4ba589ddf85749bca8ec1a2ec9df7243
Submitter: Jenkins
Branch: master

commit cf6cf44e4ba589ddf85749bca8ec1a2ec9df7243
Author: vikram.choudhary <email address hidden>
Date: Thu Apr 28 16:53:25 2016 +0530

    Documentation

    This patch restructures existing documents for better user
    experience.

    Below changes are done as part of this patch-set:
     - Re-structured/Reworded existing documents.
     - Identified and added dummy documents for required devrefs.

    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

    Change-Id: I71d9f6107bca165d72b6f6e710ac6e0e6108fcf9

[OpenStack Infra (hudson-openstack)]

Change abandoned by Na Zhu (<email address hidden>) on branch: master
Review: https://review.openstack.org/311047
Reason: Ryan will submit a patch includes all the BGP code.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/310749
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=61a933f105cad0a764417e1cf9cee9166df8926b
Submitter: Jenkins
Branch: master

commit 61a933f105cad0a764417e1cf9cee9166df8926b
Author: vikram.choudhary <email address hidden>
Date: Thu Apr 28 17:10:36 2016 +0530

    Added alembic DB migration framework

    This patch adds alembic DB migration framework required by
    neutron-dynamic-routing repository for defining new data
    models to realize BGP dynamic routing functionality in neutron.

    Below changes are done as part of this patch-set:
     - Added new alembic migration framework.
     - PEP8 changes for alembic migration check.

    Change-Id: I395ca9541bceda40bbc0c0870e58c18231ba44e9
    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/316804

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/310747
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=d0941d4bc2b29ed6deaa28a0463669110b6ab1e5
Submitter: Jenkins
Branch: master

commit d0941d4bc2b29ed6deaa28a0463669110b6ab1e5
Author: vikram.choudhary <email address hidden>
Date: Fri May 6 15:24:09 2016 +0530

    Added OSLO config framework

    This patch adds OSLO config framework required by
    neutron-dynamic-routing repository for defining configuration
    parameters.

    Below changes are done as part of this patch-set:
     - Added OSLO config framework.
     - PEP8 changes for genconfig check.

    Change-Id: Id3db0e9c5943ce913f5b2cca8d05a9ade37b3f68
    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/312324
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=270b37f22b03469a2d5087e584864d06737bfc6c
Submitter: Jenkins
Branch: master

commit 270b37f22b03469a2d5087e584864d06737bfc6c
Author: Ryan Tidwell <email address hidden>
Date: Tue Apr 26 09:56:44 2016 -0700

    Move BGP service plugin, agent, and tests out of Neutron repo

    This patch moves the BGP service plugin, agent, driver, and
    tests out of the neutron repository and into the
    neutron-dynamic-routing repository.

    Partially-Implements: blueprint bgp-spinout
    Partial-Bug: #1560003
    Co-Authored-By: vikram.choudhary <email address hidden>

    Change-Id: I80ea28a51d7b18e67d6ed4cd2da22520f950300f

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/317844

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/312038
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=fb4c68d6203c4ba0e6e3dc5b8309ed98ddd0057e
Submitter: Jenkins
Branch: master

commit fb4c68d6203c4ba0e6e3dc5b8309ed98ddd0057e
Author: steve.ruan <email address hidden>
Date: Tue May 3 07:26:15 2016 -0500

    Add devstack for neutron-dynamic-routing stadium project

    Change-Id: I406b276f691e68adac935a4a2119efc055df31b1
    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/319067

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/319067
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=e0b214a7d43c6e2c22fc5f182ec24b3192b64fde
Submitter: Jenkins
Branch: master

commit e0b214a7d43c6e2c22fc5f182ec24b3192b64fde
Author: steve.ruan <email address hidden>
Date: Thu May 19 22:55:32 2016 -0700

    Add "neutron-bgp-dragent" to setup.cfg

    neutron-bgp-dragent will be added to "/usr/local/bin" by install
    script automatically when it's defined in entry_points section

    Change-Id: I7592de868602db8f0c09756246a044cbe73d4800
    Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/316804
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=8304964d4f5760781f9c17c2c399d7452b233687
Submitter: Jenkins
Branch: master

commit 8304964d4f5760781f9c17c2c399d7452b233687
Author: Na <email address hidden>
Date: Mon May 16 05:45:06 2016 -0700

    Add system design for neutron dynamic routing

    This patch add document about neutron dynamic routing
    system design.

    Change-Id: Idf6fa7bb9202f1f40ae669a9980f624f2c855294
    Partially-Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/317844
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=6841dfae3a0aabdfb0a9559942a37bc75ceab23b
Submitter: Jenkins
Branch: master

commit 6841dfae3a0aabdfb0a9559942a37bc75ceab23b
Author: steve.ruan <email address hidden>
Date: Wed May 18 00:36:53 2016 -0500

    Add neutron-dynamic-routing DrAgent & Scheduler document

    This document introduces how DrAgent and Scheduler works,
    and how to use it.

    Change-Id: Ib44426868cdc326ef4c92cc310a4f11409581f8d
    Partially-Implements: blueprint bgp-spinout
    Partial-Bug: #1560003

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/355753

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/351964
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=a2ebd8704baa2f23846db4a01019187d06504000
Submitter: Jenkins
Branch: master

commit a2ebd8704baa2f23846db4a01019187d06504000
Author: fumihiko kakuma <email address hidden>
Date: Thu Aug 4 13:03:33 2016 +0900

    Allow to run functional tests

    This change allows to run functional tests adding gate_hook.sh and
    post_test_hook.sh scripts.

    Partial-Bug: #1560003
    Change-Id: I1d34e4f3eaf2ea8de9aab070513c6590e8e785dd

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/313503
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=294b615702b4edd8dd5e03b1d918675e7543833e
Submitter: Jenkins
Branch: master

commit 294b615702b4edd8dd5e03b1d918675e7543833e
Author: vikram.choudhary <email address hidden>
Date: Tue Aug 16 13:31:53 2016 +0530

    Fixed "tox -e py27" warning message

    tox shows warning message as follow.

    py27 runtests: PYTHONHASHSEED='3460558810'
    py27 runtests: commands[0] | find . -type f -name *.py[c|o] -delete
    WARNING:test command found but not installed in testenv
      cmd: /usr/bin/find
      env: /home/vikram/myData/work/openstack/neutron-dynamic-routing/.tox/py27
    Maybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting.
    py27 runtests: commands[1] | find . -type d -name __pycache__ -delete
    WARNING:test command found but not installed in testenv
      cmd: /usr/bin/find
      env: /home/vikram/myData/work/openstack/neutron-dynamic-routing/.tox/py27
    Maybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting.
    py27 runtests: commands[2] | sh tools/pretty_tox.sh

    It seems that "find" command needs full path in tox.ini

    Partial-Bug: #1560003
    Change-Id: Iafd0ea9fc6a79859313c004845003c7e71c2c216

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/355753
Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=29882d27fa5f789dfae23f086d56ca5d2f8e5a07
Submitter: Jenkins
Branch: master

commit 29882d27fa5f789dfae23f086d56ca5d2f8e5a07
Author: vikram.choudhary <email address hidden>
Date: Tue Aug 16 12:25:46 2016 +0530

    Removing Alembic migration devref document

    This patch removes unwanted devref documents w.r.t. alembic migration.
    IMO, separate devref for sub-project is not required as it's already
    documented @
    http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html#independent-sub-project-tables

    Change-Id: Ida41846ff5d1be0058fd6881cceaa834c37268d7
    Partial-Bug: #1560003