-
Notifications
You must be signed in to change notification settings - Fork 526
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
Compute v2: Add the user id to the get create and delete keypair #2186
Conversation
@kayrus Let me know your thoughts on this. Generally, I'm not opposed to adding an |
Build succeeded.
|
// GetWithOpts is similar to Get, but can take a GetOpts struct. | ||
// The options within GetOpts require the client to have a microversion set. | ||
// The original Get function is retained for non-microverisoned requests. | ||
func GetWithOpts(client *gophercloud.ServiceClient, name string, opts GetOptsBuilder) (r GetResult) { |
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.
@jtopjian GetWithOpts
func looks hacky. I'd extend the Get
function, even if it breaks the previous API.
@jtopjian can you also add a |
Build failed.
|
@jtopjian thanks a lot. LGTM. |
For #2145
Supersedes #2146