Commit Graph
152 Commits
Author SHA1 Message Date
Arash Deshmeh 04259ca4ac add unit tests to task package
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
Upstream-commit: 244e28f41f
Component: cli
2017-06-29 17:57:31 -04:00
Vincent Demeester d8218cc6ef Merge pull request #227 from dnephin/expose-config
expose config credentials without needing the Cli
Upstream-commit: 74af31be7f
Component: cli
2017-06-29 19:11:37 +02:00
Harald Albers 7b10a8b5c9 system prune: only warn about volumes if --volumes is given
Signed-off-by: Harald Albers <github@albersweb.de>
Upstream-commit: 849b0e96a0
Component: cli
2017-06-29 15:07:21 +02:00
Aaron Lehmann ad9f81da7f progress: Show success message when finished counting down
Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
Upstream-commit: 82e069026b
Component: cli
2017-06-27 15:07:10 -07:00
Vincent Demeester ac616d52fb Rename NewConfigFile to New in configfile package
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Upstream-commit: 105b21d1ab
Component: cli
2017-06-27 16:31:38 +02:00
Daniel Nephin 2f4e7d39ca Move credential getting functions to the ConfigFile.
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: a3cbc70147
Component: cli
2017-06-27 13:46:47 +02:00
Daniel Nephin 4682e5be27 Move config file loading to more appropriate packages.
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: a8c70e43a3
Component: cli
2017-06-27 13:45:50 +02:00
Sebastiaan van Stijn f453764eee Merge pull request #207 from cyli/warn-swarm-ca-command
If `docker swarm ca` is not called with the `--rotate` flag, warn if other flags are passed
Upstream-commit: cfff0e2259
Component: cli
2017-06-26 20:56:54 -07:00
Victor Vieux 5323af3564 hide --detach for docker < 17.05
Signed-off-by: Victor Vieux <victorvieux@gmail.com>
Upstream-commit: db60f25561
Component: cli
2017-06-26 18:55:30 -07:00
Victor Vieux bc7c33134c Merge pull request #231 from tonistiigi/client-session-fssession
Incrementally sending build context
Upstream-commit: 0133e13353
Component: cli
2017-06-26 17:15:34 -07:00
Sebastiaan van Stijn 99068fccc9 Merge pull request #237 from aaronlehmann/progress-before-assignment
progress: Show progress of replicated tasks before they are assigned
Upstream-commit: 2f58992f85
Component: cli
2017-06-26 17:14:07 -07:00
Tonis Tiigi f1b5e42df0 Use long running session in builder
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>

Add incremental context send support

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Upstream-commit: b95638a5ac
Component: cli
2017-06-26 16:30:01 -07:00
Sebastiaan van Stijn d0a0bd4e19 Merge pull request #235 from vdemeester/234-fix-warn
Add a line break after warning 👼
Upstream-commit: 298d486d8b
Component: cli
2017-06-24 02:17:26 -07:00
Aaron Lehmann 3b769b27be progress: Show progress of replicated tasks before they are assigned
This was only showing tasks that belong to nodes that are currently up,
so that tasks on down nodes don't appear to be stuck. But this
unintentionally excludes tasks that haven't been assigned yet, so if a
task is stuck before assignment, for example because no nodes meet its
constraints, a progress bar won't even be shown. The check should only
apply to tasks that have a node assignment.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
Upstream-commit: d3d09f67b1
Component: cli
2017-06-23 17:34:59 -07:00
Tonis Tiigi ec6218872a fix build issue with updated moby
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Upstream-commit: ae8d049f9e
Component: cli
2017-06-23 16:12:20 -07:00
Daniel Nephin a2d1e59fe3 Merge pull request #236 from vdemeester/import-displayable-ports-from-moby
Import DisplayablePorts from moby/moby source
Upstream-commit: 8b6196ded7
Component: cli
2017-06-23 15:01:53 -04:00
Daniel Nephin a254f57702 Fix cyclomatic complexity of two formatters
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: c60d13e539
Component: cli
2017-06-23 11:34:22 -07:00
Vincent Demeester 61f81c51e1 Add a line break after warning 👼
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Upstream-commit: b7fad8f1dc
Component: cli
2017-06-23 20:15:00 +02:00
Vincent Demeester a9a5f68c49 Import DisplayablePorts from moby/moby source
This has nothing to do in the `api` package for moby as it's just a
cli display function.

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Upstream-commit: 9e142cadc9
Component: cli
2017-06-23 20:12:30 +02:00
Vincent Demeester f79cf3c340 Merge pull request #221 from dnephin/fix-lint-errors
Fix lint errors merged while new lint branch was in PR
Upstream-commit: 0c6b8d717d
Component: cli
2017-06-23 09:54:12 +02:00
Sebastiaan van Stijn b7706d3788 Merge pull request #202 from ripcurld0/fix_32235_moby
Unmarshal a number as a Number in RawInspectFallback
Upstream-commit: aefbc9d8f7
Component: cli
2017-06-22 23:12:50 -07:00
Daniel Nephin 2b68a9d068 Use compose volume spec parser for container volume flag
Restore testcases for Volume spec parsing.
And correctly interpret the parsed volume.

Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 732261f774
Component: cli
2017-06-21 11:16:04 -04:00
Boaz Shuster 8903bd3d57 Unmarshal a number as a Number in RawInspectFallback
Running `docker inspect --format "{{.ID}} {{.Size}}" alpine` prints
sha256:651aa95985aa4a17a38ffcf71f598ec461924ca96865facc2c5782ef2d2be07f 3983636
While `docker inspect --format "{{.Id}} {{.Size}}" alpine` prints
sha256:651aa95985aa4a17a38ffcf71f598ec461924ca96865facc2c5782ef2d2be07f 3.983636e+06

This happens because "Id" is not a field of types.ImageInspect and thus
tryRawInspectFallback is called and converts the raw response into `interface{}`
using a JSON decoder. However, by default that decoder converts numbers into
`float64` unless `UseNumber` is set.

Signed-off-by: Boaz Shuster <ripcurld.github@gmail.com>
Upstream-commit: 9a2f2d769d
Component: cli
2017-06-21 13:10:14 +03:00
Vincent Demeester 3b2ad4db09 Merge pull request #214 from adshmh/add-unit-tests-to-stack-package
add unit tests to stack package
Upstream-commit: d5d0ec56c1
Component: cli
2017-06-21 10:00:08 +02:00
Daniel Nephin edf13621d4 Fix lint errors merged while new lint branch was in PR.
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: b84e21cd05
Component: cli
2017-06-21 00:11:59 -04:00
Ying Li 8a5a7a8a1e If docker swarm ca is not called with the --rotate flag, the other
flags, including cert expiry, will be ignored, so warn if a user attempts
to use `docker swarm ca --cert-expiry` or something.

Signed-off-by: Ying Li <ying.li@docker.com>
Upstream-commit: 32b43bc21a
Component: cli
2017-06-20 18:55:10 -07:00
Brian Goff 2dbec80975 Merge pull request #176 from dnephin/new-lint
New linters
Upstream-commit: 2bfac7fcda
Component: cli
2017-06-20 16:47:45 -07:00
Brian Goff c196fc8ae5 Merge pull request #195 from Zebrilee/fix-issue-114
fix the issue `--help` doesn't work if daemon is not available
Upstream-commit: 760f847bc8
Component: cli
2017-06-20 16:38:36 -07:00
Arash Deshmeh e2861e049e add unit tests to stack package
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
Upstream-commit: 535af2d868
Component: cli
2017-06-20 14:31:00 -04:00
Dave Tucker d49d72e7d6 Allow Proxy Configuration in config.json
This commit modifies config.json to allow for any proxies allowed in
build-args to be configured. These values will then be used
by default as build-args in docker build.

Signed-off-by: Dave Tucker <dt@docker.com>
Upstream-commit: 35f1e301b5
Component: cli
2017-06-20 09:51:41 +01:00
zebrilee 9fa6954b47 comment the return of setHelpFunc in order to show --help even if the daemon is not running. Then add a if statement in isSupported function to check if the daemon is running
Signed-off-by: zebrilee <zebrilee@gmail.com>

revert change on docker.go, set HasExperimental to true in cli.go

Signed-off-by: zebrilee <zebrilee@gmail.com>
Upstream-commit: cca30cb1d9
Component: cli
2017-06-16 10:24:26 +02:00
Arash Deshmeh 3f1fb4c18f removed logging from command/formatter package tests
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
Upstream-commit: 7f71d0d979
Component: cli
2017-06-15 14:14:59 -04:00
Daniel Nephin b88ae58881 Add gosimple lint
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 3724fb7f37
Component: cli
2017-06-14 16:55:08 -07:00
Daniel Nephin 003a07af7c Add unparam linter
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 3e3934c19f
Component: cli
2017-06-14 16:55:08 -07:00
Daniel Nephin c9b100a0be Add unconvert linter
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 3bf0317fea
Component: cli
2017-06-14 16:54:27 -07:00
Daniel Nephin 16de65eef4 Add misspell lint
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: a712993e93
Component: cli
2017-06-14 16:54:27 -07:00
Daniel Nephin 2cdc30d891 Add unused linter.
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 01e1e58ada
Component: cli
2017-06-14 16:54:27 -07:00
Daniel Nephin 604510bf2e Merge pull request #182 from vdemeester/180-fix-stack-rm-config-endpoint
Do not call the config endpoint if API is lower than 1.30
Upstream-commit: 759e59e4a4
Component: cli
2017-06-14 14:31:01 -04:00
Arash Deshmeh 9a1729bb2b removed logging from service/inspect test to clean up the output of running tests
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
Upstream-commit: 751278abe6
Component: cli
2017-06-14 12:38:31 -04:00
Vincent Demeester 1319ac06c7 Merge pull request #187 from dnephin/small-cleanup-to-image-remove
Fix some problems with image force remove
Upstream-commit: d209929aab
Component: cli
2017-06-14 09:59:32 +02:00
Vincent Demeester 544efe0207 Merge pull request #115 from jsoref/spelling
Spelling
Upstream-commit: 4ffefe4a1f
Component: cli
2017-06-14 09:58:55 +02:00
Vincent Demeester f3827b078b Do not call the config endpoint if API is lower than 1.30
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Upstream-commit: 2128b3f112
Component: cli
2017-06-14 09:56:53 +02:00
Aaron Lehmann 774e3b7a1d Merge pull request #72 from thaJeztah/fix-prefix-matching
Fix prefix-matching for service ps
Upstream-commit: 275c488734
Component: cli
2017-06-13 14:59:37 -07:00
Aaron Lehmann 809225a7dc Merge pull request #184 from yongtang/167-network-inspect-scope
Use scope=swarm for service related network inspect and revendor docker/docker
Upstream-commit: 25bc9f1ce7
Component: cli
2017-06-13 14:47:20 -07:00
Yong Tang a55a518566 Use scope=swarm for service related network inspect.
This fix use `scope=swarm` for service related network inspect.
The purpose is that, in case multiple networks with the same
name exist in different scopes, it is still possible to obtain
the network for services.

This fix is related to moby/moby#33630 and docker/cli#167

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Upstream-commit: 657457ee2c
Component: cli
2017-06-13 20:26:12 +00:00
Daniel Nephin 8af6b6b3cd Fix some problems with image remove force.
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Upstream-commit: 676b71eaaa
Component: cli
2017-06-13 12:12:54 -07:00
Arash Deshmeh 45b64dbaab fixed the output leak from error test case for config/remove
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
Upstream-commit: 006b9b126d
Component: cli
2017-06-13 12:24:42 -04:00
Vincent Demeester 59b1e60ea5 Fix remove_test output duplication
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Upstream-commit: dd924ebf4f
Component: cli
2017-06-13 14:02:56 +02:00
Vincent Demeester f30d636fb1 Merge pull request #160 from e11137/master
fixes #46 Always exit 0 when remove image with force option
Upstream-commit: ecc8e0a204
Component: cli
2017-06-13 14:02:02 +02:00
Vincent Demeester 2810834b91 Merge pull request #172 from adshmh/fix-secret-remove-testcase-error-message-leak
fixed the output leak from the error test case for secret/remove command
Upstream-commit: dd585ad4fb
Component: cli
2017-06-09 21:03:33 +02:00