-
Notifications
You must be signed in to change notification settings - Fork 676
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
ibm_resource_access_tag name is not consistent #5522
Comments
as an user, I was confused about how to attach an access tag to an instance, thought |
Hi, I'm a content designer and I was asked to review this name change 🙂 For this change: Would these be the steps to create and attach an access management tag?
Ultimately I think this renaming makes sense 👍 I think we would need to update our documentation here: https://cloud.ibm.com/docs/account?topic=account-tag&interface=terraform#create-access-terraform — who can I contact to review this and confirm the updates needed? |
@addison-martin1 You can contact me. However before updating that documentation, we need to make the code changes and update the Terraform documentation here https://registry.terraform.io/providers/IBM-Cloud/ibm/latest/docs/resources/resource_access_tag. |
I think this rename makes sense to avoid the confusion for the end user. |
Community Note
Terraform CLI and Terraform IBM Provider Version
Affected Resource(s)
Terraform Configuration Files
Please include all Terraform configurations required to reproduce the bug. Bug reports without a functional reproduction may be closed without investigation.
Debug Output
Panic Output
Expected Behavior
Actual Behavior
It has recently been added the possibility to create an access management tag, which is good. However, the name is inconsistent in my opinion. Let me explain why:
There is the
ibm_resource_tag
Terraform resource that handles tag attachment with a resource. So, it models the relation between an user or access tag with a resource.The new resource
ibm_resource_access_tag
is not modelling a relation between an access tag and a resource. Instead, it models an access tag. So, its name should have beenibm_iam_access_tag
, as I suggested some time ago #2608 (comment).Who reviewed the documentation and the consistency? Was the Global Search Tagging (GhoST) team involved in the development of this new feature? GhoST is the component in the cloud that owns tagging.
Steps to Reproduce
terraform apply
Important Factoids
References
The text was updated successfully, but these errors were encountered: