Re: [PATCH v4] virsh: add [--domain DOMAIN] option to domxml-to-native DOMAIN COMMAND

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



My apologies. I didn't read the function logic.
The second error was introduced by my "fix".
The error with help parameter is happening.

If --domain is required, I believe that moving to STRING
is enough. As I said, VSH_OT_DATA requires
VSH_OFLAG_REQ. See vsh.c line 746.

     {.name = "domain",
-     .type = VSH_OT_DATA,
+     .type = VSH_OT_STRING,
      .flags = VSH_OFLAG_REQ_OPT,
      .help = N_("domain name, id or uuid")
     },
     {.name = "xml",
-     .type = VSH_OT_DATA,
+     .type = VSH_OT_STRING,
      .help = N_("xml data file to export from")

2017-06-24 17:54 GMT-03:00 Julio Faracco <jcfaracco@xxxxxxxxx>:
> Hi guys,
>
> I updated the source this weekend, I missed the ability of calling help.
> virsh # domxml-to-native --help
>   NAME
>     domxml-to-native - Convert domain XML to native config
>
>   SYNOPSIS
>     domxml-to-native <format> [<domain>] [<xml>]
>
>   DESCRIPTION
>     Convert domain XML config to a native guest configuration format.
>
>   OPTIONS
>     [--format] <string>  target config data type format
> error: internal error: bad options in command: 'domxml-to-native'
>
>
> This is why:
> --- a/tools/virsh-domain.c
> +++ b/tools/virsh-domain.c
> @@ -9858,11 +9858,11 @@ static const vshCmdOptDef opts_domxmltonative[] = {
>      },
>      {.name = "domain",
>       .type = VSH_OT_DATA,
> -     .flags = VSH_OFLAG_REQ_OPT,
> +     .flags = VSH_OFLAG_REQ,
>       .help = N_("domain name, id or uuid")
>      },
>      {.name = "xml",
> -     .type = VSH_OT_DATA,
> +     .type = VSH_OT_STRING,
>       .help = N_("xml data file to export from")
>      },
>      {.name = NULL}
>
> VSH_OT_DATA requires VSH_OFLAG_REQ.
> So, since XML is not required...
> This diff fits this case. But I'm still confused.
> Because I cannot check my XML files right now.
>
> virsh # domxml-to-native qemu-argv /home/julio/WINDOWS_7.xml
> error: failed to get domain '/home/julio/WINDOWS_7.xml'
> error: Domain not found: no domain with matching name
> '/home/julio/WINDOWS_7.xml'
>
> 2017-06-23 6:38 GMT-03:00 Martin Kletzander <mkletzan@xxxxxxxxxx>:
>> On Thu, Jun 22, 2017 at 06:21:49PM -0400, John Ferlan wrote:
>>>
>>>
>>> [...]
>>>
>>>>>>>
>>>>>>
>>>>>> There was no change, it is an additional variable, the original one is
>>>>>> below.  The number of differences would be the same, I believe.
>>>>>>
>>>>>
>>>>> If edit the file and change "xml" to "xmlFile" and change the 3 changed
>>>>> xml variable references things work... Like I said, nit, IDC if it's
>>>>> changed or not...
>>>>>
>>>>
>>>> My bad, I misread that, you're right.
>>>
>>>
>>> In order to "close" on this, if a squash the attach patch does that work
>>> for everyone?
>>>
>>> John
>>
>>
>> WFM
>>
>> Reviewed-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
>>
>> --
>> libvir-list mailing list
>> libvir-list@xxxxxxxxxx
>> https://www.redhat.com/mailman/listinfo/libvir-list

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux