Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(370)

Issue 322640043: [plaso] Script used to create Jenkins slaves in GCE. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 6 months ago by romaing
Modified:
6 years, 3 months ago
Reviewers:
onager
CC:
Joachim Metz, jberggren, kiddi, log2timeline-dev_googlegroups.com, aaronp
Visibility:
Public.

Description

[plaso] Script used to create Jenkins slaves in GCE. start_slave.py will be used by the Jenkins master to start new instances (Linux or Windows), replacing the current start-slave.py script. The accompanying bootstrap-vm.ps1 PowerShell script helps install all needed dependencies in a new Windows VM for plaso, as well as the jenkins agent.

Patch Set 1 #

Total comments: 40

Patch Set 2 : Comments #

Patch Set 3 : +pyinstaller #

Total comments: 4

Patch Set 4 : update per comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+453 lines, -0 lines) Patch
A config/jenkins/start_slave.py View 1 2 3 1 chunk +254 lines, -0 lines 0 comments Download
A config/jenkins/windows/bootstrap-vm.ps1 View 1 2 1 chunk +81 lines, -0 lines 0 comments Download
A config/jenkins/windows/bootstrap-vm.ps1.orig View 1 2 1 chunk +118 lines, -0 lines 0 comments Download

Messages

Total messages: 10
onager
https://codereview.appspot.com/322640043/diff/1/config/jenkins/start_slave.py File config/jenkins/start_slave.py (right): https://codereview.appspot.com/322640043/diff/1/config/jenkins/start_slave.py#newcode22 config/jenkins/start_slave.py:22: """Create a new SlaveManager.""" +args https://codereview.appspot.com/322640043/diff/1/config/jenkins/start_slave.py#newcode30 config/jenkins/start_slave.py:30: def _Debug(self, ...
6 years, 6 months ago (2017-09-08 08:36:25 UTC) #1
romaing
https://codereview.appspot.com/322640043/diff/1/config/jenkins/start_slave.py File config/jenkins/start_slave.py (right): https://codereview.appspot.com/322640043/diff/1/config/jenkins/start_slave.py#newcode22 config/jenkins/start_slave.py:22: """Create a new SlaveManager.""" On 2017/09/08 08:36:24, onager wrote: ...
6 years, 6 months ago (2017-09-11 12:06:33 UTC) #2
romaing
Code updated.
6 years, 6 months ago (2017-09-11 12:11:44 UTC) #3
romaing
Code updated.
6 years, 6 months ago (2017-09-15 12:58:28 UTC) #4
onager
On 2017/09/15 12:58:28, romaing wrote: > Code updated. I think we can get read of ...
6 years, 6 months ago (2017-09-21 15:31:06 UTC) #5
onager
https://codereview.appspot.com/322640043/diff/40001/config/jenkins/start_slave.py File config/jenkins/start_slave.py (right): https://codereview.appspot.com/322640043/diff/40001/config/jenkins/start_slave.py#newcode24 config/jenkins/start_slave.py:24: project(str): the GCE project name. + space before the ...
6 years, 6 months ago (2017-09-22 08:37:52 UTC) #6
romaing
Code updated.
6 years, 4 months ago (2017-11-09 16:21:31 UTC) #7
romaing
https://codereview.appspot.com/322640043/diff/40001/config/jenkins/start_slave.py File config/jenkins/start_slave.py (right): https://codereview.appspot.com/322640043/diff/40001/config/jenkins/start_slave.py#newcode24 config/jenkins/start_slave.py:24: project(str): the GCE project name. On 2017/09/22 08:37:51, onager ...
6 years, 4 months ago (2017-11-09 16:21:39 UTC) #8
onager
LGTM
6 years, 4 months ago (2017-11-29 11:10:07 UTC) #9
romaing
6 years, 3 months ago (2017-12-11 14:04:57 UTC) #10
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: review.py close start-jenkins-slaves
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b