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

Bucket region cannot be set by specifying region #615

Merged
merged 9 commits into from
Apr 28, 2021

Conversation

tomarv2
Copy link
Contributor

@tomarv2 tomarv2 commented Apr 28, 2021

Tried this, we cannot set bucket region, it throws an error, also changed the bucket name for end users understanding:

38:   region = "eu-west-1"

Computed attributes cannot be set, but a value was set for "region".

@nfx
Copy link
Contributor

nfx commented Apr 28, 2021

@tomarv2 can you look at couple more places where aws_s3_bucket is defined and also change those there?

@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #615 (38b6d62) into master (f14b825) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #615   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files          85       85           
  Lines        7796     7796           
=======================================
  Hits         6447     6447           
  Misses        879      879           
  Partials      470      470           

@tomarv2
Copy link
Contributor Author

tomarv2 commented Apr 28, 2021

@tomarv2 can you look at couple more places where aws_s3_bucket is defined and also change those there?

verified there are 10 references to s3 resource and all look ok except this. Will keep a check if I notice something.

@nfx nfx merged commit 54c2b65 into databricks:master Apr 28, 2021
@nfx nfx mentioned this pull request May 12, 2021
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
* spelling correction
* updating readme
* moving images to images directory
* updating docs based on feedback
* updating bucket name and removing region as its not supported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants