brooklyncentral / brooklyn

This project has moved and is now part of the ASF
https://github.com/apache/incubator-brooklyn
72 stars 27 forks source link

Fix some DynamicCluster issues #1437

Closed grkvlt closed 10 years ago

grkvlt commented 10 years ago

The propagation is needed because essentially the state of the cluster is dependant on the state of the members, so when a failed member is removed and replaced with a new working one, the state should not still be ON_FIRE

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2388 SUCCESS This pull request looks good (what's this?)

grkvlt commented 10 years ago

I'm not going to merge this yet as it's a fairly small, simple change.

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2390 SUCCESS This pull request looks good (what's this?)

grkvlt commented 10 years ago

Right, so we probably want a smarter mechanism that sets the state based on both the cluster controller and the cluster members (keeping in mind quorum numbers) and also whether we are stopping this entity. I'll work on that...

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2392 SUCCESS This pull request looks good (what's this?)

grkvlt commented 10 years ago

@aledsage Is this better?

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2394 SUCCESS This pull request looks good (what's this?)

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2411 SUCCESS This pull request looks good (what's this?)

grkvlt commented 10 years ago

Fixed issue where state did not revert to RUNNING properly. @aledsage Any issues with merging this now?

buildhive commented 10 years ago

Brooklyn Central » brooklyn #2423 SUCCESS This pull request looks good (what's this?)