Metrics announcement#146
Merged
Merged
Conversation
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
reviewed
Jun 10, 2025
markdboyd
reviewed
Jun 10, 2025
| ## Available S3 Metrics | ||
|
|
||
| ### Storage and Capacity Metrics | ||
| - **Bucket Size in Bytes** |
Contributor
There was a problem hiding this comment.
Suggested change
| - **Bucket Size in Bytes** | |
| - **Bucket Size in Bytes**, metric name: `XXX` |
Can we add the metric name to each bullet, which will make it easier for people to search them?
markdboyd
reviewed
Jun 10, 2025
markdboyd
reviewed
Jun 10, 2025
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+23
to
+25
| - Overall domain CPU usage | ||
| - Master node CPU usage | ||
| - Helps identify processing load and potential performance bottlenecks |
Contributor
There was a problem hiding this comment.
Suggested change
| - Overall domain CPU usage | |
| - Master node CPU usage | |
| - Helps identify processing load and potential performance bottlenecks | |
| These metrics help identify processing load and potential performance bottlenecks. | |
| - Overall domain CPU usage | |
| - Master node CPU usage |
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+28
to
+31
| - JVM Memory Pressure | ||
| - Old Generation JVM Memory Pressure | ||
| - Master Node Memory Pressure | ||
| - Monitors memory consumption and potential memory-related issues |
Contributor
There was a problem hiding this comment.
Suggested change
| - JVM Memory Pressure | |
| - Old Generation JVM Memory Pressure | |
| - Master Node Memory Pressure | |
| - Monitors memory consumption and potential memory-related issues | |
| These metrics monitor memory consumption and other potential memory-related issues. | |
| - JVM Memory Pressure | |
| - Old Generation JVM Memory Pressure | |
| - Master Node Memory Pressure |
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+46
to
+48
| - Search Queue Count | ||
| - Search Queue Rejected Requests | ||
| - Monitors background processing and potential system overload |
Contributor
There was a problem hiding this comment.
Suggested change
| - Search Queue Count | |
| - Search Queue Rejected Requests | |
| - Monitors background processing and potential system overload | |
| These metrics monitor background processing and potential system overload. | |
| - Search Queue Count | |
| - Search Queue Rejected Requests |
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+96
to
+100
| ## Benefits | ||
| 1. Domain monitoring | ||
| 2. Early detection of resource constraints | ||
| 3. Access to S3 bucket sizes | ||
|
|
Contributor
There was a problem hiding this comment.
Suggested change
| ## Benefits | |
| 1. Domain monitoring | |
| 2. Early detection of resource constraints | |
| 3. Access to S3 bucket sizes |
markdboyd
reviewed
Jun 10, 2025
| ### Storage and Capacity Metrics | ||
| - **Bucket Size in Bytes**, metric name: `BucketSizeBytes` | ||
|
|
||
| ## Available Domain Metrics |
Contributor
There was a problem hiding this comment.
Suggested change
| ## Available Domain Metrics | |
| ## Available OpenSearch/Elasticsearch Metrics |
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+100
to
+107
| ## Benefits | ||
|
|
||
| 1. Domain monitoring | ||
|
|
||
| 2. Early detection of resource constraints | ||
|
|
||
| 3. Access to S3 bucket sizes | ||
|
|
Contributor
There was a problem hiding this comment.
Suggested change
| ## Benefits | |
| 1. Domain monitoring | |
| 2. Early detection of resource constraints | |
| 3. Access to S3 bucket sizes |
These benefits feel pretty well explained already by the rest of the post to me
markdboyd
reviewed
Jun 10, 2025
Comment on lines
+37
to
+40
| #### Free Storage Space, metric name: `FreeStorageSpace` | ||
| This metric helps track storage capacity and plan for scaling. | ||
|
|
||
| - Available storage in gigabytes |
Contributor
There was a problem hiding this comment.
Suggested change
| #### Free Storage Space, metric name: `FreeStorageSpace` | |
| This metric helps track storage capacity and plan for scaling. | |
| - Available storage in gigabytes | |
| This metric helps track storage capacity and plan for scaling. | |
| - Available storage in gigabytes, metric name: `FreeStorageSpace` |
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
reviewed
Jun 10, 2025
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov> Signed-off-by: Jason Gambino <jason.gambino3@gmail.com>
markdboyd
approved these changes
Jun 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes proposed in this pull request:
security considerations
None