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

Docs BucketLifecycleConfiguration: Incorrect deprecation warning of rule.filter.prefix #885

Closed
thekaleidoscope opened this issue Sep 20, 2023 · 3 comments · Fixed by #1331
Labels
bug Something isn't working needs:triage

Comments

@thekaleidoscope
Copy link
Contributor

What happened?

Documentation of rule-filter-prefix warns its deprecated and recommends to use the same filter.prefix field which it's warning for.

rule.filter.prefix: DEPRECATED Use filter instead. This has been deprecated by Amazon S3. Prefix identifying one or more objects to which the rule applies. Defaults to an empty string ("") if filter is not specified.

How can we reproduce it?

Go to https://marketplace.upbound.io/providers/upbound/provider-aws/v0.40.0/resources/s3.aws.upbound.io/BucketLifecycleConfiguration/v1beta1#doc:spec-forProvider-rule-filter-prefix

What environment did it happen in?

N/A

  • Provider Version: v0.40.0
@thekaleidoscope thekaleidoscope added bug Something isn't working needs:triage labels Sep 20, 2023
@turkenf
Copy link
Collaborator

turkenf commented Oct 11, 2023

@thekaleidoscope, this parameter seems to be deprecated, please see here for more information.
I am closing this now, feel free to reopen.

@turkenf turkenf closed this as completed Oct 11, 2023
@applike-ss
Copy link

@turkenf If i am not reading the terraform docs wrong, then it is deprecated on rule.prefix, but not on rule.filter.0.prefix.
See: https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_lifecycle_configuration

How else would one set a prefix here?

AWS Doc also doesn't show a deprecation for the parameter: https://docs.aws.amazon.com/AmazonS3/latest/API/API_PutBucketLifecycleConfiguration.html

Please re-open or tell me where i'm wrong.

@turkenf
Copy link
Collaborator

turkenf commented May 29, 2024

Hey @applike-ss, you are right, I appreciate your warning 🙏 , I opened a PR that fixed it.

@turkenf turkenf reopened this May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs:triage
Projects
None yet
3 participants