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

Remove deprecated params from bucket #1970

Merged
merged 1 commit into from
Jan 30, 2023
Merged

Conversation

KneeShard
Copy link
Contributor

The acl and versioning parameters are deprecated on the aws_s3_bucket resource. The docs recommend using aws_s3_bucket_acl and aws_s3_bucket_versioning resources instead. From what I can gather, this helps with drift detection in Terraform.

The `acl` and `versioning` parameters are deprecated on the aws_s3_bucket resource. The docs recommend using aws_s3_bucket_acl and aws_s3_bucket_versioning resources instead. From what I can gather, this helps with drift detection in Terraform.
@codecov-commenter
Copy link

Codecov Report

Merging #1970 (7711224) into master (8660c0f) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1970   +/-   ##
=======================================
  Coverage   90.29%   90.29%           
=======================================
  Files         146      146           
  Lines       11906    11906           
=======================================
  Hits        10751    10751           
  Misses        746      746           
  Partials      409      409           

Copy link
Contributor

@alexott alexott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@nfx nfx merged commit d20aea2 into databricks:master Jan 30, 2023
@nfx nfx mentioned this pull request Feb 3, 2023
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
The `acl` and `versioning` parameters are deprecated on the aws_s3_bucket resource. The docs recommend using aws_s3_bucket_acl and aws_s3_bucket_versioning resources instead. From what I can gather, this helps with drift detection in Terraform.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants