Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JENKINS-18461 Allow the user to not allocate a public ip address for a slave in a cloudstack cloud #45

Merged
merged 2 commits into from Aug 27, 2013

Conversation

spark404
Copy link
Member

This patch adds an advanced option where the user can select to assign a public ip address to a jenkins slave. This option is useful when cloudstack is running in advanced networking mode and the jenkins server is located in the same isolated network or the isolated network is bridged to a physical vlan. In those cases the slave will use the internal IP address of the slave to connect, this saves a public IPv4 address.

spark404 and others added 2 commits August 14, 2013 15:33
address.

Use this option to use the internal ip address when running in an
isolated network with advanced networking.
@buildhive
Copy link

Jenkins » jclouds-plugin #72 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@abayer
Copy link
Member

abayer commented Aug 14, 2013

So 1.6.2-incubating isn't quite out yet - it's in RC right now, and actually, we're about to cut another RC due to a showstopper bug. Otherwise, this sounds good. Will read through it in more detail once we have a real 1.6.2-incubating out.

@spark404
Copy link
Member Author

Thanks. I have it running on a couple of jenkins servers talking to a CloudStack 4.1.1 cloud. So far it seems to work like a charm with the 1.6.2-incubating version.

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@abayer
Copy link
Member

abayer commented Aug 27, 2013

1.6.2-incubating is out today - I'll re-run this PR once it's in Maven Central and then merge/release.

abayer added a commit that referenced this pull request Aug 27, 2013
JENKINS-18461 Allow the user to not allocate a public ip address for a slave in a cloudstack cloud
@abayer abayer merged commit 78b9181 into jenkinsci:master Aug 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants