Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

On GH Actions windows nodes, ensure there is no automatic docker login before running ACI E2E tests#953

Merged
gtardif merged 1 commit intomainfrom
windows_e2e_docker_logout
Nov 23, 2020
Merged

On GH Actions windows nodes, ensure there is no automatic docker login before running ACI E2E tests#953
gtardif merged 1 commit intomainfrom
windows_e2e_docker_logout

Conversation

@gtardif
Copy link
Copy Markdown
Contributor

@gtardif gtardif commented Nov 23, 2020

ACI currently has some flaky issue where deploying a payload containing docker credentials just freezes forever without error message

Signed-off-by: Guillaume Tardif guillaume.tardif@docker.com

What I did

  • in windows GHActions workflow, docker logout before running ACI E2E tests

Related issue
https://github.com/docker/compose-cli/pull/952/checks?check_run_id=1442133013

/test-aci
/test-windows

(not mandatory) A picture of a cute animal, if possible in relation with what you did

…n before running ACI E2E tests

Signed-off-by: Guillaume Tardif <guillaume.tardif@docker.com>
@github-actions github-actions Bot added the ci Continuous integration label Nov 23, 2020
@gtardif gtardif requested review from chris-crone and rumpl November 23, 2020 13:49
@gtardif gtardif merged commit 97560f8 into main Nov 23, 2020
@gtardif gtardif deleted the windows_e2e_docker_logout branch November 23, 2020 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci Continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants