-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[apple] Mobile failures in System.Diagnostics.DiagnosticSource
runtime metrics
#105202
Comments
Tagging subscribers to 'os-ios': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger |
Tagging subscribers to 'os-tvos': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger |
Tagging subscribers to 'os-maccatalyst': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger |
Possibly related to the new tests introduced in #104680. Do you know if these tests should be disabled or same changes are needed for them to work on Apple mobile targets? @stevejgordon @tarekgh ? |
@matouskozak I think we can safely skip this platform in the runtime metrics tests. |
Thank you, would you like to open a PR to disable them or should I take a look? Do you know if this is an issue that should be fixed on the Mono runtime side or is it by design of the test? It looks like only a few tests are failing so I suppose the |
@matouskozak I've opened a PR and took a different approach after reviewing the failures more closely. |
Build Information
Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=747439
Build error leg or test failing:
System.Diagnostics.DiagnosticSource.Tests
Affected CI:
[ios][tvos]simulator_x64_release_allsubsets_mono
,maccatalyst_[x64/arm64]_release_allsubsests_mono
(runtime-extra-platforms
)Range of commits: 0806470...7cdd2e9
Stack trace example from MacCatalyst x64:
Error Message
Fill the error message using step by step known issues guidance.
Known issue validation
Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=747439
Error message validated:
[Expected value to be greater than zero. Actual value was: 0.
]Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 7/21/2024 4:35:30 PM UTC
Report
Summary
The text was updated successfully, but these errors were encountered: