toolshed/docker-cli
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
9,301 Commits 3 Branches 295 Tags
1df7161b4b2a8ee5df0a9b7d5c4cbce521fe93ed
Commit Graph
9301 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
Paweł GronowskiandGitHub 1df7161b4b Merge pull request #4756 from dvdksn/fix-run-flag-example-heading-levels 
docs: fix incorrect heading levels in docker run reference
 2024-01-05 16:52:29 +01:00
David Karlsson 909111b3ad docs: fix incorrect heading levels in docker run reference 
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2024-01-05 16:41:26 +01:00
Sebastiaan van StijnandGitHub 26e3eb32ce Merge pull request #4753 from thaJeztah/bump_engine 
vendor: github.com/docker/docker v25.0.0-rc.1
 2024-01-05 14:56:51 +01:00
Sebastiaan van StijnandGitHub b36c16d38d Merge pull request #4754 from thaJeztah/fix_seccomp_defaults 
cli/command/container: parseSecurityOpts: fix --security-opt seccomp=builtin
 2024-01-05 14:30:33 +01:00
Sebastiaan van StijnandGitHub bce868bdfd Merge pull request #4252 from ChrisChinchilla/chrisward/cmd-build-refresh 
Docker build command docs refresh
 2024-01-05 14:30:05 +01:00
Sebastiaan van Stijn b43ea528b8 vendor: github.com/docker/docker v25.0.0-rc.1 
full diff: https://github.com/docker/docker/compare/v25.0.0-beta.3...v25.0.0-rc.1

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2024-01-05 13:23:48 +01:00
Sebastiaan van Stijn 6d0aa0a52d cli/command/container: parseSecurityOpts: fix --security-opt seccomp=builtin 
Docker v23.0 and up allow the daemon to be configured to have seccomp disabled
by default (using the "unconfined" profile as default), and introduced a new
"builtin" profile-name for the default (see [moby@f8795ed364586acd][1] and
[mnoby@ac449d6b5ad29a50][2]).

However, the CLI had no special handling for the "builtin" profile, which
resulted in it trying to load it as a file, which would fail;

    docker run -it --rm --security-opt seccomp=builtin busybox
    docker: opening seccomp profile (builtin) failed: open builtin: no such file or directory.
    See 'docker run --help'.

This patch adds a special case for the "builtin" profile, to allow using the
default profile on daemons with seccomp disabled (unconfined) by default.

[1]: https://github.com/moby/moby/commit/f8795ed364586acd93f72e206a409e7e0e27edcc
[2]: https://github.com/moby/moby/commit/ac449d6b5ad29a5086824729ce54eec6b0cc8545

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2024-01-05 12:19:00 +01:00
Chris ChinchillaandSebastiaan van Stijn 9be600a97b Docker build command docs refresh 
Co-authored-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Chris Chinchilla <chris.ward@docker.com>
 2024-01-05 00:07:34 +01:00
Sebastiaan van StijnandGitHub 8812e0ad3a Merge pull request #4749 from dvdksn/docs-cli-format-example-links
build / prepare (push) Has been cancelled
Details
build / build (push) Has been cancelled
Details
build / prepare-plugins (push) Has been cancelled
Details
build / plugins (push) Has been cancelled
Details
codeql / codeql (push) Has been cancelled
Details
e2e / e2e (19.03-dind, non-experimental) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, connhelper-ssh) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, experimental) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, non-experimental) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, connhelper-ssh) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, experimental) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, non-experimental) (push) Has been cancelled
Details
test / ctn (push) Has been cancelled
Details
test / host (macos-11) (push) Has been cancelled
Details
validate / validate (lint) (push) Has been cancelled
Details
validate / validate (shellcheck) (push) Has been cancelled
Details
validate / validate (update-authors) (push) Has been cancelled
Details
validate / validate (validate-vendor) (push) Has been cancelled
Details
validate / validate-md (push) Has been cancelled
Details
validate / validate-make (manpages) (push) Has been cancelled
Details
validate / validate-make (yamldocs) (push) Has been cancelled
Details 
docs: add links to volume ls, network ls, stack ps formatting examples
v25.0.0-rc.1 		2024-01-03 17:00:26 +01:00
David Karlsson 878b1c55b7 docs: add links to volume ls, network ls, stack ps formatting examples 
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2024-01-03 16:53:47 +01:00
Sebastiaan van StijnandGitHub 3c0c68a45a Merge pull request #4743 from thaJeztah/bump_mapstructure 
vendor: github.com/mitchellh/mapstructure v1.5.0
 2023-12-27 17:46:53 +01:00
Sebastiaan van Stijn c1016c05cf vendor: github.com/mitchellh/mapstructure v1.5.0 
note that this repository will be sunset, and the "endorsed" fork will be
maintened by "go-viper"; see [mapstructure#349][1]

[1]: https://github.com/mitchellh/mapstructure/issues/349

full diff: https://github.com/mitchellh/mapstructure/compare/v1.3.2...v1.5.0

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-27 17:42:31 +01:00
Sebastiaan van StijnandGitHub eed2d9c765 Merge pull request #4742 from thaJeztah/bump_runewidth 
vendor: github.com/mattn/go-runewidth v0.0.15
 2023-12-27 17:05:40 +01:00
Sebastiaan van StijnandGitHub 38d141b900 Merge pull request #4741 from thaJeztah/bump_compress 
vendor: github.com/klauspost/compress v1.17.4
 2023-12-27 17:05:06 +01:00
Sebastiaan van Stijn 58524685da vendor: github.com/mattn/go-runewidth v0.0.15 
no code-changes, but project updated CI to test against current
Go versions;

https://github.com/mattn/go-runewidth/compare/v0.0.14...v0.0.15

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-27 10:40:19 +01:00
Sebastiaan van Stijn 1e38fc3b9d vendor: github.com/klauspost/compress v1.17.4 
full diff: https://github.com/klauspost/compress/compare/v1.17.2...v1.17.4

v1.17.4:

- huff0: Speed up symbol counting
- huff0: Remove byteReader
- gzhttp: Allow overriding decompression on transport
- gzhttp: Clamp compression level
- gzip: Error out if reserved bits are set

v1.17.3:

- fse: Fix max header size
- zstd: Improve better/best compression
- gzhttp: Fix missing content type on Close

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-27 10:34:40 +01:00
Sebastiaan van StijnandGitHub ae41fd913e Merge pull request #4739 from thaJeztah/bump_engine 
vendor: github.com/docker/docker v25.0.0-beta.3
 2023-12-21 17:18:38 +01:00
Sebastiaan van StijnandGitHub bbb922226c Merge pull request #4738 from thaJeztah/update_mailmap_authors 
update authors and mailmap
 2023-12-21 17:17:50 +01:00
Sebastiaan van Stijn 0fa3a365f7 vendor: github.com/docker/docker v25.0.0-beta.3 
no diff, just the tag (which is the same as the previous commit);
https://github.com/moby/moby/compare/7bc56c53657d...v25.0.0-beta.3

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-21 16:41:36 +01:00
David KarlssonandGitHub 33827cdd08 Merge pull request #4733 from dvdksn/docs-engine-plugins-refresh 
docs: add kathara network plugin
 2023-12-21 09:25:05 +01:00
Sebastiaan van Stijn d1a51582e0 update authors and mailmap 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-21 01:12:19 +01:00
Sebastiaan van StijnandGitHub 8dd2cda041 Merge pull request #1450 from lifubang/servicelsdoc
build / prepare (push) Has been cancelled
Details
build / build (push) Has been cancelled
Details
build / prepare-plugins (push) Has been cancelled
Details
build / plugins (push) Has been cancelled
Details
codeql / codeql (push) Has been cancelled
Details
e2e / e2e (19.03-dind, non-experimental) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, connhelper-ssh) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, experimental) (push) Has been cancelled
Details
e2e / e2e (alpine, stable-dind, non-experimental) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, connhelper-ssh) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, experimental) (push) Has been cancelled
Details
e2e / e2e (debian, stable-dind, non-experimental) (push) Has been cancelled
Details
test / ctn (push) Has been cancelled
Details
test / host (macos-11) (push) Has been cancelled
Details
validate / validate (lint) (push) Has been cancelled
Details
validate / validate (shellcheck) (push) Has been cancelled
Details
validate / validate (update-authors) (push) Has been cancelled
Details
validate / validate (validate-vendor) (push) Has been cancelled
Details
validate / validate-md (push) Has been cancelled
Details
validate / validate-make (manpages) (push) Has been cancelled
Details
validate / validate-make (yamldocs) (push) Has been cancelled
Details 
fixes id and name filter in service ls is the prefix not a part
v25.0.0-beta.3 		2023-12-21 01:10:11 +01:00
Sebastiaan van StijnandGitHub 771bc0afce Merge pull request #3558 from gqqnbig/patch-1 
Fix wrong docker run command
 2023-12-21 01:07:58 +01:00
LifubangandSebastiaan van Stijn 60af715d65 id and name filter in service ls is the prefix not a part 
Signed-off-by: Lifubang <lifubang@acmcoder.com>
 2023-12-21 01:04:56 +01:00
QQ喵andSebastiaan van Stijn 90a1887d3c Fix wrong docker run command 
It forgot the image name ubuntu

Signed-off-by: QQ喵 <gqqnb2005@gmail.com>
 2023-12-21 00:58:20 +01:00
Sebastiaan van StijnandGitHub dd3ba7396d Merge pull request #4166 from ChrisChinchilla/chrisward/network-context 
Add networking reference link
 2023-12-21 00:51:23 +01:00
Chris ChinchillaandSebastiaan van Stijn 64e9cadd1c Add networking reference link 
Co-authored-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Chris Chinchilla <chris.ward@docker.com>
 2023-12-21 00:31:02 +01:00
Sebastiaan van StijnandGitHub 21682ebd5c Merge pull request #4685 from ndeloof/export_runstats 
export RunStat, StatsOptions, and add Filters option
 2023-12-20 23:45:18 +01:00
Sebastiaan van StijnandGitHub 3cc1361f01 Merge pull request #4737 from thaJeztah/noomore-score-adjust 
docs/deprecated: update "oom-score-adjust" status for daemon
 2023-12-20 23:44:04 +01:00
Sebastiaan van Stijn 924ec7f4ca docs/deprecated: update "oom-score-adjust" status for daemon 
Using this option on the daemon will now produce an error (flag will
be removed entirely in v26.0).

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 23:31:15 +01:00
Sebastiaan van StijnandGitHub 5777c1bd53 Merge pull request #4735 from thaJeztah/fix_deprecated_anchor 
deprecated: fix anchor and add missing version
 2023-12-20 23:31:03 +01:00
Sebastiaan van Stijn 800a728e12 deprecated: fix anchor and add missing version 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 23:14:51 +01:00
Sebastiaan van StijnandGitHub 6f2f54715c Merge pull request #4734 from akerouanton/deprecated-short-cid-in-aliases 
vendor: github.com/docker/docker 7bc56c53657d (v25.0.0-dev)
 2023-12-20 23:12:36 +01:00
Albin KerouantonandSebastiaan van Stijn c1c6198aed docs: deprecate short cid in Aliases field 
Related to https://github.com/moby/moby/pull/46853

Co-Authored-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Albin Kerouanton <albinker@gmail.com>
 2023-12-20 23:07:22 +01:00
Albin Kerouanton 336787c50a vendor: github.com/docker/docker 7bc56c53657d (v25.0.0-dev) 
full diff: https://github.com/docker/docker/compare/388216fc45aba43d54b950400c5106ecd71fb3db...7bc56c53657dbf405db89e55d4b120f695848058

Signed-off-by: Albin Kerouanton <albinker@gmail.com>
 2023-12-20 22:51:51 +01:00
David Karlsson 53a129e3d4 docs(plugins): remove Weave engine network plugin 
Linked URL returned 404, looks like there's a v2 plugin now instead

Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2023-12-20 21:17:21 +01:00
David Karlsson 3ecd6b6c7f docs(plugins): remove inline CSS markup 
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2023-12-20 21:17:21 +01:00
David Karlsson a4d805f9cf docs(plugins): add Kathará engine network plugin 
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2023-12-20 21:17:09 +01:00
David Karlsson 1dcd5487f0 docs(plugins): update Docker Store to Docker Hub 
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
 2023-12-20 21:16:55 +01:00
Nicolas De LoofandSebastiaan van Stijn 46355a1941 export RunStat, StatsOptions, and add Filters option 
The filter option is not currently exposed on the command-line,
but can be added as a flag in future. It will be used by compose
to filter the list of containers to include based on compose
labels.

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 17:42:49 +01:00
Sebastiaan van StijnandGitHub 8b53402125 Merge pull request #4731 from thaJeztah/stats_godoc 
cli/command/container: statsOptions: add GoDoc
 2023-12-20 17:42:19 +01:00
Sebastiaan van Stijn e9a5590add cli/command/container: statsOptions: add GoDoc 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 17:38:49 +01:00
Sebastiaan van StijnandGitHub 05a2dc401f Merge pull request #4730 from thaJeztah/fix_stats_hang 
prevent "docker stats" from hanging if the initial API call fails
 2023-12-20 17:38:29 +01:00
Sebastiaan van Stijn b642078c87 prevent "docker stats" from hanging if the initial API call fails 
When running `docker stats` without a list of containers, `runStats` collects
an initial list of containers. If that API call fails, the error is sent to the
`closeChan`, however, `closeChan` is non-buffered, and nothing is reading the
channel until we received the initial list and start collecting stats.

This patch rewrites the code that gets the initial list of containers to
return the error if the API call fails. The `getContainerList` closure is
also removed and inlined to make the logic somewhat easier to read.

Before this patch, the command would hang without producing output;

    docker stats
    # hangs; no output

With this patch, the error is printed, and the CLI exits:

    docker stats
    Error response from daemon: some error occurred

    echo $?
    1

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 16:22:11 +01:00
Sebastiaan van StijnandGitHub 3b57acb236 Merge pull request #4729 from thaJeztah/stats_cleanup 
cli/command/container: runStats(): refactor and (linting) fixes
 2023-12-20 13:04:51 +01:00
Sebastiaan van Stijn 633ba88c26 cli/command/container: runStats(): move code to where it's used 
The monitorContainerEvents and getContainerList closures where only
used when collecting "all" containers, so let's define them in that
branch of the code.

Also move some of the other variables closer to where they're used.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 12:07:56 +01:00
Sebastiaan van Stijn 5ed6c128e8 cli/command/container: runStats(): don't register unused event handlers 
We were unconditionally registering event-handlers for these events, but
the handler itself would ignore the event depending on the "all" option.

This patch skips registering the event handlers, so that we're not handling
them (saving some resources).

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 12:07:56 +01:00
Sebastiaan van Stijn 2389768fb7 cli/command/container: runStats(): minor cleanup and fixes 
- memoize the API-client in a local variable.
- use struct-literals in some places.
- rename some variables for clarity and to prevent colliding with imports.
- make use of the event-constants (events.ContainerEventType).
- fix some grammar
- fix some minor linting warnings

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-20 12:07:51 +01:00
Sebastiaan van StijnandGitHub 1f97a34ac2 Merge pull request #4728 from thaJeztah/update_engine 
vendor: github.com/docker/docker 388216fc45ab (v25.0.0-dev)
 2023-12-19 21:41:56 +01:00
Sebastiaan van Stijn 4d434dc691 vendor: github.com/docker/docker 388216fc45ab (v25.0.0-dev) 
full diff: https://github.com/moby/moby/compare/f3cc93630ed8138a6775cbf150c6bfb341cb337b...388216fc45aba43d54b950400c5106ecd71fb3db

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-12-19 20:11:21 +01:00