Skip to content

Use public Deck defaults for ship-help-bot CI - #84719

Closed
redhat-chai-bot wants to merge 1 commit into
openshift:mainfrom
redhat-chai-bot:ship-help-bot-public-deck-20260908-clean
Closed

Use public Deck defaults for ship-help-bot CI#84719
redhat-chai-bot wants to merge 1 commit into
openshift:mainfrom
redhat-chai-bot:ship-help-bot-public-deck-20260908-clean

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Make CI activity for openshift-eng/ship-help-bot use the public Deck defaults instead of the private Deck routing.

Changes

  • Remove the repository-specific private artifact storage and browsing configuration.
  • Remove the repository-specific private job and PR status URLs.
  • Remove the repository-specific private PR report template.
  • Remove the repository-specific private tenant assignment.

Validation

  • make prow-config passed twice with byte-identical output.
  • YAML parsing and targeted assertions passed.
  • git diff --check origin/main...HEAD passed.
  • The branch comparison contains exactly one file changed with 17 deletions and no additions.

AI-generated. Review for accuracy.

@jupierce requested via Chai Bot

Summary by CodeRabbit

  • Updates openshift-eng/ship-help-bot CI to use public Deck defaults.
  • Removes private artifact browsing and storage settings, status URL overrides, PR report templates, and tenant assignment.
  • Validation passed with no generated output changes.

@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: redhat-chai-bot
Once this PR has been reviewed and has the lgtm label, please assign bear-redhat for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@coderabbitai

coderabbitai Bot commented Sep 8, 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: Team

Run ID: 2fbe8f30-ccc3-47c1-a168-6b9d0bd165e6

📥 Commits

Reviewing files that changed from the base of the PR and between d68bb7c and 1e302a1.

📒 Files selected for processing (1)
  • core-services/prow/02_config/_config.yaml
💤 Files with no reviewable changes (1)
  • core-services/prow/02_config/_config.yaml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


Walkthrough

The Prow configuration removes all openshift-eng/ship-help-bot entries from artifact browsing, decoration, job URLs, report templates, tenant assignment, and PR status URLs.

Changes

Prow configuration cleanup

Layer / File(s) Summary
Remove repository-specific Prow entries
core-services/prow/02_config/_config.yaml
Removes six openshift-eng/ship-help-bot mappings from Prow configuration sections related to artifacts, jobs, dashboards, tenants, and PR status links.

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

Merge Risk: ⚪ Minimal · up to 1e302

This change removes repository-specific private Prow routing so ship-help-bot uses public Deck defaults. No current merge-blocking risk is identified.

🚥 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 accurately and clearly summarizes the main change: configuring ship-help-bot CI to use public Deck defaults.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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 PASS. The pull request changes only core-services/prow/02_config/_config.yaml and removes 17 repository-specific Prow configuration entries. It does not add or modify Ginkgo tests or test titles. Th…
Test Structure And Quality ✅ Passed PASS: The pull request changes only core-services/prow/02_config/_config.yaml and deletes 17 repository-specific Prow configuration entries. The diff contains no Ginkgo test code, test files, It b…
Microshift Test Compatibility ✅ Passed PASS. The pull request changes only core-services/prow/02_config/_config.yaml, with 17 deletions and no additions. The diff removes Prow routing and tenant settings for openshift-eng/ship-help-bot
Single Node Openshift (Sno) Test Compatibility ✅ Passed The custom check is not applicable. The pull request changes only core-services/prow/02_config/_config.yaml, with 17 deletions and no additions. The diff removes repository-specific Prow routing ent…
Topology-Aware Scheduling Compatibility ✅ Passed PASS: The pull request changes only core-services/prow/02_config/_config.yaml and deletes 17 repository-specific Prow Deck, URL, report-template, and tenant settings. It adds or modifies no deployme…
Ote Binary Stdout Contract ✅ Passed PASS. The pull request changes only core-services/prow/02_config/_config.yaml, with 17 YAML deletions and no additions. The deletions remove private Prow/Deck routing entries for `openshift-eng/ship…
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PASS. The pull request changes only core-services/prow/02_config/_config.yaml and removes 17 configuration lines. The diff adds no Ginkgo tests or other test code, so the IPv4 and external-connectiv…
No-Weak-Crypto ✅ Passed PASS. The PR changes only core-services/prow/02_config/_config.yaml and contains 0 additions and 17 deletions. The deletions remove repository-specific Prow routing, URLs, report configuration, and …
Container-Privileges ✅ Passed PASS. The pull request changes only core-services/prow/02_config/_config.yaml and deletes 17 lines. The diff has zero additions. The deleted lines configure Deck routing, GCS storage, report URLs, a…
No-Sensitive-Data-In-Logs ✅ Passed The pull request adds no logging or log content. The verified diff contains one YAML file with 17 deletions and zero additions. It removes repository-specific private Deck, artifact, URL, report-templ…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Sep 8, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@redhat-chai-bot: 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 Sep 8, 2026

Copy link
Copy Markdown
Contributor

@redhat-chai-bot: 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.

@jupierce jupierce closed this Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants