Skip to content

Update repository list and bot authors in fetch-data.sh#650

Open
iamkirkbater wants to merge 1 commit into
mainfrom
iamkirkbater-patch-1
Open

Update repository list and bot authors in fetch-data.sh#650
iamkirkbater wants to merge 1 commit into
mainfrom
iamkirkbater-patch-1

Conversation

@iamkirkbater

@iamkirkbater iamkirkbater commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Updates repo names in the PR Dashboard

Summary by CodeRabbit

  • Chores
    • Updated the repository list to include the new rosa-hyperfleet repositories.
    • Adjusted bot filtering so “needs-ok-to-test” continues to recognize the correct automation accounts.

@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The dashboard fetch-data script now targets the rosa-hyperfleet* repositories and matches rosa-hyperfleet-ci in the bot-author filter.

Changes

Dashboard fetch-data target update

Layer / File(s) Summary
Repository list and bot filter
dashboard/fetch-data.sh
The repository target array now uses rosa-hyperfleet* names, and the needs-ok-to-test bot-author regex now matches rosa-hyperfleet-ci.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main fetch-data.sh change: updated repository names and bot author filtering.
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.

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

✨ 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 iamkirkbater-patch-1

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.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@dashboard/fetch-data.sh`:
- Line 51: The bot author identity used by the `/ok-to-test` flow is out of sync
between the dashboard script and the IC process docs. Update the
`BOT_AUTHORS`-related guidance in `docs/process/ic.md` to use
`rosa-hyperfleet-ci` so it matches the behavior defined in `fetch-data.sh`,
keeping the references to the `/ok-to-test` handling and bot allowlist
consistent.
- Around line 15-18: The dashboard shorthand mapping is out of date and only
recognizes the old rosa-regional-platform repository names, so update the SHORT
mapping in dashboard/index.html to include the new rosa-hyperfleet variants
emitted by REPOS. Keep the existing label behavior, but add or replace the
entries so the repository-name-to-display-name lookup in the mapping correctly
handles rosa-hyperfleet, rosa-hyperfleet-api, rosa-hyperfleet-cli, and
rosa-hyperfleet-internal.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 497244bb-3a84-486f-9e83-1e69908f269c

📥 Commits

Reviewing files that changed from the base of the PR and between 337be0f and be3324c.

📒 Files selected for processing (1)
  • dashboard/fetch-data.sh

Comment thread dashboard/fetch-data.sh
Comment on lines +15 to +18
openshift-online/rosa-hyperfleet
openshift-online/rosa-hyperfleet-api
openshift-online/rosa-hyperfleet-cli
openshift-online/rosa-hyperfleet-internal

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.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Update frontend repo shorthand mapping to match new repo names.

REPOS now emits rosa-hyperfleet* repository names, but dashboard/index.html still maps only rosa-regional-platform* in SHORT (Line 257-301). This can regress display labels in the dashboard.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@dashboard/fetch-data.sh` around lines 15 - 18, The dashboard shorthand
mapping is out of date and only recognizes the old rosa-regional-platform
repository names, so update the SHORT mapping in dashboard/index.html to include
the new rosa-hyperfleet variants emitted by REPOS. Keep the existing label
behavior, but add or replace the entries so the repository-name-to-display-name
lookup in the mapping correctly handles rosa-hyperfleet, rosa-hyperfleet-api,
rosa-hyperfleet-cli, and rosa-hyperfleet-internal.

Comment thread dashboard/fetch-data.sh
fetch_label "discussion-needed" > /tmp/dn.json

BOT_AUTHORS="app/dependabot|rosa-regional-platform-ci|rrp-bot"
BOT_AUTHORS="app/dependabot|rosa-hyperfleet-ci|rrp-bot"

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.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Sync IC process docs with the new bot author identity.

BOT_AUTHORS now matches rosa-hyperfleet-ci, but docs/process/ic.md still references rosa-regional-platform-ci (Line 26-29). Please update the doc so /ok-to-test guidance matches dashboard behavior.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@dashboard/fetch-data.sh` at line 51, The bot author identity used by the
`/ok-to-test` flow is out of sync between the dashboard script and the IC
process docs. Update the `BOT_AUTHORS`-related guidance in `docs/process/ic.md`
to use `rosa-hyperfleet-ci` so it matches the behavior defined in
`fetch-data.sh`, keeping the references to the `/ok-to-test` handling and bot
allowlist consistent.

@slopezz

slopezz commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

/lgtm

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.

2 participants