Skip to content

docs: fix Alertmanager port in amtool config routes example#5312

Open
s3onghyun wants to merge 1 commit into
prometheus:mainfrom
s3onghyun:docs-amtool-port
Open

docs: fix Alertmanager port in amtool config routes example#5312
s3onghyun wants to merge 1 commit into
prometheus:mainfrom
s3onghyun:docs-amtool-port

Conversation

@s3onghyun

Copy link
Copy Markdown

The "View routing tree of remote Alertmanager" example uses --alertmanager.url=http://localhost:9090, but 9090 is Prometheus's port — Alertmanager's default is 9093 (cmd/alertmanager/main.go, cli/root.go). Every other URL in the README uses 9093, so this line was the lone outlier.

Signed-off-by: s3onghyun <s3onghyun.hong@gmail.com>
@s3onghyun s3onghyun requested a review from a team as a code owner June 18, 2026 07:02
@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 34320217-2e63-4201-a98d-ca43c73a15db

📥 Commits

Reviewing files that changed from the base of the PR and between f29d44c and d1cfb38.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The amtool config routes example command in README.md had its Alertmanager URL port corrected from 9090 to 9093, which is the default Alertmanager port.

Changes

README Port Fix

Layer / File(s) Summary
Correct amtool example port
README.md
The --alertmanager.url flag in the amtool config routes example is updated from http://localhost:9090 to http://localhost:9093.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description clearly explains the issue, the fix, and provides context. However, it does not follow the repository's template structure and missing checklist items and release-notes section. Follow the repository's description template by including the Pull Request Checklist and release-notes sections to indicate which category of change this is.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing an incorrect Alertmanager port in a documentation example for amtool.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant