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

Kill Servo processes left around #123

Merged
merged 1 commit into from Oct 1, 2015

Conversation

@metajack
Copy link
Contributor

metajack commented Oct 1, 2015

After running wpt tests, we sometimes end up with zombie servo processes
at 100% CPU. This kills those off to reset machine state.

Review on Reviewable

After running wpt tests, we sometimes end up with zombie servo processes
at 100% CPU. This kills those off to reset machine state.
@metajack
Copy link
Contributor Author

metajack commented Oct 1, 2015

Already tested in production.

@Manishearth
Copy link
Member

Manishearth commented Oct 1, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Oct 1, 2015

📌 Commit c1d30a7 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Oct 1, 2015

Testing commit c1d30a7 with merge 41e879b...

bors-servo pushed a commit that referenced this pull request Oct 1, 2015
Kill Servo processes left around

After running wpt tests, we sometimes end up with zombie servo processes
at 100% CPU. This kills those off to reset machine state.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/123)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 1, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit c1d30a7 into servo:master Oct 1, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@metajack metajack deleted the metajack:kill-leftover-servos branch Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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