[PATCH] libxl: Fix driver reload
Michal Prívozník
mprivozn at redhat.com
Tue Sep 14 10:23:08 UTC 2021
On 9/14/21 12:10 AM, Jim Fehlig wrote:
> On reload, the libxl driver calls virDomainObjListLoadAllConfigs to load
> all configs from /etc/libvirt/libxl/ but incorrectly passes 'true' for
> the liveStatus parameter, resulting in error messages such as
>
> libvirtd[21053]: XML error: unexpected root element <domain>, expecting <domstatus>
> libvirtd[21053]: Failed to load config for domain 'sles15sp3'
>
> Fix by not requesting live status when re-reading the persistent VM config
> files.
>
> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
> ---
> src/libxl/libxl_driver.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
> index 6a3938ead4..c5dbcaafa5 100644
> --- a/src/libxl/libxl_driver.c
> +++ b/src/libxl/libxl_driver.c
> @@ -821,7 +821,7 @@ libxlStateReload(void)
> virDomainObjListLoadAllConfigs(libxl_driver->domains,
> cfg->configDir,
> cfg->autostartDir,
> - true,
> + false,
> libxl_driver->xmlopt,
> NULL, libxl_driver);
>
>
Reviewed-by: Michal Privoznik <mprivozn at redhat.com>
But I've noticed that on reload, below this hunk is a line that
autostarts domains. This is a deviation from other drivers - should we
autostart domains on daemon reload?
Michal
More information about the libvir-list
mailing list