-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Scheduler Enhancement] Increase the retention timeout for the blackbox action. #5266
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5266 +/- ##
===========================================
- Coverage 80.09% 65.71% -14.38%
===========================================
Files 238 238
Lines 14080 14122 +42
Branches 576 589 +13
===========================================
- Hits 11277 9280 -1997
- Misses 2803 4842 +2039
Continue to review full report at Codecov.
|
stay using data.copy(activeDuringFlush = false) | ||
else | ||
cleanUpActorsAndGotoRemoved(data) | ||
|
||
case Event(GracefulShutdown, data: FlushingData) => | ||
completeAllActivations(data.reason, ContainerCreationError.whiskErrors.contains(data.error)) | ||
completeAllActivations(data.reason, isWhiskError(data.error)) |
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.
Our downstream didn't have this statement
completeAllActivations(data.reason, isWhiskError(data.error))
So, just add completeAllActivations
, is it for safe?
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.
Nice catch. I added the case back.
LGTM |
…ox action. (apache#5266) * Increase the retention timeout for the blackbox action. * Fix test cases. * Apply scalaFmt. * Add GracefulShutdown case back. * Increase the blackbox timeout for test cases. * Access the private method directly. * Replace Thread.sleep with awaitAssert * Add the missing configuration. * Enhance the test code. * Remove thread.sleep. * Fix test cases. * Fix test cases.
Description
For the blackbox actions, it may take more than 20 seconds which is a default in-progress timeout, to pull an image and it makes the activation timed out.
A memory queue no longer drop black-box activations in such a case with this change.
Related issue and scope
My changes affect the following components
Types of changes
Checklist: