Use rhobs/obs-mcp toolset for Prometheus/Alertmanager (replaces observability mcp)#226
Open
saswatamcode wants to merge 4 commits intoopenshift:mainfrom
Open
Use rhobs/obs-mcp toolset for Prometheus/Alertmanager (replaces observability mcp)#226saswatamcode wants to merge 4 commits intoopenshift:mainfrom
saswatamcode wants to merge 4 commits intoopenshift:mainfrom
Conversation
Member
saswatamcode
commented
Apr 14, 2026
saswatamcode
commented
|
Important Review skippedToo many files! This PR contains 295 files, which is 145 over the limit of 150. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (295)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: saswatamcode The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
matzew
reviewed
Apr 14, 2026
matzew
reviewed
Apr 14, 2026
Member
|
We have no evals / mcpchecker here - but I saw this based on discussions on slack CC @Cali0707 |
matzew
reviewed
Apr 14, 2026
Member
|
One of the things to have in mind when using the go modules, instead of directly prviding tools there is that it might be easier to miss tool renaming. E.g. the old version used to have: |
…vability mcp) Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
matzew
reviewed
Apr 16, 2026
| github.com/miekg/dns v1.1.68 | ||
| github.com/modelcontextprotocol/go-sdk v1.5.0 | ||
| github.com/prometheus/client_golang v1.23.2 | ||
| github.com/rhobs/obs-mcp v0.1.0-rc.1.0.20260414161157-6067303e23c9 |
Member
There was a problem hiding this comment.
Any idea when the actual release will be out ? 😅
Member
Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
Member
Author
|
Thanks! Also pinned the module to latest release rc |
|
@saswatamcode no evals for testing the obs-mcp toolset? |
Member
Author
|
@mvinkler evals are here rhobs/obs-mcp#34 |
|
@matzew is there any convention regarding toolset naming? This one is now called obs-mcp, while no other has -mcp suffix. |
Member
|
Ah. Good catch
the actual name of the tool should not have -mcp in its name.
The go module however could
Sent from Gmail Mobile
…On Thu 16. Apr 2026 at 11:48, Michal Vinkler ***@***.***> wrote:
*mvinkler* left a comment (openshift/openshift-mcp-server#226)
<#226 (comment)>
@matzew <https://github.com/matzew> is there any convention regarding
toolset naming? This one is now called obs-mcp, while no other has -mcp
suffix.
—
Reply to this email directly, view it on GitHub
<#226 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABGPTVPSKNV666D4U24UWD4WCT2FAVCNFSM6AAAAACXYNSKCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DENJZGAZDCNZRGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
saswatamcode
added a commit
to saswatamcode/obs-mcp
that referenced
this pull request
Apr 16, 2026
As per conventions in openshift-mcp-server, we need to rename the toolset to something more suitable as per openshift/openshift-mcp-server#226 (comment) Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
mvinkler
reviewed
Apr 16, 2026
| - `filter` (`string`) - Optional filter using Alertmanager filter syntax. Examples: 'alertname=Watchdog', 'severity=critical', 'namespace=openshift-monitoring' | ||
| - `inhibited` (`boolean`) - Include inhibited alerts in the results. Default: false | ||
| - `silenced` (`boolean`) - Include silenced alerts in the results. Default: false | ||
| <summary>obs-mcp</summary> |
There was a problem hiding this comment.
This should be also renamed back to observability?
|
Due to unified testing approach, I need the evals to be also in this repo. At least in the release-0.3 branch. |
Member
would be good if those evals were part of this repo. CC @Cali0707 |
slashpai
reviewed
Apr 16, 2026
| | helm | Tools for managing Helm charts and releases | | | ||
| | kcp | Manage kcp workspaces and multi-tenancy features | | | ||
| | kubevirt | KubeVirt virtual machine management tools, check the [KubeVirt documentation](https://github.com/containers/kubernetes-mcp-server/blob/main/docs/kubevirt.md) for more details. | | | ||
| | obs-mcp | Toolset for querying Prometheus and Alertmanager endpoints in efficient ways. | | |
Member
There was a problem hiding this comment.
Should we mention Thanos also?
|
From https://github.com/rhobs/obs-mcp/tree/main: obs-mcp has many new flags, for example: These flags will be merged to the openshift-mcp-server? What is their description? |
saswatamcode
added a commit
to saswatamcode/obs-mcp
that referenced
this pull request
Apr 16, 2026
As per conventions in openshift-mcp-server, we need to rename the toolset to something more suitable as per openshift/openshift-mcp-server#226 (comment) Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
Signed-off-by: Saswata Mukherjee <saswataminsta@yahoo.com>
|
@saswatamcode: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.