-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
updated with latest redshift service account ID's #1614
Conversation
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.
LGTM for the additions. Just left a comment to discuss :)
Thanks! 👍
@@ -33,6 +33,6 @@ data "aws_redshift_service_account" "main" { } | |||
|
|||
const testAccCheckAwsRedshiftServiceAccountExplicitRegionConfig = ` | |||
data "aws_redshift_service_account" "regional" { | |||
region = "eu-west-1" | |||
region = "eu-west-2" |
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.
Is there a reason to change this test?
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.
Possibly not - I'm new to updating the terraform source code
I thought it would help validate the new Redshift service account (eu-west-2) being added - remove if not required
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.
No worries, was just wondering :)
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.
$ make testacc TEST=./aws TESTARGS='-run=TestAccAWSRedshiftServiceAccount_basic'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -run=TestAccAWSRedshiftServiceAccount_basic -timeout 120m
=== RUN TestAccAWSRedshiftServiceAccount_basic
--- PASS: TestAccAWSRedshiftServiceAccount_basic (58.93s)
PASS
ok github.com/terraform-providers/terraform-provider-aws/aws 58.968s
LGTM, thanks again @david-wells-1 ! 👍
updated with latest redshift service account ID's
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
AWS Redshift service accounts IDs added to included newest regions where Redshift is now supported
Documentation: http://docs.aws.amazon.com/fr_fr/redshift/latest/mgmt/db-auditing.html