Skip to content
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

runtime: relax timeout for CreateVM + BootVM in CLH #268

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

Sumynwa
Copy link

@Sumynwa Sumynwa commented Dec 11, 2024

Merge Checklist
  • Followed patch format from upstream recommendation: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format
    • Included a single commit in a given PR - at least unless there are related commits and each makes sense as a change on its own.
  • Aware about the PR to be merged using "create a merge commit" rather than "squash and merge" (or similar)
  • The upstream/missing label (or upstream/not-needed) has been set on the PR.
Summary

This commit introduces changes merged in upstream PR 9153 of relaxing the timeout for calling CLH's CreateVM+BootVM APIs. Further, the commit increases the timeout to 100s to handle guest boot with large memory requests.

Test Methodology

This commit introduces changes merged in upstream PR 9153
of relaxing the timeout for calling CLH's CreateVM+BootVM
APIs. Further, the commit increases the timeout to 100s to
handle guest boot with large memory requests.

Signed-off-by: Sumedh Alok Sharma <[email protected]>
@Sumynwa Sumynwa requested review from a team as code owners December 11, 2024 05:55
@Sumynwa Sumynwa merged commit 24cb087 into msft-main Dec 12, 2024
48 of 106 checks passed
@Sumynwa Sumynwa deleted the sumsharma/min_timeout branch December 12, 2024 05:06
sprt pushed a commit that referenced this pull request Feb 10, 2025
runtime: relax timeout for CreateVM + BootVM in CLH
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants