Skip to content

Experiment: FOSS as external modules - #300

Open
nettle wants to merge 5 commits into
Ericsson:mainfrom
nettle:bazel-foss-external
Open

Experiment: FOSS as external modules#300
nettle wants to merge 5 commits into
Ericsson:mainfrom
nettle:bazel-foss-external

Conversation

@nettle

@nettle nettle commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Experimenting with new approach: FOSS repos as external Bazel modules

Why:
The existing Python/Bash FOSS tests are hard to
understand and extend, they are also very slow
and cannot be integrated into Bazel environment.

What:
Add test/bazel/ with a foss_test() macro
that generates py_test() targets for FOSS projects.

Test:
bazel test //test/bazel/...

nettle added 5 commits August 19, 2026 23:43
Why:
The existing Python/Bash FOSS tests are hard to
understand and extend, they are also very slow
and cannot be integrated into Bazel environment.

What:
Add test/bazel/ with a foss_test() macro
that generates sh_test() targets for FOSS projects.

Test:
bazel test //test/bazel/...

Convert to py_test()

Fix buildifier issues
@Szelethus
Szelethus requested review from Szelethus and furtib August 24, 2026 11:07
@Szelethus Szelethus added the test ☑️ Adding or refactoring tests label Aug 24, 2026
@furtib

furtib commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

This solution is much slower (360sec) than the one in #209 (120sec).
I don't fully understand why, but that solution in particular is fast; any other like this one, or #235, runs really slowly.

@nettle

nettle commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator Author

This solution is much slower (360sec) than the one in #209 (120sec). I don't fully understand why, but that solution in particular is fast; any other like this one, or #235, runs really slowly.

The reason is simple - we download, extract and run bazel build not once (setUpClass) but for every test (setUp).
Only in #209 this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test ☑️ Adding or refactoring tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants