From 26c7ee144fc4c200d9ced18675ccbee8d0946045 Mon Sep 17 00:00:00 2001 From: huyz Date: Sat, 4 Feb 2023 04:03:23 -0800 Subject: [PATCH] Document lack of support for parallel `docker login`. Addresses hashicorp/packer-plugin-docker#141 --- docs/post-processors/docker-push.mdx | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/docs/post-processors/docker-push.mdx b/docs/post-processors/docker-push.mdx index f08685b..ce0cdf0 100644 --- a/docs/post-processors/docker-push.mdx +++ b/docs/post-processors/docker-push.mdx @@ -54,6 +54,13 @@ this flag is optional if you specify the correct ECR Public URL in the - `login` (boolean) - Defaults to false. If true, the post-processor will login prior to pushing. For log into ECR see `ecr_login`. + Note that a corresponding `logout` will be performed right after the push. + This may conflict with other parallel post-processor instances that also + perform login/logout (e.g. if you're building and pushing several images + in parallel). + In that case, it's better to keep `login` set to `false` and have + the `docker login` and `docker logout` performed out of band, separately + from Packer. - `login_username` (string) - The username to use to authenticate to login.