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

Update slice modeling for 1.6 #898

Merged
merged 1 commit into from
Sep 16, 2021

Conversation

scottmbaker
Copy link

Somewhat preliminary Aether-4.x modeling, based on what was discussed in the OPs/Core/Fabric sync-up.

CHANGELOG.md contains a summary of the differences.

config-models/aether-4.x/CHANGELOG.md Show resolved Hide resolved
config-models/aether-4.x/files/yang/application.yang Outdated Show resolved Hide resolved
to match is returned. An implicit DENY ALL lies at the end.";
}

leaf upf {
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm hoping upf will be auto-configured and we will be able to drop this and the upf.yang

Copy link
Author

Choose a reason for hiding this comment

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

I think we still need to store it somewhere, even if it is autoconfigured (unless we have a convention where the UPF is named in a such a specific way, that there's no need to store the mapping). Also, we discussed last week the possibility of needing to preallocate a "pool" of UPFs that the user can associate with slices as necessary.

@scottmbaker scottmbaker force-pushed the feature/aether-4.0 branch 3 times, most recently from 7bfc177 to ad40177 Compare September 14, 2021 23:11
Copy link
Contributor

@SeanCondon SeanCondon left a comment

Choose a reason for hiding this comment

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

Looks good

Also this might be a good opportunity to tackle
https://jira.opennetworking.org/browse/AETHER-2203

It would require addiing onf- at the front of the YANG file names. See issue for more details

config-models/aether-4.x/files/aether-4.0.0.tree Outdated Show resolved Hide resolved
config-models/aether-4.x/files/yang/site.yang Outdated Show resolved Hide resolved
config-models/aether-4.x/files/yang/application.yang Outdated Show resolved Hide resolved
@scottmbaker scottmbaker force-pushed the feature/aether-4.0 branch 3 times, most recently from f469eee to f630a20 Compare September 15, 2021 23:32
Copy link
Contributor

@SeanCondon SeanCondon left a comment

Choose a reason for hiding this comment

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

LGTM

@SeanCondon SeanCondon merged commit 09ae39c into onosproject:master Sep 16, 2021
@SeanCondon SeanCondon deleted the feature/aether-4.0 branch September 16, 2021 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants