Skip to content

update boilerplate revision for tekton build - #615

Closed
bmeng wants to merge 1 commit into
openshift:masterfrom
bmeng:konflux-job-update
Closed

update boilerplate revision for tekton build#615
bmeng wants to merge 1 commit into
openshift:masterfrom
bmeng:konflux-job-update

Conversation

@bmeng

@bmeng bmeng commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated the Boilerplate pipeline revision used by pull request validation.
    • Updated the Boilerplate pipeline revision used by push validation.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 0bbdc73d-de1f-4b8a-9a9f-914f65f114c7

📥 Commits

Reviewing files that changed from the base of the PR and between 06a50a7 and 3fff0db.

📒 Files selected for processing (2)
  • .tekton/managed-cluster-validating-webhooks-pko-pull-request.yaml
  • .tekton/managed-cluster-validating-webhooks-pko-push.yaml

Walkthrough

Both Tekton pipeline manifests update their Boilerplate pipeline reference from revision 5b58d5f... to a33736f....

Changes

Pipeline revision update

Layer / File(s) Summary
Update Tekton pipeline resolvers
.tekton/managed-cluster-validating-webhooks-pko-pull-request.yaml, .tekton/managed-cluster-validating-webhooks-pko-push.yaml
The pull-request and push pipelines now reference Boilerplate revision a33736f60d9b52d50dd5a3964e4c8a734ec57444.

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

Mergeability Score: ⚪ Minimal · up to 3fff0

This change is limited to updating the build boilerplate revision, and no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related PRs

Suggested reviewers: mitalibhalla

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the update to the Boilerplate revision used by the Tekton build pipelines.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Stable And Deterministic Test Names ✅ Passed The patch changes only two Tekton Boilerplate revision values; it adds no Ginkgo test titles or dynamic test-name construction.
Test Structure And Quality ✅ Passed The verified PR diff changes only two Tekton YAML revision values; it introduces no Ginkgo test code or test behavior to assess.
Microshift Test Compatibility ✅ Passed The diff changes only Boilerplate revision values in two Tekton YAML files. It adds no Ginkgo e2e tests, APIs, resources, or MicroShift-incompatible assumptions.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The pull request changes only two Tekton Boilerplate revision values; it adds no Ginkgo e2e tests, so the SNO test-compatibility check is not applicable.
Topology-Aware Scheduling Compatibility ✅ Passed The PR only changes the pinned Boilerplate revision in two Tekton PipelineRuns; it adds no deployment, operator, controller, or scheduling constraint.
Ote Binary Stdout Contract ✅ Passed The diff only changes two Tekton Boilerplate revision values; no OTE binary, suite setup, or process-level stdout code changed, and the PKO image is a scratch package manifest.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The diff changes only two Tekton Boilerplate revision values; it adds no Ginkgo tests, IPv4 assumptions, or test external-connectivity requirements.
No-Weak-Crypto ✅ Passed The PR only changes the Boilerplate pipeline revision in two Tekton YAML files; it introduces no MD5, SHA1, DES, RC4, 3DES, Blowfish, ECB, custom crypto, or secret comparison.
Container-Privileges ✅ Passed The diff only updates Boilerplate revisions. Both changed manifests lack flagged privilege fields, and the referenced pipeline defaults privileged-nested to false; root settings were present in bot...
No-Sensitive-Data-In-Logs ✅ Passed The PR changes only the pinned Boilerplate revision in two Tekton files; the old/new pipeline diff adds no sensitive-data logging or logging statements.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@openshift-ci

openshift-ci Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bmeng

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:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 13, 2026
@openshift-ci

openshift-ci Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

@bmeng: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@samanthajayasinghe

Copy link
Copy Markdown
Contributor

/konflux-pr-override-test

@openshift-ci openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 15, 2026
@openshift-ci

openshift-ci Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@bmeng

bmeng commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

/close

@openshift-ci openshift-ci Bot closed this Aug 18, 2026
@openshift-ci

openshift-ci Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@bmeng: Closed this PR.

Details

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants