Skip to content

🐛 Fix Docker build by switching PlantUML source to GitHub releases#1708

Merged
ubmarco merged 1 commit into
masterfrom
fix/docker-plantuml-url
May 20, 2026
Merged

🐛 Fix Docker build by switching PlantUML source to GitHub releases#1708
ubmarco merged 1 commit into
masterfrom
fix/docker-plantuml-url

Conversation

@ubmarco
Copy link
Copy Markdown
Member

@ubmarco ubmarco commented May 20, 2026

Summary

  • The Docker-Image workflow has been failing because the Dockerfile hardcodes a specific Sourceforge mirror (netcologne.dl.sourceforge.net) that is no longer reachable, causing wget to exit with code 4 (network failure). See failed run.
  • Switched the download URL to the official GitHub releases endpoint https://github.com/plantuml/plantuml/releases/latest/download/plantuml.jar, which is mirror-free and always serves the latest plantuml.jar asset.
  • Preserves unpinned-latest behavior of the original line.

Test plan

  • CI Docker-Image workflow triggered by this PR (path filter docker/**) completes successfully for both sphinxdoc/sphinx:latest and sphinxdoc/sphinx-latexpdf:latest base images.

The Sourceforge mirror netcologne.dl.sourceforge.net used in the
Dockerfile is no longer reachable, causing the Docker-Image workflow
to fail with wget exit code 4 (network failure). Switch to the
official GitHub releases URL, which is reliable and always serves
the latest plantuml.jar.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.32%. Comparing base (4e10030) to head (2fe5e97).
⚠️ Report is 282 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1708      +/-   ##
==========================================
+ Coverage   86.87%   89.32%   +2.44%     
==========================================
  Files          56       72      +16     
  Lines        6532    10347    +3815     
==========================================
+ Hits         5675     9242    +3567     
- Misses        857     1105     +248     
Flag Coverage Δ
pytests 89.32% <ø> (+2.44%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ubmarco ubmarco requested a review from chrisjsewell May 20, 2026 10:01
@ubmarco ubmarco merged commit 9fa98e6 into master May 20, 2026
26 checks passed
@ubmarco ubmarco deleted the fix/docker-plantuml-url branch May 20, 2026 10:54
@ubmarco ubmarco added this to the 8.1.1 milestone May 20, 2026
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.

2 participants