Re: [libvirt] [PATCH] Activate virtual networks initialized in custom test driver.

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

 



On Fri, Jun 19, 2009 at 12:37:33PM -0400, Cole Robinson wrote:
> If specifying a custom test driver, virtual networks were not 'activated'
> on driver init. This differs from the behavior of domains and storage pools,
> so fix it. Also improve a couple error messages in that area.
> 
> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> ---
>  src/test.c |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)

ACK

> 
> diff --git a/src/test.c b/src/test.c
> index 1661144..6f07462 100644
> --- a/src/test.c
> +++ b/src/test.c
> @@ -462,7 +462,8 @@ static int testOpenFromFile(virConnectPtr conn,
>      if (!(xml = xmlReadFd(fd, file, NULL,
>                            XML_PARSE_NOENT | XML_PARSE_NONET |
>                            XML_PARSE_NOERROR | XML_PARSE_NOWARNING))) {
> -        testError(NULL, VIR_ERR_INTERNAL_ERROR, "%s", _("host"));
> +        testError(NULL, VIR_ERR_INTERNAL_ERROR,
> +                  _("Invalid XML in file '%s'"), file);
>          goto error;
>      }
>      close(fd);
> @@ -470,7 +471,8 @@ static int testOpenFromFile(virConnectPtr conn,
>  
>      root = xmlDocGetRootElement(xml);
>      if ((root == NULL) || (!xmlStrEqual(root->name, BAD_CAST "node"))) {
> -        testError(NULL, VIR_ERR_XML_ERROR, "%s", _("node"));
> +        testError(NULL, VIR_ERR_XML_ERROR, "%s",
> +                  _("Root element is not 'node'"));
>          goto error;
>      }
>  
> @@ -622,6 +624,7 @@ static int testOpenFromFile(virConnectPtr conn,
>              goto error;
>          }
>          net->persistent = 1;
> +        net->active = 1;
>          virNetworkObjUnlock(net);
>      }
>      VIR_FREE(networks);
> -- 
> 1.6.3.2
> 
> --
> Libvir-list mailing list
> Libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

--
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]