Skip to content

Conversation

@fuweid
Copy link
Member

@fuweid fuweid commented Nov 8, 2023

Fixes: #9334

@fuweid fuweid requested a review from henry118 November 8, 2023 15:41
@fuweid
Copy link
Member Author

fuweid commented Nov 8, 2023

@henry118 I can't reproduce it in my local for release/1.6. Would you please help verify this? Thanks

Copy link
Member

@henry118 henry118 left a comment

Choose a reason for hiding this comment

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

This issue only occurs on io.containerd.runc.v1.

I cherry-picked this PR to 1.6 and ran the test case locally 100x and it looked solid. Thanks for the fix!

TEST_RUNTIME=io.containerd.runc.v1 go test -count 100 -test.root -run TestIssue9103

@henry118 henry118 added cherry-pick/1.6.x cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels Nov 8, 2023
@estesp estesp added this pull request to the merge queue Nov 8, 2023
Merged via the queue into containerd:main with commit c2b39b1 Nov 8, 2023
@fuweid fuweid added cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch and removed cherry-pick/1.6.x cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels Nov 9, 2023
@samuelkarp
Copy link
Member

Somehow my notifications got turned off and I missed this. Thank you for fixing it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TestIssue9103/should_be_stopped_status_if_init_has_been_killed flaky on release/1.6 in "Linux Integration (io.containerd.runc.v1, runc)"

4 participants