Skip to content

ROX-33223: Rename list clusters tool#122

Merged
mtodor merged 1 commit intomainfrom
mtodor/ROX-33223-rename-list-cluster
Apr 29, 2026
Merged

ROX-33223: Rename list clusters tool#122
mtodor merged 1 commit intomainfrom
mtodor/ROX-33223-rename-list-cluster

Conversation

@mtodor
Copy link
Copy Markdown
Collaborator

@mtodor mtodor commented Apr 28, 2026

Description

Rename list clusters tool to avoid tool name collision with other MCP servers. (i.e. ACM, or Openshift)

Validation

  • unit tests
  • e2e tests in CI pipeline

@mtodor mtodor requested a review from janisz as a code owner April 28, 2026 14:25
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 28, 2026

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
361 2 359 12
View the full list of 2 ❄️ flaky test(s)
::policy 1

Flake rate in main: 100.00% (Passed 0 times, Failed 28 times)

Stack Traces | 0s run time
- test violation 1
- test violation 2
- test violation 3
::policy 4

Flake rate in main: 100.00% (Passed 0 times, Failed 28 times)

Stack Traces | 0s run time
- testing multiple alert violation messages 1
- testing multiple alert violation messages 2
- testing multiple alert violation messages 3

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@github-actions
Copy link
Copy Markdown

E2E Test Results

Commit: bfbc1a4
Workflow Run: View Details
Artifacts: Download test results & logs

=== Evaluation Summary ===

  ✓ cve-cluster-does-exist (assertions: 3/3)
  ✓ cve-cluster-does-not-exist (assertions: 3/3)
  ✓ cve-detected-clusters (assertions: 3/3)
  ✓ cve-log4shell (assertions: 3/3)
  ✓ cve-clusters-general (assertions: 3/3)
  ✗ cve-nonexistent (assertions: 3/3)
      one or more verification steps failed
  ✓ cve-cluster-list (assertions: 3/3)
  ✓ cve-detected-workloads (assertions: 3/3)
  ✓ cve-multiple (assertions: 3/3)
  ~ rhsa-not-supported (assertions: 1/2)
      - MaxToolCalls: Too many tool calls: expected <= 4, got 8
  ✓ list-clusters (assertions: 3/3)

Tasks:      10/11 passed (90.91%)
Assertions: 31/32 passed (96.88%)
Tokens:     ~59425 (estimate - excludes system prompt & cache)
MCP schemas: ~12562 (included in token total)
Agent used tokens:
  Input:  17478 tokens
  Output: 22491 tokens
Judge used tokens:
  Input:  39975 tokens
  Output: 38767 tokens

@mtodor mtodor merged commit ea5c0e1 into main Apr 29, 2026
10 checks passed
@mtodor mtodor deleted the mtodor/ROX-33223-rename-list-cluster branch April 29, 2026 08:57
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.

3 participants