On Thu, Aug 31, 2023 at 02:14:49AM -0700, Andrea Bolognani wrote: > On Thu, Aug 31, 2023 at 09:50:07AM +0100, Daniel P. Berrangé wrote: > > I don't much like it to be honest, as I was hoping we would get away > > from having it exist in any new installs, such that people were not > > mis-directed into trying to use it. > > > > I'm not seeing a good way to deal with the upgrade problem though. > > Yeah. The two alternatives that I was able to come up with are > > 1) documenting in the release notes for Fedora and RHEL that > people using the monolithic daemon need to run > > $ dnf mark install libvirt-daemon > > before performing the upgrade; > > 2) in libvirt-daemon's %postinst, detect whether libvirtd is > enabled and if so run the command above. > > I don't like 1) because it's really easy to miss something like that, > and if we're being honest most people don't even read the release > notes before performing an upgrade. The expectation is that things > will just work without user intervention, and I don't think it's an > unreasonable one. > > 2) would work but it feels so hacky that I didn't really consider > proposing it as an actual patch.
Isnt %postinst too late in the upgrade process for this to work ? > > Possibly the next step would be to stop building libvirtd by > > default in upstream releases[1], and figure out a way to attempt > > to auto switch installs to modular daemons during upgrade. > > We have purposefully avoided converting monolithic deployments to > modular ones so far, but if we want to ever be able to drop the > monolithic daemon I'm afraid that at some point we'll have to take > the plunge. The alternatives are breaking all monolithic deployments > or carrying it around forever. Yep, we'll have to bite the bullet eventually and make a decision, but for now lets keep our heads in the sand a bit longer :-) Meanwhile to be explicit about your current patch Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> With 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 :|
