-
Notifications
You must be signed in to change notification settings - Fork 3
OptimalEdgeDiscovery r1.2 Fall25 M4 #21
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
Changes from all commits
60cd86b
bee7080
e14d3b8
8c67ffa
3fd19ad
a1403ba
cc12157
0fe8c3e
e360966
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,11 +1,16 @@ | ||||||||||||||||
| @Optimal_Edge_Discovery | ||||||||||||||||
| Feature: CAMARA Optimal Edge Discovery API, vwip - Operations for discovering optimal edge cloud zones | ||||||||||||||||
| Feature: CAMARA Optimal Edge Discovery API, v0.1.0 - Operations for discovering optimal edge cloud zones | ||||||||||||||||
|
|
||||||||||||||||
| # Input to be provided by the implementation to the tests | ||||||||||||||||
| # | ||||||||||||||||
| # Implementation indications: | ||||||||||||||||
| # * apiRoot: API root of the server URL | ||||||||||||||||
| # | ||||||||||||||||
| # References to OAS spec schemas refer to schemas specified in optimal-edge-discovery.yaml | ||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you @Kevsy for the comments . I have updated optimal-edge-discovery.feature |
||||||||||||||||
|
|
||||||||||||||||
| Background: Common Optimal Edge Discovery setup | ||||||||||||||||
| Given the resource "{apiroot}/optimal-edge-discovery/vwip" as base-url | ||||||||||||||||
| Given an environment at "apiRoot" | ||||||||||||||||
| And the resource "/optimal-edge-discovery/v0.1" | ||||||||||||||||
| And the header "Content-Type" is set to "application/json" | ||||||||||||||||
| And the header "Authorization" is set to a valid access token | ||||||||||||||||
| And the header "x-correlator" complies with the schema at "#/components/schemas/XCorrelator" | ||||||||||||||||
|
|
||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You need to confirm that the API description is up-to-date across README, API yaml info.description, CAMARA website and API description in wiki.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @tanjadegroot made updates to wiki page to elaborate more and added/updated use cases and benefits.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @tanjadegroot Could you please let me know if we are all good with this. and if we have the approval to merge the code and create the public release.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
@maheshc01 yes that understanding is correct. The website is updated by the CAMARA marketing team based on the text in the Wiki in your API description page (link in the API Readiness Checklist. The actual update needs to be confirmed in the corresponding issue #22 (which was done) - thanks !
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
@maheshc01 All done - you can proceed with the merge and release :-) thanks ! |
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.
All the error response tests could also include the line
as well as
could be done for the next version.