Details

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

      Description

      The default scripts are stored at http://whirr.s3.amazonaws.com/, but this location should be parametrized by release version. So in the next release the scripts would be in http://whirr.s3.amazonaws.com/0.2.0-incubating-SNAPSHOT while in development, and then http://whirr.s3.amazonaws.com/0.2.0-incubating once released. Note that these are the defaults and may be overridden to use another location.

      1. WHIRR-90.patch
        21 kB
        Adrian Cole
      2. WHIRR-90.patch
        23 kB
        Tom White

        Activity

        Hide
        Tom White added a comment -

        Here's a patch that adds the version number the the runurl. To do this it introduces the idea of a default properties file, called whirr-default.properties. Also, the base URL for runurls is specified as a part of ClusterSpec (which now has unit tests).

        Show
        Tom White added a comment - Here's a patch that adds the version number the the runurl. To do this it introduces the idea of a default properties file, called whirr-default.properties. Also, the base URL for runurls is specified as a part of ClusterSpec (which now has unit tests).
        Hide
        Adrian Cole added a comment -

        path naming convention ties this to maven, which facilitates easy maintenance

        Show
        Adrian Cole added a comment - path naming convention ties this to maven, which facilitates easy maintenance
        Hide
        Adrian Cole added a comment -

        fixed patch ready and tested on hadoop, cassandra, and zk

        Show
        Adrian Cole added a comment - fixed patch ready and tested on hadoop, cassandra, and zk
        Hide
        Tom White added a comment -

        I've just committed this. Thanks for fixing the bug in the original version, Adrian.

        Show
        Tom White added a comment - I've just committed this. Thanks for fixing the bug in the original version, Adrian.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development