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

The runningOnBuildServer check for detecting the branch name on Jenkins / Hudson is out-of-date / incomplete #326

Closed
TheSnoozer opened this issue Sep 23, 2017 · 0 comments
Labels
Milestone

Comments

@TheSnoozer
Copy link
Collaborator

While testing the issue for #325 I noticed that the printenv command might not include HUDSON_URL or JENKINS_URL which is being used for detecting the branch name on Jenkins / Hudson.

[Pipeline] Running shell script
+ printenv
JENKINS_HOME=/var/lib/jenkins
MAIL=/var/mail/jenkins
LANGUAGE=en_CA:en
USER=jenkins
RUN_CHANGES_DISPLAY_URL=http://unconfigured-jenkins-location/job/Pipeline/20/display/redirect?page=changes
NODE_LABELS=master
SHLVL=1
branch=master
HOME=/var/lib/jenkins
HUDSON_COOKIE=892bd668-0a0d-4869-93f5-0659da361f41
JENKINS_SERVER_COOKIE=durable-32e8fcd6d197da46d2c1ee47a5761321
WORKSPACE=/var/lib/jenkins/workspace/Pipeline
QT_QPA_PLATFORMTHEME=appmenu-qt5
LOGNAME=jenkins
NODE_NAME=master
_=/usr/bin/daemon
EXECUTOR_NUMBER=1
GIT_BRANCH=master
XDG_SESSION_ID=c2
BUILD_DISPLAY_NAME=#20
HUDSON_HOME=/var/lib/jenkins
JOB_BASE_NAME=Pipeline
PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games:/snap/bin
BUILD_ID=20
XDG_RUNTIME_DIR=/run/user/121
BUILD_TAG=jenkins-Pipeline-20
LANG=en_CA.UTF-8
BUILD_NUMBER=20
JENKINS_NODE_COOKIE=d95dd56b-a3e6-4a01-8e9c-ea077e82be81
SHELL=/bin/bash
RUN_DISPLAY_URL=http://unconfigured-jenkins-location/job/Pipeline/20/display/redirect
HUDSON_SERVER_COOKIE=82e17db30b128800
JOB_DISPLAY_URL=http://unconfigured-jenkins-location/job/Pipeline/display/redirect
JOB_NAME=Pipeline
PWD=/var/lib/jenkins/workspace/Pipeline
[Pipeline]
@TheSnoozer TheSnoozer added the bug label Sep 23, 2017
TheSnoozer pushed a commit to TheSnoozer/git-commit-id-maven-plugin that referenced this issue Sep 23, 2017
…ranch name on Jenkins / Hudson now checks 'HUDSON_HOME' and 'JENKINS_HOME'
TheSnoozer pushed a commit that referenced this issue Sep 23, 2017
Improve properties replacement and allow to change the case of letters (#317) and fix the runningOnBuildServer check for detecting the branch name on Jenkins / Hudson to also check 'HUDSON_HOME' and 'JENKINS_HOME' (#326)
@TheSnoozer TheSnoozer added this to the 2.2.4 milestone Sep 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant