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.

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

Block-builder-scheduler: RPC interface #10022

Closed
wants to merge 27 commits into from
Closed

Block-builder-scheduler: RPC interface #10022

wants to merge 27 commits into from
+2,632 −15

Conversation

seizethedave
Copy link
Contributor

What this PR does

Which issue(s) this PR fixes or relates to

Fixes #

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

@seizethedave
Merge from davidgrant/block-builder-scheduler-reconcile
f457910
@seizethedave
Don't need to get fancy with critical section: no contention is possi…
8f9d996 
…ble yet.
@seizethedave
Document s.committed.
6712985
@seizethedave
Merge
f97e4cb
@seizethedave
Fix merge slop.
4cfeed9
@seizethedave
Add proto defs.
eaed668
@seizethedave
Initial RPC protos.
d37b1f2
@seizethedave
Log an error if flushing fails.
35d1868
@seizethedave
Include RPC forms of assignJob/updateJob.
81dee72
@seizethedave
Merge remote-tracking branch 'origin/main' into davidgrant/block-buil…
3772a12 
…der-scheduler-kafka-flush
@seizethedave
Fleshing out the client module.
7c3f2da
@seizethedave
Appease the linter.
db44d45
@seizethedave
workerID param not used.
5cc39ce
@seizethedave
CompleteJob: no context needed.
3c6a566
@seizethedave
The linter.
91b8e83
@seizethedave
More logs.
7355de9
@seizethedave
Fix circular reference. Initialize client.
c34617d
@seizethedave
schedulerproto -> schedulerpb
9f67caa
@seizethedave
merge
47b5662
@seizethedave
Fix compile stuff.
188217f
@seizethedave
Getting builder/scheduler up in docker-compose.
179018e
@seizethedave
Clean unneeded flags.
f789447
@seizethedave
Fixes for BB<>scheduler communications:
bad7dc2 
- accidentally wasn't calling scheduler.assignJob in the RPC.
- Add logging for jobQueue operations.
- Experimentally tear into blockbuilder to begin consuming jobs from scheduler.
@seizethedave
Fix block-builder's scheduler_config so it can connect to a scheduler.
762012f
@seizethedave
Shorter update interval for testing.
671f4b3
@seizethedave
Fix unloggable key.
f760fa8
@seizethedave
Longer job working time.
df7e43e
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
@seizethedave