Skip to content

feat: add Observability, simplified mapping schema, PR labels and Jira ID tracking#110

Closed
vmrh21 wants to merge 2 commits intoambient-code:mainfrom
angaduom:add-observability-and-pr-labels
Closed

feat: add Observability, simplified mapping schema, PR labels and Jira ID tracking#110
vmrh21 wants to merge 2 commits intoambient-code:mainfrom
angaduom:add-observability-and-pr-labels

Conversation

@vmrh21
Copy link
Copy Markdown
Contributor

@vmrh21 vmrh21 commented Apr 16, 2026

Summary

Simplified mapping schema (from PR #108)

Applies the simplified repos[] schema replacing the old nested repositories + container_to_repo_mapping structure. Removes 6 unused components (AI Core Dashboard, Model Serving, Notebooks Images, AI Pipelines, Notebooks Server, Training Kubeflow). Fixes repo types: opendatahub-io/* → midstream, red-hat-data-services/* → downstream.

Observability component (from PR #103)

Converts the Observability component (14 stolostron repos) from the old schema to the simplified format. Containers assigned to their respective repos.

PR label for dashboard tracking

Every fix PR now gets --label cve-fixer-automated so the dashboard can query all workflow-created PRs with gh pr list --label cve-fixer-automated. Falls back gracefully if the label doesn't exist in the target repo.

Jira IDs in PR body

Clarified that both plain (PROJ-12345) and linked ([PROJ-12345](url)) formats are acceptable. Added note that IDs are required for dashboard tracking.

Test plan

  • Run /cve.find Observability — verify it queries ACM Jira tickets
  • Run /cve.fix — verify PRs get cve-fixer-automated label
  • Verify PR body includes Jira issue IDs

🤖 Generated with Claude Code

vmrh21 and others added 2 commits April 16, 2026 13:51
…acking

component-repository-mappings.json:
- Apply simplified schema (repos[] instead of nested repositories/container_to_repo_mapping)
- Remove unused components (AI Core Dashboard, Model Serving, Notebooks Images,
  AI Pipelines, Notebooks Server, Training Kubeflow)
- Fix repo types: opendatahub-io=midstream, red-hat-data-services=downstream, others=upstream
- Add Observability component (14 stolostron repos with ACM containers)
  from PR ambient-code#103 converted to new simplified schema

cve.fix.md:
- Add --label cve-fixer-automated to every gh pr create call with
  graceful fallback if label doesn't exist in the target repo
- Allow both plain and linked Jira issue IDs in PR body (both are fine)
- Add note that Jira IDs are required for dashboard tracking

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 7de03a1d-0ce4-4107-83fa-da29f6e22a54

📥 Commits

Reviewing files that changed from the base of the PR and between 774a5eb and 5c9a982.

📒 Files selected for processing (3)
  • workflows/cve-fixer/.claude/commands/cve.fix.md
  • workflows/cve-fixer/.claude/commands/onboard.md
  • workflows/cve-fixer/component-repository-mappings.json

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting


Walkthrough

The pull request updates the CVE-fixer workflow by modifying command invocations and PR creation guidance in documentation, restructuring the component repository mappings from a per-component schema with container mappings to a unified repos array format, and shifting the onboarding flow from manual container image collection to auto-discovery via Jira labels.

Changes

Cohort / File(s) Summary
CVE Command Documentation
workflows/cve-fixer/.claude/commands/cve.fix.md
Updated Jira issue reference examples to permit markdown links and multiple IDs; enhanced PR creation invocation with base branch, title template, body, and conditional label handling with fallback.
Onboarding Workflow Documentation
workflows/cve-fixer/.claude/commands/onboard.md
Shifted from manual container image input to auto-discovery from Jira pscomponent: labels; replaced per-component JSON mapping schema with simplified repos array; added Step 5 to generate per-repo CVE guidance files from PR history analysis.
Component Repository Mappings Configuration
workflows/cve-fixer/component-repository-mappings.json
Restructured schema from per-component objects with container_to_repo_mapping to flat repos array; removed AI Core Dashboard component; simplified repository fields (removed github_url, protected_branches, branch_strategy, cve_fix_workflow); retained url, type, default_branch, active_branches, containers; updated metadata timestamp to 2026-04-16.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~65 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

@vmrh21
Copy link
Copy Markdown
Contributor Author

vmrh21 commented Apr 16, 2026

Closing — changes moved to PR #108 (simplify-mapping-and-onboard)

@vmrh21 vmrh21 closed this Apr 16, 2026
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.

1 participant