- Notifications
You must be signed in to change notification settings - Fork925
chore(enterprise/coderd): use filesystem mirror for providers in TestWorkspaceTagsTerraform#16155
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
…mirror for providers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Nice 👍
// downloadProviders is a test helper that creates a temporary file and writes a | ||
// terraform CLI config file with a provider_installation stanza for coder/coder | ||
// using dev_overrides. It also fetches the latest provider release from GitHub | ||
// and extracts the binary to the temporary dir. It is the responsibility of the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Love the line length alignment here 😍, too bad about the third line missing one char 🤣
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I did that specifically to annoy you ;-P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
3a81aac
intomainUh oh!
There was an error while loading.Please reload this page.
Fixescoder/internal#266 (hopefully)
Each instance of this test has to download the coder/coder Terraform provider.
To mitigate this, only download the providers once using a
filesystem_mirror
(ref:https://developer.hashicorp.com/terraform/cli/config/config-file#provider-installation)