-
Notifications
You must be signed in to change notification settings - Fork 7
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
41 enhancement netapp ontap storage volume need to support import #91
41 enhancement netapp ontap storage volume need to support import #91
Conversation
CHANGELOG.md
Outdated
@@ -1,4 +1,8 @@ | |||
|
|||
## 1.1.0 () | |||
ENHANCEMENTS: | |||
* **netapp-ontap_storage_volume_resource**: Add support for import ([#72](https://github.com/NetApp/terraform-provider-netapp-ontap/issues/41)) |
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.
this need to be 41
if len(idParts) != 3 || idParts[0] == "" || idParts[1] == "" || idParts[2] == "" { | ||
resp.Diagnostics.AddError( | ||
"Unexpected Import Identifier", | ||
fmt.Sprintf("Expected import identifier with format: attr_one,attr_two,attr_three. Got: %q", req.ID), |
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.
Shall we update the error message on attr_one,attr_two... with real parameter names? That will help the user know what need to put.
Expected import identifier with format: name,svm_name,cx_proflie_name. Got: xxx
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.
Yes we should do that.
@@ -164,4 +164,86 @@ Optional: | |||
- `policy_name` (String) The tiering policy that is to be associated with the volume | |||
|
|||
## Import | |||
Import is currently not support for this Resource. | |||
This resource supports import, which allows you to import existing volumes into the state of this resource. | |||
Import require a unique ID composed of the volume name, the volume name, separated by a comma. |
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.
"the volume name, the volume name" ?
…age_volume-need-to-support-import
…age_volume-need-to-support-import
This complete #41