chore(deps): update build-tools digest to 5307281 - #1048
Open
red-hat-konflux[bot] wants to merge 1 commit into
Open
red-hat-konflux[bot] wants to merge 1 commit into
red-hat-konflux[bot] wants to merge 1 commit into
Conversation
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Contributor
Summary by CodeRabbit
WalkthroughThe ChangesBuild tools update
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description provides a dependency update summary and relevant links, but it does not follow the repository template. It omits the required description format, UI impact statement, screenshots section, checklist, and issue link information. The referenced digest also conflicts with the raw summary and PR objectives. Resolution Rewrite the description using the repository template. Include a 2-3 sentence summary, state that no UI is impacted if applicable, complete the screenshots and checklist sections, add the tracked issue link when applicable, and correct the build-tools digest values so they match the actual change.
✨ Finishing Touches 💡 2⚔️ Resolve merge conflicts 💡
🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@build-tools`:
- Line 1: Update the build-tools gitlink from revision 72c2bef to
979ac563bf567f96a04e08358538e797f197eeef, preserving the submodule/gitlink
structure.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| @@ -1 +1 @@ | |||
| Subproject commit a96ba3d39e112031b7ce93056c6df7e013a4237e | |||
| Subproject commit 72c2bef5808c5dbbfda833de240a9d9057dc1ebe | |||
Contributor
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
expected_prefix="979ac56"
actual="$(git ls-tree HEAD build-tools | awk '{print $3}')"
url="$(git config -f .gitmodules --get submodule.build-tools.url)"
case "$actual" in
"$expected_prefix"*) ;;
*)
echo "Expected build-tools to start with $expected_prefix, got $actual" >&2
exit 1
;;
esac
tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
git init -q "$tmp"
git -C "$tmp" remote add origin "$url"
git -C "$tmp" fetch --quiet --depth=1 origin "$actual"
git -C "$tmp" cat-file -e "$actual^{commit}"
git -C "$tmp" cat-file -e "$actual:Dockerfile"Repository: RedHatInsights/notifications-frontend
Length of output: 264
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "== submodule configuration =="
git config -f .gitmodules --get-regexp '^submodule\.build-tools\.(path|url)$' || true
echo "== checked-in gitlink =="
git ls-tree HEAD build-tools
echo "== references to build-tools and target prefix =="
rg -n -C 2 'build-tools|979ac56|72c2bef' .gitmodules .tekton README.md 2>/dev/null || true
url="$(git config -f .gitmodules --get submodule.build-tools.url)"
tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
git init -q "$tmp"
git -C "$tmp" remote add origin "$url"
git -C "$tmp" fetch --quiet --depth=1 origin 72c2bef5808c5dbbfda833de240a9d9057dc1ebe 979ac56 || true
echo "== fetched commit resolution =="
for sha in 72c2bef5808c5dbbfda833de240a9d9057dc1ebe 979ac56; do
if git -C "$tmp" cat-file -e "$sha^{commit}" 2>/dev/null; then
git -C "$tmp" show -s --format='%H%n%P%n%s' "$sha"
if git -C "$tmp" cat-file -e "$sha:Dockerfile" 2>/dev/null; then
echo "Dockerfile: present"
git -C "$tmp" show "$sha:Dockerfile" | sha256sum
else
echo "Dockerfile: absent"
fi
else
echo "$sha: not resolved as a commit"
fi
doneRepository: RedHatInsights/notifications-frontend
Length of output: 2516
🏁 Script executed:
#!/bin/bash
set -euo pipefail
repo="https://api.github.com/repos/RedHatInsights/insights-frontend-builder-common"
for ref in 72c2bef5808c5dbbfda833de240a9d9057dc1ebe 979ac56; do
echo "== API lookup: $ref =="
curl -sS -L -H 'Accept: application/vnd.github+json' "$repo/commits/$ref" |
jq -r 'if .sha then [.sha, .commit.message, .commit.tree.sha] | `@tsv` else [.status, .message] | `@tsv` end'
done
echo "== advertised heads and tags matching either prefix =="
git ls-remote --heads --tags https://github.com/RedHatInsights/insights-frontend-builder-common.git |
grep -E '72c2bef|979ac56' || true
echo "== recent remote refs =="
git ls-remote --heads --tags https://github.com/RedHatInsights/insights-frontend-builder-common.git |
tail -n 20Repository: RedHatInsights/notifications-frontend
Length of output: 2513
🏁 Script executed:
#!/bin/bash
set -euo pipefail
repo="https://api.github.com/repos/RedHatInsights/insights-frontend-builder-common"
old="72c2bef5808c5dbbfda833de240a9d9057dc1ebe"
new="979ac563bf567f96a04e08358538e797f197eeef"
tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
for pair in "old:$old" "new:$new"; do
name="${pair%%:*}"
sha="${pair#*:}"
curl -sS -L -H 'Accept: application/vnd.github+json' \
"$repo/contents/Dockerfile?ref=$sha" |
jq -r '.content' | tr -d '\n' | base64 -d > "$tmp/$name.Dockerfile"
echo "== $name Dockerfile =="
sha256sum "$tmp/$name.Dockerfile"
sed -n '1,160p' "$tmp/$name.Dockerfile"
done
echo "== Dockerfile diff =="
diff -u "$tmp/old.Dockerfile" "$tmp/new.Dockerfile" || trueRepository: RedHatInsights/notifications-frontend
Length of output: 8906
Update the build-tools gitlink to 979ac563bf567f96a04e08358538e797f197eeef.
The current gitlink points to 72c2bef. The required revision changes the Node image, installs pnpm, and adds dependency_helpers.sh to the build image.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@build-tools` at line 1, Update the build-tools gitlink from revision 72c2bef
to 979ac563bf567f96a04e08358538e797f197eeef, preserving the submodule/gitlink
structure.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
7aae40d→5307281Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ 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.
To execute skipped test pipelines write comment
/ok-to-test.Documentation
Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.