Skip to content

api/types: remove some redundant imports#49355

Merged
vvoland merged 2 commits intomoby:masterfrom
thaJeztah:api_types_less_deps
Jan 29, 2025
Merged

api/types: remove some redundant imports#49355
vvoland merged 2 commits intomoby:masterfrom
thaJeztah:api_types_less_deps

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

api/types/registry: use stdlib errors package

It was the only use of github.com/pkg/errors inside api/types,
which is the package that's imported by external users.

api/types/container: remove use of errdefs package in test

It was the only use of errdefs inside api/types, which is the package
that's imported by external users.

- Description for the changelog

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

It was the only use of github.com/pkg/errors inside api/types,
which is the package that's imported by external users.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code area/go-sdk labels Jan 29, 2025
@thaJeztah thaJeztah self-assigned this Jan 29, 2025
It was the only use of errdefs inside api/types, which is the package
that's imported by external users.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

New flaky test; TestAccessPublishedPortFromAnotherNetwork
https://github.com/moby/moby/actions/runs/13030476533/job/36348946770?pr=49355

Added in

=== Failed
=== FAIL: amd64.integration.networking TestAccessPublishedPortFromAnotherNetwork/IPv4/Gateway (0.59s)
    port_mapping_linux_test.go:877: assertion failed: string "" does not contain "foobar": Payload was not received by the server container
    --- FAIL: TestAccessPublishedPortFromAnotherNetwork/IPv4/Gateway (0.59s)

=== FAIL: amd64.integration.networking TestAccessPublishedPortFromAnotherNetwork (3.31s)

@thaJeztah
Copy link
Copy Markdown
Member Author

This one also remains quite flaky;

=== Failed
=== FAIL: github.com/docker/docker/integration/container TestWaitRestartedContainer/not-running (60.37s)
    wait_test.go:205: assertion failed: error is not nil: error during connect: Post "http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.48/containers/create": EOF

=== FAIL: github.com/docker/docker/integration/container TestWaitRestartedContainer (0.02s)

@vvoland vvoland merged commit e273a56 into moby:master Jan 29, 2025
@vvoland vvoland added this to the 28.0.0 milestone Jan 29, 2025
@thaJeztah thaJeztah deleted the api_types_less_deps branch January 29, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/go-sdk kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants