Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: remove registry_auth for Docker Terraform module #957

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

Nirusu
Copy link
Contributor

@Nirusu Nirusu commented Jan 12, 2023

Proposed change(s)

  • Remove registry_auth for Docker Terraform provider

This seems to be a leftover from back when the repoistory and images were private, but causes Mini Constellation to fail when no config.json for Docker exists under the defined path ~/docker/config.json.

Related issue

Trying to run MiniConstellation on a relatively fresh Docker installation where this config file does not exist, you will get this error:

./constellation-linux-amd64 mini up
A config file already exists in the current workspace.
Do you want to overwrite it? [y/n]: y
Creating cluster in QEMU   
An error occurred: exit status 1

Error: Error loading registry auth config: could not open config file from filePath: /home/ubuntu/.docker/config.json. Error: open /home/ubuntu/.docker/config.json: no such file or directory

  with provider["registry.terraform.io/kreuzwerker/docker"],
  on main.tf line 18, in provider "docker":
  18: provider "docker" {

Checklist

@Nirusu Nirusu added the bug fix Fixing a bug label Jan 12, 2023
@netlify
Copy link

netlify bot commented Jan 12, 2023

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit 5bec81f
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/63c01587c301eb00090b8e64

@Nirusu Nirusu changed the title qemu: Remove registry_auth for Docker Terraform provider cli: Remove registry_auth for Docker Terraform provider Jan 12, 2023
@malt3
Copy link
Contributor

malt3 commented Jan 12, 2023

Nice catch! I added a checklist item to backport this to v2.4.1 (@3u13r)

@katexochen katexochen changed the title cli: Remove registry_auth for Docker Terraform provider cli: remove registry_auth for Docker Terraform module Jan 12, 2023
@Nirusu Nirusu merged commit b3c3c2f into main Jan 12, 2023
@Nirusu Nirusu deleted the fix/docker-config branch January 12, 2023 14:47
malt3 pushed a commit that referenced this pull request Jan 12, 2023
malt3 pushed a commit that referenced this pull request Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants