Details

    • Type: Improvement
    • Status: Resolved
    • Priority: 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.

        Attachments

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

          Issue Links

            Activity

              People

              • Assignee:
                savu.andrei Andrei Savu
                Reporter:
                dr-alves David Alves
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: