Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: cli
    • Labels:
      None

      Description

      It seems like when I have committed WHIRR-258 I have managed to break the logging config for the command line interface.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2m 7s 1 Andrei Savu 26/Aug/11 06:34
        Patch Available Patch Available Resolved Resolved
        3d 10h 27m 1 Tom White 29/Aug/11 17:01
        Tom White made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Tom White added a comment -

        I've just committed this.

        Andrei - When I first tried to commit this I got a permission denied message too. I had to open up permissions for Whirr committers on the new SVN path, and then it worked. Let me know if you continue to have permissions problems when committing.

        Show
        Tom White added a comment - I've just committed this. Andrei - When I first tried to commit this I got a permission denied message too. I had to open up permissions for Whirr committers on the new SVN path, and then it worked. Let me know if you continue to have permissions problems when committing.
        Hide
        Andrei Savu added a comment -

        I've tried to commit this but I no longer have write permissions on the new repo location.

        Show
        Andrei Savu added a comment - I've tried to commit this but I no longer have write permissions on the new repo location.
        Hide
        Andrei Savu added a comment -

        Without this patch ./bin/whirr destroy-cluster --config recipes/zookeeper-ec2.properties shows a list of log4j warnings.

        Show
        Andrei Savu added a comment - Without this patch ./bin/whirr destroy-cluster --config recipes/zookeeper-ec2.properties shows a list of log4j warnings.
        Hide
        Tom White added a comment -

        +1 looks good. How did you test this?

        Show
        Tom White added a comment - +1 looks good. How did you test this?
        Andrei Savu made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Andrei Savu made changes -
        Field Original Value New Value
        Attachment WHIRR-377.patch [ 12491736 ]
        Hide
        Andrei Savu added a comment -

        Simple patch that fixes the issue.

        Show
        Andrei Savu added a comment - Simple patch that fixes the issue.
        Andrei Savu created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development