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

New resource and data source: vsphere_tag #171

Merged
merged 5 commits into from
Sep 25, 2017
Merged

Conversation

vancluever
Copy link
Contributor

@vancluever vancluever commented Sep 20, 2017

This adds the vsphere_tag resource and data source, allowing the ability to manage and pull in tags.

Next step is adding the ability to tag all of the objects in the provider that can be tagged!

PS: This also includes an update to the vic package which was also missing tag update support.

Looks like UpdateTag needed to be added too. This is now done, will
re-vendor off of master if something comes up.
This adds the vsphere_tag resource, which can be used to set up and
manage tags. Data source and docs will be coming soon.

Note that the import for this function takes a JSON string, as both tags
and categories can support special characters, so there is not much for
us to really use as a delimiter.
This is the data source counterpart to the vsphere_tag resource, similar
in workflow to the vsphere_tag_category data source.
@vancluever vancluever added enhancement Type: Enhancement new-resource Feature: New Resource labels Sep 23, 2017
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM with a minor nit

* `name` - (String, required) The display name of the tag. The name must be
unique within its category.
* `description` - (String, optional) A description for the tag.
* `category_id` - (String, required, forces new resource) The unique identifier
Copy link
Member

Choose a reason for hiding this comment

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

Thoughts on putting the required arguments next to each other?

@vancluever
Copy link
Contributor Author

Applied doc change and pushed - thanks @mbfrahry! Merging now

@vancluever vancluever merged commit 7430b1e into master Sep 25, 2017
@vancluever vancluever deleted the f-tag-resource branch September 25, 2017 22:25
@ghost ghost locked and limited conversation to collaborators Apr 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Type: Enhancement new-resource Feature: New Resource
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants