Closed ahgittin closed 10 years ago
Looks good. Only concern is allowRemoteTimestampInMemento
- what do we need that for? Would be nice if tests behaved same way as production code.
Brooklyn Central » brooklyn #2537 FAILURE Looks like there's a problem with this pull request (what's this?)
Brooklyn Central » brooklyn #2538 SUCCESS This pull request looks good (what's this?)
Hello, and thank you for your contribution to Brooklyn. We've moved house, so unfortunately we can't accept your pull request on this repository. You can read our new How to contribute guide, and read How to migrate your brooklyncentral fork to Apache. Our Community is available to help you if you have any trouble with the migration - or simply post a comment here and I will respond.
Thank you again for your contribution and we hope that you will submit your contribution again at our new home!
... that to determine which mgmt nodes are live
means we aren't reliant on hosts having their time synched
probably needs more testing (which i'll do) but wanted to share this now for review