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

Mongodbatlas add metrics #5921

Merged
merged 13 commits into from
Nov 16, 2021

Conversation

zenmoto
Copy link
Contributor

@zenmoto zenmoto commented Oct 26, 2021

Description:
Adding metrics processing to MongoDB Atlas receiver

@zenmoto zenmoto requested review from a team and anuraaga October 26, 2021 01:05
@zenmoto zenmoto force-pushed the mongodbatlas_add_metrics branch from 2ad27b7 to 1972614 Compare November 2, 2021 16:36
receiver/mongodbatlasreceiver/metadata.yaml Outdated Show resolved Hide resolved
extended_documentation: MongoDB Metric BACKGROUND_FLUSH_AVG
unit: 1
data:
type: gauge
Copy link
Member

Choose a reason for hiding this comment

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

Sum? Or this is per_second?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately I've found very little about this particular metric save the name. The documentation from MongoDB is what's here, I've seen reference in other documentation to it being in once place a counter, and in another the time the flush took in milliseconds. I've been unable to see this one live with ad-hoc testing to see what it reports, but I've seen enough questions of people using it for performance diagnoses that I don't want to ignore it.

receiver/mongodbatlasreceiver/metadata.yaml Outdated Show resolved Hide resolved
receiver/mongodbatlasreceiver/metadata.yaml Show resolved Hide resolved
receiver/mongodbatlasreceiver/metadata.yaml Outdated Show resolved Hide resolved
Copy link
Member

@bogdandrutu bogdandrutu left a comment

Choose a reason for hiding this comment

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

Please rebase, "labels" were renamed to "attributes".

@zenmoto zenmoto force-pushed the mongodbatlas_add_metrics branch from 3c4e678 to 8a5aaa6 Compare November 12, 2021 22:36
@zenmoto zenmoto force-pushed the mongodbatlas_add_metrics branch from ea6dc95 to 4ce9637 Compare November 13, 2021 00:05
@zenmoto
Copy link
Contributor Author

zenmoto commented Nov 13, 2021

@bogdandrutu done.

@bogdandrutu bogdandrutu merged commit 6467131 into open-telemetry:main Nov 16, 2021
@zenmoto zenmoto deleted the mongodbatlas_add_metrics branch November 16, 2021 22:15
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.

4 participants