Skip to content

Add channel jobs - #233

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-online:mainfrom
chenz4027:add-custom-channel-ci-status-jobs
Aug 19, 2026
Merged

Add channel jobs#233
openshift-merge-bot[bot] merged 1 commit into
openshift-online:mainfrom
chenz4027:add-custom-channel-ci-status-jobs

Conversation

@chenz4027

@chenz4027 chenz4027 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)
  • CI/CD or tooling change

Testing

  • Unit tests pass (make test)
  • Integration tests pass (if applicable)
  • Manual verification completed

Checklist

  • My code follows the project's coding conventions
  • I have updated documentation as needed
  • I have added tests that prove my fix/feature works
  • All new and existing tests pass

Summary by CodeRabbit

  • New Features
    • Added STS Custom Channel CI jobs to the OCM FVT Classic staging and integration test categories.
    • Included the corresponding CI job identifiers for automated status tracking.

Signed-off-by: Maggie Chen <magchen@redhat.com>
@openshift-ci
openshift-ci Bot requested review from gdbranco and tiwillia August 19, 2026 17:07
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: 00e4ccc4-05f1-46bf-ba2f-7ddebe4f6f27

📥 Commits

Reviewing files that changed from the base of the PR and between 4a7d182 and 26f48a7.

📒 Files selected for processing (1)
  • configs/ci-status-jobs.yaml

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


Walkthrough

The CI status configuration now includes STS Custom Channel jobs in the OCM FVT Classic staging and integration categories.

Changes

STS Custom Channel CI status jobs

Layer / File(s) Summary
Add STS Custom Channel job identifiers
configs/ci-status-jobs.yaml
Added the staging and integration Prow job identifiers to the corresponding OCM FVT Classic CI status categories.

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

Merge Risk: ⚪ Minimal · up to 26f48

This localized CI configuration change has no actionable merge-blocking risk remaining and is merge-ready after normal checks and review.

Suggested reviewers: gdbranco, tiwillia, dustman9000

🚥 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 describes the main change: adding channel CI jobs.
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 HEAD^..HEAD diff adds only two YAML CI job entries for STS Custom Channel; no weak-crypto primitive, custom crypto, or secret comparison is introduced.
Container-Privileges ✅ Passed The diff adds only two name/prow_job entries in a CI status YAML file. It adds no privileged, host namespace, SYS_ADMIN, or escalation settings.
No-Sensitive-Data-In-Logs ✅ Passed The diff adds only two CI display names and Prow job identifiers; it introduces no logging code or passwords, tokens, API keys, PII, hostnames, or customer data.
No-Hardcoded-Secrets ✅ Passed The four added lines contain only CI job names and Prow identifiers; no secret assignments, credentials, private keys, encoded secrets, or credential-bearing URLs are present.
No-Injection-Vectors ✅ Passed The exact diff adds only four static YAML job fields; no SQL, eval/exec, unsafe YAML load, os.system, shell=True, pickle, or HTML injection is introduced.
Ai-Attribution ✅ Passed The sole PR commit and supplied PR description do not mention AI tools; no Assisted-by, Generated-by, or AI Co-Authored-By check is triggered.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@dustman9000

Copy link
Copy Markdown
Collaborator

/lgtm
/approve

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

openshift-ci Bot commented Aug 19, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chenz4027, dustman9000

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 19, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit facada6 into openshift-online:main Aug 19, 2026
11 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.

2 participants