sagemath/sage

tox -e docker-...-incremental

mkoeppe opened this issue · 30 comments

This does an incremental build of Sage on top of a prebuilt image published at ghcr.io (https://github.com/orgs/sagemath/packages?tab=packages)

For example:

$ tox -e docker-fedora-31-standard-incremental

CC: @dimpase @kliem

Component: porting

Author: Matthias Koeppe

Branch/Commit: 85ce9a6

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/34228

Commit: a9eaae5

New commits:

a9eaae5tox.ini, build/bin/write-dockerfile.sh: Add 'tox -e docker-...-incremental'

Description changed:

--- 
+++ 
@@ -1,2 +1,8 @@
 This does an incremental build of Sage on top of a prebuilt image published at ghcr.io (https://github.com/orgs/sagemath/packages?tab=packages)
 
+For example:
+
+```
+$ tox -e docker-fedora-31-standard-incremental
+```
+

Author: Matthias Koeppe

Branch pushed to git repo; I updated commit sha1. New commits:

b9bfbf9tox.ini: Add comment

Changed commit from a9eaae5 to b9bfbf9

Changed commit from b9bfbf9 to 4c0d7f5

Branch pushed to git repo; I updated commit sha1. New commits:

4c0d7f5tox.ini: Use FROM_DOCKER_REPOSITORY

Branch pushed to git repo; I updated commit sha1. New commits:

a07874dbuild/bin/write-dockerfile.sh: In incremental build, keep logs

Changed commit from 4c0d7f5 to a07874d

comment:6

Ready for review

Changed commit from a07874d to ae269d1

Branch pushed to git repo; I updated commit sha1. New commits:

ae269d1tox.ini (docker-incremental): Do not include '-incremental' in the Docker image name
comment:8

does not work with devtoolset

Branch pushed to git repo; I updated commit sha1. New commits:

b354a38tox.ini, .github/workflows/docker.yml: Reimplement -incremental via SKIP_SYSTEM_PACKAGES
5decaddbuild/bin/write-dockerfile.sh: Do not use persistent env var PACKAGES (except on nix)

Changed commit from ae269d1 to 5decadd

Changed commit from 5decadd to a90608f

Branch pushed to git repo; I updated commit sha1. New commits:

adddf8eMerge tag '9.7.beta8' into t/34228/tox__e_docker_____incremental
a90608fbuild/bin/write-dockerfile.sh: Invoke sage-package directly, do not go through ./sage

Branch pushed to git repo; I updated commit sha1. New commits:

85ce9a6Merge tag '9.7.rc0' into t/34228/tox__e_docker_____incremental

Changed commit from a90608f to 85ce9a6

comment:14

Let's get this in please

comment:15

ping

comment:16

sorry, can't review, docker issues on one machine I have it.

(hilariosly, another laptop, with macOS, has docker just fine, but it's misplaced somewhere in our messy house, hopefully I'll bump into it soon...)

comment:18

lgtm

Reviewer: Dima Pasechnik

comment:19

Thank you!

comment:20

Marking it as critical because a long chain of tickets depends on it