Uploaded image for project: 'Apache Whirr (retired)'
  1. Apache Whirr (retired)
  2. WHIRR-246

Single place to store/load cluster state

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 0.5.0
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: