[libvirt] [PATCH] virsh: Accept UUID as an argument for storage commands

Affected commands are: pool-start, vol-create-from. --- tools/virsh.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 1303da9..4737948 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -11286,7 +11286,7 @@ static const vshCmdInfo info_pool_start[] = { }; static const vshCmdOptDef opts_pool_start[] = { - {"pool", VSH_OT_DATA, VSH_OFLAG_REQ, N_("name of the inactive pool")}, + {"pool", VSH_OT_DATA, VSH_OFLAG_REQ, N_("name or uuid of the inactive pool")}, {NULL, 0, 0, NULL} }; @@ -11295,18 +11295,18 @@ cmdPoolStart(vshControl *ctl, const vshCmd *cmd) { virStoragePoolPtr pool; bool ret = true; + const char *name = NULL; if (!vshConnectionUsability(ctl, ctl->conn)) return false; - if (!(pool = vshCommandOptPoolBy(ctl, cmd, "pool", NULL, VSH_BYNAME))) + if (!(pool = vshCommandOptPool(ctl, cmd, "pool", &name))) return false; if (virStoragePoolCreate(pool, 0) == 0) { - vshPrint(ctl, _("Pool %s started\n"), - virStoragePoolGetName(pool)); + vshPrint(ctl, _("Pool %s started\n"), name); } else { - vshError(ctl, _("Failed to start pool %s"), virStoragePoolGetName(pool)); + vshError(ctl, _("Failed to start pool %s"), name); ret = false; } @@ -11629,7 +11629,7 @@ static const vshCmdInfo info_vol_create_from[] = { }; static const vshCmdOptDef opts_vol_create_from[] = { - {"pool", VSH_OT_DATA, VSH_OFLAG_REQ, N_("pool name")}, + {"pool", VSH_OT_DATA, VSH_OFLAG_REQ, N_("pool name or uuid")}, {"file", VSH_OT_DATA, VSH_OFLAG_REQ, N_("file containing an XML vol description")}, {"vol", VSH_OT_DATA, VSH_OFLAG_REQ, N_("input vol name or key")}, {"inputpool", VSH_OT_STRING, 0, N_("pool name or uuid of the input volume's pool")}, @@ -11648,7 +11648,7 @@ cmdVolCreateFrom(vshControl *ctl, const vshCmd *cmd) if (!vshConnectionUsability(ctl, ctl->conn)) goto cleanup; - if (!(pool = vshCommandOptPoolBy(ctl, cmd, "pool", NULL, VSH_BYNAME))) + if (!(pool = vshCommandOptPool(ctl, cmd, "pool", NULL))) goto cleanup; if (vshCommandOptString(cmd, "file", &from) <= 0) { -- 1.7.7.3

On 06/04/2012 02:35 AM, Osier Yang wrote:
Affected commands are: pool-start, vol-create-from. --- tools/virsh.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-)
Nice usability enhancement. ACK. -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org

On 2012年06月05日 03:11, Eric Blake wrote:
On 06/04/2012 02:35 AM, Osier Yang wrote:
Affected commands are: pool-start, vol-create-from. --- tools/virsh.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-)
Nice usability enhancement.
ACK.
Thanks, pushed. Osier
participants (2)
-
Eric Blake
-
Osier Yang