Re: [PATCH] scripts: Allow sha512 in clean_functional_cache.py, too

["Daniel P. Berrangé" <berrange@redhat.com>]

On Tue, Jan 20, 2026 at 03:10:34PM +0100, Thomas Huth wrote:
> On 20/01/2026 14.02, Daniel P. Berrangé wrote:
> > On Tue, Jan 20, 2026 at 01:18:49PM +0100, Thomas Huth wrote:
> > > From: Thomas Huth <thuth@redhat.com>
> > > 
> > > The functional testing frameworks also supports assets that are
> > > identified by their sha512 checksum (instead of only using sha25),
> > > and at least one of the tests (tests/functional/ppc64/test_fadump.py)
> > > is already using such a checksum, so adjust the clean_functional_cache
> > > script to support these checksums, too.
> > 
> > I wrote this code to match what we had in Avocado, but now I'm really
> > struggling to justify why we should choose different checksum algorithms
> > per asset. With only 1 test diverging from sha256, I'd rather we just
> > dropped sha512 support
> 
> Fine for me, too ... could you suggest a patch, or want me to do it?

I'll let you do it

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 :|