restate icon indicating copy to clipboard operation
restate copied to clipboard

Attach to cluster controller through Networking

Open AhmedSoliman opened this issue 1 year ago • 4 comments

Attach to cluster controller through Networking

This PR includes:

  • Attachment of PPM to cluster controller now use Networking
  • PPM observes partition processor status through a buffered watch mechanism
  • PPM can now send control messages (unused at this PR) to processor for future use.
  • PPM collects state information from running processors. This will be used in a follow PR to response to controller requests about partitions.

Cluster controller grpc service is kept for external tooling integration (CLI, etc.)


Stack created with Sapling. Best reviewed with ReviewStack.

  • #1514
  • -> #1500
  • #1511

AhmedSoliman avatar May 10 '24 10:05 AhmedSoliman

Test Results

 99 files  ±0   99 suites  ±0   8m 5s :stopwatch: -51s  84 tests ±0   82 :white_check_mark: ±0  2 :zzz: ±0  0 :x: ±0  215 runs   - 1  209 :white_check_mark:  - 1  6 :zzz: ±0  0 :x: ±0 

Results for commit 4f22a938. ± Comparison against base commit 1d7ac91b.

This pull request removes 1 and adds 1 tests. Note that renamed tests count towards both.
dev.restate.e2e.JavaWorkflowAPITest ‑ Workflow cannot be submitted more than once
dev.restate.e2e.runtime.IngressTest ‑ Idempotent send then attach/getOutput

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar May 10 '24 10:05 github-actions[bot]

Linter error is on generated code, this will be fixed in the next PR in this stack.

AhmedSoliman avatar May 10 '24 12:05 AhmedSoliman

Significant changes are incoming to this PR, putting this back as WIP.

AhmedSoliman avatar May 13 '24 10:05 AhmedSoliman

This should be ready for review. PTAL @tillrohrmann.

AhmedSoliman avatar May 14 '24 12:05 AhmedSoliman