Skip to content

fix: add prow trigger plugin config for openshift-priv/dra-driver-sriov - #83508

Merged
openshift-merge-robot merged 1 commit into
openshift:mainfrom
Prucek:fix-trigger-plugin-openshift-priv
Aug 17, 2026
Merged

fix: add prow trigger plugin config for openshift-priv/dra-driver-sriov#83508
openshift-merge-robot merged 1 commit into
openshift:mainfrom
Prucek:fix-trigger-plugin-openshift-priv

Conversation

@Prucek

@Prucek Prucek commented Aug 17, 2026

Copy link
Copy Markdown
Member

Summary

🤖 Generated with Claude Code

Summary by CodeRabbit

Adds the missing Prow plugin configuration for openshift-priv/dra-driver-sriov. This enables the repository’s approval, LGTM, trigger, and Bugzilla plugin behavior and satisfies the Prow trigger plugin configuration check.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 04c9a9c9-50f3-4faf-a211-3efd170d3957

📥 Commits

Reviewing files that changed from the base of the PR and between 51b6bf9 and 622744b.

📒 Files selected for processing (1)
  • core-services/prow/02_config/openshift-priv/dra-driver-sriov/_pluginconfig.yaml

Included review availability: Your plan includes up to 12 reviews per rolling hour; 11 remain after this review.


Walkthrough

Added Prow configuration for openshift-priv/dra-driver-sriov. The configuration enables approval and LGTM behavior, registers repository plugins, and trusts openshift-merge-bot for triggers.

Changes

Prow configuration

Layer / File(s) Summary
Repository plugin behavior
core-services/prow/02_config/openshift-priv/dra-driver-sriov/_pluginconfig.yaml
Configures approval behavior, review-based LGTM, enabled plugins, and trusted repository triggers.

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

Merge Risk: 🔵 Low · up to 62274

The PR adds repository-specific Prow automation settings, but Bugzilla automation may remain incomplete if the required repository mapping is missing; the change is mergeable with explicit owner verification.

Possibly related PRs

🚥 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 and concisely describes the added Prow trigger plugin configuration for the specified repository.
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 parent-to-HEAD diff adds only one Prow YAML file and contains no Ginkgo test files or test titles.
Test Structure And Quality ✅ Passed The PR adds only a 41-line Prow YAML file; the diff introduces no Ginkgo tests, cluster operations, waits, or assertions to review.
Microshift Test Compatibility ✅ Passed The PR adds only a 41-line Prow YAML file. The diff adds no Ginkgo e2e tests or MicroShift-incompatible API references.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The diff adds only a 41-line Prow YAML configuration file and adds no Ginkgo e2e tests or multi-node assumptions.
Topology-Aware Scheduling Compatibility ✅ Passed The PR adds only a Prow _pluginconfig.yaml with approval, LGTM, plugin, and trigger settings; it adds no deployment, controller, or scheduling constraint.
Ote Binary Stdout Contract ✅ Passed The commit adds only a Prow YAML file; it changes no OTE binary or process-level code and introduces no stdout write.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The commit adds only a 41-line Prow YAML configuration; it adds no Ginkgo e2e tests, IPv4 assumptions, or external connectivity.
No-Weak-Crypto ✅ Passed The pull request adds only one Prow YAML file. Its added lines contain no MD5, SHA1, DES, RC4, 3DES, Blowfish, ECB, custom crypto, or secret comparison code.
Container-Privileges ✅ Passed The diff adds only a Prow _pluginconfig.yaml fragment. It contains no privileged, hostPID, hostNetwork, hostIPC, SYS_ADMIN, root, or allowPrivilegeEscalation settings.
No-Sensitive-Data-In-Logs ✅ Passed The changed file only adds Prow approval, LGTM, plugin, and trigger configuration; it contains no logging directives or passwords, tokens, API keys, PII, hostnames, or customer data.
✨ 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 requested review from jmguzik and pruan-rht August 17, 2026 08:44
@openshift-ci

openshift-ci Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Prucek

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 17, 2026
@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Aug 17, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@Prucek: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

@openshift-ci

openshift-ci Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

@Prucek: 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.

@openshift-merge-robot
openshift-merge-robot merged commit 6366dd7 into openshift:main Aug 17, 2026
11 of 12 checks passed
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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants