-
Notifications
You must be signed in to change notification settings - Fork 807
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
Update OTLP Metrics Exporter with latest Metrics SDK #2774
Comments
What kind of discussion this and #2773 requires? |
This will depend on #2775 |
Any update on this as #2775 seems to be closed now. Seems this no longer works with any OTEL Collector past |
We are approaching a useable release of the metrics rewrite and this is one of the last things that will be required before we can release an alpha so I expect it to be on deck soon |
I'd like to 'pick this one up if nobody is working on this yet. 🙂 |
After #2871 has merged, the exporters still need to be updated to use the I will do that in a follow-up PR to unblock everyone who is currently waiting on #2871. |
Since #2871 technically updates the OTLP Metrics Exporters to use the latest SDK. Should I create follow up issues to track changes that are still needed in the exporters? 🤔 I think there a a few steps left to get all exporters into a more manageable state:
|
@pichlermarc you can track all of the items that need to be done by listing them in the OP. Or if you prefer tracking them in separate issues, I'm fine with either way. |
Thanks! I created a single issue with all of these items on it (#2886) 🙂 |
This issue is done and can be closed 🙂 |
Well done @pichlermarc 🎉 |
After the major refactoring of Metrics SDK, we should update the OTLP Metrics Exporter to adopt the changes.
The text was updated successfully, but these errors were encountered: