Re: [PATCH 17/26] conf: implement support for autostart once feature

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

 



On Wed, Jan 08, 2025 at 19:42:50 +0000, Daniel P. Berrangé wrote:
> This is maintained in the same way as the autostart flag, using a
> symlink. The difference is that instead of '.xml', the symlink
> suffix is '.xml.once'. The link is also deleted immediately after
> it has been read.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> ---
>  src/conf/domain_conf.c      | 6 +++++-
>  src/conf/domain_conf.h      | 1 +
>  src/conf/virdomainobjlist.c | 7 ++++++-
>  3 files changed, 12 insertions(+), 2 deletions(-)

[...]

> diff --git a/src/conf/virdomainobjlist.c b/src/conf/virdomainobjlist.c
> index 72207450c5..ba2f9f544d 100644
> --- a/src/conf/virdomainobjlist.c
> +++ b/src/conf/virdomainobjlist.c
> @@ -487,9 +487,10 @@ virDomainObjListLoadConfig(virDomainObjList *doms,
>  {
>      g_autofree char *configFile = NULL;
>      g_autofree char *autostartLink = NULL;
> +    g_autofree char *autostartOnceLink = NULL;
>      g_autoptr(virDomainDef) def = NULL;
>      virDomainObj *dom;
> -    int autostart;
> +    int autostart, autostartOnce;

Preferrably one per line.

>      g_autoptr(virDomainDef) oldDef = NULL;
>  
>      configFile = virDomainConfigFile(configDir, name);

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>




[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