-
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
Add a documentation for warmed containers configurations. #5282
Conversation
9c9aacb
to
0d6061f
Compare
Codecov Report
@@ Coverage Diff @@
## master #5282 +/- ##
==========================================
- Coverage 80.08% 75.35% -4.74%
==========================================
Files 238 238
Lines 14122 14122
Branches 589 589
==========================================
- Hits 11310 10642 -668
- Misses 2812 3480 +668
Continue to review full report at Codecov.
|
|
||
## Configuration | ||
|
||
The configurations are only effective with the FPC scheduler. |
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.
this isn't totally true since some of the configurations already existed right?
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.
warmedContainerKeepingCount
and warmedContainerKeepingTimeout
are only effective in FPContainerProxy
.
allowedKinds: Option[Set[String]] = None, | ||
storeActivations: Option[Boolean] = None, | ||
warmedContainerKeepingCount: Option[Int] = None, | ||
warmedContainerKeepingTimeout: Option[String] = None) |
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.
This overrides the system wide idle timeout? I didn't even know this existed
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.
Yes, so do I. So I added them.
LGTM |
Description
This is to add documentation about the configurations for warmed containers.
Related issue and scope
My changes affect the following components
Types of changes
Checklist: