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

Killing servo processes doesn't always work #124

Closed
jdm opened this issue Oct 4, 2015 · 4 comments
Closed

Killing servo processes doesn't always work #124

jdm opened this issue Oct 4, 2015 · 4 comments

Comments

@jdm
Copy link
Member

@jdm jdm commented Oct 4, 2015

From http://build.servo.org/builders/linux-rel/builds/367:

pkill -x servo
 in dir /home/servo/buildbot/slave/linux-rel/build (timeout 1200 secs)
 watching logfiles {}
 argv: ['pkill', '-x', 'servo']
 environment:
  HOME=/home/servo
  PATH=/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin
  PWD=/home/servo/buildbot/slave/linux-rel/build
  TERM=linux
  UPSTART_INSTANCE=
  UPSTART_JOB=buildbot-slave
 using PTY: False

command timed out: 1200 seconds without output running ['pkill', '-x', 'servo'], attempting to kill
SIGKILL failed to kill process
using fake rc=-1
program finished with exit code -1

remoteFailed: [Failure instance: Traceback from remote host -- Traceback (most recent call last):
Failure: exceptions.RuntimeError: SIGKILL failed to kill process
exceptions.RuntimeError: SIGKILL failed to kill process
]
@metajack
Copy link
Contributor

@metajack metajack commented Oct 5, 2015

Some initial Googling didn't turn up any results for pkill hanging. Hopefully this isn't going to happen often.

@metajack
Copy link
Contributor

@metajack metajack commented Nov 2, 2015

@metajack
Copy link
Contributor

@metajack metajack commented Nov 2, 2015

We had to turn this off because it kept happening. See #146

@edunham
Copy link
Contributor

@edunham edunham commented Jul 5, 2017

The pkill step made it back in during the Buildbot refactor, and nobody has reported this issue continuing to happen, so I'm going to close this issue. If it is actually still a problem, please feel free to reopen with some recent examples.

@edunham edunham closed this Jul 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.