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

Adding roles and permissions resources #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sumitAgrawal007
Copy link
Owner

Added 3 resources

data_source_vsphere_role.go:
resource_vsphere_role.go
resource_vsphere_entity_permission.go

main.tf Show resolved Hide resolved
"strings"
)

func resourceVsphereRole() *schema.Resource {

Choose a reason for hiding this comment

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

how are global permissions/roles managed?

Copy link
Owner Author

Choose a reason for hiding this comment

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

They are not supported as of now, only entity level permissions are supported.
I couldn't find an API for global permissions.

Choose a reason for hiding this comment

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

Is it a requirement to add support for global permissions? Maybe a good idea to check with Santosh.

Choose a reason for hiding this comment

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

Yes, I highly suggest that the ability to add global permissions is indeed important. I've already run into instances where I need to automate this for use by vRealize Suite service accounts that need this scope of access across Cloud Foundation workload domains. For example, vRealize Automation required global permissions to use the content library based on the hierarchy.

sumitAgrawal007 pushed a commit that referenced this pull request Sep 16, 2020
* Add import for vsphere_virtual_disk
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.

3 participants