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]: worker as a separate entry point for cluster primary #835

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

snewcomer
Copy link
Contributor

@snewcomer snewcomer commented Jun 1, 2021

ref #833

This PR is intended to be an initial migration towards a vision for the fastboot app server that ensures extensibility and reliability.

What this PR includes

  • Two separate entry points: cluster setup with separate worker entry point - docs. This reduces conflation between primary and worker APIs as well as ensure all application state and information is the responsibility of the worker. Moreover, it moves us to completely exposing the worker to end user configuration and setup.
  • HTTP server should be cluster worker concerns and not passed as top level to primary process.

What this PR does not include (future follow on PRs)

  • Exposing worker for configuration and setup script
  • extend message handling to more than a hard coded set of messages
  • Event emitter API to receive messages from http server or primary; bi directional communication
  • Further improvements to download and notify API
  • Middleware should be loaded via DAG.

See repo for ideas included in this PR.

https://github.com/snewcomer/ember-alt-fastboot-app-server

@snewcomer snewcomer self-assigned this Jun 1, 2021
@snewcomer snewcomer force-pushed the sn/worker-separate branch 7 times, most recently from 4bca7d2 to 714c0e2 Compare June 1, 2021 18:08
@snewcomer snewcomer force-pushed the sn/worker-separate branch from 18706eb to 3db8208 Compare May 8, 2022 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant