-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
exporter/azureblobexporter First PR #11871
exporter/azureblobexporter First PR #11871
Conversation
…y-collector-contrib into eedorenko/azureblobexporter-1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmm-sumo please review
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
pmm-sumo is not longer involved in the project. @Aneurysm9 as the next name on the sponsors list, please review this component. |
Removing pmm-sumo, moving @Aneurysm9 and @mx-psi to the bottom of the rotation after sponsoring: open-telemetry#11872 open-telemetry#11871
Removing pmm-sumo, moving @Aneurysm9 and @mx-psi to the bottom of the rotation after sponsoring: #11872 #11871
…y-collector-contrib into eedorenko/azureblobexporter-1
ping @Aneurysm9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will also need to add your component to github issues dropdowns. See #16603 for reference.
Co-authored-by: Antoine Toulme <[email protected]>
Co-authored-by: Antoine Toulme <[email protected]>
Foresight Summary
View More Details⭕ build-and-test-windows workflow has finished in 4 seconds (46 minutes 36 seconds less than
|
Job | Failed Steps | Tests | |
---|---|---|---|
windows-unittest-matrix | - 🔗 | N/A | See Details |
windows-unittest | - 🔗 | N/A | See Details |
✅ tracegen workflow has finished in 1 minute 6 seconds (2 minutes 3 seconds less than main
branch avg.) and finished at 7th Jan, 2023.
Job | Failed Steps | Tests | |
---|---|---|---|
build-dev | - 🔗 | N/A | See Details |
publish-stable | - 🔗 | N/A | See Details |
publish-latest | - 🔗 | N/A | See Details |
✅ check-links workflow has finished in 1 minute 19 seconds (1 minute 28 seconds less than main
branch avg.) and finished at 7th Jan, 2023.
Job | Failed Steps | Tests | |
---|---|---|---|
changed files | - 🔗 | N/A | See Details |
check-links | - 🔗 | N/A | See Details |
✅ changelog workflow has finished in 2 minutes 3 seconds (6 minutes 35 seconds less than main
branch avg.) and finished at 7th Jan, 2023.
Job | Failed Steps | Tests | |
---|---|---|---|
changelog | - 🔗 | N/A | See Details |
✅ prometheus-compliance-tests workflow has finished in 12 minutes 28 seconds (⚠️ 3 minutes 22 seconds more than main
branch avg.) and finished at 7th Jan, 2023.
Job | Failed Steps | Tests | |
---|---|---|---|
prometheus-compliance-tests | - 🔗 | ✅ 21 ❌ 0 ⏭ 0 🔗 | See Details |
✅ load-tests workflow has finished in 21 minutes 5 seconds (⚠️ 4 minutes 45 seconds more than main
branch avg.) and finished at 7th Jan, 2023.
Job | Failed Steps | Tests | |
---|---|---|---|
loadtest (TestTraceAttributesProcessor) | - 🔗 | ✅ 3 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestIdleMode) | - 🔗 | ✅ 1 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestMetric10kDPS|TestMetricsFromFile) | - 🔗 | ✅ 6 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestTraceNoBackend10kSPS|TestTrace1kSPSWithAttrs) | - 🔗 | ✅ 8 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestMetricResourceProcessor|TestTrace10kSPS) | - 🔗 | ✅ 12 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestTraceBallast1kSPSWithAttrs|TestTraceBallast1kSPSAddAttrs) | - 🔗 | ✅ 10 ❌ 0 ⏭ 0 🔗 | See Details |
loadtest (TestBallastMemory|TestLog10kDPS) | - 🔗 | ✅ 19 ❌ 0 ⏭ 0 🔗 | See Details |
setup-environment | - 🔗 | N/A | See Details |
❌ build-and-test workflow has finished in 8 minutes 54 seconds (43 minutes 55 seconds less than main
branch avg.) and finished at 1st Feb, 2023. 2 jobs failed.
Job | Failed Steps | Tests | |
---|---|---|---|
check-collector-module-version | Check Collector Module Version 🔗 | N/A | See Details |
checks | Components dropdown in issue templates 🔗 | N/A | See Details |
*You can configure Foresight comments in your organization settings page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, one nit.
Co-authored-by: Antoine Toulme <[email protected]>
…y-collector-contrib into eedorenko/azureblobexporter-1
@Aneurysm9, could you please review this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies, looks like I had reviewed most of this and then <squirrel!>...
This looks fine modulo the collector module versions that need to be updated.
go.opentelemetry.io/collector v0.68.0 | ||
go.opentelemetry.io/collector/component v0.68.0 | ||
go.opentelemetry.io/collector/pdata v1.0.0-rc2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These will need to be updated to require current versions.
Hey, wanted to check if there was any progress on this? Looking to use this :). Happy to contribute something if there's still work to be done and the author (@eedorenko) is no longer involved. Let me know! Is the only required change to update the |
Please rebase. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
Description:
azureblobexporter exports logs and traces to Azure Blob Storage
Fixes #8823
Documentation:
Readme describing exporter functionality and configuration.