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

fix: failing Docker images from CI #2568

Merged
merged 1 commit into from
Jan 25, 2025
Merged

fix: failing Docker images from CI #2568

merged 1 commit into from
Jan 25, 2025

Conversation

ferferga
Copy link
Member

After the merge of #2539, the Docker builds from CI stopped working properly. However, local and Codespace builds (even locally built Docker images) still are working fine, so no idea what's going on. This PR is a playground to test stuff since it can only be reproduced in CI-produced artifacts. When the fix is found, this same PR is going to contain the fixes.

@jellyfin-bot
Copy link

jellyfin-bot commented Jan 25, 2025

Cloudflare Pages deployment

Latest commit 02251e8
Status ✅ Deployed!
Preview URL https://d5a96fdc.jf-vue.pages.dev
Preview alias https://test-docker.jf-vue.pages.dev
Type 🔀 Preview

View build logs

@ferferga ferferga changed the title wip: test docker fix: failing Docker images from CI Jan 25, 2025
@jellyfin-bot jellyfin-bot added the ci Pull requests that update Continuos Integration-related code label Jan 25, 2025
@jellyfin-bot jellyfin-bot removed the ci Pull requests that update Continuos Integration-related code label Jan 25, 2025
@ferferga
Copy link
Member Author

ferferga commented Jan 25, 2025

The issue is only reproducible when using docker buildx

For some reason, the client throwed an error about incorrect attribute
when using Docker buildx's docker-container driver.
The build worked flawlessly when using the normal docker build command
(which is now an alias to docker buildx build, but with the docker driver)
and in the host system using normal commands, and that's why it went unnoticed.

Really strange error... Perhaps something encoding related in
the outputted files?

Either way, this commit fixes it by removing the trailing double quotes
from the component!
@ferferga ferferga merged commit 540d8eb into master Jan 25, 2025
20 of 23 checks passed
@ferferga ferferga deleted the test-docker branch January 25, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vue Pull requests that edit or add Vue files
Projects
Development

Successfully merging this pull request may close these issues.

2 participants