Skip to content

MM-69311: Document new metric: mattermost_cluster_reliable_fallback_tcp#9068

Open
agarciamontoro wants to merge 2 commits into
masterfrom
MM-69311.new.metric
Open

MM-69311: Document new metric: mattermost_cluster_reliable_fallback_tcp#9068
agarciamontoro wants to merge 2 commits into
masterfrom
MM-69311.new.metric

Conversation

@agarciamontoro

Copy link
Copy Markdown
Member

Summary

Document new metric: mattermost_cluster_reliable_fallback_tcp

The new metric will land on v11.9, and was cherry-picked to v11.7 and v11.8 as well. I still don't know what the docs policy is with regards to the releases. Should I change the target branch to be v11.9-documentation?

Ticket Link

https://mattermost.atlassian.net/browse/MM-69311

@agarciamontoro agarciamontoro added 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor labels Jun 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA d04eadc

Comment thread source/administration-guide/scale/performance-monitoring-metrics.rst Outdated
…cs.rst

Co-authored-by: Amy Blais <29708087+amyblais@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 774807a5-2cfe-4932-a8db-fe622768771f

📥 Commits

Reviewing files that changed from the base of the PR and between 6e6ca68 and 769c214.

📒 Files selected for processing (1)
  • source/administration-guide/scale/performance-monitoring-metrics.rst

📝 Walkthrough

Walkthrough

A new documentation bullet was added for mattermost_cluster_reliable_fallback_tcp, describing it as a histogram for UDP SendBestEffort messages that fall back to TCP SendReliable when message size limits are exceeded, along with its bucket range and guidance text.

Changes

Cluster metrics documentation

Layer / File(s) Summary
Add fallback metric entry
source/administration-guide/scale/performance-monitoring-metrics.rst
Adds a new mattermost_cluster_reliable_fallback_tcp documentation bullet with its metric semantics, bucket range, and inline implementation reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely states the main change: documenting the new metric mattermost_cluster_reliable_fallback_tcp.
Description check ✅ Passed The description is directly related to documenting the new metric and adds relevant release-branch context.
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
  • Commit unit tests in branch MM-69311.new.metric

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

@github-actions

Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA 769c214

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

Labels

1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants