-
Notifications
You must be signed in to change notification settings - Fork 33
Add SFS interfaces #37
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.
Some nits inline, otherwise looks good to me
openstack/client.go
Outdated
return sc, err | ||
} | ||
|
||
//NewSFSV2 creates a service client taht is used for Huawei cloud for SFS , it replaces the EVS type. |
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.
typo, s/taht/that
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.
Done
//NewSFSV2 creates a service client taht is used for Huawei cloud for SFS , it replaces the EVS type. | ||
func NewHwSFSV2(client *golangsdk.ProviderClient, eo golangsdk.EndpointOpts) (*golangsdk.ServiceClient, error) { | ||
sc, err := initClientOpts(client, eo, "evs") | ||
sc.Endpoint = strings.Replace(sc.Endpoint, "evs", "sfs", 1) |
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.
better to leave a TODO here to use real service type when we add the endpoint for SFS.
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.
Comment
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.
Done
What this PR does / why we need it: This PR adds SFS interfaces
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #Special notes for your reviewer:NONE
Release note:NONE