Support for the --kill-nailguns option was inadvertently removed, this puts it back.

Review Request #1352 - Created Nov. 18, 2014 and submitted

Information
Eric Ayers
pants
zundel/nailgun-timeouts-on-ci
803
cb9122a...
Reviewers
pants-reviews
benjyw, ity, patricklaw

The kill-nailguns flag was being read through old_options.cleanup_nailguns

Also updated the nailgun timeout logic to try and pinpoint transient errors in CI if it
happens again.

This reverts the change at https://rbcommons.com/s/twitter/r/1345/ which is redundant with this one (and not as effective.)

CI Passed.

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Patrick Lawson
Eric Ayers
Eric Ayers
Eric Ayers
Ity Kaul
Patrick Lawson
Eric Ayers
Review request changed

Status: Closed (submitted)

Change Summary:

commit 126360f
Loading...