Skip to content

AO3-7340 Production-ized Dockerfile#5836

Draft
brianjaustin wants to merge 6 commits into
otwcode:masterfrom
brianjaustin:AO3-7340-docker
Draft

AO3-7340 Production-ized Dockerfile#5836
brianjaustin wants to merge 6 commits into
otwcode:masterfrom
brianjaustin:AO3-7340-docker

Conversation

@brianjaustin
Copy link
Copy Markdown
Member

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-7340

Purpose

Creates a new dockerfile that can be used to run in the archive in production. As part of this, I also removed old migrations to avoid dependency issues with Departure -- we shouldn't need it now that MariaDB has online schema migrations

Credit

What name and pronouns should we use to credit you in the Archive of Our Own's Release Notes?

Brian Austin (they/he)

@github-actions github-actions Bot added Has Migrations Contains migrations and therefore needs special attention when deploying Gem Updates Awaiting Review labels May 23, 2026
Comment thread Dockerfile
# Install runtime dependencies, e.g. for downloads generation and online database migrations
RUN apt-get update -qq && \
apt-get install --no-install-recommends -y calibre libmariadb3 percona-toolkit shared-mime-info zip && \
rm -rf /var/lib/apt/lists /var/cache/apt/archives
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would prefer

apt autoremove && apt clean

Comment thread Dockerfile
# Install packages needed to build gems
RUN apt-get update -qq && \
apt-get install --no-install-recommends -y build-essential git libmariadb-dev libyaml-dev pkg-config && \
rm -rf /var/lib/apt/lists /var/cache/apt/archives
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

as below

Comment thread Dockerfile
RUN apt-get update -qq && \
apt-get install --no-install-recommends -y curl libjemalloc2 libvips sqlite3 && \
ln -s /usr/lib/$(uname -m)-linux-gnu/libjemalloc.so.2 /usr/local/lib/libjemalloc.so && \
rm -rf /var/lib/apt/lists /var/cache/apt/archives
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

and again

@zz9pzza zz9pzza added Reviewed: Ready to Merge and removed Awaiting Review Has Migrations Contains migrations and therefore needs special attention when deploying labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants