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

Issue 9065 #6

Closed
wants to merge 1 commit into from
Closed

Issue 9065 #6

wants to merge 1 commit into from

Conversation

mwhudson
Copy link
Member

Hi,

This fixes http://issues.jenkins-ci.org/browse/JENKINS-9065 by adding a log statement when the instance cap is the reason for not launching an instance.

This is my first time using git, github or developing in Java, so it's also a bit of a test for me, so please let me know if I did anything wrong.

Cheers,
Michael

@kohsuke
Copy link
Member

kohsuke commented Mar 16, 2011

Thanks. Merged toward 1.11. I've also added you as a committer so that you can directly push to the repository for simple changes like this. Thanks!

csilvers added a commit to Khan/ec2-plugin that referenced this pull request Oct 20, 2014
Summary:
Sometimes slave.jar dies due to an OOM error or the like.  In those
cases, make it automatically restart.  Jenkins doesn't notice that
slaves are dead in order to restart it themselves.

Test Plan:
Fingers crossed.  I'll install this on jenkins, terminate all the
jenkins-slave machines on ec2, and run make-allcheck, and make sure
the slaves start up properly.  I don't know what
com.trilead.ssh2.Connection.execCommand does, but it looks like it
just runs the given command in the shell, based on code example jenkinsci#6 at
   http://www.programcreek.com/java-api-examples/index.php?api=com.trilead.ssh2.Session
   >>> sess.execCommand("uname -a && date && uptime && who");

Reviewers: jlfwong

Reviewed By: jlfwong

Subscribers: chris

Differential Revision: http://phabricator.khanacademy.org/D14010
daniel-beck pushed a commit that referenced this pull request May 6, 2020
BindingStep fixes enabled by Workflow 1.5
This pull request was closed.
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.

2 participants