fix(logs): correct metricIncomingLogEvents metric name from IncomingLogs to IncomingLogEvents#37458
fix(logs): correct metricIncomingLogEvents metric name from IncomingLogs to IncomingLogEvents#37458syukawa-gh wants to merge 3 commits intoaws:mainfrom
Conversation
There was a problem hiding this comment.
The pull request linter fails with the following errors:
❌ Fixes must contain a change to an integration test file and the resulting snapshot.
If you believe this pull request should receive an exemption, please comment and provide a justification. A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed, add Clarification Request to a comment.
✅ A exemption request has been requested. Please wait for a maintainer's review.
…ogs to IncomingLogEvents The metricIncomingLogEvents() method was using 'IncomingLogs' as the metric name, but the correct CloudWatch metric name per AWS documentation is 'IncomingLogEvents'. This caused alarms created with this method to have no datapoints. Closes aws#36815
77a0c0f to
e351e81
Compare
|
Exemption Request: This change is a minimal fix that does not alter CloudFormation resource behavior in a way that requires a new integration test. The fix is covered by unit tests. |
|
Exemption Request: This fix corrects the metric name from |
|
|
||||||||||||||
|
|
||||||||||||||
Issue
Closes #36815
Reason for this change
The
metricIncomingLogEvents()method was using'IncomingLogs'as the metric name, but the correct CloudWatch metric name per AWS documentation is'IncomingLogEvents'. This caused alarms created with this method to have no datapoints.Description of changes
Changed the metric name from
'IncomingLogs'to'IncomingLogEvents'inLogGroup.metricIncomingLogEvents().Description of how you validated changes
Verified against the AWS CloudWatch Logs metrics documentation.
Checklist