-
Notifications
You must be signed in to change notification settings - Fork 8
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(jimmctl migrate): enable users to specify model names #1500
feat(jimmctl migrate): enable users to specify model names #1500
Conversation
d124ac8
to
6494529
Compare
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.
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.
just a single comment
internal/jimm/jimm.go
Outdated
if err != nil { | ||
s := strings.Split(modelNameOrUUID, "/") | ||
if len(s) != 2 { | ||
return jujuparams.InitiateMigrationResult{}, errors.E(op, "missing model name") |
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 is more "invalid target" right? because we don't know if the model or the user name is wrong
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.
Yeah true, I'll update it
Description
Enables users to specify model names when migrating. Fixes #1371
Engineering checklist
Test instructions