Skip to content

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 8, 2025

This test was recently rewritten from testing plugin upgrade with DCT enabled to just "plugin upgrade", but there's a fair amount of complexity in the e2e tests that set up different daemons and registries.

It's possible that tests are affecting each-other, and some state (config) is left behind.

Let's skip the test for now, and add a tracking ticket to dig deeper.

=== FAIL: e2e/global TestPromptExitCode/plugin_upgrade (7.55s)
    cli_test.go:205: assertion failed:
        Command:  docker plugin push registry:5000/plugin-upgrade-test:latest
        ExitCode: 1
        Error:    exit status 1
        Stdout:   The push refers to repository [registry:5000/plugin-upgrade-test]
        459089aa5943: Preparing
        adc41078d1d9: Preparing
        d7bff979db13: Preparing
        459089aa5943: Preparing

        Stderr:   error pushing plugin: failed to do request: Head "https://registry:5000/v2/plugin-upgrade-test/blobs/sha256:adc41078d1d937495df2f90444e5414a01db31e5a080f8aa4f163c64d41abd11": http: server gave HTTP response to HTTPS client

        Failures:
        ExitCode was 1 expected 0
        Expected no error

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

This test was recently rewritten from testing plugin upgrade with
DCT enabled to just "plugin upgrade", but there's a fair amount of
complexity in the e2e tests that set up different daemons and registries.

It's possible that tests are affecting each-other, and some state (config)
is left behind.

Let's skip the test for now, and add a tracking ticket to dig deeper.

    === FAIL: e2e/global TestPromptExitCode/plugin_upgrade (7.55s)
        cli_test.go:205: assertion failed:
            Command:  docker plugin push registry:5000/plugin-upgrade-test:latest
            ExitCode: 1
            Error:    exit status 1
            Stdout:   The push refers to repository [registry:5000/plugin-upgrade-test]
            459089aa5943: Preparing
            adc41078d1d9: Preparing
            d7bff979db13: Preparing
            459089aa5943: Preparing

            Stderr:   error pushing plugin: failed to do request: Head "https://registry:5000/v2/plugin-upgrade-test/blobs/sha256:adc41078d1d937495df2f90444e5414a01db31e5a080f8aa4f163c64d41abd11": http: server gave HTTP response to HTTPS client

            Failures:
            ExitCode was 1 expected 0
            Expected no error

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah requested review from Benehiko and vvoland August 11, 2025 12:28
@thaJeztah thaJeztah merged commit 8324b17 into docker:master Aug 11, 2025
99 checks passed
@thaJeztah thaJeztah deleted the skip_TestPromptExitCode branch August 11, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants