Commit graph

148 commits

Author SHA1 Message Date
aiordache
87245ef727 rename packages ( amazon to ecs and azure to aci )
Signed-off-by: aiordache <anca.iordache@docker.com>
2020-07-30 11:34:34 +02:00
aiordache
648f0139df add e2e test for compose up on default context
Signed-off-by: aiordache <anca.iordache@docker.com>
2020-07-29 17:33:23 +02:00
Christopher Crone
eac315c6ec tests: Fix logout test on Windows
The macOS/Linux output when logging out from a registry that you're not
logged into is different to that of Windows. As the login test doesn't
really test if the registry login works, the test now just checks that
the command is properly parsed.

Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-07-17 16:55:12 +02:00
Christopher Crone
d1a929eb7f cli: Fix logout when specifying a registry
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
2020-07-17 16:29:12 +02:00
Djordje Lukic
4480d27595 Don't set the container name in the test
This will help us catch errors such as
https://github.com/docker/desktop-microsoft/issues/42
2020-07-16 10:39:41 +02:00
Guillaume Tardif
1fbdbbbbe3
Merge pull request #381 from docker/fix-logs-windows
Get the real width of the terminal
2020-07-10 13:06:07 +02:00
Guillaume Tardif
d21fd7b8cf Do not fail (divide by zero) if terminal width is not set. Improved a bit the ACI e2e test, happy that it catches this error (this was going to break the log gRPC API) 2020-07-10 13:00:41 +02:00
Guillaume Tardif
52c2d09eae
Merge pull request #376 from docker/logout
Implement azure logout
2020-07-10 09:48:39 +02:00
Djordje Lukic
26b4ece77d
Merge pull request #373 from docker/chore-guava
Update guava version to remove security warning
2020-07-09 15:02:39 +02:00
Djordje Lukic
ef2d304762 Allow non-interactive exec on ACI
If the request is for a non-interactive exec we don't attach the stdin
when executing.
2020-07-09 11:47:22 +02:00
Guillaume Tardif
492ce96dd4 Implement azure logout 2020-07-08 17:48:10 +02:00
Guillaume Tardif
7cd0b3c159 Separate ACI e2e tests, allowing to run a single one 2020-07-08 16:54:45 +02:00
Guillaume Tardif
a2ea2347ba Remove It() in e2e tests, simplified step declaration. Nicer test output for all tests 2020-07-08 16:54:45 +02:00
Djordje Lukic
f5f01a8729 Update guava version to remove security warning 2020-07-08 15:07:12 +02:00
Guillaume Tardif
c5cd0f44f8 Fix ACI e2e tests : do not use console if no tty option set in exec 2020-07-07 14:05:25 +02:00
Guillaume Tardif
16acc36033
Merge pull request #356 from docker/fix-non-interactive
Add check on exec looking for arguments
2020-07-07 12:41:44 +02:00
Guillaume Tardif
f7daa70c87 Add ACI e2e test starting mysql with typical env vars 2020-07-07 12:22:56 +02:00
Guillaume Tardif
d7f8724dba
Merge pull request #354 from docker/aci_fix_compose_working_dir
Fix `docker compose up` with default project name being derived from current working dir
2020-07-07 11:27:49 +02:00
Ulysses Souza
35cbb621fb Add check on exec looking for arguments
In the case where the command has arguments
it fails because ACI implementation does not
support commands with arguments

Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-07-07 09:32:52 +02:00
Guillaume Tardif
e4c1a1f945 Fix docker compose up with default project name being derived from current working dir. has been fixed upstream in compose-go (see https://github.com/compose-spec/compose-go/pull/56) 2020-07-06 16:56:30 +02:00
Guillaume Tardif
7296290408 Add ”Platform” in docker inspect result, showing container OS (this field is used by VSCode) 2020-07-06 12:16:02 +02:00
Guillaume Tardif
76cb73c5c2 ACI : allow docker compose down —project-name xxx without requiring the compose file 2020-07-03 10:24:06 +02:00
Guillaume Tardif
ae76e0cf27
Merge pull request #325 from docker/error_status_context_needs_login
Specific exit code when command fails because azure login is required
2020-07-02 17:02:57 +02:00
Guillaume Tardif
4a1b50a9cd
Merge pull request #326 from docker/run_follow_logs
Run containers and attach to logs by default, detach with -d option
2020-07-02 17:02:33 +02:00
Guillaume Tardif
5eea111ecf Run containers and attach to logs by default, detach with -d option 2020-07-02 16:45:04 +02:00
Guillaume Tardif
a9ef20eefd In ACI e2e tests, do not wait for full resource group deletion at the end, let ACI clean things UP (timeframe ~45 sec). This is costing GH Action minutes for nothing, and slows test results. 2020-07-02 16:13:38 +02:00
Guillaume Tardif
964e331a10 Specific exit code when command fails because azure login is required. Will be used by VSCode Docker integration 2020-07-02 13:52:57 +02:00
Guillaume Tardif
8bdb4dee77 Remove debug stuff from web images, that is causing apt-get install issues 2020-07-01 15:47:41 +02:00
Guillaume Tardif
7cde96e8b6 Update guava to fix CVE 2020-07-01 14:59:23 +02:00
Djordje Lukic
94a530b05f Make log stream test more readable 2020-06-30 18:26:38 +02:00
Guillaume Tardif
a48c4fb41c
Merge pull request #307 from docker/aci_e2e_group
Random ACI group name for tests, otherwise we are conflicting with each other + the CI
2020-06-30 18:18:16 +02:00
Guillaume Tardif
f850dc557d Random ACI group name for tests, otherwise we are conflicting with each other + the CI.
Same for storage account name (outside of the scope of resource group)
2020-06-30 18:13:07 +02:00
Djordje Lukic
fdc2bd34d7 Add test for logs --follow on ACI 2020-06-30 17:13:51 +02:00
Guillaume Tardif
2268f0c598 Renamed Login method specific to tests, to make things more explicit 2020-06-30 10:11:06 +02:00
Guillaume Tardif
eda438aaed Added basic support for service principal login, run ACI e2e tests with it 2020-06-29 20:35:23 +02:00
Guillaume Tardif
2b5e6f0038
Merge pull request #278 from docker/limits
Limits for CPU and Memory
2020-06-29 13:32:25 +02:00
Guillaume Tardif
c1b0dd2e31
Merge pull request #284 from docker/aci_e2e_compose_multiport_update
Uses compose update e2e test to also test compose exposing multiple ports
2020-06-29 10:37:50 +02:00
Ulysses Souza
1deb2b8b09 Integrate limits on inspect
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
2020-06-29 10:32:28 +02:00
Guillaume Tardif
d97f9d68fd Add missing f shorthand for version command 2020-06-26 12:35:51 +02:00
Guillaume Tardif
839b1b0b44 Delegate to Moby CLI, to allow executing ecs CLI plugin if user has switched to the AWS context (created by the plugin) 2020-06-25 11:49:28 +02:00
Guillaume Tardif
2df807abca Uses compose update e2e test to also test compose exposing multiple ports 2020-06-24 18:20:27 +02:00
Guillaume Tardif
9789ce119b
Merge pull request #268 from docker/compose_expose_miltiple_ports
Allow ACI compose to expose several ports
2020-06-23 16:17:39 +02:00
Guillaume Tardif
7306821804 Allow ACI compose to expose several ports 2020-06-23 16:00:52 +02:00
Guillaume Tardif
fa8b911cd8 Allow running docker -v or --version even if context is not default 2020-06-23 15:13:43 +02:00
Guillaume Tardif
4197ef8b68
Merge pull request #263 from docker/fix_host_flag
Shell out to Moby cli if user uses -H, regardless of context set.
2020-06-23 14:57:54 +02:00
Guillaume Tardif
ab39310771 Allow --format on docker context ls. Fixing VS Code extension shellout 2020-06-23 13:25:43 +02:00
guillaume.tardif
4afdc69bb7 e2e test running on windows, default endpoint is npipe on windows 2020-06-23 12:49:02 +02:00
Guillaume Tardif
38d4f8d25a Shell out to Moby cli if user uses -H, regardless of context set. Moby cli will refuse both options (context & host) to be set at the same time 2020-06-23 12:00:46 +02:00
Guillaume Tardif
03f11e2953
Merge pull request #247 from docker/main_command_cleanup
Main command cleanup
2020-06-23 10:35:07 +02:00
Guillaume Tardif
c3924f2e8e Add -H flag to our own version command & login command, & also —-log-level (-l) flag. 2020-06-22 18:22:18 +02:00