Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.3.0
    • Component/s: core
    • Labels:
      None

      Description

      From WHIRR-117 a cluster may run multiple services, making the service-name property redundant, since the role names in the instance template drive the bootstrapping and configuration of nodes.

      We should remove it, or at least make it optional, since there may still be a need for different implementations of Service (e.g. WHIRR-125).

        Activity

        Tom White created issue -
        Hide
        Tom White added a comment -

        Here's a straightforward patch which makes the service name optional. It passes tests.

        Show
        Tom White added a comment - Here's a straightforward patch which makes the service name optional. It passes tests.
        Tom White made changes -
        Field Original Value New Value
        Attachment WHIRR-157.patch [ 12466490 ]
        Hide
        Andrei Savu added a comment - - edited

        +1 Looks good. All unit tests are passing for me (& zookeeper integration tests on ec2).

        Show
        Andrei Savu added a comment - - edited +1 Looks good. All unit tests are passing for me (& zookeeper integration tests on ec2).
        Hide
        Tom White added a comment -

        I've just committed this.

        Show
        Tom White added a comment - I've just committed this.
        Tom White made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Tom White [ tomwhite ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        10d 23h 38m 1 Tom White 20/Dec/10 22:55

          People

          • Assignee:
            Tom White
            Reporter:
            Tom White
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development