Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhancement(aws s3 and azure blob sinks) Emit a counter for when aws s3 and azure blob storage requests are made #20400

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

flavioc
Copy link
Contributor

@flavioc flavioc commented Apr 30, 2024

We are exposing two new metrics to track requests into AWS S3 and Azure Blob Storage:

  • aws_s3_requests_sent_total
  • azure_blob_requests_sent_total

@flavioc flavioc requested a review from a team as a code owner April 30, 2024 03:32
@github-actions github-actions bot added the domain: sinks Anything related to the Vector's sinks label Apr 30, 2024
@flavioc flavioc changed the title (enhancement) (aws s3 and azure blob sinks) Emit a counter for when aws s3 and azure blob storage requests are made enhancement(aws s3 and azure blob sinks) Emit a counter for when aws s3 and azure blob storage requests are made Apr 30, 2024
Copy link
Member

@jszwedko jszwedko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @flavioc !

Thanks for opening this. We actually already have a metric for HTTP requests, https://vector.dev/docs/reference/configuration/sources/internal_metrics/#http_client_requests_sent_total. Though, maybe it isn't being correctly published by the blob storage sinks. Did you attempt to use that metric already? I know it won't have the same tags you have here so maybe would be insufficient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: sinks Anything related to the Vector's sinks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants