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

[receiver/googlecloudpubsub] Turn noisy warn in reset metric (#37571) #37583

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

Conversation

alexvanboxel
Copy link
Contributor

Description

The receiver uses the Google Cloud Pub/Sub StreamingPull API and keeps a open connection. The Pub/Sub servers
recurrently close the connection after a time period to avoid a long-running sticky connection. Before the
receiver logged warn log lines everytime this happened. These log lines are moved to debug so that fleets with
lots of collectors with the receiver don't span logs at warn level. To keep track of the resets, whenever a
connection reset happens a otelcol_receiver_googlecloudpubsub_stream_restarts metric is increased by one.

Link to tracking issue

Fixes 37571

Documentation

Documentation is auto generated for the metric

@alexvanboxel alexvanboxel force-pushed the pubsubreceiver-resetmetric branch 8 times, most recently from 8228531 to 5466619 Compare February 1, 2025 09:15
@alexvanboxel alexvanboxel force-pushed the pubsubreceiver-resetmetric branch from 5466619 to 51ac85d Compare February 1, 2025 10:01
@alexvanboxel alexvanboxel marked this pull request as ready for review February 1, 2025 10:40
@alexvanboxel alexvanboxel requested a review from a team as a code owner February 1, 2025 10:40
Comment on lines +98 to +99
attribute.String("otelcol.component.kind", "receiver"),
attribute.String("otelcol.component.id", handler.settings.ID.String()),
Copy link
Member

Choose a reason for hiding this comment

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

@djaglowski do the attributes here look good?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants