Skip to content

Comments

Update Dockerfile for BLAST 2.15.0#743

Merged
yannickwurm merged 1 commit intowurmlab:masterfrom
nathanweeks:docker-2.15.0
May 29, 2024
Merged

Update Dockerfile for BLAST 2.15.0#743
yannickwurm merged 1 commit intowurmlab:masterfrom
nathanweeks:docker-2.15.0

Conversation

@nathanweeks
Copy link
Contributor

SequenceServer 3.0.1 supports BLAST 2.15.0, though the Dockerfile currently specifies 2.14.0:

ARG BLAST_VERSION=2.14.0

This PR proposes updating the Dockerfile for BLAST 2.15.0, with several other ancillary modifications:

  • Facilitate arm64 container builds by using ncbi/blast-static (which has linux/arm64 images)
  • Use ruby "bookworm" base images instead of "bullseye" to fix resulting GLIBCXX_<version> not found errors
  • While devops best practices generally involve pinning dependencies as precisely as possible for reproducibility, suggest not pinning ruby patch version (i.e., (ruby:3.2 instead of ruby:3.2.<patch>) so newer/patched versions are more easily/likely used during development & production builds
  • Update node base image from (EOL) version 15 to (LTS) version 20.
  • Explicitly specify container registry for build environments where docker.io is not the default

@yannickwurm yannickwurm requested a review from tadast May 27, 2024 16:45
Copy link
Collaborator

@tadast tadast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks @nathanweeks 🙌

@yannickwurm yannickwurm merged commit e84332b into wurmlab:master May 29, 2024
@nathanweeks nathanweeks deleted the docker-2.15.0 branch May 29, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants