Skip to content

Conversation

@cheina97
Copy link
Member

@cheina97 cheina97 commented Jul 12, 2025

#Description

This PR optimizes the building speed of liqo components for single and multi-arch builds.

It moves the binary build outside the Docker container and then pushes it inside. This allows us to fully utilize the multi-arch build feature of Golang, eliminating the need for Qemu dependencies.

Additionally, it provides a faster way to build images for development, reducing the time taken to several seconds.

The most significant time reduction is observed in multi-arch builds for AMD64, ARM64, and ARM32 platforms, which have been reduced from 50 minutes to just 3 minutes.

@adamjensenbot
Copy link
Collaborator

Hi @cheina97. Thanks for your PR!

I am @adamjensenbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

  • /rebase: Rebase this PR onto the master branch (You can add the option test=true to launch the tests
    when the rebase operation is completed)
  • /merge: Merge this PR into the master branch
  • /build Build Liqo components
  • /test Launch the E2E and Unit tests
  • /hold, /unhold Add/remove the hold label to prevent merging with /merge

Make sure this PR appears in the liqo changelog, adding one of the following labels:

  • feat: 🚀 New Feature
  • fix: 🐛 Bug Fix
  • refactor: 🧹 Code Refactoring
  • docs: 📝 Documentation
  • style: 💄 Code Style
  • perf: 🐎 Performance Improvement
  • test: ✅ Tests
  • chore: 🚚 Dependencies Management
  • build: 📦 Builds Management
  • ci: 👷 CI/CD
  • revert: ⏪ Reverts Previous Changes

@github-actions github-actions bot added the ci Changes to CI configuration files or scripts label Jul 12, 2025
@cheina97 cheina97 force-pushed the frc/buildpipeline branch 19 times, most recently from 99bb317 to d331a36 Compare July 13, 2025 09:48
@cheina97 cheina97 marked this pull request as ready for review July 13, 2025 09:54
@cheina97 cheina97 force-pushed the frc/buildpipeline branch 7 times, most recently from b943cff to 6e205a6 Compare July 13, 2025 11:06
@cheina97 cheina97 force-pushed the frc/buildpipeline branch 5 times, most recently from 257a593 to 643c86e Compare July 14, 2025 17:24
@cheina97 cheina97 requested review from aleoli, claudiolor and fra98 July 14, 2025 17:29
@cheina97
Copy link
Member Author

/rebase

@cheina97
Copy link
Member Author

/rebase test=true

1 similar comment
@cheina97
Copy link
Member Author

/rebase test=true

@cheina97 cheina97 force-pushed the frc/buildpipeline branch from e712e7c to f509e59 Compare July 16, 2025 10:39
@cheina97 cheina97 force-pushed the frc/buildpipeline branch 4 times, most recently from 3a23d03 to 01d8357 Compare July 16, 2025 10:54
@cheina97 cheina97 force-pushed the frc/buildpipeline branch from 01d8357 to d049864 Compare July 16, 2025 12:57
@cheina97 cheina97 requested a review from claudiolor July 16, 2025 12:57
@cheina97
Copy link
Member Author

/rebase test=true

@cheina97 cheina97 merged commit 6855735 into liqotech:master Jul 25, 2025
10 of 12 checks passed
@cheina97 cheina97 deleted the frc/buildpipeline branch July 25, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to CI configuration files or scripts size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants