Details

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

      Description

      List cluster should also display role names for each node. This feature is going to be really useful when the cluster state is no longer persisted on the local machine.

      1. WHIRR-289.patch
        12 kB
        Andrei Savu
      2. WHIRR-289.patch
        8 kB
        Andrei Savu

        Issue Links

          Activity

          Hide
          savu.andrei Andrei Savu added a comment -

          Trivial patch tested on ec2.

          Show
          savu.andrei Andrei Savu added a comment - Trivial patch tested on ec2.
          Hide
          tomwhite Tom White added a comment -

          How about changing ClusterController.getNodes() to return Cluster.Instance objects? It shouldn't really be returning NodeMetadata in the public interface anyway, and making this change would help other callers of the interface (so they can find roles), not just the CLI.

          Show
          tomwhite Tom White added a comment - How about changing ClusterController.getNodes() to return Cluster.Instance objects? It shouldn't really be returning NodeMetadata in the public interface anyway, and making this change would help other callers of the interface (so they can find roles), not just the CLI.
          Hide
          tomwhite Tom White added a comment -

          We could do this by adding a getInstances() method and deprecating getNodes().

          Show
          tomwhite Tom White added a comment - We could do this by adding a getInstances() method and deprecating getNodes().
          Hide
          savu.andrei Andrei Savu added a comment -

          Great idea! I will update the code.

          Show
          savu.andrei Andrei Savu added a comment - Great idea! I will update the code.
          Hide
          savu.andrei Andrei Savu added a comment -

          Updated patch. Tom, let me know what you think.

          Show
          savu.andrei Andrei Savu added a comment - Updated patch. Tom, let me know what you think.
          Hide
          tomwhite Tom White added a comment -

          +1

          Show
          tomwhite Tom White added a comment - +1
          Hide
          savu.andrei Andrei Savu added a comment -

          I've just committed this. Tom, thanks for taking the time to review.

          Show
          savu.andrei Andrei Savu added a comment - I've just committed this. Tom, thanks for taking the time to review.

            People

            • Assignee:
              savu.andrei Andrei Savu
              Reporter:
              savu.andrei Andrei Savu
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development