mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-23 13:05:27 +00:00
libxl: use job functions in libxlDomainSetAutostart
Setting autostart is a modify operation that needs to wait in the queue of modify jobs. Signed-off-by: Jim Fehlig <jfehlig@suse.com>
This commit is contained in:
parent
85ff3d7aec
commit
7d9ff81603
@ -3866,40 +3866,43 @@ libxlDomainSetAutostart(virDomainPtr dom, int autostart)
|
||||
if (virDomainSetAutostartEnsureACL(dom->conn, vm->def) < 0)
|
||||
goto cleanup;
|
||||
|
||||
if (libxlDomainObjBeginJob(driver, vm, LIBXL_JOB_MODIFY) < 0)
|
||||
goto cleanup;
|
||||
|
||||
if (!vm->persistent) {
|
||||
virReportError(VIR_ERR_OPERATION_INVALID,
|
||||
"%s", _("cannot set autostart for transient domain"));
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
}
|
||||
|
||||
autostart = (autostart != 0);
|
||||
|
||||
if (vm->autostart != autostart) {
|
||||
if (!(configFile = virDomainConfigFile(cfg->configDir, vm->def->name)))
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
if (!(autostartLink = virDomainConfigFile(cfg->autostartDir, vm->def->name)))
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
|
||||
if (autostart) {
|
||||
if (virFileMakePath(cfg->autostartDir) < 0) {
|
||||
virReportSystemError(errno,
|
||||
_("cannot create autostart directory %s"),
|
||||
cfg->autostartDir);
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
}
|
||||
|
||||
if (symlink(configFile, autostartLink) < 0) {
|
||||
virReportSystemError(errno,
|
||||
_("Failed to create symlink '%s to '%s'"),
|
||||
autostartLink, configFile);
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
}
|
||||
} else {
|
||||
if (unlink(autostartLink) < 0 && errno != ENOENT && errno != ENOTDIR) {
|
||||
virReportSystemError(errno,
|
||||
_("Failed to delete symlink '%s'"),
|
||||
autostartLink);
|
||||
goto cleanup;
|
||||
goto endjob;
|
||||
}
|
||||
}
|
||||
|
||||
@ -3907,6 +3910,10 @@ libxlDomainSetAutostart(virDomainPtr dom, int autostart)
|
||||
}
|
||||
ret = 0;
|
||||
|
||||
endjob:
|
||||
if (!libxlDomainObjEndJob(driver, vm))
|
||||
vm = NULL;
|
||||
|
||||
cleanup:
|
||||
VIR_FREE(configFile);
|
||||
VIR_FREE(autostartLink);
|
||||
|
Loading…
x
Reference in New Issue
Block a user