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

RD auto-assigment based on roque Suggestion #172

Merged
merged 2 commits into from Sep 17, 2020
Merged

RD auto-assigment based on roque Suggestion #172

merged 2 commits into from Sep 17, 2020

Conversation

sbarguil
Copy link
Collaborator

@sbarguil sbarguil commented Jul 2, 2020

No description provided.

@sbarguil sbarguil mentioned this pull request Jul 2, 2020
@sbarguil sbarguil requested a review from boucadair July 2, 2020 13:30
yang/IETF-04/ietf-vpn-common.yang Outdated Show resolved Hide resolved
yang/IETF-04/ietf-vpn-common.yang Show resolved Hide resolved
@boucadair
Copy link
Collaborator

boucadair commented Jul 3, 2020

@oscargdd: Please link this PR to #114 so that issue is closed automatically when this PR is merged. Thanks.

@oscargdd oscargdd linked an issue Jul 3, 2020 that may be closed by this pull request
@oscargdd
Copy link
Collaborator

oscargdd commented Jul 9, 2020

In the auto-asignment cases add a read-only leaf with the asigned value, so it can be retrieved

@oscargdd
Copy link
Collaborator

oscargdd commented Jul 9, 2020

Proposal:
choice rd-choice {
case directly-assigned {
leaf rd {
type rt-types:route-distinguisher;
}
}
case auto-assigned {
leaf auto-rd {
type empty;
}
read only leaf rd-asigned {
type rt-types:route-distinguisher;
config false;
}
}
case pool-assigned {
leaf rd-pool-name {
type string;
}
leaf rd-asigned {
type rt-types:route-distinguisher;
config false;
}
}
case no-rd {
leaf no-rd {
type empty;
}
}
}

@oscargdd oscargdd merged commit 14371d7 into master Sep 17, 2020
@sbarguil sbarguil deleted the pr162 branch April 12, 2021 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rd semantics
4 participants