Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Reporting exceeded CPU limit #4851

Merged
merged 4 commits into from
Nov 6, 2019
Merged

Reporting exceeded CPU limit #4851

merged 4 commits into from
Nov 6, 2019

Conversation

kmazurek
Copy link
Contributor

@kmazurek kmazurek commented Nov 4, 2019

Part of GR-179

Implements informing the requestor about CPU budget overruns during task computation. This fact is reported in the TaskFailure message using the reason field (with value BudgetExceeded).

@kmazurek kmazurek self-assigned this Nov 4, 2019
@kmazurek kmazurek marked this pull request as ready for review November 4, 2019 16:53
@kmazurek kmazurek requested a review from Wiezzel as a code owner November 4, 2019 16:53
@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #4851 into develop will increase coverage by 1.91%.
The diff coverage is 78.26%.

@@             Coverage Diff             @@
##           develop    #4851      +/-   ##
===========================================
+ Coverage    87.44%   89.35%   +1.91%     
===========================================
  Files          227      227              
  Lines        20776    20758      -18     
===========================================
+ Hits         18167    18549     +382     
+ Misses        2609     2209     -400

@kmazurek kmazurek changed the title Reporting exceeded CPU budget Reporting exceeded CPU limit Nov 6, 2019
Copy link
Contributor

@shadeofblue shadeofblue left a comment

Choose a reason for hiding this comment

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

incredible_contribution

@kmazurek kmazurek merged commit b49093d into develop Nov 6, 2019
@kmazurek kmazurek deleted the budget-exceeded-reporting branch November 6, 2019 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants