Skip to content

Conversation

@thaJeztah
Copy link
Member

It was added in Docker v1.3.0 through bd130e7, but never used, and its behavior never documented. There are no publicly visible external consumers of this function, so let's remove it.

- Description for the changelog

Go SDK: pkg/ioutils: remove NewReaderErrWrapper as it was never used.

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

It was added in Docker [v1.3.0] through bd130e7,
but never used, and its behavior never documented. There are no publicly
visible external consumers  of this function, so let's remove it.

[v1.3.0]: https://github.com/moby/moby/releases/tag/v1.3.0

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.0.0 milestone Jan 10, 2025
@thaJeztah thaJeztah self-assigned this Jan 10, 2025
@thaJeztah
Copy link
Member Author

This test is quite flaky on Windows; tracked in #48882

=== Failed
=== FAIL: github.com/docker/docker/integration/networking TestNatNetworkICC/User_defined_nat_network (8.61s)
    nat_windows_test.go:62: assertion failed: error is not nil: Post "http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.48/containers/8519f1a3cf94d13f7e2462f525b263cff7b52bd0377d612bfb80adcbddd5020d/start": context deadline exceeded
    panic.go:629: assertion failed: error is not nil: Error response from daemon: error while removing network: network mynat id dbbcc7ffd874e0e72311aa4502c50327aded37eacf08a6543ed76232fb9c37bc has active endpoints

@tianon tianon merged commit df596e9 into moby:master Jan 10, 2025
161 checks passed
@thaJeztah thaJeztah deleted the ioutils_remove_NewReaderErrWrapper branch January 10, 2025 22:45
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