Skip to content

chore(Boilerplate update)#255

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
devppratik:update-boilerplate
May 18, 2026
Merged

chore(Boilerplate update)#255
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
devppratik:update-boilerplate

Conversation

@devppratik
Copy link
Copy Markdown
Contributor

@devppratik devppratik commented May 13, 2026

What type of PR is this?

(boilerplate update)

What this PR does / why we need it?

Updating the repo by running make boilerplate-update to sync latest changes.

Which Jira/Github issue(s) this PR fixes?

Fixes #

Special notes for your reviewer:

Pre-checks (if applicable):

  • Tested latest changes against a cluster
  • Ran make generate command locally to validate code changes
  • Included documentation changes with PR

Summary by CodeRabbit

  • Chores
    • Updated team member access within the project.
    • Updated container runtime base images to latest versions.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

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: b5b49ae0-777e-4662-8460-c15fd4c0cc96

📥 Commits

Reviewing files that changed from the base of the PR and between 29435a7 and 6e8da31.

⛔ Files ignored due to path filters (2)
  • boilerplate/_data/last-boilerplate-commit is excluded by !boilerplate/**
  • boilerplate/openshift/golang-osd-operator/OWNERS_ALIASES is excluded by !boilerplate/**
📒 Files selected for processing (3)
  • OWNERS_ALIASES
  • build/Dockerfile
  • build/Dockerfile.olm-registry

Walkthrough

This PR adds a GitHub handle to a team alias and updates the Red Hat UBI base image tag in two Dockerfiles to a newer patch version. Three independent changes across configuration and build files.

Changes

Infrastructure and Team Updates

Layer / File(s) Summary
Team membership
OWNERS_ALIASES
Added BATMAN-JD to the srep-functional-team-aurora team alias list.
Base image version bumps
build/Dockerfile, build/Dockerfile.olm-registry
Updated Red Hat UBI minimal base image from 9.7-1778461551 to 9.7-1778562320 in both Dockerfiles' FROM directives.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 10 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Test Structure And Quality ⚠️ Warning Tests lack AfterEach cleanup for gomock controllers (6 handler tests, fleetnotification controller test), lack meaningful assertion messages, and have redundant error assertions. Add AfterEach with mockCtrl.Finish() to handler/controller tests. Add messages to Expect() calls. Remove duplicate error assertions.
Title check ❓ Inconclusive The title 'chore(Boilerplate update)' is vague and generic, using non-descriptive terms that don't convey specific details about what was actually updated. Consider revising the title to be more specific about the changes, such as 'chore: Update base image versions and add GitHub handle to OWNERS_ALIASES' to provide clearer context.
✅ Passed checks (10 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 No violations in explicitly listed test types. All main test titles are static and deterministic. Dynamic content only in ginkgo.By() steps, not in the check specification.
Microshift Test Compatibility ✅ Passed No new Ginkgo e2e tests were added. This boilerplate PR only modifies OWNERS_ALIASES and Docker base image versions. The check is not applicable.
Single Node Openshift (Sno) Test Compatibility ✅ Passed No new Ginkgo e2e tests were added in this PR. Changes consist of OWNERS_ALIASES config update and Docker base image version updates. SNO compatibility check is not applicable.
Topology-Aware Scheduling Compatibility ✅ Passed PR contains only boilerplate updates: OWNERS_ALIASES edits and Dockerfile base image version bumps. No deployment manifests, operator code, or scheduling constraints are introduced or modified.
Ote Binary Stdout Contract ✅ Passed PR adds a Kubernetes operator, not an OTE test binary. The check applies only to openshift-tests extension binaries. Not applicable.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed This PR does not add any Ginkgo e2e tests. The changes are limited to OWNERS_ALIASES configuration and Dockerfile base image updates. The check is not applicable to this PR.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@openshift-ci openshift-ci Bot requested review from ravitri and vaidehi411 May 13, 2026 08:03
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 13, 2026

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.04%. Comparing base (29435a7) to head (6e8da31).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #255   +/-   ##
=======================================
  Coverage   66.04%   66.04%           
=======================================
  Files          23       23           
  Lines        1546     1546           
=======================================
  Hits         1021     1021           
  Misses        447      447           
  Partials       78       78           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tkong-redhat
Copy link
Copy Markdown
Contributor

/lgtm

@tkong-redhat
Copy link
Copy Markdown
Contributor

/approve

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 18, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 18, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: devppratik, tkong-redhat

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 May 18, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit cc46d50 into openshift:master May 18, 2026
14 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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants