Customize pants settings for Jenkins.

Review Request #4100 — Created July 20, 2016 and submitted

jsirois
pants
jsirois/issues/3684/custom-jenkins-pants.ini
3684, 3689
4104
24bf343...
pants-reviews
kwlzn, patricklaw
Jenkinsfile       |  3 +++
 pants.jenkins.ini | 21 +++++++++++++++++++++
 2 files changed, 24 insertions(+)
CI went green here:
  http://jenkins.pantsbuild.org/job/pantsbuild/job/pants/job/PR-3689/6/

This seems like it can't hurt + provides clarity, but the real
fix is over here:
  https://rbcommons.com/s/twitter/r/4101/
KW
  1. Ship It!
  2. 
      
JS
Review request changed

Status: Closed (submitted)

Change Summary:

Now on master:

git log -1 origin/master
commit cc877f6f06a162efec8b60db556670419b1ac7c0
Author: John Sirois <john.sirois@gmail.com>
Date:   Tue Jul 19 20:25:52 2016 -0600

    Customize pants settings for Jenkins.
    
    Testing Done:
    CI went green here:
      http://jenkins.pantsbuild.org/job/pantsbuild/job/pants/job/PR-3689/6/
    
    This seems like it can't hurt + provides clarity, but the real
    fix is over here:
      https://rbcommons.com/s/twitter/r/4101/
    
    Bugs closed: 3684, 3689
    
    Reviewed at https://rbcommons.com/s/twitter/r/4100/
ZU
  1. 
      
  2. pants.jenkins.ini (Diff revision 1)
     
     

    We have --kill-nailguns turned on in our repo and it prevents a lot of problems.

    [GLOBAL]
    kill_nailguns: True
    

    I'm not sure exactly what problems nailgun is running into, but maybe later we can experiment with leaving this setting on but turning on nailgun explicitly for [compile.zinc].

    1. Agreed, running a few combos and getting timings would be a good idea in case some combo is faster on CI. At present I'm focused on the quest for stable green master, but after travis is dead, this seems like a worthy task to save $$$.
  3. 
      
ST
  1. 
      
  2. pants.jenkins.ini (Diff revision 1)
     
     

    One more use of this: [compile.zinc] worker_count

    1. Thanks - I'll circle back and get that in a new RB.
    2. https://rbcommons.com/s/twitter/r/4104/
  3. 
      
Loading...