search

nodesource / nsolid

N|Solid Runtime
Other
61 stars 4 forks source link

Multiple gRPC fixes #222

Closed santigimeno closed 4 days ago

santigimeno commented 1 week ago 
agents: fix ExitEvent condition handling

A couple of fixes here:
- Both the `uv_mutex_t` and `uv_cond_t` should be initialized before
  defining the lambda.
- Add a boolean variable to make sure we're not waiting forever in the
  case the lambda code is run before reaching the `uv_cond_wait()`.
agents: fix synchronized code in GrpcAgent

It should also cover the `start_profiling` fn execution.
test: unflake test-nsolid asset tests

Increase the timeouts a bit to make sure the heapSampling and the
cpuProfile are actually stopped before trying to start a new one.
santigimeno commented 4 days ago

Landed in fef5776e1b368bc8e229a446b3f067677505fce7...04a1aa08f4444cb95c9f01fc33e036779ce071b4