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

refactor: first-pass at runtime interface, signals, shutdown, and more #180

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

zeeshanlakhani
Copy link
Contributor

This work precedes subsequent work on running workflows from the Homestar runner/server/runtime but sets everything into gear.

Includes:

  • overall refactoring of events and event handling, streamlining how to work with senders, etc
  • introduce a runner which can
    • interact with task/batch specific-abort handles introduced in joinsets within a worker
    • interact with commands, which will happen through the CLI interface, i.e. star, run-workflow, akin to something like temporal-server
    • start-up event handler, websocket server, takes in db and can mock via implementation
  • lots of cleanups

@codecov
Copy link

codecov bot commented Jul 4, 2023

Codecov Report

Merging #180 (d39ae60) into main (af9b1d1) will increase coverage by 3.45%.
The diff coverage is 42.18%.

❗ Current head d39ae60 differs from pull request most recent head a6d3157. Consider uploading reports for the commit a6d3157 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #180      +/-   ##
==========================================
+ Coverage   47.97%   51.42%   +3.45%     
==========================================
  Files          50       56       +6     
  Lines        2193     2349     +156     
  Branches      539      571      +32     
==========================================
+ Hits         1052     1208     +156     
+ Misses        817      773      -44     
- Partials      324      368      +44     
Impacted Files Coverage Δ
homestar-core/src/ipld/link.rs 0.00% <ø> (ø)
homestar-runtime/src/db.rs 44.44% <ø> (+5.55%) ⬆️
homestar-runtime/src/lib.rs 100.00% <ø> (ø)
homestar-runtime/src/logger.rs 0.00% <ø> (ø)
homestar-runtime/src/main.rs 2.94% <0.00%> (-1.41%) ⬇️
homestar-runtime/src/network/ipfs.rs 30.00% <0.00%> (ø)
homestar-runtime/src/tasks/wasm.rs 63.63% <ø> (ø)
homestar-runtime/src/workflow.rs 65.38% <0.00%> (ø)
homestar-runtime/src/event_handler/swarm_event.rs 11.11% <11.11%> (ø)
homestar-runtime/src/event_handler/event.rs 12.00% <12.00%> (ø)
... and 15 more

... and 1 file with indirect coverage changes

@zeeshanlakhani
Copy link
Contributor Author

@mriise will hold off merging this until #172 is in.

This work precedes subsequent work on running workflows from the Homestar runner/server/runtime but sets everything into gear.

Includes:

* overall refactoring of events and event handling, streamlining how to work with senders, etc
* introduce a runner which can
  - interact with task/batch specific-abort handles introduced in joinsets within a worker
  - interact with commands, which will happen through the CLI interface, i.e. star, run-workflow, akin to something like temporal-server
  - start-up event handler, websocket server, takes in db and can mock via implementation
* lots of cleanups
@zeeshanlakhani zeeshanlakhani merged commit 86ca3e4 into main Jul 7, 2023
8 checks passed
@zeeshanlakhani zeeshanlakhani deleted the zl/cli+shutdown-start branch July 7, 2023 06:07
@release-plz-ipvm-wg release-plz-ipvm-wg bot mentioned this pull request Oct 9, 2023
This was referenced Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant