[SREP-2141] Add basic MCP server for two read-only commands

[bergmannf]

This should allow AI tools to use osdctl to get some additional information.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bergmannf

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [bergmannf]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[RaphaelBut]

looks pretty good to me
if you want you could separate the refactors and the mcp into two PRs :)

[openshift-ci]

@bergmannf: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions 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.

[bergmannf]

looks pretty good to me if you want you could separate the refactors and the mcp into two PRs :)

I felt that the changes I made would be way uglier if I didn't refactor as well, as all our commands make everything private and make it unnecessarily hard to call them from elsewhere.

If you feel strongly about it I can split the PR though.

[joshbranham]

looks pretty good to me if you want you could separate the refactors and the mcp into two PRs :)

I felt that the changes I made would be way uglier if I didn't refactor as well, as all our commands make everything private and make it unnecessarily hard to call them from elsewhere.

If you feel strongly about it I can split the PR though.

I have always wanted to slowly push our cmd/ packages to be thin and move all the business logic into pkg/cluster/context (for example) type packages. Then the CLI is a small shim over calling actual exported code that could be consumed by other projects, vs them calling our CLI functions.

Do with that what you will, by no means saying you have to, but figured I'd mention it.

[bergmannf]

looks pretty good to me if you want you could separate the refactors and the mcp into two PRs :)

I felt that the changes I made would be way uglier if I didn't refactor as well, as all our commands make everything private and make it unnecessarily hard to call them from elsewhere.
If you feel strongly about it I can split the PR though.

I have always wanted to slowly push our cmd/ packages to be thin and move all the business logic into pkg/cluster/context (for example) type packages. Then the CLI is a small shim over calling actual exported code that could be consumed by other projects, vs them calling our CLI functions.

Do with that what you will, by no means saying you have to, but figured I'd mention it.

Absolutely - essentially I'd like the CLI to really be the thinnest layer of input - essentially right now we have all the logic in the view layer of MVC and the goal should be moving to the C layer (which we don't have).
This was really just a quick refactor to make it less ugly to expose it for now.

[bergmannf]

/hold
I am holding this for now as there are guidelines form the ProdSec team coming along we want to make sure we adhere with before merging and releasing this.

[openshift-merge-robot]

PR needs rebase.

Details

Instructions 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.