-
Notifications
You must be signed in to change notification settings - Fork 7
Release r2.1 with NSB v0.2.0-rc.1 and NSA v0.1.0-rc.1 #93
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
base: main
Are you sure you want to change the base?
Conversation
tanjadegroot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi all,
quite some comments on the new API
I recommend to
- change this to a DRAFT Release PR, do the changes on main in normal PRs.
- wait for the Release PRs of Commonalities and ICM to be available and do the relevant adjustments (if any)
- then re-open and resynch the Release PR only after all required changes are done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r2.1 needs to be added to the table of contents at the top of the file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the Release information section
- the links need to be updater to r2.1
- the links for the new NSA API need to be added.
| As a specific rule, error `501 - NOT_IMPLEMENTED` can be only a possible error response if it is explicitly documented in the API. | ||
| version: wip | ||
| version: v0.1.0-rc.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| version: v0.1.0-rc.1 | |
| version: 0.1.0-rc.1 |
| version: wip | ||
| version: v0.1.0-rc.1 | ||
| x-camara-commonalities: 0.6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spring26 APIs need to release against Commonalities 0.7 (and ICM 0.5), not 0.6. (0.4)
To align with Spring26 Commonalities and ICM the release PRs need to be available, but there is a small delay.
Alternatively you can check the scope issue of Commonalities and ICM to see which updates might apply to your API.
See
| As a specific rule, error `501 - NOT_IMPLEMENTED` can be only a possible error response if it is explicitly documented in the API. | ||
| version: wip | ||
| version: v0.2.0-rc.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| version: v0.2.0-rc.1 | |
| version: 0.2.0-rc.1 |
| @@ -1,4 +1,4 @@ | |||
| Feature: CAMARA Network Slice Booking API vwip - Operation deleteSlice | |||
| Feature: CAMARA Network Slice Booking API v0.2.0-rc.1 - Operation deleteSlice | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not the same operation(Id) as in the filename
| * It provides the API consumer with the ability to: | ||
| * reserve, dynamically provisioning, query, dynamically delete a slice with customized SLA assurance capabilities, customized service duration, expected slice covered locations. | ||
| * reserve, dynamically provisioning, query, dynamically delete a slice with customized SLA assurance capabilities, customized service duration, expected slice covered locations. | ||
| * manage user subscription to the network slices. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| * manage user subscription to the network slices. | |
| * manage user subscription by assigning their devices to the network slices. |
| | 10 | API release numbering convention applied | M | M | M | M | Y | | | ||
| | 11 | Change log updated | M | M | M | M | Y | [link](/CHANGELOG.md) | | ||
| | 12 | Previous public release was certified | O | O | O | M | N | | | ||
| | 13 | API description (for marketing) | O | O | M | M | Y | [Wiki link]( https://lf-camaraproject.atlassian.net/wiki/spaces/CAM/pages/81166982/NetworkSliceBooking+API+description) | | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the Wiki page is not yet updated to include the assignment API (or a dedicated file could be done).
either put "N" in the checklist and update later, or create/update it before release.
| | 3 | Guidelines from ICM applied | O | M | M | M | Y | [r3.4](https://github.com/camaraproject/IdentityAndConsentManagement/releases/tag/r3.4) | | ||
| | 4 | API versioning convention applied | M | M | M | M | Y | v0.1.0-rc.1 | | ||
| | 5 | API documentation | M | M | M | M | Y | Embed documentation into API spec - [link](/code/API_definitions/network-slice-assignment.yaml) | | ||
| | 6 | User stories | O | O | O | M | Y | [UserStory for Network Slicing Booking](/documentation/API_documentation/NSA_User_Story.md) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| | 6 | User stories | O | O | O | M | Y | [UserStory for Network Slicing Booking](/documentation/API_documentation/NSA_User_Story.md) | | |
| | 6 | User stories | O | O | O | M | Y | [UserStory for Network Slice Assignement](/documentation/API_documentation/NSA_User_Story.md) | |
| # API Readiness Checklist | ||
|
|
||
| Checklist for network-slicing-book v0.1.0 in r1.2 | ||
| Checklist for network-slicing-book v0.2.0-rc.1 in r2.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Checklist for network-slicing-book v0.2.0-rc.1 in r2.1 | |
| Checklist for network-slicing-booking v0.2.0-rc.1 in r2.1 |
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Release r2.1 with network-slice-booking v0.2.0-rc.1 and network-slice-assignment v0.1.0-rc.1
Which issue(s) this PR fixes:
Fixes #
Special notes for reviewers:
Changelog input