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

Option to disable default tags #1141

Open
mikael-lindstrom opened this issue Feb 8, 2024 · 7 comments · May be fixed by #1474
Open

Option to disable default tags #1141

mikael-lindstrom opened this issue Feb 8, 2024 · 7 comments · May be fixed by #1474
Labels
enhancement New feature or request needs:triage

Comments

@mikael-lindstrom
Copy link

What problem are you facing?

We are using a S3 compatible service called Scality Ring. It supports most S3 functionality, however it does not support tags which causes a problem with this provider since it sets three tags by default:

  • crossplane-kind
  • crossplane-name
  • crossplane-providerconfig

The tags seems to be set by the AddExternalTagsField function which adds a Tagger if the resource supports tags and there is no way to disable this. I have made a fork and built a custom version based on the v1.0.0 release where I simply commented out AddExternalTagsField from the withDefaultResourceOptions here. We are now using this version in our clusters which works great.

How could Official AWS Provider help solve your problem?

By providing an option to disable the "default" crossplane tags the provider would work with more S3 compatible services. I would be happy to help with implementing this but I'm not familiar enough with the codebase to know how this should be implemented.

@ulucinar
Copy link
Collaborator

Hi @mikael-lindstrom,
So, my understanding is that you are using upbound/provider-aws against the Scality Ring service which is mostly compatible with the AWS API (at least for the resources you are interested in and some other provider-level APIs like STS, or maybe the service is built on top these). And you can use the Bucket.s3 managed resource against this service's "S3 buckets" if you disable the default tags at compile time. Is this true?

And the option you are suggesting is a new feature which you can use while provisioning the resource at runtime, to disable the default tagging.

What happens when you use the vanilla upbound/provider-aws provider against this service? Do you get some errors from the Scality Ring service complaining about not being able to set tags or something like an HTTP bad request (HTTP 400) error?

@mikael-lindstrom
Copy link
Author

Hi @ulucinar,

That is correct, Scality supports most S3 functionality and if I disable the default tags everything works great. I have a fork we are using which only comments out the line I mentioned above here.

If I just use the AWS CLI to try an tag a bucket Scality returns 501 Not implemented

$ aws s3api put-bucket-tagging --bucket my-test-bucket --tagging 'TagSet=[{Key=test,Value=test}]'

An error occurred (NotImplemented) when calling the PutBucketTagging operation: A header you provided implies functionality that is not implemented.

If there was an option to disable the default tags we would be able to switch to the vanilla provider and not maintain our own fork. I know there are other S3-compatible services which does not support bucket tags so I think this would be useful feature.

Copy link

github-actions bot commented Aug 8, 2024

This provider repo does not have enough maintainers to address every issue. Since there has been no activity in the last 90 days it is now marked as stale. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

@github-actions github-actions bot added the stale label Aug 8, 2024
Copy link

This issue is being closed since there has been no activity for 14 days since marking it as stale. If you still need help, feel free to comment or reopen the issue!

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 23, 2024
@mikael-lindstrom mikael-lindstrom linked a pull request Aug 27, 2024 that will close this issue
3 tasks
@turkenf turkenf reopened this Aug 28, 2024
@github-actions github-actions bot removed the stale label Aug 29, 2024
Copy link

This provider repo does not have enough maintainers to address every issue. Since there has been no activity in the last 90 days it is now marked as stale. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

@github-actions github-actions bot added the stale label Nov 27, 2024
Copy link

This issue is being closed since there has been no activity for 14 days since marking it as stale. If you still need help, feel free to comment or reopen the issue!

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 11, 2024
@erikgb
Copy link

erikgb commented Jan 24, 2025

/reopen

@turkenf turkenf reopened this Jan 27, 2025
@github-actions github-actions bot removed the stale label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs:triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants