-
Notifications
You must be signed in to change notification settings - Fork 225
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
[YUNIKORN-2652] Add Resource Usage details to getApplication endpoint #892
Conversation
Add logic to the getApplication webservice handler to provide expanded details about an application, including resource usage, if the `details=true` query parameter is specified on the endpoint request.
Add logic to the getApplication webservice handler to provide expanded details about an application, including resource usage, if the `details=true` query parameter is specified on the endpoint request.
…yunikorn-core into getapplication-resource-usage
Add logic to the getApplication webservice handler to provide expanded details about an application, including resource usage, if the `details=true` query parameter is specified on the endpoint request.
…yunikorn-core into getapplication-resource-usage
Add logic to the getApplication webservice handler to provide expanded details about an application, including resource usage, if the `details=true` query parameter is specified on the endpoint request.
…yunikorn-core into getapplication-resource-usage
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #892 +/- ##
==========================================
- Coverage 78.52% 78.31% -0.22%
==========================================
Files 97 97
Lines 12206 12241 +35
==========================================
+ Hits 9585 9586 +1
- Misses 2323 2356 +33
- Partials 298 299 +1 ☔ View full report in Codecov by Sentry. |
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.
Please enhance the unit tests to ensure proper coverage.
@richscott should this PR be closed and we continue with #897 ? |
I was out on holiday for the last two weeks - this other PR (#897) looks like it would work fine, I will close this PR. Thanks. |
What is this PR for?
These changes enhance the
getApplication()
function inpkg/webservice/
to include some additional fields that may be valuable to users viewing completed applications - specifically, the response now includedStartTime
,ResourceUsage
,PreemptedResource
, andPlaceholderResource
members.What type of PR is it?
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2652
How should this be tested?
Any request to the endpoint(s) that use the
getApplication()
function can be queried, with an additional boolean query parameter ofdetails
set to true, e.g.