Skip to content

Fix TestNewBinaryIOCleanup failing with gotip#10346

Merged
estesp merged 1 commit intocontainerd:mainfrom
mauri870:hotfix/gotip-test
Jul 23, 2024
Merged

Fix TestNewBinaryIOCleanup failing with gotip#10346
estesp merged 1 commit intocontainerd:mainfrom
mauri870:hotfix/gotip-test

Conversation

@mauri870
Copy link
Contributor

This PR ignores a new pidfd file descriptor that is introduced in gotip (future 1.23) and should not be considered when detecting fd leaks.

Fixes #10345

@k8s-ci-robot
Copy link

Hi @mauri870. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@akhilerm
Copy link
Member

@mauri870 Can you sign your commits, so that the tests can proceed.

@samuelkarp samuelkarp added kind/test go Pull requests that update Go code labels Jul 19, 2024
This PR ignores a new pidfd file descriptor that is introduced in
gotip (future 1.23) and should not be considered when detecting fd leaks.

Fixes containerd#10345

Signed-off-by: Mauri de Souza Meneguzzo <[email protected]>
@mauri870 mauri870 force-pushed the hotfix/gotip-test branch from 921a8cc to f0aecaa Compare July 19, 2024 21:50
@mauri870
Copy link
Contributor Author

@mauri870 Can you sign your commits, so that the tests can proceed.

Thanks for the reminder, I just signed the commit.

@samuelkarp
Copy link
Member

/ok-to-test

@kzys kzys added this pull request to the merge queue Jul 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 23, 2024
@estesp estesp added this pull request to the merge queue Jul 23, 2024
Merged via the queue into containerd:main with commit 0fe79b6 Jul 23, 2024
@zhsj
Copy link
Contributor

zhsj commented Aug 6, 2024

/cherrypick release/1.7

@k8s-infra-cherrypick-robot

@zhsj: only containerd org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually.

Details

In response to this:

/cherrypick release/1.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@akhilerm akhilerm 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 labels Aug 7, 2024
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 go Pull requests that update Go code kind/test ok-to-test size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TestNewBinaryIOCleanup failing on gotip

8 participants