Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@cj-chung
Copy link
Contributor

This design document includes concept, data schemas, APIs, and detailed workflows design for the management of routers, routing tables and routing rules in Alcor for both Neutron and VPC scenarios.

@cj-chung cj-chung self-assigned this Sep 10, 2020
@xieus xieus added the documentation Improvements or additions to documentation label Sep 10, 2020
@xieus xieus added this to the Version 0.9.2020.09.30 milestone Sep 10, 2020
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look great in general. One comment is to continue on designing the Subnet Mgr path, and keep APIs up to update.

== Overview

The Route Manager is an Alcor microservice which manages the route tables and route entries for all VPCs and Subnets.
It also provides router-related functionalities to compatible with OpenStack Neutron's router APIs.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor: to be compatible with

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

. Router resource managed by an owner, Neutron or VPC.
. Route table resource managed by an owner, VPC or Subnet.
. Route entry should include destination, nexthop, target, and priority.
. Warning message should be raised if same route entry with different priority has been inserted by users.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more requirement:

  • Warning message should be raised when adding an invalid routing rule

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

** API Gateway
//*** xref:_api_gateway.adoc[API Gateway]
** xref:mgmt_services/mgmt_services_overview.adoc[Resource Management Services]
*** xref:mgmt_services/vpc_manager.adoc[VPC Manager]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will need a new route_manager_l3_neighbors_update_5.PNG to remove the grammar check warning.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PNG is recaptured without grammar check warning.

@xieus xieus self-requested a review September 11, 2020 21:25
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @cj-chung for putting all the design details together.

@xieus xieus merged commit 677b536 into futurewei-cloud:master Sep 11, 2020
@cj-chung cj-chung deleted the cj-alcor branch December 2, 2020 18:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants