Skip to content

HYPERFLEET-668 | Add the Manual Trigger in Prow Dashboard#91

Merged
yingzhanredhat merged 1 commit intoopenshift-hyperfleet:mainfrom
yingzhanredhat:ying-test
Feb 27, 2026
Merged

HYPERFLEET-668 | Add the Manual Trigger in Prow Dashboard#91
yingzhanredhat merged 1 commit intoopenshift-hyperfleet:mainfrom
yingzhanredhat:ying-test

Conversation

@yingzhanredhat
Copy link
Contributor

@yingzhanredhat yingzhanredhat commented Feb 25, 2026

Summary by CodeRabbit

  • Documentation
    • Expanded Manual Trigger guidance in the Prow dashboard: how team-based rerun permissions are configured, YAML example fields, verification steps, and note that only teams (no individual users) are currently configured.
    • Added command-based Manual Trigger instructions, token acquisition notes, and curl examples for triggering periodic and PR rehearse jobs.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026

Walkthrough

Adds detailed Prow Dashboard documentation describing GitHub rerun permissions configured via rerun_auth_configs in _config.yaml. Includes a YAML example for openshift-hyperfleet/hyperfleet-e2e with github_team_slugs (and optional github_users), enumerates configured teams, notes github_users is empty, and provides verification steps. Also documents manual trigger via command including how to obtain an authentication token and curl examples for triggering periodic jobs and rehearse runs.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • yasun1
  • rh-amarin
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding Manual Trigger documentation to the Prow Dashboard, which aligns with the changeset content.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md`:
- Line 191: The sentence "After this configuration is merged, members of these
teams will be able to rerun failed CI jobs on PRs in the
`openshift-hyperfleet/hyperfleet-e2e` repository directly from the Prow
dashboard" is missing a period; edit that line in
add-hyperfleet-e2e-ci-job-in-prow.md to append a trailing period so it reads
with proper punctuation.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 76cd9ea and e916e1f.

📒 Files selected for processing (1)
  • hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md

@yasun1
Copy link
Contributor

yasun1 commented Feb 26, 2026

/lgtm

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

♻️ Duplicate comments (1)
hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md (1)

191-191: ⚠️ Potential issue | 🟡 Minor

Add terminal punctuation to complete the sentence.

Line 191 is missing a period at the end, which hurts documentation quality slightly.

✏️ Suggested fix
-After this configuration is merged, members of these teams will be able to rerun and abort CI jobs under the `openshift-hyperfleet/hyperfleet-e2e` repository directly from the Prow dashboard
+After this configuration is merged, members of these teams will be able to rerun and abort CI jobs under the `openshift-hyperfleet/hyperfleet-e2e` repository directly from the Prow dashboard.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md` at line 191,
The sentence "After this configuration is merged, members of these teams will be
able to rerun and abort CI jobs under the `openshift-hyperfleet/hyperfleet-e2e`
repository directly from the Prow dashboard" is missing terminal punctuation;
edit hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md and append a
period to that sentence so it ends with "Prow dashboard.".
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md`:
- Line 191: The sentence "After this configuration is merged, members of these
teams will be able to rerun and abort CI jobs under the
`openshift-hyperfleet/hyperfleet-e2e` repository directly from the Prow
dashboard" is missing terminal punctuation; edit
hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md and append a period
to that sentence so it ends with "Prow dashboard.".

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 51809dc and a87abe2.

📒 Files selected for processing (1)
  • hyperfleet/test-release/add-hyperfleet-e2e-ci-job-in-prow.md

@yingzhanredhat yingzhanredhat merged commit 18dc4f4 into openshift-hyperfleet:main Feb 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants