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

Kinetica exporter/pr 2 metrics #27239

Merged

Conversation

am-kinetica
Copy link
Contributor

Description: Added metrics handling

Link to tracking Issue: #24294

Testing: No new tests

Documentation: No new documentation

@am-kinetica am-kinetica reopened this Sep 27, 2023
@am-kinetica
Copy link
Contributor Author

Merged with changes from main

@TylerHelmuth TylerHelmuth added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 3, 2023
@TylerHelmuth TylerHelmuth removed the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 3, 2023
@atoulme
Copy link
Contributor

atoulme commented Nov 29, 2023

I think this PR looks good now. @TylerHelmuth mind taking another look?

Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

@atoulme we're waiting on some unit tests for metrics_exporter.go.

Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

@am-kinetica wanted to chime in an say I'm still keeping an eye on this PR, excited to see the progress made on the unit tests.

exporter/kineticaexporter/exporter_metric_test.go Outdated Show resolved Hide resolved
@TylerHelmuth TylerHelmuth merged commit e7bf6ab into open-telemetry:main Jan 9, 2024
85 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 9, 2024
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
**Description:** Added metrics handling

**Link to tracking Issue:**
open-telemetry#24294

**Testing:** No new tests

**Documentation:** No new documentation
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.

4 participants