chore(deps): update projectbluefin/actions digest to 9c69981 - #36
mergeraptor[bot] wants to merge 3 commits into
Conversation
e8f695c to
753c3ac
Compare
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: contract Raw failure evidence (from check-run annotations): Remove the |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: sec-check[bot] <sec-check[bot]@users.noreply.github.com>
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
hanthor
left a comment
There was a problem hiding this comment.
A digest bump is carrying an unrelated package-policy change
The PR describes exactly one change: projectbluefin/actions digest eb4c546 → 9c69981. The diff is 5 files, and one of them is packages/utah.toml:
@@ -102,7 +102,10 @@ packages = [
packages = [
"evolution-ews-core",
+ "firefox",
"fish",
+ "grub2-efi-x64-cdboot",
+ "pipewire-libs-extra",
"tailscale",
"zsh",
]That is the [unavailable] list — it changes which packages Utah stops trying to install. It has nothing to do with an action digest, it is not mentioned in the PR body, and Renovate did not author it. #29 and #30 carry the byte-identical hunk, which is how you can tell it came from the branch base rather than from the update.
Why it is there, and why it is now wrong
This branch is based on 0a5cbd2a, while main is at 636b48e3. In the interim main picked up those same three packages, and tailscale moved out of [unavailable]. Current main:
$ python3 -c "import tomllib;print(tomllib.load(open('packages/utah.toml','rb'))['unavailable']['packages'])"
['evolution-ews-core', 'firefox', 'fish', 'grub2-efi-x64-cdboot', 'pipewire-libs-extra', 'zsh']Six entries, no tailscale. So merging this branch as-is would re-add tailscale to [unavailable], silently dropping it from the image — and system_files/shared/usr/share/ublue-os/privileged-setup.hooks.d/10-tailscale.sh expects it to be there. That is a functional regression smuggled in by a dependency bump.
This is presumably why mergeable_state is dirty: the branch conflicts with main and cannot merge without a rebase.
What this needs
Rebase onto 636b48e3. After a rebase the packages/utah.toml hunk should vanish entirely — main already contains those three entries — and the PR should reduce to the workflow-file digest changes it claims to be.
If any packages/utah.toml change survives the rebase, it wants its own PR with a rationale, since [unavailable] entries carry tracking issues by convention in that file.
Also worth checking before merge
The automerge label is on this PR. Given the branch carries an undeclared manifest change and is currently conflicted, automerge on a stale Renovate branch is the mechanism that would land the tailscale regression without anyone reading it. The needs-human label is doing the right thing here; please keep it until after the rebase.
Verify the target digest. projectbluefin/actions@9c699818ddf1043b8f5af8b862f59f783c6c7bda is consumed by reusable-build.yml, reusable-sync-branches.yml and the promote/release workflows — it is the whole build topology. Worth a look at the eb4c546...9c69981 compare before merging, since a digest bump here changes more behaviour than a typical action update.
CI cannot validate this right now anyway. main@636b48e is red at Containerfile:81 (install: cannot create regular file '/usr/local/libexec/utah-install-packages', run 34654001732). #56 needs to land first for a green build to be achievable at all.
Same applies to #29 and #30, which carry the identical stray hunk.
Generated by Claude Code
|
Picking this up because of the 1. The digest it proposes is long stale. 2. 3. Its non-workflow payload is already on Worth noting for its own sake: this started life as a four-file workflow-digest bump and acquired a fifth file, a package-availability change, somewhere along the way. I flagged the same shape on projectbluefin/server#172 today, where CI pushes a freedesktop-sdk point-release bump into whatever Recommendation: tick the rebase box in the description. Renovate will re-resolve to the current tip and drop the now-redundant toml hunk, and the result should be a clean four-file workflow-only diff that is quick to review. Closing also works — the body says closing suppresses future reminders for this update, and a fresh PR for the current digest would supersede it anyway — but rebasing keeps the automation on the rails. Not posting a formal review verdict, since there is nothing here worth verifying until it is rebased. Generated by Claude Code |
|
Closing per hanthor's assessment above, re-verified today: the proposed digest |
This PR contains the following updates:
eb4c546→9c69981Configuration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate CLI.