-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add storage_profile_id
to vcd_catalog and introduce datasource vcd_storage_profile
#602
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
vcd/resource_vcd_catalog.go
Outdated
if storageProfileId != "" { | ||
storageProfileReference, err := adminOrg.GetStorageProfileReferenceById(storageProfileId, false) | ||
if err != nil { | ||
return fmt.Errorf("couuld not proces storage profile '%s': %s", storageProfileId, err) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return fmt.Errorf("couuld not proces storage profile '%s': %s", storageProfileId, err) | |
return fmt.Errorf("could not process Storage Profile '%s': %s", storageProfileId, err) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now, thanks!
Fixes #598 ,
This PR adds
storage_profile_id
attribute tovcd_catalog
resource and datasource (for create and update)Additionally it introduces
vcd_storage_profile
datasource for storage profile ID lookup.It also allows to change
description
field forvcd_catalog
Note. Acceptance tests passed on at least one version (also
VCD_TEST_ORG_USER=1 go test -timeout 30m -v -tags "org catalog"
)