Skip to content

Conversation

@joshrotenberg
Copy link
Collaborator

Description

Implements comprehensive alert management commands for Redis Enterprise API, addressing issue #300.

Features Added

Alert Commands

  • List alerts - List all alerts with filtering by type and severity
  • Get specific alert - Retrieve details for a specific alert by UID
  • Cluster alerts - View cluster-level alerts
  • Node alerts - View node-specific alerts
  • Database alerts - View database-specific alerts
  • Alert settings - Get and update alert configuration settings

Key Capabilities

  • Filter alerts by type (cluster, node, bdb)
  • Filter by severity (info, warning, error, critical)
  • Full JMESPath query support for output filtering
  • Support for all output formats (JSON, YAML, Table)
  • Comprehensive alert settings management

Implementation Details

  • Uses the three alert endpoints: /v1/cluster/alerts, /v1/nodes/alerts, /v1/bdbs/alerts
  • Combines alerts from all sources with type annotation
  • Alert settings managed through cluster configuration
  • Follows established patterns for command structure and error handling

Testing

All commands tested against local Docker instance:

  • alerts list - Returns empty array when no alerts present ✅
  • alerts settings-get - Returns full alert configuration ✅
  • All filtering and query functionality verified ✅

Documentation

Comprehensive mdBook documentation added covering:

  • All command usage examples
  • Alert types and descriptions
  • Common monitoring scenarios
  • Script examples for automation

Closes #300

- List and filter alerts by type and severity
- Get specific alerts by UID
- View cluster, node, and database alerts
- Manage alert settings
- Full JMESPath query support
- Comprehensive mdBook documentation
@joshrotenberg joshrotenberg merged commit 3569109 into main Sep 15, 2025
22 checks passed
@joshrotenberg joshrotenberg deleted the feat/enterprise-alerts-commands branch September 15, 2025 14:54
joshrotenberg added a commit that referenced this pull request Sep 15, 2025
## What's Changed

### New Features
- feat(enterprise): add comprehensive alerts management commands (#307)
- feat(enterprise): add license management commands (#308)
- feat(enterprise): implement bootstrap commands for cluster initialization (#309)
- feat(enterprise): add debug info collection commands (#309)
- feat(enterprise): implement LDAP integration commands (#309)
- feat(enterprise): add OCSP certificate validation commands (#309)
- feat(enterprise): implement service management commands (#309)

### Improvements
- chore: simplify crates.io publishing workflow and remove manual release scripts (#310)
- docs: add comprehensive documentation for all new enterprise commands (#310)
- chore: update CLAUDE.md with documentation requirements for new features (#310)

### Coverage
- Enterprise API coverage increased to 74% (29 handlers implemented)
- All new commands include comprehensive integration tests with wiremock

### Breaking Changes
None

### Full Changelog
v0.4.1...v0.5.0
joshrotenberg added a commit that referenced this pull request Sep 15, 2025
## What's Changed

### New Features
- feat(enterprise): add comprehensive alerts management commands (#307)
- feat(enterprise): add license management commands (#308)
- feat(enterprise): implement bootstrap commands for cluster initialization (#309)
- feat(enterprise): add debug info collection commands (#309)
- feat(enterprise): implement LDAP integration commands (#309)
- feat(enterprise): add OCSP certificate validation commands (#309)
- feat(enterprise): implement service management commands (#309)

### Improvements
- chore: simplify crates.io publishing workflow and remove manual release scripts (#310)
- docs: add comprehensive documentation for all new enterprise commands (#310)
- chore: update CLAUDE.md with documentation requirements for new features (#310)

### Coverage
- Enterprise API coverage increased to 74% (29 handlers implemented)
- All new commands include comprehensive integration tests with wiremock

### Breaking Changes
None

### Full Changelog
v0.4.1...v0.5.0
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.

feat(enterprise): implement alerts management commands

2 participants