-
Notifications
You must be signed in to change notification settings - Fork 6
PERFSCALE-4192: Jira mcp integration #55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
58b9813 to
85ec230
Compare
vishnuchalla
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This implementation is introducing tight integration with custom JIRA MCP server which is not the intended purpose.
Please consider BugZooka to be a tool that provides “Bring your own MCP” flexibility with minimal code changes and update these changes.
|
Hi @vishnuchalla thank you for the review and sorry it took a while to circle back to this! |
|
Hey @abraham2512 thanks for the changes. Can you rebase please, so that I can take a look ASAP? |
57a1d1e to
7db9c39
Compare
|
@vishnuchalla rebased, ptal when you can 😄 |
| @@ -0,0 +1,42 @@ | |||
| apiVersion: apps/v1 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This deployment should be in its own namespace, idea being it can be commonly reused by others also to avoid tight coupling with BugZooka.
| - configmap-prompts.yaml | ||
| - configmap-mcp-config.yaml | ||
| - deployment.yaml | ||
| - deployment-jira-mcp.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shoudn't be part of this kustomize. Please see how we recently deployed Orion MCP for understanding.
| export QUAY_CRED='<base64 encoded pull secret>' | ||
| export BUGZOOKA_IMAGE='<bugzooka image tag>' | ||
| export BUGZOOKA_NAMESPACE='<your namespace>' | ||
| export JIRA_MCP_IMAGE='<jira mcp server image>' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as other comments about JIRA MCP isolation.
| INFERENCE_MAX_TOOL_ITERATIONS, | ||
| ) | ||
| from bugzooka.integrations.inference import InferenceAPIUnavailableError | ||
| from bugzooka.analysis.prompts import JIRA_TOOL_PROMPT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of the changes here are related to formatting, can we avoid them in this PR please and only keep the real code changes? Those optimizations can be taken care in other follow ups if requried.
| "user": "Here is the log summary from an OpenShift environment:\n\n{summary}\n\nBased on this summary, provide a structured breakdown of:\n- The OpenShift component likely affected (e.g., etcd, kube-apiserver, ingress, SDN, Machine API)\n- The probable root cause\n- Steps to verify the issue further\n- Suggested resolution, including OpenShift-specific commands or configurations.", | ||
| "assistant": "**Affected Component:** <Identified component>\n\n**Probable Root Cause:** <Describe why this issue might be occurring>\n\n**Verification Steps:**\n- <Step 1>\n- <Step 2>\n- <Step 3>\n\n**Suggested Resolution:**\n- <OpenShift CLI commands>\n- <Relevant OpenShift configurations>" | ||
| }, | ||
| "OPENSHIFT_PROMPT": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange, do you have an inbuilt linter in our IDE? Almost all the changes are being formatted.
This PR integrates jira mcp client and prompt updates to fetch relevant jira issues.
Sample output