Whirr
  1. Whirr
  2. WHIRR-102

unknown service NPEs cli, should print the bad service to console

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.2.0
    • Component/s: core
    • Labels:
      None

      Description

      specifying an unknown service results in npe.

      1. WHIRR-102.patch
        1.0 kB
        Patrick Hunt
      2. WHIRR-102.patch
        16 kB
        Patrick Hunt

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Patch Available Patch Available Open Open
        4d 2h 28m 1 Patrick Hunt 28/Sep/10 08:21
        Open Open Patch Available Patch Available
        2m 2 Patrick Hunt 28/Sep/10 08:22
        Patch Available Patch Available Resolved Resolved
        2d 12h 56m 1 Patrick Hunt 30/Sep/10 21:19
        Patrick Hunt made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Tom White added a comment -

        +1

        Show
        Tom White added a comment - +1
        Patrick Hunt made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Patrick Hunt made changes -
        Attachment WHIRR-102.patch [ 12455814 ]
        Hide
        Patrick Hunt added a comment -

        adjusted patch for review comments.

        Show
        Patrick Hunt added a comment - adjusted patch for review comments.
        Patrick Hunt made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]
        Hide
        Tom White added a comment -

        +1

        • DestroyClusterCommand suffers from the same problem - can we move the code into ClusterSpecCommand?
        • It should be fairly easy to write a unit test for this.
        • Nit: indentation should be two spaces.
        Show
        Tom White added a comment - +1 DestroyClusterCommand suffers from the same problem - can we move the code into ClusterSpecCommand? It should be fairly easy to write a unit test for this. Nit: indentation should be two spaces.
        Patrick Hunt made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Patrick Hunt made changes -
        Field Original Value New Value
        Attachment WHIRR-102.patch [ 12455461 ]
        Hide
        Patrick Hunt added a comment -

        this patch prints the bad service name rather than npe

        Show
        Patrick Hunt added a comment - this patch prints the bad service name rather than npe
        Patrick Hunt created issue -

          People

          • Assignee:
            Patrick Hunt
            Reporter:
            Patrick Hunt
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development