Skip to content

0.19.12-beta.4 was build with debug profile #5718

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

Closed
Nutomic opened this issue May 30, 2025 · 7 comments · Fixed by #5719
Closed

0.19.12-beta.4 was build with debug profile #5718

Nutomic opened this issue May 30, 2025 · 7 comments · Fixed by #5719
Labels
bug Something isn't working

Comments

@Nutomic
Copy link
Member

Nutomic commented May 30, 2025

Relevant build log

For comparison 0.19.11

For some reason the parameter RUST_RELEASE_MODE=release was not passed properly, instead it was set to *=RUST_RELEASE_MODE=release (log for 0.19.12-beta.4, log for 0.19.11).

I noticed this because a debug assertion was triggered on lemmy.ml. It can also be seen that all Docker images starting from 0.19.12-beta.0 have a size of 92 mb, compared to 69 mb for previous versions.

For now I built a release image locally and uploaded it directly to lemmy.ml. Not sure what caused this problem or how to fix it, as there were no changes to that part of .woodpecker.yml. There was a new version 6.0 for woodpeckerci/plugin-docker-buildx but the changelog doesnt mention anything relevant.

Edit: Change is described in this issue, lets see if that works (ci link).

@Nutomic Nutomic added the bug Something isn't working label May 30, 2025
@Nothing4You
Copy link
Collaborator

the issue you linked indicates that this was changed from a list to a mapping. odd that that wouldn't be mentioned in the release notes.
you seem to not have included the woodpecker change in the latest tag though.

@Nutomic
Copy link
Member Author

Nutomic commented May 30, 2025

The fix is here: deca323

But its still passing the build arg wrong: https://woodpecker.join-lemmy.org/repos/129/pipeline/14002/5#L102

@Nothing4You
Copy link
Collaborator

https://codeberg.org/woodpecker-plugins/docker-buildx/pulls/229

BREAKING CHANGES:

cache_from is now a list instead of a string
args is now an object instead of a string

that should probably have been part of the release notes directly.

the correct change would be changing

      build_args:
        - RUST_RELEASE_MODE=release

to

      build_args:
        RUST_RELEASE_MODE: release

in both places where this is used.

@Nutomic
Copy link
Member Author

Nutomic commented May 30, 2025

Ah right its used twice, fixed now.

@Nothing4You
Copy link
Collaborator

even if it does work, it probably should still use a mapping rather than just a string, as the mapping is the documented way to use it: https://codeberg.org/woodpecker-plugins/docker-buildx/src/branch/main/docs.md#examples

@Nutomic
Copy link
Member Author

Nutomic commented May 30, 2025

Right it was still failing, now I got it.

https://woodpecker.join-lemmy.org/repos/129/pipeline/14010/5#L102

@Nothing4You Nothing4You linked a pull request May 30, 2025 that will close this issue
@Nutomic
Copy link
Member Author

Nutomic commented May 30, 2025

Yep looks fine now, docker image is back down to 69mb and working on lemmy.ml without problems.

dessalines pushed a commit that referenced this issue May 30, 2025
* Fix build_args for release builds (fixes #5718)

* fix syntax
dessalines added a commit that referenced this issue May 30, 2025
* Fix build_args for release builds (fixes #5718)

* fix syntax

* Remove nightly build from ci

---------

Co-authored-by: Dessalines <dessalines@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants