diff options
| author | Sebastiaan van Stijn <github@gone.nl> | 2021-02-28 16:03:54 +0100 |
|---|---|---|
| committer | Sebastiaan van Stijn <github@gone.nl> | 2021-05-04 16:01:22 +0200 |
| commit | 4d3f64da6348c20f57efe01ca483b130bcf5dd9b (patch) | |
| tree | 616d051de374830833408f15553ba34b3eb4cb7f /testutil | |
| parent | 5a82cee6d499e51bc903a743e4f0f205adb161a9 (diff) | |
| download | docker-4d3f64da6348c20f57efe01ca483b130bcf5dd9b.tar.gz | |
API-client: remove unused ContainerListOptions.Quiet field
This option was originally added in d05aa418b0466553a24d42896f99176cfa29765f,
and moved in 8b15839ee85b291266d07f97d9ad6ca0326d1339 (after which it temporarily
went to the docker/engine-api repository, and was brought back in this repository
in 91e197d614547f0202e6ae9b8a24d88ee131d950).
However, it looks like this field was never used; the API always returns the standard
information, and the "--quiet" option for `docker ps` is implemented on the CLI
side, which uses different formatting when setting this option;
https://github.com/moby/moby/blob/2ec468e2844365f0d7e2afd744f322cde25c077e/api/client/ps.go#L73-L79
This patch removes the unused field,
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Diffstat (limited to 'testutil')
| -rw-r--r-- | testutil/environment/clean.go | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/testutil/environment/clean.go b/testutil/environment/clean.go index 23d434f0bd..415615e877 100644 --- a/testutil/environment/clean.go +++ b/testutil/environment/clean.go @@ -50,7 +50,6 @@ func getPausedContainers(ctx context.Context, t testing.TB, client client.Contai filter.Add("status", "paused") containers, err := client.ContainerList(ctx, types.ContainerListOptions{ Filters: filter, - Quiet: true, All: true, }) assert.Check(t, err, "failed to list containers") @@ -85,8 +84,7 @@ func deleteAllContainers(t testing.TB, apiclient client.ContainerAPIClient, prot func getAllContainers(ctx context.Context, t testing.TB, client client.ContainerAPIClient) []types.Container { t.Helper() containers, err := client.ContainerList(ctx, types.ContainerListOptions{ - Quiet: true, - All: true, + All: true, }) assert.Check(t, err, "failed to list containers") return containers |