Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
On Fri, 2017-03-03 at 14:08 +0100, Pavel Grunt wrote:
> Theoretically a VM name can be a valid VM id or uuid. In that case
> connecting to the VMs may be problematic since virt-viewer selects
> the VM by its id then by uuid if not found then by its name.
>
> Introduce new command line options to cover this situation:
> "--id" to connect to the VM by its id
> "--uuid" to connect to the VM by its uuid
> "--domain-name" to connect to the VM by its name
> The options are mutually exclusive
>
> Resolves: rhbz#1399077
> ---
> v3: Avoided switch
> ---
> man/virt-viewer.pod | 14 ++++++++++
> src/virt-viewer.c | 78
> ++++++++++++++++++++++++++++++++++++++++++++---------
> 2 files changed, 80 insertions(+), 12 deletions(-)
>
> diff --git a/man/virt-viewer.pod b/man/virt-viewer.pod
> index 9abf03c..30af8db 100644
> --- a/man/virt-viewer.pod
> +++ b/man/virt-viewer.pod
> @@ -122,6 +122,20 @@ kiosk-quit option to "on-disconnect" value,
> virt-viewer will quit
> instead. Please note that --reconnect takes precedence over this
> option, and will attempt to do a reconnection before it quits.
>
> +=item --id, --uuid, --domain-name
> +
> +Connect to the virtual machine by its id, uuid or name. These
> options
> +are mutual exclusive. For example the following command may
> sometimes
> +connect to a virtual machine with the id 2 or with the name 2
> (depending
> +on the number of running machines):
> +
> + virt-viewer 2
> +
> +To always connect to the virtual machine with the name "2" use the
> +"--domain-name" option:
> +
> + virt-viewer --domain-name 2
> +
> =back
>
> =head1 CONFIGURATION
> diff --git a/src/virt-viewer.c b/src/virt-viewer.c
> index 1f99552..22a952a 100644
> --- a/src/virt-viewer.c
> +++ b/src/virt-viewer.c
> @@ -82,6 +82,45 @@ static gboolean opt_attach = FALSE;
> static gboolean opt_waitvm = FALSE;
> static gboolean opt_reconnect = FALSE;
>
> +typedef enum {
> + DOMAIN_SELECTION_ID = (1 << 0),
> + DOMAIN_SELECTION_UUID = (1 << 1),
> + DOMAIN_SELECTION_NAME = (1 << 2),
> + DOMAIN_SELECTION_DEFAULT = DOMAIN_SELECTION_ID |
> DOMAIN_SELECTION_UUID | DOMAIN_SELECTION_NAME,
> +} DomainSelection;
> +
> +static const gchar* domain_selection_to_opt[] = {
> + [DOMAIN_SELECTION_ID] = "--id",
> + [DOMAIN_SELECTION_UUID] = "--uuid",
> + [DOMAIN_SELECTION_NAME] = "--domain-name",
> +};
> +
> +static DomainSelection domain_selection_type =
> DOMAIN_SELECTION_DEFAULT;
> +
> +static gboolean
> +opt_domain_selection_cb(const gchar *option_name,
> + const gchar *value G_GNUC_UNUSED,
> + gpointer data G_GNUC_UNUSED,
> + GError **error)
> +{
> + guint i;
> + if (domain_selection_type != DOMAIN_SELECTION_DEFAULT) {
> + g_set_error(error, G_OPTION_ERROR, G_OPTION_ERROR_FAILED,
> + "selection type has been already set");
> + return FALSE;
> + }
> +
> + for (i = DOMAIN_SELECTION_ID; i <=
> G_N_ELEMENTS(domain_selection_to_opt); i++) {
> + if (g_strcmp0(option_name, domain_selection_to_opt[i]) == 0)
> {
> + domain_selection_type = i;
> + return TRUE;
> + }
> + }
> +
> + g_assert_not_reached();
> + return FALSE;
> +}
> +
> static void
> virt_viewer_add_option_entries(VirtViewerApp *self, GOptionContext
> *context, GOptionGroup *group)
> {
> @@ -96,6 +135,12 @@ virt_viewer_add_option_entries(VirtViewerApp
> *self, GOptionContext *context, GOp
> N_("Wait for domain to start"), NULL },
> { "reconnect", 'r', 0, G_OPTION_ARG_NONE, &opt_reconnect,
> N_("Reconnect to domain upon restart"), NULL },
> + { "domain-name", '\0', G_OPTION_FLAG_NO_ARG,
> G_OPTION_ARG_CALLBACK, opt_domain_selection_cb,
> + N_("Select the virtual machine only by its name"), NULL },
> + { "id", '\0', G_OPTION_FLAG_NO_ARG, G_OPTION_ARG_CALLBACK,
> opt_domain_selection_cb,
> + N_("Select the virtual machine only by its id"), NULL },
> + { "uuid", '\0', G_OPTION_FLAG_NO_ARG, G_OPTION_ARG_CALLBACK,
> opt_domain_selection_cb,
> + N_("Select the virtual machine only by its uuid"), NULL },
> { G_OPTION_REMAINING, '\0', 0, G_OPTION_ARG_STRING_ARRAY,
> &opt_args,
> NULL, "-- DOMAIN-NAME|ID|UUID" },
> { NULL, 0, 0, G_OPTION_ARG_NONE, NULL, NULL, NULL }
> @@ -131,15 +176,16 @@ virt_viewer_local_command_line
> (GApplication *gapp,
> }
>
>
> - if (opt_waitvm) {
> + if (opt_waitvm || domain_selection_type !=
> DOMAIN_SELECTION_DEFAULT) {
> if (!self->priv->domkey) {
> - g_printerr(_("\nNo DOMAIN-NAME|ID|UUID was specified for
> '--wait'\n\n"));
> + g_printerr(_("\nNo DOMAIN-NAME|ID|UUID was specified for
> '%s'\n\n"),
> + opt_waitvm ? "--wait" :
> domain_selection_to_opt[domain_selection_type]);
> ret = TRUE;
> *status = 1;
> goto end;
> }
>
> - self->priv->waitvm = TRUE;
> + self->priv->waitvm = opt_waitvm;
> }
>
> virt_viewer_app_set_direct(app, opt_direct);
> @@ -303,24 +349,32 @@ virt_viewer_lookup_domain(VirtViewer *self)
> {
> char *end;
> VirtViewerPrivate *priv = self->priv;
> - int id;
> virDomainPtr dom = NULL;
> - unsigned char uuid[16];
>
> if (priv->domkey == NULL) {
> return NULL;
> }
>
> - id = strtol(priv->domkey, &end, 10);
> - if (id >= 0 && end && !*end) {
> - dom = virDomainLookupByID(priv->conn, id);
> + if (domain_selection_type & DOMAIN_SELECTION_ID) {
> + long int id = strtol(priv->domkey, &end, 10);
> + if (id >= 0 && end && !*end) {
> + dom = virDomainLookupByID(priv->conn, id);
> + }
> }
> - if (!dom && virt_viewer_parse_uuid(priv->domkey, uuid) == 0) {
> - dom = virDomainLookupByUUID(priv->conn, uuid);
> +
> + if (domain_selection_type & DOMAIN_SELECTION_UUID) {
> + unsigned char uuid[16];
> + if (dom == NULL && virt_viewer_parse_uuid(priv->domkey,
> uuid) == 0) {
> + dom = virDomainLookupByUUID(priv->conn, uuid);
> + }
> }
> - if (!dom) {
> - dom = virDomainLookupByName(priv->conn, priv->domkey);
> +
> + if (domain_selection_type & DOMAIN_SELECTION_NAME) {
> + if (dom == NULL) {
> + dom = virDomainLookupByName(priv->conn, priv->domkey);
> + }
> }
> +
> return dom;
> }
>
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
