Move node info to a script.

Review Request #3842 — Created May 7, 2016 and submitted

jsirois
pants
jsirois/issues/3343/access_node_info_from_script
3356, 3361
4f3e5de...
pants-reviews
patricklaw
Previously, the Jenkinsfile node info was always being taken from the
Jenkins master.

 Jenkinsfile                                                     |  18 +++++-------------
 build-support/aws/ec2/ci/print_node_info.sh                     |  31 +++++++++++++++++++++++++++++++
 build-support/{ => aws/ec2}/packer/README.md                    |   4 ++--
 build-support/{ => aws/ec2}/packer/images/check-ami-success.png | Bin
 build-support/{ => aws/ec2}/packer/jenkins-slave.customize.sh   |   0
 build-support/{ => aws/ec2}/packer/jenkins-slave.json           |   0
 build-support/{ => aws/ec2}/packer/jenkins-slave.packages.sh    |   0
 7 files changed, 38 insertions(+), 15 deletions(-)
Tested this script on my machine, mac-mini1, jenkins.pantsbuild.org and 1 slave.

CI went green here:
  http://jenkins.pantsbuild.org/job/pantsbuild/job/pants/branch/PR-3361/2/
ST
  1. 
      
  2. build-support/aws/ec2/ci/print_node_info.sh (Diff revision 1)
     
     
     

    Since this supports running in places other than ec2, it might move to a more general location.

  3. 
      
JS
JS
JS
Review request changed

Status: Closed (submitted)

Change Summary:

Now on master:

git log -1 origin/master
commit f1fb836f86857445f8440dd87ae994632cd96cf2
Author: John Sirois <john.sirois@gmail.com>
Date:   Sun May 8 12:37:12 2016 -0600

    Move node info to a script.
    
    Previously, the Jenkinsfile node info was always being taken from the
    Jenkins master.
    
    Testing Done:
    Tested this script on my machine, mac-mini1, jenkins.pantsbuild.org and 1 slave.
    
    CI went green here:
      http://jenkins.pantsbuild.org/job/pantsbuild/job/pants/branch/PR-3361/2/
    
    Bugs closed: 3356, 3361
    
    Reviewed at https://rbcommons.com/s/twitter/r/3842/
Loading...