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(a)suse.com&gt; > --- > 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(a)redhat.com&gt;

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?