Skip to content

Commit

Permalink
[FIXED JENKINS-11744] Debian/Ubuntu init script does not wait long en…
Browse files Browse the repository at this point in the history
…ough during stop operation

Increase stop timeout from 5 to 20 seconds.
  • Loading branch information
oldelvet authored and kohsuke committed Nov 30, 2011
1 parent a1714c1 commit 487b945
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions changelog.html
Expand Up @@ -76,6 +76,9 @@
<li class=bug>
Fixed a bug in Windows registry enumeration with large data.
(<a href="https://issues.jenkins-ci.org/browse/JENKINS-11767">issue 11767</a>)
<li class=bug>
Debian/Ubuntu init script does not wait long enough during stop operation
(<a href="https://issues.jenkins-ci.org/browse/JENKINS-11744">issue 11744</a>)
</ul>
</div><!--=TRUNK-END=-->

Expand Down
2 changes: 1 addition & 1 deletion debian/debian/jenkins.init
Expand Up @@ -147,7 +147,7 @@ do_stop()
0)
$DAEMON $DAEMON_ARGS --stop || return 2
# wait for the process to really terminate
for n in 1 2 3 4 5; do
for n in 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20; do

This comment has been minimized.

Copy link
@neclimdul

neclimdul Dec 14, 2011

It seems like you could use seq for a lot more compact and maintainable line.

for n in $(seq 20); do
sleep 1
$DAEMON $DAEMON_ARGS --running || break
done
Expand Down

0 comments on commit 487b945

Please sign in to comment.