Commit 66b0294
authored
feat(enterprise): implement comprehensive stats and metrics commands (#240)
* feat(enterprise): implement comprehensive stats and metrics commands
Implements issue #156 - comprehensive statistics and metrics collection commands
for Redis Enterprise API.
## Commands Added
### Database Statistics
- `redisctl enterprise stats database <id>` - Get database statistics
- `redisctl enterprise stats database-shards <id>` - Get database shard statistics
- `redisctl enterprise stats database-metrics <id> --interval 1h` - Get database metrics over time
### Node Statistics
- `redisctl enterprise stats node <id>` - Get node statistics
- `redisctl enterprise stats node-metrics <id> --interval 1h` - Get node metrics over time
### Cluster Statistics
- `redisctl enterprise stats cluster` - Get cluster-wide statistics
- `redisctl enterprise stats cluster-metrics --interval 1h` - Get cluster metrics over time
### Listener & Export
- `redisctl enterprise stats listener` - Get listener statistics
- `redisctl enterprise stats export --format json --interval 1h` - Export statistics in various formats
## Implementation Notes
- Uses redis-enterprise StatsHandler for all operations
- Supports various time intervals (1m, 5m, 1h, 1d)
- Handles large datasets efficiently with proper JSON serialization
- Consistent error handling and output formatting
- Standard JMESPath filtering and output format support
## Testing
- Compiles cleanly with proper type safety
- Follows existing enterprise command patterns
- Uses established connection management and output utilities
This provides essential monitoring capabilities for Redis Enterprise deployments,
complementing the existing database and node management commands.
* chore: add #[allow(dead_code)] to suppress false positive warnings
These functions are used by the binary but appear unused from the library's perspective.
* fix(enterprise): correct LastStatsResponse structure to match actual API
The API returns stats fields directly with stime/etime, not wrapped in a
metrics object. This fixes the 'error decoding response body' issue when
running stats commands.
* fix(enterprise): fix stats test data format
The LastStatsResponse struct uses serde(flatten) to include metrics
at the top level, not in a nested 'metrics' field. Updated test data
to match the actual API response format.1 parent 4050023 commit 66b0294
6 files changed
+395
-28
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
| 96 | + | |
96 | 97 | | |
97 | 98 | | |
98 | | - | |
99 | | - | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
100 | 102 | | |
101 | | - | |
| 103 | + | |
102 | 104 | | |
103 | 105 | | |
104 | 106 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
110 | | - | |
111 | | - | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
112 | 112 | | |
113 | 113 | | |
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
117 | | - | |
118 | | - | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
126 | 126 | | |
127 | 127 | | |
128 | 128 | | |
129 | 129 | | |
130 | 130 | | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
139 | 139 | | |
140 | 140 | | |
141 | 141 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
996 | 996 | | |
997 | 997 | | |
998 | 998 | | |
| 999 | + | |
| 1000 | + | |
| 1001 | + | |
| 1002 | + | |
999 | 1003 | | |
1000 | 1004 | | |
1001 | 1005 | | |
| |||
2549 | 2553 | | |
2550 | 2554 | | |
2551 | 2555 | | |
| 2556 | + | |
| 2557 | + | |
| 2558 | + | |
| 2559 | + | |
| 2560 | + | |
| 2561 | + | |
| 2562 | + | |
| 2563 | + | |
| 2564 | + | |
| 2565 | + | |
| 2566 | + | |
| 2567 | + | |
| 2568 | + | |
| 2569 | + | |
| 2570 | + | |
| 2571 | + | |
| 2572 | + | |
| 2573 | + | |
| 2574 | + | |
| 2575 | + | |
| 2576 | + | |
| 2577 | + | |
| 2578 | + | |
| 2579 | + | |
| 2580 | + | |
| 2581 | + | |
| 2582 | + | |
| 2583 | + | |
| 2584 | + | |
| 2585 | + | |
| 2586 | + | |
| 2587 | + | |
| 2588 | + | |
| 2589 | + | |
| 2590 | + | |
| 2591 | + | |
| 2592 | + | |
| 2593 | + | |
| 2594 | + | |
| 2595 | + | |
| 2596 | + | |
| 2597 | + | |
| 2598 | + | |
| 2599 | + | |
| 2600 | + | |
| 2601 | + | |
| 2602 | + | |
| 2603 | + | |
| 2604 | + | |
| 2605 | + | |
| 2606 | + | |
| 2607 | + | |
| 2608 | + | |
| 2609 | + | |
| 2610 | + | |
| 2611 | + | |
| 2612 | + | |
| 2613 | + | |
| 2614 | + | |
| 2615 | + | |
| 2616 | + | |
| 2617 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| 17 | + | |
17 | 18 | | |
0 commit comments