Whirr
  1. Whirr
  2. WHIRR-90

Scripts should be versioned

        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. Text File
            WHIRR-90.patch
            21 kB
            Adrian Cole
          2. Text File
            WHIRR-90.patch
            23 kB
            Tom White

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            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