Skip to content

[release/0.9] Fix graceful termination test errors (#1687)#1695

Merged
kiashok merged 1 commit intomicrosoft:release/0.9from
kiashok:portGracefulTermTestFix
Mar 15, 2023
Merged

[release/0.9] Fix graceful termination test errors (#1687)#1695
kiashok merged 1 commit intomicrosoft:release/0.9from
kiashok:portGracefulTermTestFix

Conversation

@kiashok
Copy link
Copy Markdown
Contributor

@kiashok kiashok commented Mar 15, 2023

This PR ports graceful termination test fix from main.

  • Loosen the time interval check that measures the time taken to
    stop the container with -t command to account for cloud test delays
  • Add to check that OS version is V21H2Server since the graceful
    termination test images are based on servercore and nanoserver 2022.

(cherry picked from commit 5e3a6df)

@kiashok kiashok requested a review from a team as a code owner March 15, 2023 00:51
Copy link
Copy Markdown
Contributor

@ambarve ambarve left a comment

Choose a reason for hiding this comment

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

Looks like there are some test failures. Can you fix those before merging?

- Loosen the time interval check that measures the time taken to
stop the container with -t command to account for cloud test delays
- Add to check that OS version is V21H2Server since the graceful
termination test images are based on servercore and nanoserver 2022.

Signed-off-by: Kirtana Ashok <[email protected]>
Co-authored-by: Kirtana Ashok <[email protected]>
(cherry picked from commit 5e3a6df)
Signed-off-by: Kirtana Ashok <[email protected]>
@kiashok kiashok force-pushed the portGracefulTermTestFix branch from 26cab91 to 8aa8c7a Compare March 15, 2023 20:58
@kiashok kiashok merged commit 43468fc into microsoft:release/0.9 Mar 15, 2023
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.

2 participants