USHIFT-7041: MicroShift CI Doctor directory restructure - #181
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ggiguash The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
WalkthroughThis PR systematically refactors the CI Doctor workflow's on-disk artifact organization by consolidating flat ChangesArtifact Directory Structure Refactoring
🎯 3 (Moderate) | ⏱️ ~20 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 markdownlint-cli2 (0.22.1)docs/claude/microshift-ci.mdmarkdownlint-cli2 v0.22.1 (markdownlint v0.40.0) ... [truncated 1076 characters] ... node:internal/modules/esm/resolve:271:11) plugins/lvms-ci/skills/doctor/SKILL.mdmarkdownlint-cli2 v0.22.1 (markdownlint v0.40.0) ... [truncated 1086 characters] ... node:internal/modules/esm/resolve:271:11) plugins/microshift-ci/skills/doctor/SKILL.mdmarkdownlint-cli2 v0.22.1 (markdownlint v0.40.0) ... [truncated 1092 characters] ... node:internal/modules/esm/resolve:271:11)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@plugins/microshift-ci/skills/create-bugs/SKILL.md`:
- Line 4: Update the SKILL.md frontmatter description for the create-bugs skill
to be action-oriented (start with "Use when…") instead of generic prose; locate
the current description key in SKILL.md (the line containing description: Create
JIRA bugs from CI failure reports...) and replace it with a concise "Use when…"
sentence that tells when to run the skill (for example: "Use when a CI failure
report should create or deduplicate JIRA bugs across releases; runs in dry‑run
by default"). Ensure the new text preserves the existing intent: CI failure →
create JIRA bugs with cross-release deduplication and dry-run default.
🪄 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: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 023f9778-2424-438e-b420-f98caadd74a2
📒 Files selected for processing (13)
docs/claude/microshift-ci.mdplugins/lvms-ci/skills/doctor/SKILL.mdplugins/microshift-ci/scripts/continue-session.shplugins/microshift-ci/scripts/search-bugs.pyplugins/microshift-ci/skills/close-stale-bugs/SKILL.mdplugins/microshift-ci/skills/continue-session/SKILL.mdplugins/microshift-ci/skills/create-bugs/SKILL.mdplugins/microshift-ci/skills/doctor-refresh/SKILL.mdplugins/microshift-ci/skills/doctor/SKILL.mdplugins/microshift-ci/skills/fix-test-bugs/SKILL.mdplugins/shared/scripts/aggregate.pyplugins/shared/scripts/create-report.pyplugins/shared/scripts/doctor.sh
|
/lgtm |
New Directory Structure
Summary by CodeRabbit
Release Notes
Documentation
Chores
jobs/andbugs/) with consistent, deterministic naming patterns.