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

feat: have otel collect our docker stats #1516

Closed

Conversation

chesedo
Copy link
Contributor

@chesedo chesedo commented Jan 10, 2024

Description of change

Cause otel to push docker metrics to a S3 bucket

How has this been tested? (if applicable)

Will on staging

@chesedo chesedo changed the title Feature/engn 44 have otel collect our docker stats feat: have otel collect our docker stats Jan 10, 2024
@@ -68,17 +74,28 @@ exporters:
endpoint: "api.honeycomb.io:443"
headers:
"x-honeycomb-team": ${env:HONEYCOMB_API_KEY}
awss3/metrics:
Copy link
Member

Choose a reason for hiding this comment

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

How is the access to this bucket handled ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On AWS it will just use the instance metadata endpoint. The otel-s3-doc says it uses the default from the go-sdk which fallbacks to the "EC2 Instance Role Credentials" 1

Copy link
Member

Choose a reason for hiding this comment

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

If it fallbacks to the EC2 role, then does it mean that the user application containers will inherit these permissions as well without having to do anything ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh dang, yes!! I recommend that we then rather find a way to install the collector directly on the VM 😄

collection_interval: 1m
container_labels_to_metric_labels:
shuttle.idle_minutes: shuttle.idle_minutes
shuttle.project: shuttle.project_name
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be shuttle.project.name? And perhaps the same convention for the project id and idle minutes, but we haven't implemented that elsewhere in our code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed, I'll make this change

@oddgrd
Copy link
Contributor

oddgrd commented Jan 15, 2024

By the way, do we know that this will not cause issues similar to what we saw in the past when we enabled Datadog docker stats collection? IIRC it was previously collecting every 10s, and this is once every minute. Are there other differences here?

@chesedo
Copy link
Contributor Author

chesedo commented Jan 16, 2024

By the way, do we know that this will not cause issues similar to what we saw in the past when we enabled Datadog docker stats collection? IIRC it was previously collecting every 10s, and this is once every minute. Are there other differences here?

We don't know 😅 There are also no other differences.

However, I just realized that the old collector of 10s might be active. Hopefully it is not active and the cause of our Docker issues 😅

@chesedo
Copy link
Contributor Author

chesedo commented Jan 16, 2024

Btw, this is a bit on ice while I'm working on the RDS subscriptions

@chesedo chesedo closed this May 24, 2024
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.

None yet

3 participants