Skip to content

feat: add rhdh-pr-review skill for testing operator PRs on live clusters#20

Open
Fortune-Ndlovu wants to merge 6 commits intoredhat-developer:mainfrom
Fortune-Ndlovu:feat/rhdh-pr-review
Open

feat: add rhdh-pr-review skill for testing operator PRs on live clusters#20
Fortune-Ndlovu wants to merge 6 commits intoredhat-developer:mainfrom
Fortune-Ndlovu:feat/rhdh-pr-review

Conversation

@Fortune-Ndlovu
Copy link
Copy Markdown
Member

Automates the PR review workflow: fetches CI-built images from GH Actions comments, ensures a running RHDH cluster (provisioning via rhdh-test-instance if needed), swaps the operator image (CSV patch for OLM, oc set image for non-OLM), and generates a targeted review checklist from the diff.

Automates the PR review workflow: fetches CI-built images from GH Actions
comments, ensures a running RHDH cluster (provisioning via rhdh-test-instance
if needed), swaps the operator image (CSV patch for OLM, oc set image for
non-OLM), and generates a targeted review checklist from the diff.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@schultzp2020
Copy link
Copy Markdown
Collaborator

I'll take a look!

Fortune-Ndlovu and others added 2 commits May 7, 2026 15:49
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@schultzp2020 schultzp2020 left a comment

Choose a reason for hiding this comment

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

Nice work, Fortune! A few questions and suggestions inline.

Comment thread skills/rhdh-pr-review/SKILL.md Outdated
Comment thread skills/rhdh-pr-review/SKILL.md
Comment thread skills/rhdh-pr-review/SKILL.md
Comment thread skills/rhdh-pr-review/workflows/review-operator-pr.md
Comment thread skills/rhdh-pr-review/workflows/review-operator-pr.md Outdated
Comment thread skills/rhdh/SKILL.md
Comment thread tests/unit/test_review_pr_skill.py Outdated
Comment thread skills/rhdh-pr-review/SKILL.md
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
… set image) paths.

Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
…ocumentation

- Introduced a new reference document for provisioning RHDH clusters via the rhdh-test-instance PR workflow.
- Updated the review workflow documentation to reference the new cluster provisioning guide and streamlined the process descriptions.
- Removed outdated success criteria from the SKILL.md and linked to the updated checklist in the new reference.

Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
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