-
Notifications
You must be signed in to change notification settings - Fork 27
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
[Console API] Add basic get status API #903
Conversation
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #903 +/- ##
============================================
- Coverage 74.93% 74.54% -0.40%
- Complexity 1933 1961 +28
============================================
Files 334 343 +9
Lines 13387 13607 +220
Branches 773 786 +13
============================================
+ Hits 10032 10143 +111
- Misses 2870 2977 +107
- Partials 485 487 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
Signed-off-by: Tanner Lewis <[email protected]>
...c/main/docker/migrationConsole/console_api/console_api/apps/orchestrator/tests/test_views.py
Outdated
Show resolved
Hide resolved
...rSolution/src/main/docker/migrationConsole/console_api/console_api/apps/orchestrator/urls.py
Show resolved
Hide resolved
Signed-off-by: Tanner Lewis <[email protected]>
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.
lgtm
Description
This change introduces a simple get status API to our Console API for retrieving the status of an OSI migration. This logic currently only returns the status and statusMessage from the OSI
getPipeline
API with the anticipation to add more details like progress in future iterations.Also included is some reformatting of the response given by each of the API endpoints to keep a consistent response structure on error.
Issues Resolved
https://opensearch.atlassian.net/browse/MIGRATIONS-1945
Testing
Unit testing and manual cloud testing
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.