[JIRA] (JENKINS-7830) using environment variable for local port of ssh tunnel

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[JIRA] (JENKINS-7830) using environment variable for local port of ssh tunnel

JIRA noreply@jenkins-ci.org

    [ https://issues.jenkins-ci.org/browse/JENKINS-7830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=161118#comment-161118 ]

Lorrin Nelson commented on JENKINS-7830:

Hi Ross, thanks again.

Based on GitHub history, looks like the change went into 1.13 so went with that.

In src/main/java/hudson/plugins/sauce_ondemand/SauceOnDemandBuildWrapper.java I think the regex is too restrictive: {code}Pattern ENVIRONMENT_VARIABLE_PATTERN = Pattern.compile("[$|%]([A-Z]+)");{code}

(I tend to have underscores in my environment variables.) This [StackOverflow discussion|http://stackoverflow.com/questions/2821043/allowed-characters-in-linux-environment-variable-names] suggests {code}[a-zA-Z_][a-zA-Z0-9_]*{code} might be a good regex.

Also the help text for the port should say port/_PORT instead of host/_HOST in this line: "The value of the host will be stored in the SAUCE_ONDEMAND_HOST environment variable."

Not working for me yet, but possibly config error on my end to iron out.

> using environment variable for local port of ssh tunnel
> -------------------------------------------------------
>                 Key: JENKINS-7830
>                 URL: https://issues.jenkins-ci.org/browse/JENKINS-7830
>             Project: Jenkins
>          Issue Type: New Feature
>          Components: sauce-ondemand
>            Reporter: scytacki
>            Assignee: Ross Rowe
>             Fix For: current
> I use an environment variable to set the port used by the local server that I want to test.  I do this because we want to be able to build/test the hudson job on multiple slave nodes at the same time.  We currently have those slave nodes running on the same machine. If the port is hard coded then I can't startup multiple local servers at the same time because the port will be the same for each and will conflict.
> So it would be helpful if the sauce-ondemand plugin would allow using environment variables for the "Local Port" setting.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jenkins-ci.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira