Skip to content

Conversation

@Meggielqk
Copy link
Collaborator

No description provided.

@Meggielqk Meggielqk added this to the 5.9.2 milestone Sep 9, 2025
@Meggielqk Meggielqk requested review from HJianBo and Copilot September 9, 2025 05:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR corrects the alarm list documentation by consolidating the separate open source and enterprise edition tables into a single unified table that includes all available alarms.

  • Removed duplicate alarm entries that were separated between open source and enterprise editions
  • Added missing alarm types (mnesia_transaction_manager_overload and broker_pool_overload) to the consolidated table
  • Unified the alarm documentation structure for better maintainability

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
zh_CN/observability/alarms.md Consolidated Chinese alarm table by removing enterprise-specific section and adding missing alarms
en_US/observability/alarms.md Unified English alarm table structure and added previously missing alarm types

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@HJianBo HJianBo merged commit 2a45d7a into release-5.9 Sep 23, 2025
7 checks passed
@HJianBo HJianBo deleted the r59-update-alarm-list branch September 23, 2025 04:08
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.

3 participants