-
Notifications
You must be signed in to change notification settings - Fork 148
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(instance): support snapshot based instance #3787
base: master
Are you sure you want to change the base?
Conversation
bdc9f1a
to
7f521e5
Compare
serverType *instance.ServerType | ||
) | ||
if args.Image != "none" { | ||
getImageResponse, err := apiInstance.GetImage(&instance.GetImageRequest{ |
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.
getImageResponse
is shadowing the one created in previous context.
getImageResponse, err := apiInstance.GetImage(&instance.GetImageRequest{ | |
var err error | |
getImageResponse, err = apiInstance.GetImage(&instance.GetImageRequest{ |
|
||
serverType := getServerType(apiInstance, serverReq.Zone, serverReq.CommercialType) | ||
serverType := getServerType(apiInstance, serverReq.Zone, serverReq.CommercialType) |
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.
serverType
shadows the one created in previous context
serverType := getServerType(apiInstance, serverReq.Zone, serverReq.CommercialType) | |
serverType = getServerType(apiInstance, serverReq.Zone, serverReq.CommercialType) |
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.
Oops thanks a lot. Missed that. That should explain the failing CI.
This PR allow users to create an instance from a snapshot directly without creating an image. It's already supported by the APIs. Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
7f521e5
to
ad04273
Compare
Hi,
In this PR I tried to allow the CLI to deploy an instance directly from a snapshot without calling the marketplace API.
Community Note
Release note for CHANGELOG: