On Mon, Jan 18, 2021 at 05:13:16PM +0100, Gerd Hoffmann wrote: > Hi, > > > Or we could have a more generic "display-reload" command, which has > > fields indicating what aspect to reload. eg a 'tls-certs: bool' field > > to indicate reload of TLS certs in the display. This could be useful > > if we wanted the ability to reload authz access control lists, though > > we did actually wire them up to auto-reload using inotify. > > Maybe we should just use inotify-based reload for the certs too?
The authz access control is easy because it is just one file. When updating the certs though, we have 1-4 files that need loading, and they can only be reloaded once all of them are updated on disk. This gives a synchronization challenge for use of inotify, as when we see 1 updated, we don't know if we need to wait for the others to be updated or not. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
