Commit Graph

4934 Commits

Author SHA1 Message Date
Djordje Lukic 693aa7b849 Remove golangci-lint action, use executable
The github action we were using had to be rebuilt on each build, we now
start from base go 1.14 and just download the golangci-tool and run it.
This saves around 20-30 seconds on each build.
2020-05-15 09:17:01 +02:00
Djordje Lukic 15fb6f63c3 Add licenses 2020-05-15 09:14:52 +02:00
Djordje Lukic 0765b08309 Put each compose command in own file 2020-05-15 09:14:51 +02:00
Djordje Lukic 962efef48c Commands run and rm only call a func 2020-05-15 09:14:51 +02:00
Djordje Lukic 42c72c365c Split context comands into own files 2020-05-15 09:14:51 +02:00
Djordje Lukic 0d3ddaa490
Merge pull request #94 from rumpl/chore-rm-setup-dev
Remove unused setup-dev.sh, add protoc to the readme
2020-05-15 09:14:01 +02:00
Djordje Lukic f464d98f5f
Merge pull request #95 from docker/fix-make-protos
Fix `make protos`
2020-05-15 09:13:47 +02:00
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