Skip to content

Conversation

@direnakkoc
Copy link
Collaborator

Description

Add support for S3 alarms and dashboard for the metrics are

  • FirstByteLatency

  • HeadRequests

  • 5xxErrors

  • 4xxErrors

  • TotalRequestLatency

  • AllRequests

Motivation and Context

  • FirstByteLatency : The time in seconds until the first byte of a response is returned. Useful to detect latency issues.

  • HeadRequests : The number of HEAD requests made to the bucket. Can be used to detect unusual access patterns.

  • 5xxErrors : The number of server-side errors (5XX). Indicates internal issues in S3.

  • 4xxErrors : The number of client-side errors (4XX). Can indicate permission or request format issues.

  • TotalRequestLatency : The total time in seconds that S3 takes to process requests. Helps monitor overall performance.

  • AllRequests : The total number of requests made to the bucket. Useful to track traffic trends.

How Has This Been Tested?

  • Unit tests added for all key functions and metrics
  • Deployed the Serverless test project and ran stress tests to verify metrics and dashboards

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@coveralls
Copy link

Coverage Status

coverage: 97.662% (+0.05%) from 97.615%
when pulling 067bfc3 on feat/s3-metrics
into d532930 on main.

@direnakkoc direnakkoc requested a review from eoinsha January 16, 2026 09:07
@direnakkoc direnakkoc merged commit ddf4da4 into main Jan 20, 2026
5 checks passed
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