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

warmup and backup enhancements, now rest endpoint #14

Merged
merged 1 commit into from Jul 19, 2016
Merged

Conversation

ipapapa
Copy link
Contributor

@ipapapa ipapapa commented Jul 19, 2016

This PR does two things:
(a) enhances the warm up process by

  • delaying the start of Dynomite. We do not need to start Dynomite before the warm up finishes. The reason is that replicated data are received by peer nodes even though the node is out of Discovery. That may increase the memory footprint of Dynomite.
  • asking the local rack for the nodes with the same token

(b) adds a REST endpoint in Florida in which information about the status of the node (and the processes running) in JSON format. The reason, we added this feature is: to have a single point of ground truth about the node without having to parse logs (especially useful for S3 backups). It will allow us to check before we move on.
For now, the following are supported:

  • warmup (status: not started, completed, pending, unsuccessful)
  • backup (status: not started, completed, pending, unsuccessful)
  • restore (status: not started, completed, pending, unsuccessful)
  • dynomiteAlive
  • storageAlive
  • healthy

(c) restructured backups and restores so that we can extend them to other platforms as well. The main benefit from that would be OSS folks who use Dynomite-manager with Google Cloud etc.

@ipapapa ipapapa merged commit 1036d0f into dev Jul 19, 2016
@ipapapa ipapapa deleted the warmv2backupv2 branch July 19, 2016 18:41
@ipapapa
Copy link
Contributor Author

ipapapa commented Jul 19, 2016

JSON would like the following image:

screen shot 2016-07-19 at 11 42 29 am

The new Status REST endpoint integrates very well with Spinnaker as well.

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.

None yet

1 participant