-
Notifications
You must be signed in to change notification settings - Fork 154
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
feat(dds): support to modify params for instance #4822
feat(dds): support to modify params for instance #4822
Conversation
/lgtm |
return diag.FromErr(err) | ||
} | ||
|
||
d.SetId(instId) |
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 ID is error, if group or node params are modified, then the instance ID may same, but entity ID is different, so the resource ID should contains entity ID if the valuie of entity_id is not empty
} | ||
} | ||
|
||
func resourceDDSInstanceParametersModifyRead(ctx context.Context, _ *schema.ResourceData, _ interface{}) diag.Diagnostics { |
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.
read method shoud be added, API:
https://support.huaweicloud.com/api-dds/dds_api_0108.html
huaweicloud/services/dds/resource_huaweicloud_dds_instance_parameters_modify.go
Show resolved
Hide resolved
5a92786
to
75bb2e8
Compare
75bb2e8
to
7380565
Compare
/lgtm |
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 PR has been approved by: @houpeng80
/lgtm
/approve
What this PR does / why we need it:
Support to modify params for instance.
PR Checklist
Acceptance Steps Performed