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

[pull] main from open-telemetry:main #300

Merged
merged 32 commits into from
May 30, 2024
Merged

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 25, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label Mar 25, 2024
keith and others added 28 commits March 25, 2024 19:09
Co-authored-by: Tom Tan <[email protected]>
Co-authored-by: Marc Alff <[email protected]>
Co-authored-by: Lalit Kumar Bhasin <[email protected]>
Co-authored-by: Marc Alff <[email protected]>
…ool (#2553)


Addresses two issues -
1. Fix the use of a conditional variable where a wait on the variable might not be in flight when a notify is called. This is fixed by ensuring that an associated lock is aquired before calling the notify.
2. Instead of relying on a lock an a boolean, replace the use wit a single atomic boolean.

---------

Co-authored-by: Ehsan Saei <[email protected]>
Co-authored-by: Marc Alff <[email protected]>
Co-authored-by: Marc Alff <[email protected]>
* Upgrade Maintainers CI to ubuntu-24.04

* Disable IWYU
@lalitb lalitb merged commit e4d4bd7 into lalitb:main May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants