Navigation Menu

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

Configure Jenkins location and admin email #14

Merged
merged 1 commit into from Jul 18, 2017
Merged

Configure Jenkins location and admin email #14

merged 1 commit into from Jul 18, 2017

Conversation

nre-ableton
Copy link
Contributor

This commit creates jenkins.model.JenkinsLocationConfiguration.xml,
which is automatically generated by Jenkins but only after one
manually saves the system configuration. Until this file is generated,
several Jenkins-related environment variables use
"http://unconfigured-jenkins-location" as the server URL.

This change also introduces the jenkins_admin variable for the
administrator's email address, since that is contained in the same XML
file.

This commit creates jenkins.model.JenkinsLocationConfiguration.xml,
which is automatically generated by Jenkins but only after one
manually saves the system configuration. Until this file is generated,
several Jenkins-related environment variables use
"http://unconfigured-jenkins-location" as the server URL.

This change also introduces the jenkins_admin variable for the
administrator's email address, since that is contained in the same XML
file.
@nre-ableton
Copy link
Contributor Author

ping @emmetog

@emmetog
Copy link
Owner

emmetog commented Jul 18, 2017

Thanks for this @nre-ableton! Sorry for the delay in getting merged.

@emmetog emmetog merged commit bb7eb8b into emmetog:master Jul 18, 2017
@nre-ableton
Copy link
Contributor Author

@emmetog No problem, I just wanted to make sure that this PR didn't get lost in your inbox (as has happened to me on more than one occasion 😄 ). Thanks for the merge!

@nre-ableton nre-ableton deleted the nre_master_jenkins-location branch July 18, 2017 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants