Skip to content

MGMT-21873: Update MCP to get log analyzer tool#229

Merged
openshift-merge-bot[bot] merged 1 commit intorh-ecosystem-edge:mainfrom
keitwb:log-analyzer-mcp
Oct 14, 2025
Merged

MGMT-21873: Update MCP to get log analyzer tool#229
openshift-merge-bot[bot] merged 1 commit intorh-ecosystem-edge:mainfrom
keitwb:log-analyzer-mcp

Conversation

@keitwb
Copy link
Copy Markdown
Contributor

@keitwb keitwb commented Oct 14, 2025

  • Enable it by default in local dev env
  • Also adding an ignore for a summary archive that should have been already.

Summary by CodeRabbit

  • New Features

    • Enabled built-in troubleshooting tools in the service.
  • Chores

    • Updated internal component reference with no functional changes.
    • Excluded archive artifacts from version control.

@openshift-ci openshift-ci bot requested a review from eranco74 October 14, 2025 14:22
@keitwb keitwb changed the title Update MCP to get log analyzer tool MGMT-21873: Update MCP to get log analyzer tool Oct 14, 2025
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 14, 2025

@keitwb: This pull request references MGMT-21873 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.21.0" version, but no target version was set.

Details

In response to this:

  • Enable it by default in local dev env
  • Also adding an ignore for a summary archive that should have been already.

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 openshift-eng/jira-lifecycle-plugin repository.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 14, 2025

Walkthrough

Added ENABLE_TROUBLESHOOTING_TOOLS=1 to the assisted-service-mcp container in assisted-chat-pod.yaml, updated the assisted-service-mcp submodule commit reference, and expanded scripts/archives/.gitignore to ignore summaries.tar.gz.

Changes

Cohort / File(s) Summary of changes
Deployment config
assisted-chat-pod.yaml
Added environment variable ENABLE_TROUBLESHOOTING_TOOLS="1" to assisted-service-mcp container.
Submodule update
assisted-service-mcp
Updated submodule pointer from 986abb7a0... to f25ad9429....
Git ignore updates
scripts/archives/.gitignore
Added summaries.tar.gz to ignored files (alongside from-s3/, extracted/, summaries/).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

approved, lgtm

Suggested reviewers

  • eranco74
  • carbonin

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title concisely identifies the primary change—updating the MCP to include the log analyzer tool—while referencing the relevant Jira ticket. It accurately reflects the main purpose of the changeset without extraneous detail.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6667b84 and 1ca0a6f.

📒 Files selected for processing (3)
  • assisted-chat-pod.yaml (1 hunks)
  • assisted-service-mcp (1 hunks)
  • scripts/archives/.gitignore (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Red Hat Konflux / assisted-chat-saas-main-on-pull-request
  • GitHub Check: Red Hat Konflux / assisted-chat-test-image-saas-main-on-pull-request
🔇 Additional comments (3)
assisted-chat-pod.yaml (1)

54-55: Looks good—thanks for enabling this in dev.
No further action required.

scripts/archives/.gitignore (1)

4-4: Ignore entry makes sense.
Helps keep generated tarball out of git.

assisted-service-mcp (1)

1-1: Submodule pointer update looks good.

No concerns with the refreshed commit reference.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

- Enable it by default in local dev env
- Also adding an ignore for a summary archive that should have been
  already.
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 14, 2025

@keitwb: This pull request references MGMT-21873 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.21.0" version, but no target version was set.

Details

In response to this:

  • Enable it by default in local dev env
  • Also adding an ignore for a summary archive that should have been already.

Summary by CodeRabbit

  • New Features

  • Enabled built-in troubleshooting tools in the service.

  • Chores

  • Updated internal component reference with no functional changes.

  • Excluded archive artifacts from version control.

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Oct 14, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: carbonin, keitwb

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:

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

@openshift-merge-bot openshift-merge-bot bot merged commit d786204 into rh-ecosystem-edge:main Oct 14, 2025
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants