-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[chore] use macos-14 for arm testing #10540
[chore] use macos-14 for arm testing #10540
Conversation
Similar to open-telemetry/opentelemetry-collector-contrib#33921 Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10540 +/- ##
==========================================
- Coverage 92.39% 92.26% -0.13%
==========================================
Files 393 395 +2
Lines 18614 18703 +89
==========================================
+ Hits 17199 17257 +58
- Misses 1058 1086 +28
- Partials 357 360 +3 ☔ View full report in Codecov by Sentry. |
I think after this change what we claim on https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/platform-support.md#tier-2--secondary-support is no longer correct. But realistically, the current arm runners do not provide good test coverage, they are too flaky, so maybe we can just downgrade our support for Linux ARM for now |
Pinging @atoulme for review on this as well since he's the owner of linux/arm |
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@mx-psi as per your suggestion, i've updated the supported platforms doc to reflect this change |
I am acknowledging this change. I didn't get a chance to review and weigh in. I will take further actions to bring back support of linux/arm64 back to tier 2. Thanks. |
Similar to open-telemetry/opentelemetry-collector-contrib#33921