Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decouple PCE from local NSA configuration. #21

Open
jmacauley opened this issue Nov 17, 2016 · 3 comments
Open

Decouple PCE from local NSA configuration. #21

jmacauley opened this issue Nov 17, 2016 · 3 comments
Assignees

Comments

@jmacauley
Copy link
Member

The PCE currently utilized the DDS "local" document API to retrieve the NSA description document associated with the local NSA instance. The NSA identifier from this context is then used to perform control plane path finding and populate path results with the next hop NSA for a reservation request.

The primary issue is that the PCE is bound to a single NSA as it can only correctly respond to path requests for the NSA running the DDS to which the PCE is bound. Any time a request comes into the PCE it uses the local NSA context from the DDS as the root of the graph for the control plane path computation. Removing the dependency on the DDS "local" document API means the PCE could use any DDS available on the network. In addition, having the client of the PCE pass in the NSA context would allow a single PCE to service path requests from any number of NSA instances.

The following changes would be required:

  1. The northbound path API would need to pass in the NSA identifier providing context for the control plane path computation.
  2. The PCE would no longer use the DDS /local API for determining the local NSA identifier, and instead use the NSA identifier passed in through the northbound API.
@jmacauley jmacauley self-assigned this Nov 17, 2016
@erikrozendaal
Copy link
Member

We can pass the NSA identifier in a new top-level field aggregatorNSA in the PathComputationRequest. Is this OK with you?

@jmacauley
Copy link
Member Author

Can we just call it nsaId since the PCE could be used in a non-aggregator context?

Thanks!

erikrozendaal added a commit to BandwidthOnDemand/nsi-safnari that referenced this issue Dec 8, 2016
@erikrozendaal
Copy link
Member

Safnari now passes its ID in the nsaId top-level field as part of the path computation request: BandwidthOnDemand/nsi-safnari@8a0ff0e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants