Whirr
  1. Whirr
  2. WHIRR-246

Single place to store/load cluster state

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.5.0
    • Component/s: None
    • Labels:
      None

      Description

      Right now cluster state is written and read from the ~/.whirr/cluster-name/instances file and logic to write and update it is spread between the Service and DestroyInstanceCommand (that I know of).

      Since for WHIRR-214 the file must be updated (in another class) and for WHIRR-238 an altogether different method of storing cluster state will be required I think it might be time to move the read/write state logic to its own file (or factory in the future).

      I'll attach a very preliminary patch just to get some feedback.

      1. WHIRR-246.patch
        7 kB
        David Alves
      2. WHIRR-246.patch
        11 kB
        David Alves
      3. WHIRR-246.patch
        17 kB
        Andrei Savu
      4. WHIRR-246.patch
        27 kB
        Andrei Savu

        Issue Links

          Activity

          No work has yet been logged on this issue.

            People

            • Assignee:
              Andrei Savu
              Reporter:
              David Alves
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development