fix(rh-developer): correct stale parameter names for prometheus_queryin incident-triage

[ikrispin]

Summary

Fix according to readme: https://github.com/openshift/openshift-mcp-server

Pack(s) affected

• rh-sre
• rh-developer
• ocp-admin
• rh-virt
• rh-ai-engineer
• Other / repo-wide

Change type

• New skill
• New agent
• New pack
• Update existing skill / agent
• MCP server config (mcps.json)
• Docs / README
• CI / tooling

CLAUDE.md compliance

• Agents orchestrate skills; no direct MCP/tool calls in agents
• Skills are single-purpose task executors
• Skills encapsulate all tool access (MCP tools invoked only inside skills)
• Document consultation: file is read with the Read tool, then declared to the user
• No credentials hardcoded; env vars used via ${VAR} references
• Human-in-the-loop confirmation added for any destructive or critical operations

Validation

• make validate passes locally
• New/changed skills have valid YAML frontmatter (name, description)
• New/changed agents have valid YAML frontmatter (name, description)

[dmartinol]

@jordigilh could you please tal?

[jordigilh]

Hey @ikrispin, good catch on the stale parameter names — match and metric were leftovers from the old get_metric_names/get_metric_metadata tools that no longer exist.

While you're at it, there's one more artifact from the same rename in the Verification Steps section (lines 56-57). There are two nearly identical lines:

1. Check `openshift` server is configured in `mcps.json`
2. Check `openshift` server is configured in `mcps.json` with `observability` in its `--toolsets`

Line 1 is redundant — line 2 already covers it (and adds the observability toolset check). Could you remove line 1 and renumber the remaining steps? That way both leftovers are cleaned up in a single PR.

[dmartinol]

Thank you! /lgtm