Skip to content

Add version '5.1' to ACTIVE_OCP_VERSIONS list - #3221

Merged
joepvd merged 1 commit into
mainfrom
fgallott-patch-1
Aug 4, 2026
Merged

Add version '5.1' to ACTIVE_OCP_VERSIONS list#3221
joepvd merged 1 commit into
mainfrom
fgallott-patch-1

Conversation

@fgallott

@fgallott fgallott commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features
    • Added support for OpenShift Container Platform version 5.1 in the list of actively supported releases.

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: bf554398-d2c4-44c9-b0ce-0e73da9dc069

📥 Commits

Reviewing files that changed from the base of the PR and between b3f2c06 and 7439884.

📒 Files selected for processing (1)
  • artcommon/artcommonlib/constants.py

Walkthrough

Changes

OpenShift version support

Layer / File(s) Summary
Active version list update
artcommon/artcommonlib/constants.py
ACTIVE_OCP_VERSIONS now includes "5.1".

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

Suggested reviewers: ashwindasr, thegreyd

🚥 Pre-merge checks | ✅ 11
✅ Passed checks (11 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 addition of version 5.1 to the ACTIVE_OCP_VERSIONS list.
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.
No-Weak-Crypto ✅ Passed The PR adds only the literal version string "5.1" to ACTIVE_OCP_VERSIONS; it adds no cryptographic algorithm, custom crypto, or secret comparison.
Container-Privileges ✅ Passed The PR modifies only a Python constants file, adding version "5.1" to ACTIVE_OCP_VERSIONS list. No container or Kubernetes manifests are changed, so the container-privileges check does not apply.
No-Sensitive-Data-In-Logs ✅ Passed The commit adds only the literal "5.1" to ACTIVE_OCP_VERSIONS; it adds no logging calls or sensitive-data values to log output.
No-Hardcoded-Secrets ✅ Passed The PR adds only the literal version "5.1"; focused scans found no secrets, credential-bearing URLs, suspicious assignments, or long base64 strings.
No-Injection-Vectors ✅ Passed The parent-to-HEAD diff adds only the literal string "5.1" to ACTIVE_OCP_VERSIONS; the changed file contains no flagged SQL, shell, eval/exec, pickle, YAML, os.system, or HTML injection pattern.
Ai-Attribution ✅ Passed The PR has one commit with no AI-tool mention, no attribution trailer, and no Co-Authored-By line; the diff only adds version 5.1.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fgallott-patch-1

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

@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

@fgallott: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/art-pre-commit-check 7439884 link false /test art-pre-commit-check
ci/prow/security 7439884 link false /test security

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.

@joepvd joepvd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: joepvd

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 4, 2026
@joepvd
joepvd merged commit 0415a37 into main Aug 4, 2026
7 of 10 checks passed
@fgallott
fgallott deleted the fgallott-patch-1 branch August 5, 2026 15:45
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.

2 participants