From 6cc9b890b53dc5a12ec9dba02a2ecb9ed7107534 Mon Sep 17 00:00:00 2001 From: Stephane Jourdan Date: Tue, 2 Feb 2021 10:39:52 +0100 Subject: [PATCH 1/4] add LIMITATIONS.md --- doc/LIMITATIONS.md | 6 ++++++ doc/README.md | 3 +++ 2 files changed, 9 insertions(+) create mode 100644 doc/LIMITATIONS.md diff --git a/doc/LIMITATIONS.md b/doc/LIMITATIONS.md new file mode 100644 index 00000000..6403b7b8 --- /dev/null +++ b/doc/LIMITATIONS.md @@ -0,0 +1,6 @@ +# Known Issues and Limitations + +## AWS Regions Limits + +- The user needs to use the same AWS region for both the scanned infrastructure than the S3 bucket where the Terraform state is stored (ie.: a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work.). + - See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). diff --git a/doc/README.md b/doc/README.md index f98344c6..ecd35f04 100644 --- a/doc/README.md +++ b/doc/README.md @@ -22,3 +22,6 @@ Details of reported data can be found [here](./cmd/flags/error-reporting.md) - [Iac sources](cmd/scan/iac_source.md) - [Completion](cmd/completion/script.md) +## Issues + +- [Known Issues & Limitations](LIMITATIONS.md) \ No newline at end of file From 4ca2a3147b3787832133e2ba29abc032f0674c78 Mon Sep 17 00:00:00 2001 From: Stephane Jourdan Date: Tue, 2 Feb 2021 10:45:47 +0100 Subject: [PATCH 2/4] update text --- doc/LIMITATIONS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/LIMITATIONS.md b/doc/LIMITATIONS.md index 6403b7b8..3d633ae3 100644 --- a/doc/LIMITATIONS.md +++ b/doc/LIMITATIONS.md @@ -2,5 +2,5 @@ ## AWS Regions Limits -- The user needs to use the same AWS region for both the scanned infrastructure than the S3 bucket where the Terraform state is stored (ie.: a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work.). +- The user needs to use the same AWS region for both the scanned infrastructure and the S3 bucket where the Terraform state is stored (for example, a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work.). - See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). From eb34ab7cf694861662086c4d88df647b905a386d Mon Sep 17 00:00:00 2001 From: Stephane Jourdan Date: Tue, 2 Feb 2021 18:37:04 +0100 Subject: [PATCH 3/4] cleaner syntax --- doc/LIMITATIONS.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/doc/LIMITATIONS.md b/doc/LIMITATIONS.md index 3d633ae3..591038b2 100644 --- a/doc/LIMITATIONS.md +++ b/doc/LIMITATIONS.md @@ -2,5 +2,4 @@ ## AWS Regions Limits -- The user needs to use the same AWS region for both the scanned infrastructure and the S3 bucket where the Terraform state is stored (for example, a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work.). - - See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). +- The user needs to use the same AWS region for both the scanned infrastructure and the S3 bucket where the Terraform state is stored (for example, a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work). See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). From 90d076e5040855bd7d673194963451a19cfd70c3 Mon Sep 17 00:00:00 2001 From: Stephane Jourdan Date: Tue, 2 Feb 2021 18:43:41 +0100 Subject: [PATCH 4/4] add the multiple aliases for a single account limitation --- doc/LIMITATIONS.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/doc/LIMITATIONS.md b/doc/LIMITATIONS.md index 591038b2..e95e67c0 100644 --- a/doc/LIMITATIONS.md +++ b/doc/LIMITATIONS.md @@ -1,5 +1,6 @@ # Known Issues and Limitations -## AWS Regions Limits +## AWS Regions & Credentials Limits -- The user needs to use the same AWS region for both the scanned infrastructure and the S3 bucket where the Terraform state is stored (for example, a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work). See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). +- The user needs to use the same AWS region and credentials for both the scanned infrastructure and the S3 bucket where the Terraform state is stored (for example, a Terraform state stored on S3 on us-east-1 for an infrastructure to be scanned on us-west-1 won't work). Think `AWS_PROFILE` for the underlying reason. See the related [GitHub Discussion](https://github.com/cloudskiff/driftctl/discussions/130). +- Driftctl currently doesn't support multiple aliased providers in a single Terraform state (like a single account but multiple regions). This will be implemented soon.