Boilerplate: Update to a8a3172411f3f2b8848f64333843e028ef4b3ed1 - #387
Conversation
Conventions: - openshift/golang-osd-operator: Update --- openshift/boilerplate@8fb7c80...a8a3172 commit: 39b903e8c5db27cee7bb5b2dcce9ff4b5a07da5e author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: c678988174f35841a156cc2bba88d043b84cea9f author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to d9beb74 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 1542cb9a6f953091ea36868f18ef42542103dba7 author: Bo Meng Add shared CodeRabbit configuration commit: 76056e0c32e5423cbc9e163c2053cd36b7435961 author: Dustin Row Update roxctl-scan task bundle to fix null jq error Updates the roxctl-scan task bundle SHA to include the fix for KONFLUX-15651, where the proccess-output step fails with "Cannot iterate over null" on scratch-based images. Fix: konflux-ci/konflux-test#906 commit: e857a1ac44cb260f0df165bc3cdc77524fcdca48 author: Dustin Row gangway-bridge: tighten POLL_OVERSHOOT and remove redundant 429 sleep POLL_OVERSHOOT now uses max(POLL_INTERVAL, 300) + 30 instead of the hardcoded 300+30+300, so the budget reflects whichever delay is larger. The extra sleep in the 429 branch is removed; the loop leading sleep already provides the backoff on the next iteration. commit: bcf83f16fe1a00029f45af9c3ac0584203090991 author: Dustin Row gangway-bridge: fix REQUIRED_DEADLINE to include INITIAL_DELAY and larger POLL_OVERSHOOT commit: 7f5eecc7db7d567023b2015c71f2786b91d77c19 author: Dustin Row gangway-bridge: longer poll interval and retry backoff cap Double default POLL_INTERVAL from 60s to 120s to reduce the baseline polling rate. With multiple concurrent jobs the polling alone can consume the 9 req/min Gangway rate limit budget. Raise the inter-retry backoff cap from 480s to 900s so later retry attempts back off more aggressively when contention is high. commit: d2a4c7a5454223b2169392c645f108e364bb57be author: Dustin Row gangway-bridge: add INITIAL_DELAY and poll 429 backoff Gangway rate-limits at 9 req/min per source IP with nodelay burst of 5. When multiple operators deploy in the same SAPM pipeline run their gangway-bridge jobs all start simultaneously and saturate the shared quota, causing trigger attempts to exhaust all retries and fail. Add INITIAL_DELAY parameter (default 0s) so callers can stagger concurrent jobs by setting different delays per target in the saas file. Also fix the status-poll loop to back off exponentially (doubling up to 300s) on 429 responses instead of silently retrying at the normal POLL_INTERVAL, which was burning rate limit budget during polling and competing with trigger retries from other concurrent jobs. commit: 03d7a11d4b501360c57e0d4aec00f0f19d556044 author: Chai Bot gangway-bridge: back off on 429 during status polling commit: b55a066cc92d9f51b06add5f046e3e90d4c463d1 author: Chai Bot Add 429 rate-limit handling with exponential backoff and jitter to gangway bridge - Increase MAX_RETRIES default from 1 to 5 and ACTIVE_DEADLINE from 14430 to 50400 to accommodate additional retry attempts with backoff - Replace fixed deadline formula with backoff-aware calculation that accounts for exponential delays and jitter - Capture HTTP response code from curl and handle 429 responses by parsing the Retry-After header (capped at 600s) before falling through to the outer retry loop - Replace fixed 30s retry delay with exponential backoff (30s base, doubling per attempt, capped at 480s) plus random jitter (0-15s) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> commit: b5c538a3d4db95d1a543c2d111686211ec803cca author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 6ded743c2641eee96ac36c242e936a68bd842516 author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to c2483a8 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 63c3b7097adf9f68250309eab02b6ac4161556a7 author: Chai Bot Fix CodeRabbit path filter override for boilerplate directory Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> commit: 22b25670188fa481117bb381e2bde9683844d031 author: Chai Bot Remove departed members from OWNERS_ALIASES Remove c-e-brumm, OliviaHY, and syncrou from the rosa-managers alias group, and Tessg22 from the srep-functional-team-thor alias group. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> commit: d547835be8964f9069a0ca23b2daf00670d518b2 author: Alice Hubenko Drop osd-cluster-ready from boilerplate subscribers. The Job is no longer deployed; keep subscriber list aligned with live operators. Co-authored-by: Cursor <cursoragent@cursor.com> commit: 2ab6bf81e5778a8804ea7ed606f1f552c4bd56c8 author: Chai Bot Add geowa4 to rosa-staff-engineers alias Closes #858 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> commit: 68e5a1af106eafcae0440ea6b085858b81719f89 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.4.0 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 342174c70548ea540a3666bfdc80f1374cb9f96b author: Alice Hubenko ROSAENG-65731: harden gangway-bridge retry and timeout handling - Validate ACTIVE_DEADLINE covers the full retry budget at startup - Return immediately from trigger_and_poll on POST failure or invalid response instead of polling an empty URL until timeout - Add --max-time to curl requests to prevent hangs beyond budget Co-authored-by: Cursor <cursoragent@cursor.com> commit: 8da3e3cea5fa0bbaa460e74757772d79bce6ae74 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 1d375dff0c3edcffcbc40873a59c65b26d8107ad author: Alice Hubenko fix: replace curl|bash codecov uploader with pinned codecov-cli Replace the deprecated `curl https://codecov.io/bash | bash` pattern with a pinned Codecov CLI binary (v11.3.1) downloaded from GitHub releases and verified via SHA256 checksum before execution. The legacy bash uploader was the vector for the April 2021 Codecov supply-chain compromise (CWE-494, CWE-829). Because boilerplate vendors this script into every subscribing OSD operator, a single upstream tampering event would yield arbitrary code execution in dozens of operator CI jobs. Files updated: - boilerplate/openshift/golang-osd-operator/codecov.sh - boilerplate/openshift/golang-codecov/codecov.sh - boilerplate/test/test-base-convention/codecov.sh Closes: ROSAENG-61296 Ref: HCMSEC-3528 Co-authored-by: Cursor <cursoragent@cursor.com> commit: 7ea236f59482d4cba85bbd637ed2b0d284aed30c author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to 8eb4bce Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: ab82d119c6080be925eddb9c20b0b52af2089d97 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: b8fba3028968377cb61b03af6844f78df706d1ba author: red-hat-konflux[bot] chore(deps): update konflux references to v0.2.6 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 163faa4dc74b5696a93a8f95bd1fb9f3013ed05b author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to 2e70a98 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: c1c11cc2d6c8228894588c689282a9f7847ffb99 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: f08967e1ae2483e7eeed71a400d1fa6566085569 author: Dustin Row Add repo-level coderabbit config to re-enable reviews The org-wide coderabbit config excludes boilerplate/** so consuming repos don't get noisy reviews of the vendored copy they pull in via `make boilerplate-update`. That same exclusion accidentally matches this repo's own source, since it all lives under boilerplate/ too, so add a repo-level override to re-include it here. commit: 01a155d3820d975e897e057af3cbb8fd54ed85cc author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 1bc39d51b568e317a7be904b04cb644e977454ac author: Daniel Hall Revert prefetch-dependencies taskRef to original bundle format with digest This reverts the changes from PR #844 and #845, restoring the original bundle reference format that includes the digest pinning: quay.io/.../task-prefetch-dependencies-oci-ta:0.9.0@sha256:4486aaa... While this format currently fails Enterprise Contract validation (due to a bug in Tekton Chains that strips version tags from bundle references in SLSA attestations), it is the correct format that should work once the upstream issue is resolved. Signed-off-by: Daniel Hall <danhall@redhat.com> commit: c36e436d593bf7dc1cabf5422eb601b11f266181 author: Daniel Hall Fix invalid 'version' field in prefetch-dependencies taskRef The previous PR #844 introduced an invalid 'version' field in the taskRef, which is not part of the Tekton API and causes pipeline validation errors: unknown field "spec.tasks[2].taskRef.version" This change fixes it by using the proper Tekton bundles resolver format WITHOUT digest pinning. The bundle reference without digest allows EC to properly match the version tag in the allow list. Changes: - Replace invalid 'version: "0.9"' field - Use bundles resolver with proper params structure - Bundle ref: task-prefetch-dependencies-oci-ta:0.9.0 (no digest) - EC key generated: oci://.../task-prefetch-dependencies-oci-ta:0.9.0 - Matches allowed entry in trusted_tasks data This should resolve both: 1. Tekton API validation (accepts bundles resolver) 2. Enterprise Contract validation (version tag in key matches allow list) Fixes: #844 Signed-off-by: Daniel Hall <danhall@redhat.com> commit: 719a063808000d152afc65d323554bd207b28a14 author: Daniel Hall Fix prefetch-dependencies-oci-ta Enterprise Contract violation Change task reference format from bundle resolver to version-based resolution to fix Enterprise Contract "untrusted task" violations. This minimal change updates only the prefetch-dependencies-oci-ta task reference from: resolver: bundles with bundle URL and digest to: taskRef with name and version The version-based format (version: "0.9") is the current approved format that passes Enterprise Contract trusted_task_rules validation. Fixes Enterprise Contract violations in downstream repos using this pipeline. Signed-off-by: Daniel Hall <danhall@redhat.com> commit: 3d0f3b2d49f63f14624458e4f99aee19301d9dd6 author: Dustin Row ROSAENG-62320: Add automatic retry to gangway bridge template Add MAX_RETRIES parameter (default 1) so the bridge retries the Prow job once on failure before reporting failure. Most infra flakes (DiskPressure, node eviction, quota) are transient and pass on retry. Also adds ACTIVE_DEADLINE parameter to set the Kubernetes Job deadline independently from the per-attempt TIMEOUT, since retries need more total wall time. commit: 56243bd0598738bea9eb26bc6260df023a414063 author: Josh Branham Update README.md commit: e6407dabdeb0546965a2f669e53bcf439e61d450 author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to 8ff5fdb Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: a33736f60d9b52d50dd5a3964e4c8a734ec57444 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 46f31fe8bc385b58904c97a032e9b98cd5b46ba2 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 90877487a11f0fed7e5252885efe73dc240049bf author: MitaliBhalla Remove Dependabot from operator conventions Dependabot docker updates shipped via golang-osd-operator and osd-container-image duplicate Konflux/MintMaker. Delete the templates and teach update to remove legacy consumer .github/dependabot.yml on the next boilerplate-update. Co-authored-by: Cursor <cursoragent@cursor.com> commit: 7aeb3f51a7519b66fe457f35e1db19d0f0b9bb0e author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: b1c1ff90d475b8fbc861c6b5bf9a3c2939238bca author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 169ef92a61302eaf80abab69d68a20bade19669d author: red-hat-konflux[bot] chore(deps): update konflux references to v0.7.0 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: d97f0e1c03122551dab459615274e4220e7c773b author: Christopher Collins Update UBI 9 base image to 9.8-1785906690 Bump both builder and runtime stages from ubi9:9.7-1778044007 / ubi9:9.8-1781496985 to ubi9:9.8-1785906690. This picks up go-toolset 1.26.5-1.el9_8 (from 1.26.3) and latest RPM security fixes. Created with assistance from Claude 🤖 <claude@anthropic.com> Signed-off-by: Christopher Collins <collins.christopher@gmail.com> commit: 28c74867d2b608eb0bf633831cbd6c7a3b7af6c2 author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to cb964eb Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 7085bc24dc6887dc9cf171507289fda9ea072f96 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 70b91bbe286c10d5c3971d28239bc972595cee8b author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: f9344eb477a50a6644671be30c679bf6a1198353 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.2 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 8045cf1a0117762736ccee318f823e038801db17 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.3 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 1f12c802193218d1da2984931ea0d949a4bc25ca author: Roland Kunkel ROSAENG-63019: Remove MateSaary from boilerplate OWNERS file commit: 0586fd1f9610d766cab5cd5e9c5c56076851f4b6 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.10.7 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 0d435dacf4e749b86163dfcb9c5ce6173cdb9ca9 author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: eac613afacc794eaf03aecd2a58378c24d8a50b5 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 9ae795a3ddfacbab96925d01f1f5adbfdad899fe author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: cb186ea257dc0860b3be4f3086ee708c9fbc17e7 author: Dustin Row ROSAENG-1344: Add make e2e-local target for local operator e2e testing Add e2e-local make target that builds the e2e binary and runs it against a cluster via KUBECONFIG or backplane. Supports GINKGO_FOCUS for focused tests and CLUSTER_ID for backplane access. Update README with examples for common local testing workflows. commit: 4590fe84db3f66975d3210f6a5ac0be55262f9fa author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: c27b52ab4baa97bcce237d1cd3e14a180ab106f8 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.3.1 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: ee25720ce5f2a5c6032ea63d9a688c27635b6f41 author: red-hat-konflux[bot] chore(deps): update konflux references to v0.10.5 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 89c9d77372f8f598d05026cb84f2988d952abea0 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: db37707e88ce21f740f8e29db193a0ef858a9382 author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 32af3865956156e28e70a0248b045f8fbb98c406 author: Dustin Row Fix pipeline ref pinning to use exported BOILERPLATE_COMMIT commit: 347afe216ff19631654c2dc94aa235b5a0b88344 author: Dustin Row Harden boilerplate-generated CI artifacts - Pin agentic SDLC pipeline ref to the boilerplate commit SHA instead of mutable master branch - Reject zero values for POLL_INTERVAL and TIMEOUT in gangway bridge template to prevent tight-loop requests - Clean up e2e README wording commit: fec76938a41415d820b1b05d90e3b3d59509b7b5 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 05ffad554a0dec0a5c6e35d77838abdf453fcb43 author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 7b1d81589e5222cac93f49baa3b737b7bbf66560 author: Dustin Row Update OWNERS_ALIASES: scope rosa alias to staff engineers Remove non-staff-engineer members from the rosa alias to reduce notification noise for service engineering. commit: 5c6ee6a812c36a575ec599bc4bc4c9dc748fac2e author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: ba3055067ac90b8caaecdbcf701e04fe186a4a34 author: Dustin Row Remove dependabot config from boilerplate repo Boilerplate uses Renovate (MintMaker) for dependency management. The dependabot.yml creates conflicting duplicate PRs for Docker base image updates. commit: 35d0599bed431c51ebfe469f05e50280a4a333ec author: Bo Meng fix trailing space commit: 5edba99b9a0e98b6f9bb955ce70b1a4dc7f13340 author: tkong Reading default branch in existing pipeline config as dynamic detection returns different value in differnt env commit: fbc38a8fcfde5b0fa56b113161c490b7249e4cbc author: Dustin Row Fix agentic-sdlc-check: auto-detect Konflux application and component names commit: 8d6dc7f4108dfe3839b590bb38c5d8e4467fb052 author: Dustin Row Harden gangway-bridge template: validate params, read-only fs, dynamic deadline commit: 4a8b2a0125f4ed52d16e327a15e34bd77b0c0384 author: Dustin Row Add gangway-bridge template to golang-osd-e2e convention commit: 0e8dacefda4939842bb35449cd47d76c6ab80c50 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 42a109f6528984b998d7bbb2ca3539947207cf3c author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 8a119cdaae30c7a5b8a1367bc8c781f5bda4966f author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 14c71960783fe68dc648d9d810a446181f2096a5 author: Trevor Nierman Refactor agentic SDLC check to use pipelineRef (ROSA-730) Move check logic from the per-repo PipelineRun template into a centralized Pipeline definition at pipelines/agentic-sdlc-check/, matching the pattern used by docker-build-oci-ta. The PipelineRun template generated by boilerplate is now a thin wrapper (~45 lines) that provides operator-specific metadata and references the pipeline via the git resolver. Check logic changes now only need a commit to boilerplate — no make boilerplate-update across subscriber repos. Non-boilerplate repos can also use the same pipelineRef since the pipeline lives in a public repo. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> commit: b9dc4be828aeeefa1779f4f9ff0faef34ca19f08 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 402d43e7862ef701e025daa37e3d349521094679 author: Trevor Nierman Derive Konflux tenant and SA from existing pipelines (ROSA-730) Some repos (configure-alertmanager-operator, configure-goalert-operator) have non-standard Konflux tenant namespaces and service accounts from early onboarding. Instead of constructing these from the operator name, read them from the existing Konflux pipeline files in .tekton/ which are the source of truth. Falls back to the <operator>-tenant and build-pipeline-<operator> conventions if no existing pipelines exist. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> commit: 217935836ab6bfeb2a6e1a5874c0d2f504aa2b68 author: tkong Set the target branch by repo default branch commit: 11ef3a8fd7a934c83a34c707e21b3474744e3c9f author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 198aea4a930a49f9505d0a375fe93583860e667b author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 9bc6892b0b1ad51e026f2cd7a9750b5cf20c1798 author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: e6d218c5d02f0a9e206106dc6c84f70a3e8285ca author: Jaime Melis Remove jmelis from OWNERS_ALIASES In order to not be in the slack handle for boilerplate commit: b35af73138dcd87ea92cec18c21a6eee3d5d3fcd author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 1000b99d5d564b21592ab8667cb764cb4246c679 author: Trevor Nierman Fix agentic SDLC check: resolve @ includes, switch to informing mode Two fixes for the agentic SDLC conformance check pipeline: 1. Resolve CLAUDE.md @ includes: repos using `@AGENTS.md` (or similar include directives) had content checks fail because the script only read CLAUDE.md itself. Now resolves one level of @ references and checks the combined content. Deduplicates with sort -u. 2. Add onError: continue to both check tasks so the pipeline always passes at the PipelineRun level. Non-conformance is still visible in the individual task status (red task in Konflux UI) but the overall check reports success. This avoids noisy red X signals on repos that are still being onboarded. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> commit: a9bc3f61dfc9c24eccdfab8d09ee34334b822c46 author: Franco Sferrazza updated sha on pipelines/docker-build-oci-ta/pipeline.yaml commit: dd7b4c340b4c37295db0c889b61f8a516146cd49 author: Franco Sferrazza bumped konflux references commit: 09664e2f43f0b521c49d467ad519ce5f15da0bbf author: Franco Sferrazza fixed regex; better fix commit: 536448e39ae052c0d5e22492093bdf8462a3227e author: Franco Sferrazza fixed boilerplate/openshift/golang-osd-operator/update commit: 6f92657b2b4d93172156b594a9db555e0e7179c4 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 329570b87fe41e6a50c4c8350d32dd6f24cc54d0 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: c5e09fc1fb75539267456bc5d94b53086aaa416d author: Trevor Nierman Add agentic SDLC conformance check pipeline (ROSA-730) Adds a Konflux pipeline template that validates ROSA-730 agentic SDLC standards on every PR to subscriber repos. The check is non-blocking (not in required_status_checks) but reports a visible red X when repos are non-conformant. The pipeline runs three tasks: 1. clone-repository: uses the Konflux catalog git-clone task 2. check-file-existence: verifies CLAUDE.md, .pre-commit-config.yaml, .codecov.yml, CONTRIBUTING.md, DEVELOPMENT.md, TESTING.md, .claude/settings.json, and golangci.yml 3. check-content-validation: verifies CLAUDE.md has required sections, codecov patch coverage >= 50%, pre-commit includes gitleaks + golangci-lint + file hygiene hooks, and docs are not stubs The template is generated during `make boilerplate-update` with the operator name substituted from config/config.go. Tested on openshift/deadmanssnitch-operator PR openshift#337. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> commit: b76bd7cb863f3d759e3294133f007c03a7ef5f84 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 177b0c8fb5808f7554464d30e2b96620af96675e author: MitaliBhalla ROSA-745: route MintMaker gomod and tekton PRs through Prow/Tide Set automerge and platformAutomerge to false for patch/minor gomod and tekton updates while keeping lgtm/approved labels and the UTC batch window. Prow/Tide-integrated Konflux repos merge dependency PRs after required checks pass instead of platform automerge. Co-authored-by: Cursor <cursoragent@cursor.com> commit: d9dfe47696ff1bbb5316dbf978bcb207602174ed author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 0d0fb98b01ce219ee56175ce0c4d52f737a29059 author: Dustin Row Update OWNERS_ALIASES to match ROSA org restructure The ROSA org has been restructured under Hybrid Platforms. The old "SREP" naming and "leads" terminology no longer applies. Alias changes: - srep-functional-leads -> rosa-staff-engineers - srep-team-leads -> rosa-team-group-managers - sre-group-leads -> rosa-org-managers - srep-architects -> hcm-architects - srep-functional-team-security (removed, moved to Platform Engineering) - srep-infra-cicd (removed, moved to Platform Engineering) Members updated from the hybrid-platforms/org repo. Convention update scripts now warn consuming repos about stale alias references in their OWNERS files. commit: bb66e6dc802572b8cd74840d709b64a47ccb620c author: red-hat-konflux[bot] chore(deps): update konflux references (#783) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: cc82140effdd8dbfc0308dfa30840e359fd965d3 author: red-hat-konflux[bot] chore(deps): update all digest updates Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 40a024a080589201385ecb4d8276865f382f4e7a author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 2153c03ccf3b2f6423642f63361d83eca72af6a5 author: red-hat-konflux[bot] chore(deps): update konflux references (#780) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 6a1e728c4cffff224bc3081150b4c2a6bb98e4dc author: Daniel Hall fix(pipelines): update git-clone-oci-ta task bundle to trusted version The previous digest was revoked, causing trusted_task.trusted violations in downstream repos (e.g. rbac-permissions-operator). Signed-off-by: Daniel Hall <danhall@redhat.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> commit: 4fa952575170159d2fcfe085357927558a938bcb author: red-hat-konflux[bot] chore(deps): update konflux references (#776) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 826fc687cb4e9cbec44852e88cf6f364dac21f93 author: Daniel Hall fix(pipelines): update Konflux task bundle digests to trusted versions Update task bundle references to match the latest trusted upstream pipeline bundle (pipeline-docker-build-oci-ta:devel). The previous digests were revoked from the Konflux Enterprise Contract trusted task set, causing EC verify failures in downstream repos. Updated tasks: - task-clair-scan:0.3 - task-sast-snyk-check-oci-ta:0.5 - task-sast-shell-check-oci-ta:0.1 - task-sast-unicode-check-oci-ta:0.4 - task-rpms-signature-scan:0.2 Signed-off-by: Daniel Hall <danhall@redhat.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> commit: 596193fcd5b46772239f90cdc83e9c617cba002d author: red-hat-konflux[bot] chore(deps): update konflux references (#774) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 9dc481b03b3b36ee57d10f12b77ddfef0fed5f19 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: d6f5d8d616cd9f6207a081b402847452fff99980 author: Josh Branham Update OWNERS_ALIASES commit: 44c5cc9aac1b9ad275235263bf006147bee51b5a author: Jason Healy tidy security team members Removes Casey, who has moved to app-sre, and updates my GitHub username. commit: 94304f2e7988c420490761559078a532207cd3fe author: Alex Smith Fix incorrect closing fi commit: 3546c2274351309a862b636af4fd5328ab48a5e7 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 8ecb257ce6c0c515fd671d3fa99ffa178642097e author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: e927b5c3a9be8f3948e92d9ace8e7235ec8751a8 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: e5c909f0670e96152d001c8eedd47239ec78e737 author: Alex Smith Compile golangci-lint in container commit: 168bc549866be3e05429bb9eee12402c7e030741 author: Christopher Mancini Update boilerplate/openshift/golang-osd-operator/update Co-authored-by: Ritu Mundhe <rmundhe@redhat.com> commit: bc326cfc3867dcccd00b78f9d70f91df97223970 author: Alex Smith Update sast-snyk-check-oci-ta task digest commit: dfe333265e5ae00e917a50061509fba4657ecf2f author: Alex Smith Bump UBI9 image to 9.8 commit: 814eca209808a086e7dc0b212686f2c31bed90c4 author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to 12b7acc Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 068f4da5d269790bc96c7bf1a96e77f0133974e7 author: red-hat-konflux[bot] chore(deps): update konflux references (#762) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 4b2ed1adbf5200151a45b2510ed4e77662590dcd author: cgong docs: replace uv curl installation script with official package managers and pip commit: ffd18ca9ec4fbd6cea1a4a0d5f035c1c497e38f0 author: red-hat-konflux[bot] chore(deps): update konflux references Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: 6a1278e0e838590f334ec02b8427cac731f558ac author: red-hat-konflux[bot] chore(deps): update registry.access.redhat.com/ubi8/ubi-minimal:latest docker digest to 5d76d0c (#759) Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com> Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com> commit: ea1887f1148cbe93fa0f8692af27ef7c92d8cb4a author: devppratik chore(remove devppratik from owners) commit: db4d8254bb80e5d3007190a60c27d1cd042056e6 author: MitaliBhalla golang-osd-e2e: derive e2e Dockerfile WORKDIR from go.mod Use the module path from go.mod instead of OperatorName so repos like MCWV build with the correct GOPATH layout. Bump default Go builder images to 1.26 to match current operator go.mod/toolchain versions. Co-authored-by: Cursor <cursoragent@cursor.com> commit: 806a6cb87c742643c0e66fbffffc4b5fc3559007 author: Kirk Bater update subscriber propose update command to work (#756) * update subscriber propose update command to work * updates subscribers * fixes subscriber steps * adds utilitiy to find subscribers commit: 78ca4f49ab071cfce931e4f116dd0afa24673f0f author: Christopher Mancini Remove srep-infra-cicd from golang-osd-operator update script [SDCICD-1761] The srep-infra-cicd team migration is complete. Operator teams should now own their Konflux setup. This update: 1. Removes automatic creation of .tekton/OWNERS with srep-infra-cicd 2. Adds cleanup logic to remove srep-infra-cicd from existing .tekton/OWNERS: - Deletes the file if it only contains srep-infra-cicd - Removes only srep-infra-cicd if other owners are present Repos will get this cleanup automatically on their next bp-update run. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> commit: 4113f0425dbcd9f4da00b1c8a56c7b39cd907926 author: vkadapar [SDCICD-1701] Address review feedback on operator precheck convention - Guard nil Spec.Replicas before deref in checkDeploymentsReady - Replace loose strings.Contains version matching with boundary-aware check - Make BeforeSuite conflict a fatal error (exit 1) instead of a warning Co-Authored-By: Claude Code <noreply@anthropic.com> commit: 37ced7454dea4b83103ecb54d97f66c2f6bf536d author: vkadapar [SDCICD-1701] Add `golang-osd-operator-precheck` boilerplate convention New convention that generates operator pre-check tests into each operator's `test/e2e/` directory. Validates operator readiness before e2e tests run via a Ginkgo `BeforeSuite`: - Auto-detects PKO (`ClusterPackage`) vs OLM (`CSV`) operator management - Validates deployed version matches expected version from image tag - Checks operator namespace exists and all deployments are healthy - Polls every 5s with 10min timeout; skips version check for `latest` tag - Uses kubernetes dynamic client to avoid API discovery rate limiting - Scopes CSV queries to operator namespace for efficiency - Reads `OperatorName` and `OperatorNamespace` from `config/config.go` Files generated by the convention: - `test/e2e/operator_precheck.go` (copied, always overwritten) - `test/e2e/{operator}_precheck_test.go` (generated `BeforeSuite` wrapper) Operators subscribe by adding `openshift/golang-osd-operator-precheck` to `boilerplate/update.cfg`. Requires `openshift/golang-osd-e2e`. ## Test Strategy Tested on cluster `2q2npckfvdh6fg2sa2d8ac6la936tbin` using osde2e ad-hoc image execution. ### Setup 1. Cloned `rbac-permissions-operator` (OLM) and `route-monitor-operator` (PKO) into `~/repo/osd-operators/` 2. Added `openshift/golang-osd-operator-precheck` to each operator's `boilerplate/update.cfg` 3. Ran `BOILERPLATE_GIT_REPO=<local-boilerplate> make boilerplate-update` to generate precheck files using the local boilerplate convention 4. Ran `GOFLAGS="-tags=osde2e" go mod tidy` to pull in dependencies 5. Built e2e images with `podman build -f test/e2e/Dockerfile` and pushed to `quay.io/vkadapar_openshift/` 6. Tagged the same image with different version tags for mismatch and health-only scenarios 7. Ran tests via osde2e VSCode launch configs pointing to the images ### Results | Test | Operator | Path | Tag | Result | |------|----------|------|-----|--------| | OLM success | `rbac-permissions-operator` | CSV match | `v0.1.494-g951a9ca` | Passed | | OLM mismatch | `rbac-permissions-operator` | CSV mismatch | `v999.0.0-gdeadbeef` | Passed (10min timeout, tests skipped) | | PKO success | `route-monitor-operator` | ClusterPackage match | `3c23881` | Passed | | PKO mismatch | `route-monitor-operator` | ClusterPackage mismatch | `v999.0.0-gdeadbeef` | Passed (10min timeout, tests skipped) | | Health-only | `rbac-permissions-operator` | Skip version | `latest` | Passed (health checks only) | ### Issues found and fixed during testing - **In-cluster config:** used `rest.InClusterConfig()` when `KUBECONFIG` unset - **API rate limiting:** switched to dynamic client with explicit GVRs - **Namespace mismatch:** read `OperatorNamespace` from `config/config.go` ### Test Images Available at quay.io for reproducing tests via osde2e ad-hoc images: **OLM (`rbac-permissions-operator`):** - `quay.io/vkadapar_openshift/rbac-permissions-operator-e2e:v0.1.494-g951a9ca` (success) - `quay.io/vkadapar_openshift/rbac-permissions-operator-e2e:v999.0.0-gdeadbeef` (mismatch) - `quay.io/vkadapar_openshift/rbac-permissions-operator-e2e:latest` (health-only) **PKO (`route-monitor-operator`):** - `quay.io/vkadapar_openshift/route-monitor-operator-e2e:3c23881` (success) - `quay.io/vkadapar_openshift/route-monitor-operator-e2e:v999.0.0-gdeadbeef` (mismatch) Co-Authored-By: Claude Code <noreply@anthropic.com>
|
Warning This pull request changes a CodeRabbit configuration file. Because it comes from a fork or its author is not a repository collaborator, reviews use only the configuration from the target branch. The proposed configuration will take effect after it is merged. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (13)
📒 Files selected for processing (7)
💤 Files with no reviewable changes (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe change updates the CI build image, review exclusions, Dependabot configuration, repository ownership, and Tekton pull request automation. A new PipelineRun invokes a pinned agentic SDLC check for pull requests targeting ChangesCI and repository governance
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This updates repository CI, governance, and pull-request automation conventions without an identified current merge-blocking risk. Sequence Diagram(s)sequenceDiagram
participant GitHub as GitHub pull request
participant PipelineRun as Tekton PipelineRun
participant Boilerplate as openshift/boilerplate pipeline
GitHub->>PipelineRun: Send pull_request event for master
PipelineRun->>PipelineRun: Evaluate CEL trigger
PipelineRun->>Boilerplate: Resolve pinned agentic-sdlc-check pipeline
Boilerplate-->>PipelineRun: Use git-url, revision, workspace, and git-auth
🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: redhat-chai-bot The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@redhat-chai-bot: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #387 +/- ##
=======================================
Coverage 13.05% 13.05%
=======================================
Files 15 15
Lines 1141 1141
=======================================
Hits 149 149
Misses 979 979
Partials 13 13 🚀 New features to boost your workflow:
|
Conventions:
openshift/boilerplate@f66d57c...a8a3172
AI-generated. Review for accuracy.
Automated by scheduled task
rosa_sre_boilerplate_update(instructions:ship_help_bot/shared/instructions/scheduled/rosa_sre_boilerplate_update.md, run:22ff8a8c, commit:20a4c3bc-dirty)Summary by CodeRabbit
New Features
masterbranch.Chores
Removed