Skip to content

fix(bazel): repin Cargo.Bazel.lock after #2040 manifest update - #2042

Merged
r-birkner merged 2 commits into
mainfrom
fix/repin-cargo-bazel-lock
Jun 8, 2026
Merged

fix(bazel): repin Cargo.Bazel.lock after #2040 manifest update#2042
r-birkner merged 2 commits into
mainfrom
fix/repin-cargo-bazel-lock

Conversation

@pietrodimarco-dfinity

Copy link
Copy Markdown
Contributor

What

Regenerate Cargo.Bazel.lock (crate_index_dre) via CARGO_BAZEL_REPIN=true bazel sync.

Why

PR #2040 ("fix(registry): support NodeRewardType type4.1-4.5") updated Cargo.toml and Cargo.lock but did not repin the Bazel crate lockfile. This left crate_index_dre's recorded digest stale, so Bazel fails at repo fetch with:

Error: Digests do not match ...
The current `lockfile` is out of date for 'crate_index_dre'.
Please re-run bazel using `CARGO_BAZEL_REPIN=true` ...

This has been breaking main CI ("Build and test", "Required checks") since #2040 merged, and it also blocks the v0.7.8 release pipeline (its test job runs bazel test without repin).

Verification

bazel query //rs/cli:dre now resolves cleanly without the digest mismatch.

PR #2040 updated Cargo.toml/Cargo.lock without regenerating the Bazel
crate lockfile, leaving crate_index_dre's digest stale and breaking
`bazel build`/`bazel test` on main (and the v0.7.8 release pipeline)
with "lockfile is out of date". Regenerate via CARGO_BAZEL_REPIN=true.
@pietrodimarco-dfinity
pietrodimarco-dfinity requested a review from a team as a code owner June 8, 2026 15:16
@pietrodimarco-dfinity
pietrodimarco-dfinity enabled auto-merge (squash) June 8, 2026 15:20
The ic-nns-governance-api bump in #2040 added a `success_value` field to
ProposalInfo. The main code path (proposal.rs) was updated, but the
test-only `gen_test_proposal` constructor was missed, breaking the build
once the Bazel lockfile is repinned. Initialize it to None.
@pietrodimarco-dfinity

Copy link
Copy Markdown
Contributor Author

Pushed a second commit: the lockfile repin exposed a real compile break that the stale lockfile was masking — #2040 added ProposalInfo.success_value and updated the main code path (proposal.rs) but missed the test-only gen_test_proposal constructor in slack-notifications. Now initialized to None.

@r-birkner
r-birkner merged commit 3ea01f9 into main Jun 8, 2026
7 checks passed
@r-birkner
r-birkner deleted the fix/repin-cargo-bazel-lock branch June 8, 2026 15:39
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.

2 participants