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 slave defaults bugfix #229

Merged
merged 2 commits into from
Dec 30, 2014

Conversation

madAndroid
Copy link
Contributor

This was causing an issue during slave process restart on CentOS7, resulting in the following:

[root@slave ~]# systemctl status jenkins-slave.service
jenkins-slave.service - SYSV: Jenkins Slave Swarm Client
   Loaded: loaded (/etc/rc.d/init.d/jenkins-slave)
   Active: failed (Result: exit-code) since Thu 2014-12-18 08:19:23 UTC; 1min 17s ago
  Process: 27562 ExecStop=/etc/rc.d/init.d/jenkins-slave stop (code=exited, status=0/SUCCESS)
  Process: 27569 ExecStart=/etc/rc.d/init.d/jenkins-slave start (code=exited, status=1/FAILURE)

Dec 18 08:19:23 slave.host systemd[1]: Starting SYSV: Jenkins Slave Swarm Client...
Dec 18 08:19:23 slave.host jenkins-slave[27569]: /etc/sysconfig/jenkins-slave: line 75: [-n: command not found
Dec 18 08:19:23 slave.host jenkins-slave[27569]: Starting Jenkins Slave...
Dec 18 08:19:23 slave.host systemd[1]: jenkins-slave.service: control process exited, code=exited status=1
Dec 18 08:19:23 slave.host systemd[1]: Failed to start SYSV: Jenkins Slave Swarm Client.
Dec 18 08:19:23 slave.host systemd[1]: Unit jenkins-slave.service entered failed state.

@jenkinsadmin
Copy link

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

@rtyler
Copy link

rtyler commented Dec 30, 2014

Weird, thanks for the fix

@rtyler rtyler added this to the 1.4.0 - Smithers milestone Dec 30, 2014
rtyler pushed a commit that referenced this pull request Dec 30, 2014
@rtyler rtyler merged commit 3880b34 into voxpupuli:master Dec 30, 2014
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.

3 participants