Modifies the prompt for the cloud name to include cloud type #6632
Conversation
| @@ -182,7 +186,7 @@ func queryName(cloudMetadataStore CloudMetadataStore, pollster *interact.Pollste | |||
| } | |||
|
|
|||
| for { | |||
| name, err := pollster.Enter("a name for the cloud") | |||
| name, err := pollster.Enter(fmt.Sprintf("a name for your %s", cloudType)) | |||
There was a problem hiding this comment.
Is this clearer as "a name for your %s cloud". It's a little funny for 'maas', but makes more sense for 'manual' and is fine with 'openstack', 'vsphere', and so on.
There was a problem hiding this comment.
Good idea, implemented.
kat-co
force-pushed
the
modify-prompt-cloudtype
branch
2 times, most recently
from
c024bbd
to
0e0411a
Compare
kat-co
force-pushed
the
modify-prompt-cloudtype
branch
from
0e0411a
to
01d862d
Compare
|
|
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
Spurious error. |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
Spurious error. |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Note that this is based upon this PR. Look at the latest commit for the (small) diff.