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

[ON HOLD] Disable TestIPFSComposeUp #3396

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

apostasie
Copy link
Contributor

This test is failing a lot.

Even despite the fact that we do retry all tests, it still regularly fail builds, with a variety of different errors (cannot start a stopped container, reuse of closed network connections, etc).

While it is unclear whether the test or the underlying code is faulty, and despite some time spent investigating it with no success, this test serves no purpose given its current condition, and there does not seem to be interest in fixing it.

Suggesting we disable it.

Signed-off-by: apostasie <[email protected]>
@apostasie apostasie force-pushed the dev-disable-compose-ipfs-testing branch from c913f68 to 0f41939 Compare September 2, 2024 18:51
@@ -31,6 +31,8 @@ import (
)

func TestIPFSComposeUp(t *testing.T) {
t.Skip("This is currently broken and failing very often. It is unclear if the test or the code is faulty.")
Copy link
Member

Choose a reason for hiding this comment

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

Needs a link to the github issue w/ failure logs

@ktock
Copy link
Member

ktock commented Sep 3, 2024

I'm trying to reproduce this flakiness on GitHub Actions but I can't reproduce this (passed 3 times / 3 attemptions). 🤔

@ktock ktock self-assigned this Sep 3, 2024
@apostasie
Copy link
Contributor Author

apostasie commented Sep 3, 2024

Thanks a lot for looking into this @ktock.

Here are a couple of links with potentially more info:

One example of failure on the above happening after a couple of tries: https://github.com/containerd/nerdctl/actions/runs/10433233259/job/28894754561?pr=3305#step:7:531

It also happens regularly on normal PRs:
#3293 (comment)

unfortunately not sure how to access the logs of this one ^ since we manually restarted it.

Will send more as soon as we hit it again.

@apostasie
Copy link
Contributor Author

apostasie commented Sep 3, 2024

I'm trying to reproduce this flakiness on GitHub Actions but I can't reproduce this (passed 3 times / 3 attemptions). 🤔

I still think we should remove the retries entirely: #3400

It would be extremely painful for a few weeks, but we would be much better off in the long run...

@apostasie apostasie marked this pull request as draft September 3, 2024 18:13
@apostasie
Copy link
Contributor Author

@ktock I just hit the "cannot start a stopped container" variant of it:

https://github.com/containerd/nerdctl/actions/runs/10690479035/job/29634950871

=== RUN   TestIPFSComposeUp/ipfs-address
    testing_linux.go:186: projectName="nerdctl-compose-test3949339011"
    testing_linux.go:188: assertion failed: res.ExitCode is not exitCode: 
        Command:  /usr/local/bin/nerdctl --namespace=nerdctl-test compose -f /tmp/nerdctl-compose-test3949339011/docker-compose.yaml --ipfs-address=/ip4/10.4.0.1/tcp/5000 up -d
        ExitCode: 1
        Error:    exit status 1
        Stdout:   
        Stderr:   time="2024-09-03T21:19:34Z" level=warning msg="/tmp/nerdctl-compose-test3949339011/docker-compose.yaml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid potential confusion"
        time="2024-09-03T21:19:34Z" level=info msg="Creating network nerdctl-compose-test3949339011_default"
        time="2024-09-03T21:19:34Z" level=info msg="Creating volume nerdctl-compose-test3949339011_db"
        time="2024-09-03T21:19:34Z" level=info msg="Creating volume nerdctl-compose-test3949339011_wordpress"
        time="2024-09-03T21:19:34Z" level=info msg="Ensuring image ipfs://bafkreif7vyfj36wyyzxi42pmcor57gamp73mh2cwtjzhio5gukoocu54ku"
        time="2024-09-03T21:19:34Z" level=info msg="Ensuring image ipfs://bafkreiep3tl4cxkt4mlhzjil24cgzi7bqcwckxjm7fy6cx7nnqh56idl44"
        time="2024-09-03T21:19:34Z" level=info msg="Creating container nerdctl-compose-test3949339011-db-1"
        time="2024-09-03T21:19:34Z" level=warning msg="volume \"nerdctl-compose-test3949339011_db\" already exists and will be returned as-is"
        time="2024-09-03T21:19:34Z" level=fatal msg="OCI runtime start failed: cannot start a container that has stopped: unknown"
        time="2024-09-03T21:19:34Z" level=fatal msg="error while creating container nerdctl-compose-test3949339011-db-1: exit status 1"
        
        
--- FAIL: TestIPFSComposeUp/ipfs-address (7.16s)
FAIL cmd/nerdctl/ipfs.TestIPFSComposeUp/ipfs-address (7.16s)
=== RUN   TestIPFSComposeUp
--- FAIL: TestIPFSComposeUp (163.63s)

@apostasie apostasie changed the title Disable TestIPFSComposeUp [ON HOLD] Disable TestIPFSComposeUp Sep 3, 2024
@apostasie
Copy link
Contributor Author

@apostasie
Copy link
Contributor Author

@apostasie
Copy link
Contributor Author

@apostasie
Copy link
Contributor Author

@ktock were you able to reproduce this locally?

@apostasie
Copy link
Contributor Author

@ktock any update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants