Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace dvisvgm builder with apt package #1770

Merged
merged 8 commits into from
Aug 18, 2024

Conversation

felixvanoost
Copy link
Contributor

@felixvanoost felixvanoost commented Aug 12, 2024

  • Replaces the kroki-builder-dvisvgm Docker builder image with the now available apt package. dvisvgm is available from Ubuntu 24.04 onwards.
  • Updates the base and builder images to use Ubuntu 24.04.

@ggrossetie
Copy link
Member

7.353 info: Selecting GID from range 1000 to 59999 ...
7.355 info: Adding group `kroki' (GID 1001) ...
7.393 fatal: The UID 1000 is already in use.

I think this is a mistake in the Dockerfile as we should use 1001 GID and not 1000. I guess the latest version of adduser is more strict and fail if the GID is already taken.

@ggrossetie
Copy link
Member

Not sure why but symbolator is failing 🤔

@ggrossetie
Copy link
Member

Still failing, my guess is that symbolator relies on a system dependency that was removed/updated. Do you reproduce this issue locally?

@felixvanoost
Copy link
Contributor Author

Still failing, my guess is that symbolator relies on a system dependency that was removed/updated. Do you reproduce this issue locally?

I've been having issues running the project locally, hence my over-reliance on the CI here. 🙃

Symbolator was failing because I forgot to update the builder image to Ubuntu 24.04, so it was compiled for Python 3.11 instead of 3.12.

@ggrossetie
Copy link
Member

Perfect, thanks!

@ggrossetie ggrossetie merged commit 3580756 into yuzutech:main Aug 18, 2024
1 check passed
@felixvanoost felixvanoost deleted the use-dvisvgm-apt-package branch August 18, 2024 14:41
jynolen pushed a commit to jynolen/kroki that referenced this pull request Dec 3, 2024
Use pre-built Docker image for erd (yuzutech#1754)

Restart browser if killed on Mermaid (yuzutech#1745)

chore(deps): update java dependencies (yuzutech#1722)

chore(deps): update structurizr packages to v2.2.0 (yuzutech#1757)

chore(docs): add Ditaa diagram options to diagram-options page (yuzutech#1781)

Co-authored-by: Guillaume Grossetie <[email protected]>

chore(node): upgrade to Node LTS 20.18.0 (yuzutech#1784)

chore(deps): use Node built-in test runner (yuzutech#1785)

chore(docs): add hugomods-kroki to list of 3rd-party tools (yuzutech#1724)

chore(node): replace mocha + chai by Node test runner and assert modules (yuzutech#1786)

chore(deps): replace mocha by Node test runner (yuzutech#1787)

chore(bpmn): upgrade dependencies (docker base image, puppeteer, bpmn, pino) (yuzutech#1788)

chore(deps): update java testing packages (yuzutech#1772)

chore(deps): update dependency rust to v1.82 (yuzutech#1791)

chore(deps): update node dependencies (yuzutech#1737)

chore(deps): update actions/checkout action to v4.2.1 (yuzutech#1790)

chore(deps): update maven plugins (yuzutech#1755)

chore(docs): introduce helm deploy method in use kubernetes section (yuzutech#1793)

Co-authored-by: Guillaume Grossetie <[email protected]>

chore(deps): update java dependencies (yuzutech#1789)

chore(deps): update structurizr packages to v3 (major) (yuzutech#1792)

chore(deps): upgrade Mermaid to 11.3.0 (yuzutech#1795)

chore(deps): bump puppeteer and pino in diagrams.net (yuzutech#1794)

chore(deps): update node dependencies (major) (yuzutech#1671)

chore(deps): update dependency org.apache.maven.plugins:maven-surefire-plugin to v3.5.2 (yuzutech#1801)

chore(deps): update java dependencies to v1.5.12 (yuzutech#1798)

chore(deps): update actions/checkout action to v4.2.2 (yuzutech#1797)

chore(deps): update dependency mermaid to v11.4.0 (yuzutech#1800)

chore(deps): update versions and distributions

fix(ditaa): pass argument values as separate arguments

ref yuzutech#1803

Version 0.26.0

chore(release): free up disk space on GitHub runner

chore(release): allow to manually push a release

chore(release): keep the v prefix when creating the release

chore(release): fix environment variable assignment

chore(release): dynamic run-name when using workflow_dispatch

feat: proposal without cheerio

fix: lint + kroki unsafe

chore(deps): update node dependencies (yuzutech#1737)

chore(deps): bump puppeteer and pino in diagrams.net (yuzutech#1794)

feat: self contains svg
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