4827 Commits

Author SHA1 Message Date
Djordje Lukic
a506b7f4e9 Fix make protos
The base target for protos was missing goimports, which we run after
making protos so that the linter doesn't fail on generated code
2020-05-14 21:53:14 +02:00
Djordje Lukic
f71ef61fef Remove unused setup-dev.sh, add protoc to the readme 2020-05-14 21:48:50 +02:00
Djordje Lukic
03095e5796
Merge pull request #93 from rumpl/chore-clean-readme
Remove gotestsum from the readme
2020-05-14 21:31:30 +02:00
Djordje Lukic
85ccba0e5e Remove gotestsum from the readme
It is not used any more
2020-05-14 21:26:13 +02:00
Djordje Lukic
17ffa84495
Merge pull request #90 from docker/chore-faster-build
Faster build
2020-05-14 21:21:50 +02:00
Djordje Lukic
7b26e8e836 Faster build
* Run  in parallel
  * lint
  * test/build/e2e test
* use cache for go
* do not use docker for building
* remove useless dependencies from the base image

Build time passes from 5 minutes to 1 minute 30 seconds
2020-05-14 21:16:31 +02:00
Djordje Lukic
6b435cf802
Merge pull request #86 from rumpl/feat-multierror
Add multierror
2020-05-14 21:14:32 +02:00
Chris Crone
747258d452
Merge pull request #91 from chris-crone/appease-linter
Fix import order of cli/options
2020-05-14 20:02:29 +02:00
Christopher Crone
19baa17246 Fix import order of cli/options
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 19:28:57 +02:00
Chris Crone
cc23137945
Merge pull request #82 from chris-crone/context-use
Add context use command
2020-05-14 19:24:03 +02:00
Christopher Crone
a1e83109cd Replace require with assert for errors test
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 19:18:33 +02:00
Christopher Crone
11b4bd19f5 Refactor global CLI options
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 18:29:09 +02:00
Christopher Crone
90e11cf349 e2e: Use new context use command
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
8720a62c37 Add context use command
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
c92a9b12d9 Add help to context commands
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
9bf2924c24 Remove logrus.Fatal
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
ad8a16a922 Do not allow changing default context
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
96907f13e1 Add forbidden error
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:24 +02:00
Christopher Crone
d46398dbef Refactor config into package
* Move CLI config management into cli/config
* Add ability to save current context
* Remove ability to rename the config file as this was never used

Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 17:10:20 +02:00
Djordje Lukic
8032a601c2
Merge pull request #88 from ulyssessouza/fix-ci-timeout
Give more time to github actions to run it
2020-05-14 15:57:58 +02:00
Justin Cormack
cf32d01be1
Merge pull request #89 from chris-crone/docs-cli-ux
Add CLI UX docs
2020-05-14 13:16:31 +01:00
Christopher Crone
44712ab3c4 Move CLI UX docs into folder
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-14 13:29:11 +02:00
Justin Cormack
dac5558885 Initial CLI notes for context and run
Some things undecided still, and more commands needed.

Signed-off-by: Justin Cormack <justin.cormack@docker.com>
2020-05-14 13:27:43 +02:00
Ulysses Souza
a98acafeee Give more time to github actions to run it
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-14 13:01:05 +02:00
Ulysses Souza
4377bceb70
Merge pull request #87 from ulyssessouza/add-run-output
Test output of `docker run`
2020-05-14 11:06:14 +02:00
Djordje Lukic
6486f199e2 Add multierror
docker/api/multierror wraps go-multierror from hashicorp with
our default error formating
2020-05-13 18:37:41 +02:00
Ulysses Souza
323601865d Test output of docker run
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 17:09:18 +02:00
Djordje Lukic
2d14bfeb38
Merge pull request #85 from ulyssessouza/add-run-output
WIP - Missing e2e - Print container name when successful
2020-05-13 14:30:58 +02:00
Guillaume Tardif
67a7c96dc4
Merge pull request #84 from rumpl/chore-remove-util
Remove util package, it was only used  by cli/main.go
2020-05-13 14:23:36 +02:00
Djordje Lukic
53bc852e3e
Merge pull request #76 from ulyssessouza/add-rm
Add rm command
2020-05-13 14:04:37 +02:00
Ulysses Souza
507caabf49 Add id to the error output
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 12:37:18 +02:00
Ulysses Souza
df48f4eb20 Implement Delete for moby
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:50:36 +02:00
Ulysses Souza
b98f45eb28 Enable "run->rm" cycle on e2e-aci
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:44:04 +02:00
Ulysses Souza
7ff1899021 Format with goimport and refactor consts
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:44:04 +02:00
Ulysses Souza
14fb12c8c8 Let make protos affect host files
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:44:04 +02:00
Ulysses Souza
40fa78ac5d Add rm command
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:44:04 +02:00
Ulysses Souza
80bc3f3620 Print container name when successful
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-05-13 10:27:39 +02:00
Djordje Lukic
33b9525c9e Remove util package, it was only used by cli/main.go
`util` is not a great name, it has no context and can quickly become a
place where we put everything and anything.
2020-05-13 10:24:16 +02:00
Guillaume Tardif
1a3365fa37
Merge pull request #81 from chris-crone/always-context
Context should always be handled by new CLI
2020-05-13 09:40:28 +02:00
Christopher Crone
c93d2e9180 Context should always be handled by new CLI
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 18:05:02 +02:00
Guillaume Tardif
4e7645aae2
Merge pull request #42 from rumpl/moby
Moby
2020-05-12 13:46:26 +02:00
Guillaume Tardif
102116315a
Merge pull request #79 from chris-crone/context-rm
Add context rm command
2020-05-12 13:19:08 +02:00
Christopher Crone
5a2a5f5f60 Use new context rm command for e2e
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 12:24:47 +02:00
Christopher Crone
1d3ffc0254 Add context rm command
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 12:24:43 +02:00
Christopher Crone
3c43606a20 Add remove function to context store
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 12:05:00 +02:00
Christopher Crone
4788dd5b93 Use common errors in context store
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 12:05:00 +02:00
Christopher Crone
b55f4b0547 Add common error definitions
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-12 12:04:57 +02:00
Christopher Crone
144ee29645 Fix function description typo
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-05-11 15:49:14 +02:00
Djordje Lukic
fe1772b84a Moby backend 2020-05-11 15:49:03 +02:00
Djordje Lukic
b5821e0e39
Merge pull request #75 from rumpl/chore-upgrade-logrus
Upgrade logrus to 1.6.0
2020-05-07 18:51:04 +02:00