Skip to content

Eventstream destruction deadlock #793

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 33 commits into from
Closed

Eventstream destruction deadlock #793

wants to merge 33 commits into from

Conversation

bretambrose
Copy link
Contributor

  • Small workaround for a particular deadlock in eventstream bindings (others exist but are out of scope)
  • Adds eventstream tests to CI using new echo server builder action
  • Update SDK formatting utility, format secure tunneling

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Bret Ambrose added 30 commits April 11, 2025 13:55
Initial resolution attempt
d3e49b0
Update
a99cd1e
Testing
904651a
Pre-format sync
bb948aa
Updated format script
8403ed4
Update formatting to python-based with clang version control
1160d10
Update to builder version with eventstream server support
b4d3bcd
Credentials setup
21abfcd
Cleanup strings per fixture tear down
4df20ff
Try ubuntu 22 to see if maven is pre-installed
6c0184c
No allocator exists during static initialization
e6fbe72
Begin test rewrite; failures on other platforms should reveal other t…
8f2417e 
…erminal status codes
Includes
5c3ccfc
Use latest configure credentials to see if retries avoid baffling error
a1b918a
Widen failure possibility based on CI
a571695
Custom eventstream server step so that we can dump server-side logs f…
9c5dd02 
…or debugging on posix systems
Install maven during setup
8639e71
Try to gather output that tells us whether the Java CRT is supported …
8085d1a 
…on this platform
Check stderr of eventstream run to verify platform Java CRT support
cf79be3
print stderr
440c493
Why is everything failing now?
e8f00fd
Don't route stdout to stdout
d80780f
Missing import
121dc44
Probe command log
7c7f7cd
Updates
2028eac
Umm let's not cause tsan warnings by blindly unlocking stuff
231e4f4
Formatting fix (but don't check in format update yet) + more test work
cc7afc7
Formatting and codegen updates
cd9289c
More tests
34b1e29
Merge branch 'main' into EventstreamDestructionDeadlock
42eb9b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bretambrose