HYPERFLEET-1614 - ci: prefetch gomod dependencies for Hermeto attribution

[ciaranRoche]

Summary

• configure push and tag pipelines to prefetch Go modules
• ensure generated SBOM packages carry Hermeto attribution before enforcement

HYPERFLEET-1614

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: cf9e49a8-03fe-452e-999b-d5ee4d26e04a

📥 Commits

Reviewing files that changed from the base of the PR and between db67637 and 11c2421.

📒 Files selected for processing (2)

• .tekton/hyperfleet-api-push.yaml
• .tekton/hyperfleet-api-tag.yaml

🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

• openshift-hyperfleet/architecture (manual)
• openshift-hyperfleet/hyperfleet-api (manual)
• openshift-hyperfleet/hyperfleet-sentinel (manual) → reviewed against open PR #261 HYPERFLEET-1614-hermeto-prefetch instead of the default branch
• openshift-hyperfleet/hyperfleet-adapter (manual) → reviewed against open PR #293 HYPERFLEET-1614-hermeto-prefetch instead of the default branch
• openshift-hyperfleet/hyperfleet-broker (manual)

Included review availability: Your plan provides up to 12 included reviews per hour; 9 remain after this review.

---

📝 Summary

Summary by CodeRabbit

• Chores
  • Updated push and tag build pipelines to prefetch Go module dependencies.
  • Improved dependency availability during pipeline execution.

Walkthrough

The push and tag-triggered Tekton PipelineRun definitions now include a prefetch-input parameter. The parameter configures dependency prefetching for Go modules with {"type": "gomod"}.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 11c24

Push and tag pipelines now prefetch Go module dependencies consistently, with no remaining merge-readiness risk identified.

🚥 Pre-merge checks | ✅ 11

✅ Passed checks (11 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the CI change: prefetching Go module dependencies for Hermeto attribution.
Description check	✅ Passed	The description directly explains the pipeline changes and their purpose for SBOM Hermeto attribution.
Docstring Coverage	✅ Passed	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 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Sec-02: Secrets In Log Output	✅ Passed	The pull request adds only the prefetch-input Tekton parameter with the value {"type": "gomod"} in two YAML PipelineRun definitions. The added lines contain no slog, log, logr, zap, or `fm…
No Hardcoded Secrets	✅ Passed	The pull request changes only two Tekton YAML files. The four added lines define prefetch-input with the non-secret literal {"type": "gomod"}. No API key, token, password, private key, embedded cr…
No Weak Cryptography	✅ Passed	The pull request changes only two Tekton YAML files. The four added lines set prefetch-input to {"type": "gomod"}. The diff adds no banned primitive, cryptographic implementation, or secret compar…
No Injection Vectors	✅ Passed	PASS. The commit changes only two Tekton YAML files and adds the hardcoded value '{"type": "gomod"}' to prefetch-input in each PipelineRun. The value is trusted and introduces no SQL construction, c…
No Privileged Containers	✅ Passed	PASS. The pull request changes only two Tekton CI PipelineRuns and adds prefetch-input: '{"type": "gomod"}' to each. The exact diff introduces no privileged: true, hostPID, hostNetwork, `hostI…
No Pii Or Sensitive Data In Logs	✅ Passed	PASS: The pull request changes only two Tekton YAML files. The complete HEAD diff adds the fixed parameter value {"type": "gomod"} to the push and tag PipelineRuns. It adds no slog, logr, zap,…

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch HYPERFLEET-1614-hermeto-prefetch

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch HYPERFLEET-1614-hermeto-prefetch

---

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

[hyperfleet-ci-bot]

Risk Score: 0 — risk/low

Signal	Detail	Points
PR size	4 lines	+0
Sensitive paths	none	+0

_{Computed by hyperfleet-risk-scorer}

[ma-hill]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ma-hill

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [ma-hill]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment