fix(EC-2188): limit release PR builds to linux amd64 by jsmid1 · Pull Request #3602 · conforma/cli

jsmid1 · 2026-10-13T10:53:40Z

Use a release-specific pre-merge build-args file when generating release branch pipelines. Keep post-merge builds unmodified so they continue to build every supported platform.

Resolves: EC-2188

coderabbitai · 2026-10-13T10:53:57Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: f1ab1c9a-cb21-440e-8697-1ce029df70c5

📥 Commits

Reviewing files that changed from the base of the PR and between d7152fd and a994b95.

📒 Files selected for processing (2)

hack/release-branch-pipeline-patch.sh
release-pre-merge-build-args.conf

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The release pipeline transformation now selects release-pre-merge-build-args.conf instead of removing the main pre-merge build-args filename. The release configuration sets BUILD_LIST to linux_amd64.

Changes: Release pre-merge build arguments

Layer / File(s)	Summary
Configure and select release build arguments `release-pre-merge-build-args.conf`, `hack/release-branch-pipeline-patch.sh`	The release configuration sets `BUILD_LIST` to `linux_amd64`. The pipeline transformation replaces the main pre-merge build-args filename with the release filename.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Suggested reviewers: dheerajodha, simonbaird

Merge Risk: ⚪ Minimal · up to a994b

Release pre-merge builds are limited to linux/amd64, while post-merge builds retain their existing platform coverage. No actionable merge-blocking risk remains.

Pre-merge checks: 5

Passed checks (5 passed)

Check name	Status	Explanation
Title check		The title clearly states that release PR builds will be limited to Linux AMD64 and includes the related ticket.
Description check		The description explains what changes, why it changes, and links the related EC-2188 ticket. It covers the required template information despite not using the template headings.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches 💡 1

🧪 Generate unit tests (beta)

Create a new PR

🛠️ Fix failing CI checks 💡

Commit to this branch
Create a new PR

_{Comment @coderabbitai help to get the list of available commands.}

fullsend-ai-review · 2026-10-13T11:02:34Z

Risk Assessment: low (1/5)

Details

Minimal two-file XS configuration change with no security sensitivity, no CI workflow impact, no dependency changes, and a single known author - composite score 1.24 rounds to 1 (low).

Previous run

Risk Assessment: low (1/5)

Details

Two-file, three-line config-only change with small blast radius, no protected or security-sensitive paths, no CI or dependency modifications, a known author with minimal churn history, and a scope that precisely matches the stated issue; all signals converge at the lowest risk tier.

Previous run (2)

Risk Assessment: low (1/5)

Details

XS configuration-only change by a seasoned author limiting release builds to linux/amd64, touching no protected paths, security-sensitive files, or CI workflow definitions, with stable file history and a single prior fix commit yielding a composite near 1.

fullsend-ai-review · 2026-10-13T11:02:38Z

Looks good to me

Previous run

Looks good to me

Previous run (2)

Looks good to me

codecov · 2026-10-13T11:09:41Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag	Coverage Δ
acceptance	`54.54% <ø> (ø)`
generative	`12.25% <ø> (ø)`
integration	`23.55% <ø> (ø)`
unit	`72.26% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

st3penta

LGTM. this probably just needs a rebase to pass CI

Superseded by updated review

Use a release-specific pre-merge build-args file when generating release branch pipelines. Keep post-merge builds unmodified so they continue to build every supported platform. Co-authored-by: OpenAI Codex <noreply@openai.com> Signed-off-by: Jan Smid <jsmid@redhat.com>

Superseded by updated review

fullsend-ai-retro · 2026-10-20T09:22:24Z

Retro on PR #3602 (fix(EC-2188): limit release PR builds to linux amd64). Clean workflow — nothing to propose.

What happened. Human-authored 2-file config change (one-line edit to hack/release-branch-pipeline-patch.sh plus a new single-line release-pre-merge-build-args.conf). The review agent (fullsend-ai-review) ran twice (initial and post-rebase), returned risk 1/5, and produced an empty v2 findings payload both times ("Looks good to me"). Two humans approved (st3penta, cuipinghuo); CI was green across Checks, CodeQL, codecov, Konflux, and CodeRabbit; the PR merged ~24h after the second human approval and a rebase.

Review quality. No human reviewer added inline change requests that the agent missed, and the agent did not flag anything a human later overruled. Low-complexity config change, correctly handled.

Rework rate. Zero iterations — one commit, one rebase, no fix cycle.

Autonomy readiness. For this class of change (small, additive release-pipeline config under hack/ with no code-path impact) the review agent matched the human reviewers; worth noting as one data point in the broader autonomy picture tracked by issues like #3508.

Existing-issue check. Related open retro-filed issues were reviewed and no new proposal is warranted:

Fix stale quick-build-args.conf reference in release-branch-pipeline-patch.sh #3533 — stale quick-build-args.conf reference in the same script this PR edits (hack/release-branch-pipeline-patch.sh); adjacent cleanup, already tracked.
Backport .fullsend/ directory to release-v0.7, release-v0.8, release-v0.9 to restore review-agent coverage #3625 — backport .fullsend/ to release branches so review-agent runs there too; relevant to the release-branch build-args surface.
Add AGENTS.md guidance for hack/ directory script conventions and review expectations #3508 — AGENTS.md guidance for hack/ directory script conventions; would cover review-agent context for this file.
Add AGENTS.md guidance for UBI base image bump workflow and release branch scope #3502, Document UBI base image bump workflow in AGENTS.md to prevent false-positive review findings #3503 — AGENTS.md guidance for release-branch scope / UBI bump workflow; related framing but not a duplicate of anything observed here.

Dispatch repo confirmed as fullsend-ai/fullsend @ ea66a0a1; agents repo confirmed as fullsend-ai/agents @ ce2eedd0.

jsmid1 requested a review from a team as a code owner October 13, 2026 10:53

github-actions Bot added the size: XS label Oct 13, 2026

fullsend-ai-review Bot added the risk/low PR risk: low label Oct 13, 2026

This comment was marked as outdated.

Sign in to view

fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label Oct 13, 2026

st3penta approved these changes Oct 19, 2026

View reviewed changes

jsmid1 force-pushed the EC-2188 branch from a994b95 to b621eab Compare October 19, 2026 13:06

This comment was marked as outdated.

Sign in to view

cuipinghuo approved these changes Oct 19, 2026

View reviewed changes

jsmid1 force-pushed the EC-2188 branch from b621eab to 429168c Compare October 20, 2026 08:42

fullsend-ai-review Bot approved these changes Oct 20, 2026

View reviewed changes

jsmid1 merged commit be22fc0 into conforma:main Oct 20, 2026
36 checks passed

Navigation Menu

Repository navigation

fix(EC-2188): limit release PR builds to linux amd64 - #3602

fix(EC-2188): limit release PR builds to linux amd64#3602
jsmid1 merged 1 commit into
conforma:mainfrom
jsmid1:EC-2188

jsmid1 commented Oct 13, 2026

Uh oh!

coderabbitai Bot commented Oct 13, 2026 •

edited

Loading

Uh oh!

fullsend-ai-review Bot commented Oct 13, 2026 •

edited

Loading

Uh oh!

fullsend-ai-review Bot commented Oct 13, 2026 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

codecov Bot commented Oct 13, 2026 •

edited

Loading

Uh oh!

st3penta left a comment

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

fullsend-ai-retro Bot commented Oct 20, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

jsmid1 commented Oct 13, 2026

Uh oh!

coderabbitai Bot commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fullsend-ai-review Bot commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fullsend-ai-review Bot commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

codecov Bot commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

st3penta left a comment

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

fullsend-ai-retro Bot commented Oct 20, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

coderabbitai Bot commented Oct 13, 2026 •

edited

Loading

fullsend-ai-review Bot commented Oct 13, 2026 •

edited

Loading

fullsend-ai-review Bot commented Oct 13, 2026 •

edited

Loading

codecov Bot commented Oct 13, 2026 •

edited

Loading