Skip to content

Conversation

@lurtz
Copy link
Contributor

@lurtz lurtz commented Jul 29, 2025

These tools are needed to run disk images and interact with them via e.g. bazel using ssh.

@lurtz lurtz changed the title added qemu-system-aarch64 and sshpass to docker image Add qemu-system-aarch64 and sshpass to docker image Aug 1, 2025
@lurtz lurtz marked this pull request as ready for review August 1, 2025 09:29
@AlexanderLanin
Copy link
Member

I can approve this, but please keep in mind that all tools should be provided by bazel.
For example CI will run via bazel and not via devcontainer.

At least that's the current approach.

@lurtz
Copy link
Contributor Author

lurtz commented Aug 1, 2025

I can approve this, but please keep in mind that all tools should be provided by bazel. For example CI will run via bazel and not via devcontainer.

At least that's the current approach.

How do you make sure that the tools downloaded via bazel are compatible with the host / environment in which they are run?

@lurtz
Copy link
Contributor Author

lurtz commented Aug 1, 2025

Does this also mean you oppose changes like this eclipse-score/communication#26 ?

@AlexanderLanin
Copy link
Member

How do you make sure that the tools downloaded via bazel are compatible with the host / environment in which they are run?

We define separate executables per environment: https://github.com/eclipse-score/tooling/blob/b8e741b7a7d2cc80999eb61dff58f6abbec1319a/tools/ruff.lock.json

@AlexanderLanin
Copy link
Member

Does this also mean you oppose changes like this eclipse-score/communication#26 ?

It's not that I oppose them, it's that we are not aligned in the project. Let's make that a dedicated topic in the infrastructure community.

Our concept so far was to use pure bazel in CI. However, due to performance reasons (and effort) we have partially abandoned that already. example

@AlexanderLanin AlexanderLanin merged commit 95699f3 into eclipse-score:main Aug 5, 2025
2 checks passed
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.

3 participants