Skip to content

bzlmod lock enforcement#80

Merged
dcalavrezo-qorix merged 3 commits intomainfrom
dcalavrezo_lock
Mar 6, 2026
Merged

bzlmod lock enforcement#80
dcalavrezo-qorix merged 3 commits intomainfrom
dcalavrezo_lock

Conversation

@dcalavrezo-qorix
Copy link
Contributor

Notes for Reviewer

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #

- add reusable lockfile check workflow

- track MODULE.bazel.lock (remove from .gitignore)

- enforce --lockfile_mode=error in CI and scripts

Addresses eclipse-score/score#2628
fix permission issue reported by Copilot

Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Mar 6, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
2026/03/06 14:58:08 Downloading https://releases.bazel.build/8.3.0/release/bazel-8.3.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 7ffc222f-bfce-4bb3-9b8e-5119cbbebfe4
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'rules_python', the root module requires module version rules_python@1.4.1, but got rules_python@1.8.3 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'bazel_skylib', the root module requires module version bazel_skylib@1.7.1, but got bazel_skylib@1.8.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_cc', the root module requires module version rules_cc@0.1.1, but got rules_cc@0.2.14 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'buildifier_prebuilt', the root module requires module version buildifier_prebuilt@7.3.1, but got buildifier_prebuilt@8.2.0.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'googletest', the root module requires module version googletest@1.14.0, but got googletest@1.14.0.bcr.1 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (31 packages loaded, 9 targets configured)

Analyzing: target //:license-check (66 packages loaded, 9 targets configured)

Analyzing: target //:license-check (83 packages loaded, 9 targets configured)

Analyzing: target //:license-check (139 packages loaded, 1148 targets configured)

Analyzing: target //:license-check (149 packages loaded, 2391 targets configured)

Analyzing: target //:license-check (160 packages loaded, 3413 targets configured)

Analyzing: target //:license-check (160 packages loaded, 3415 targets configured)

Analyzing: target //:license-check (160 packages loaded, 3415 targets configured)

Analyzing: target //:license-check (160 packages loaded, 3415 targets configured)

Analyzing: target //:license-check (164 packages loaded, 5428 targets configured)

Analyzing: target //:license-check (168 packages loaded, 5566 targets configured)
[10 / 17] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local
Analyzing: target //:license-check (189 packages loaded, 5632 targets configured)
[13 / 17] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions, 1 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 141 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
Analyzing: target //:license-check (191 packages loaded, 5640 targets configured)
[14 / 17] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 1s disk-cache, processwrapper-sandbox
Analyzing: target //:license-check (191 packages loaded, 5640 targets configured)
[15 / 17] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
Analyzing: target //:license-check (191 packages loaded, 5640 targets configured)
[16 / 17] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
Analyzing: target //:license-check (191 packages loaded, 5640 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
Analyzing: target //:license-check (192 packages loaded, 9974 targets configured)
[17 / 17] no actions running
INFO: Analyzed target //:license-check (194 packages loaded, 10088 targets configured).
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 56.010s, Critical Path: 4.34s
INFO: 17 processes: 12 internal, 4 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 17 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

The created documentation from the pull request is available at: docu-html

@vinodreddy-g
Copy link
Contributor

vinodreddy-g commented Mar 6, 2026

@dcalavrezo-qorix - May be add to readme/error log that if the build fails with this . The lock file has to be updated and pushed if the dep versions have changed for example. like this or - bazel build --lockfile_mode=update //src/...

vinodreddy-g
vinodreddy-g previously approved these changes Mar 6, 2026
implemented change suggested by Vinod

Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
@dcalavrezo-qorix dcalavrezo-qorix merged commit a45485d into main Mar 6, 2026
19 checks passed
@dcalavrezo-qorix dcalavrezo-qorix deleted the dcalavrezo_lock branch March 6, 2026 15:13
@github-project-automation github-project-automation bot moved this from Backlog to Done in ORC - Orchestration FT Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants