adler32 checksum should be optional

[davidgcameron]

Motivation

A community may decide to use md5 checksums instead of adler32, but using the generic schema, giving a None adler32 to the replica API is not allowed:

Rucio exception: Provided object does not match schema.
Details: Problem validating dids : None is not of type 'string'

Failed validating 'type' in schema['items']['properties']['adler32']:
    {'description': 'adler32',
     'pattern': '^[a-fA-F\\d]{8}$',
     'type': 'string'}

On instance[0]['adler32']:
    None

Modification

adler32 checksums should be optional at the schema level and also in the client API:

def add_replica(self, rse, scope, name, bytes, adler32, pfn=None, md5=None, meta={}):

[davidgcameron]

In addition rucio list-file-replicas should show the checksum type that is stored instead of always adler32.

[gabrielefronze]

Hello @davidgcameron ,
I worked a bit on checksums in the past and I'll try to figure how to take care of this issue!
Cheers,

Gabriele

[bari12]

The schema level can be done with a VO specific schema. But on the method level I agree we shouldn't require it.
Thanks @gabrielefronze 😃