search

umccr / htsget-rs

A server implementation of the htsget protocol for bioinformatics in Rust
https://samtools.github.io/hts-specs/htsget.html
MIT License
39 stars 9 forks source link

Tweak docker container labels (was: Docker SOCI support) #236

Closed brainstorm closed 7 months ago

brainstorm commented 7 months ago

Outcome of this PR w.r.t SOCI is that we don't have layers < 10MB (total docker image size is currently ~50MB), so no ztocs indexes are generated and the CI build fails. If and when the layers grow in the future, we can re-evaluate this.

Docker LABELs

https://github.com/opencontainers/image-spec/blob/main/annotations.md

SOCI

https://aws.amazon.com/blogs/containers/under-the-hood-lazy-loading-container-images-with-seekable-oci-and-aws-fargate/

https://github.com/awslabs/soci-snapshotter/discussions/949

brainstorm commented 7 months ago

@mmalenic I'm leaving the platforms: ${{ matrix.platforms }} in the CI as well as a reminder to switch to a job matrix once we get ARM runners (instead of QEMU): https://github.com/orgs/community/discussions/73040#discussioncomment-8658172

It shouldn't fail since there's only one platform in the matrix currently... if it does I'll comment/remove it.