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

[chore] Skip Unix datagram tests on Windows #37036

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

pjanotti
Copy link
Contributor

@pjanotti pjanotti commented Jan 6, 2025

Description

Skip tests that are always failing on Windows. The tests were added via #36608 but can't run on Windows..

Fixes #10151

Testing

Tested locally.

@pjanotti pjanotti requested review from jmacd, dmitryax and a team as code owners January 6, 2025 19:19
@pjanotti pjanotti added the Run Windows Enable running windows test on a PR label Jan 6, 2025
@github-actions github-actions bot added the receiver/statsd statsd related issues label Jan 6, 2025
@pjanotti pjanotti added Skip Changelog PRs that do not require a CHANGELOG.md entry and removed receiver/statsd statsd related issues labels Jan 6, 2025
@pjanotti
Copy link
Contributor Author

pjanotti commented Jan 6, 2025

CI failure is fixed by #36923

@pjanotti pjanotti added the ready to merge Code review completed; ready to merge by maintainers label Jan 6, 2025
Copy link

codecov bot commented Jan 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.63%. Comparing base (333f016) to head (61992f6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #37036      +/-   ##
==========================================
- Coverage   79.63%   79.63%   -0.01%     
==========================================
  Files        2223     2223              
  Lines      210283   210283              
==========================================
- Hits       167467   167459       -8     
- Misses      37206    37208       +2     
- Partials     5610     5616       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dmitryax dmitryax merged commit 5e50ee8 into open-telemetry:main Jan 6, 2025
224 of 247 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 6, 2025
@pjanotti pjanotti deleted the skip-uds-tests-on-windows branch January 6, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers Run Windows Enable running windows test on a PR Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[receiver/statsd] Test_statsdreceiver_EndToEnd test is failing on Windows
3 participants