-
Notifications
You must be signed in to change notification settings - Fork 887
Don't use parameters to pass secrets to GCP or AWS #2039
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,6 +10,13 @@ tags: [cloud, aws] | |
|
||
Pick this template in `coder templates init` and follow instructions. | ||
|
||
## Authentication | ||
|
||
This template assumes that coderd is run in an environment that is authenticated | ||
with AWS. For example, run `aws configure import` to import credentials on the | ||
system and user running coderd. For other ways to authenticate [consult the | ||
Terraform docs](https://registry.terraform.io/providers/hashicorp/aws/latest/docs#authentication-and-configuration). | ||
Comment on lines
+13
to
+18
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Currently, this isn't really be visible when creating a template via Any ideas? Perhaps it's also fair to rely on the default Terraform errors (+ these docs) to warn users. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, I don't like the idea of hijacking parameters to guide end users. If we can't expect users to read the README in the examples, and want do do some handholding, we should build out a guided |
||
|
||
## Required permissions / policy | ||
|
||
This example policy allows Coder to create EC2 instances and modify instances provisioned by Coder. | ||
|
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.
Appreciate the detail here 👍🏼