[Pkg-libvirt-maintainers] Bug#1040783: libvirt-daemon: libvirt firewalld zone is missing

[Andrea Bolognani]

Control: reassign -1 firewalld

Ugh. Sorry for the noise.

On Sun, Apr 13, 2025 at 07:35:06PM +0200, Andrea Bolognani wrote:
> Control: reassing -1 firewalld
> Control: retitle -1 firewalld: Please introduce a trigger for zones installed by other packages
> Control: affects -1 libvirt
> 
> On Sun, Apr 13, 2025 at 09:28:58AM +0200, Guido Günther wrote:
> > On Sun, Apr 13, 2025 at 01:15:28AM +0200, Andrea Bolognani wrote:
> > > I think what might be happening is that we don't have any code in the
> > > libvirt package that matches the following snippet from the upstream
> > > spec file:
> > > 
> > >   %post daemon-driver-network
> > >     %if %{with_firewalld_zone}
> > >     %firewalld_reload
> > >     %endif
> > 
> > Nice find!
> >  
> > > It would probably make sense to at least attempt to reload firewalld
> > > when the network driver is installed, the way upstream and Fedora
> > > already do.
> > 
> > Instead of all packages shipping zones doing this firewalld should
> > likely better register a trigger, like e.g. glib does for new schemas:
> > 
> > /usr/share/glib-2.0/schemas libglib2.0-0t64:amd64
> > 
> > in /var/lib/dpkg/triggers/File
> > 
> > so probably best to move the issue there?
> 
> Agreed, that would probably be the best way to handle things.

-- 
Andrea Bolognani <eof at kiyuko.org>
Resistance is futile, you will be garbage collected.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/pkg-libvirt-maintainers/attachments/20250413/95768881/attachment.sig>