[jira] Updated: (WHIRR-26) Allow script locations to be overridden

["Jeff Hammerbacher (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/WHIRR-26?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jeff Hammerbacher updated WHIRR-26:
-----------------------------------

    Attachment: WHIRR-26.patch

The attached patch allows the user to set a Java system property called {{whirr.runurl.base}} which will be prepended to all "urls" submitted to RunUrlBuilder. Note that we now require that the protocol (http://) be specified for this prefix, as the java.net.URL constructor expects a protocol to be supplied even if runurl/wget do not require it; I could change to handle that case for the user as well, but it felt a bit magical to do two rounds of URL scrubbing.

There are currently no tests for Whirr core, so I did not add a test for this new behavior. Once there's a unit testing framework in place, I might just use it, but I don't feel comfortable adding the first test.

> Allow script locations to be overridden
> ---------------------------------------
>
>                 Key: WHIRR-26
>                 URL: https://issues.apache.org/jira/browse/WHIRR-26
>             Project: Whirr
>          Issue Type: Improvement
>          Components: core
>            Reporter: Tom White
>            Assignee: Jeff Hammerbacher
>            Priority: Blocker
>             Fix For: 0.1.0
>
>         Attachments: WHIRR-26.patch
>
>
> Currently the script locations are hard-coded to an EC2 bucket. We should make this configurable.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.