Skip to content

Conversation

meisterT
Copy link
Member

@meisterT meisterT commented Oct 5, 2025

No description provided.

@meisterT meisterT added this pull request to the merge queue Oct 5, 2025
Merged via the queue into DOMjudge:main with commit 07d0ace Oct 5, 2025
3 checks passed
@meisterT meisterT deleted the 900 branch October 5, 2025 19:38
@pobrn
Copy link

pobrn commented Oct 6, 2025

Should this be already available on the docker hub? I can see that the digest shown in https://github.com/DOMjudge/domjudge-packaging/actions/runs/18263427940/job/51994437966#step:9:40 is displayed at https://hub.docker.com/r/domjudge/domserver/tags as 9.0.0. But still:

$ docker pull domjudge/domserver:9.0.0
9.0.0: Pulling from domjudge/domserver
[...]
Digest: sha256:c0a634317e7fa0f44e43504f37a33964308e0a6aab15acda917567cc6fa60652
Status: Downloaded newer image for domjudge/domserver:9.0.0
docker.io/domjudge/domserver:9.0.0
$ docker run --rm -it domjudge/domserver:9.0.0 bash -c "apt list --installed | grep bcmath"
$

What am I missing?

@nickygerritsen
Copy link
Member

Should this be already available on the docker hub? I can see that the digest shown in DOMjudge/domjudge-packaging/actions/runs/18263427940/job/51994437966#step:9:40 is displayed at hub.docker.com/r/domjudge/domserver/tags as 9.0.0. But still:

$ docker pull domjudge/domserver:9.0.0
9.0.0: Pulling from domjudge/domserver
[...]
Digest: sha256:c0a634317e7fa0f44e43504f37a33964308e0a6aab15acda917567cc6fa60652
Status: Downloaded newer image for domjudge/domserver:9.0.0
docker.io/domjudge/domserver:9.0.0
$ docker run --rm -it domjudge/domserver:9.0.0 bash -c "apt list --installed | grep bcmath"
$

What am I missing?

Good catch. I'm fixing it in #225

@nickygerritsen
Copy link
Member

docker run --rm -it domjudge/domserver:9.0.0 bash -c "apt list --installed | grep bcmath"

It should be updated now 😄 .

@pobrn
Copy link

pobrn commented Oct 6, 2025

It is, yes, thanks.

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