EPMRPP-113668 || Update Kubernetes chart in RP Docs+video link for MCP#1092
EPMRPP-113668 || Update Kubernetes chart in RP Docs+video link for MCP#1092maria-hambardzumian merged 1 commit intodevelopfrom
Conversation
pressayuliya
requested review from
AmsterGet and
maria-hambardzumian
as code owners
WalkthroughThis pull request includes two minor documentation updates: adding a YouTube video link to the MCP Server example queries section and updating the Helm repository URL in Kubernetes deployment instructions. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip You can generate walkthrough in a markdown collapsible section to save space.Enable the |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docs/external-integrations/MCPServer.mdx (1)
360-361: Consider removing the tracking parameter for a cleaner documentation URL.The YouTube video link is accessible and working. The
?si=tBQ-t5sZWzW5S8Krparameter is a YouTube sharing identifier that can be omitted for a simpler, cleaner URL in documentation.Optional: Cleaner URL without tracking parameter
-You can see practical usage examples in [this video](https://youtu.be/1WKh8WLiA3Q?si=tBQ-t5sZWzW5S8Kr). +You can see practical usage examples in [this video](https://youtu.be/1WKh8WLiA3Q).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/external-integrations/MCPServer.mdx` around lines 360 - 361, Update the YouTube link in the sentence that currently ends with "https://youtu.be/1WKh8WLiA3Q?si=tBQ-t5sZWzW5S8Kr" by removing the query parameter so it becomes "https://youtu.be/1WKh8WLiA3Q"; edit the markdown in docs/external-integrations/MCPServer.mdx where that URL is referenced to use the cleaner URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/installation-steps/DeployWithKubernetes.md`:
- Around line 22-23: The Minikube guide still references the old Helm repo URL;
open DeployWithMinikube.md, find the helm repo commands that use "helm repo add
reportportal https://reportportal.io/kubernetes" (and the subsequent "helm repo
update reportportal") and replace the old URL with the new one
"https://k8s.reportportal.io" so the Minikube instructions match
DeployWithKubernetes.md and there are no conflicting repository URLs.
---
Nitpick comments:
In `@docs/external-integrations/MCPServer.mdx`:
- Around line 360-361: Update the YouTube link in the sentence that currently
ends with "https://youtu.be/1WKh8WLiA3Q?si=tBQ-t5sZWzW5S8Kr" by removing the
query parameter so it becomes "https://youtu.be/1WKh8WLiA3Q"; edit the markdown
in docs/external-integrations/MCPServer.mdx where that URL is referenced to use
the cleaner URL.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: dff55ac9-c31b-4d3d-bbfb-899eb36a2295
📒 Files selected for processing (2)
docs/external-integrations/MCPServer.mdxdocs/installation-steps/DeployWithKubernetes.md
| helm repo add reportportal https://k8s.reportportal.io | ||
| helm repo update reportportal |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Find all references to the old Helm repository URL in documentation
# Search for the old URL pattern
rg -n "reportportal\.io/kubernetes" --type md
# Search for any helm repo add commands to see all Helm repository configurations
rg -n "helm repo add reportportal" --type md -A 1Repository: reportportal/docs
Length of output: 593
🏁 Script executed:
#!/bin/bash
# Description: Verify the new Helm repository URL is accessible
# Test if the URL is accessible
curl -I https://k8s.reportportal.io
# Try to fetch the index.yaml which should exist in a Helm repo
curl -s https://k8s.reportportal.io/index.yaml | head -20Repository: reportportal/docs
Length of output: 1568
Update the Minikube deployment guide to use the new Helm repository URL.
The new Helm repository URL (https://k8s.reportportal.io) is valid and serves the chart repository correctly. However, the Minikube deployment guide (docs/installation-steps/DeployWithMinikube.md line 45) still references the old URL (https://reportportal.io/kubernetes). This creates conflicting instructions for users depending on which installation path they follow. Update DeployWithMinikube.md to use the new URL for consistency.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/installation-steps/DeployWithKubernetes.md` around lines 22 - 23, The
Minikube guide still references the old Helm repo URL; open
DeployWithMinikube.md, find the helm repo commands that use "helm repo add
reportportal https://reportportal.io/kubernetes" (and the subsequent "helm repo
update reportportal") and replace the old URL with the new one
"https://k8s.reportportal.io" so the Minikube instructions match
DeployWithKubernetes.md and there are no conflicting repository URLs.
2 participants
Summary by CodeRabbit