Skip to content

Start/Stop Improvements#199

Merged
pnagurny merged 1 commit intobitpay:masterfrom
braydonf:start-error
Sep 8, 2015
Merged

Start/Stop Improvements#199
pnagurny merged 1 commit intobitpay:masterfrom
braydonf:start-error

Conversation

@braydonf
Copy link
Copy Markdown
Contributor

@braydonf braydonf commented Sep 4, 2015

  • A Node will shutdown if there is an error starting a service as it would lead to cascading errors.
  • node.start() needs to be called, and nolonger is called automatically when the instance is created.
  • A service will only be added to node.services after it's started
  • Stopping services that are not started will gracefully continue.
  • Logging sync status of db will only apply if the service is started.
  • Debug log about a service without a route will always include the service name
  • Closes Exit with failing service #181

@braydonf braydonf changed the title Shutdown node if there is an error in starting services. WIP Shutdown node if there is an error in starting services. Sep 4, 2015
@braydonf braydonf changed the title WIP Shutdown node if there is an error in starting services. Start/Stop Improvements Sep 5, 2015
- A Node will shutdown if there is an error starting a service as it would lead to cascading errors.
- `node.start()` needs to be called, and nolonger is called automatically when the instance is created.
- A service will only be added to node.services after it's started
- Stopping services that are not started will gracefully continue.
- Logging sync status of db will only apply if the service is started.
- Debug log about a service without a route will always include the service name
@pnagurny
Copy link
Copy Markdown
Contributor

pnagurny commented Sep 8, 2015

LGTM

pnagurny added a commit that referenced this pull request Sep 8, 2015
@pnagurny pnagurny merged commit 2469c6d into bitpay:master Sep 8, 2015
@braydonf braydonf deleted the start-error branch September 16, 2015 03:04
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.

2 participants