- Notifications
You must be signed in to change notification settings - Fork2.9k
fix: Container with Notify=healthy respectHealthStartPeriod larger thanTimeoutStartSec.#27631
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
…han TimeoutStartSecSigned-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>
[APPROVALNOTIFIER] This PR isNOT APPROVED This pull-request has been approved by:Mujib-Ahasan The full list of commands accepted by this bot can be foundhere. Needs approval from an approver in each of these files: Approvers can indicate their approval by writing |
Uh oh!
There was an error while loading.Please reload this page.
HealthStartPeriod > 0HealthStartPeriodby sending periodicEXTEND_TIMEOUT_USECmessagesHealthStartPeriod.copyInternalfunctionhttps://github.com/containers/container-libs/blob/a3b0f19c3ea12cb02f0b11788489c52d4fb02e96/common/libimage/copier.go#L392-L421suggested by maintainers.
closes#27290
Checklist
Ensure you have completed the following checklist for your pull request to be reviewed:
commits. (
git commit -s). (If needed, usegit commit -s --amend). The author email must matchthe sign-off email address. SeeCONTRIBUTING.md
for more information.
Fixes: #00000in commit message (if applicable)make validatepr(format/lint checks)Noneif no user-facing changes)Does this PR introduce a user-facing change?