Skip to content
This repository has been archived by the owner on Dec 15, 2018. It is now read-only.

Commit

Permalink
[changes] Pass diff_cluster when constructing builder
Browse files Browse the repository at this point in the history
Summary: D84153 added ability to use diff_cluster, but we have to pass this from the build step

Test Plan: unit tests

Reviewers: vishal, kylec

Reviewed By: kylec

Subscribers: wwu

Differential Revision: https://tails.corp.dropbox.com/D84245
  • Loading branch information
Akhil Ravidas committed Jan 14, 2015
1 parent 2beb0ef commit 5f55d4b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion changes/backends/jenkins/buildstep.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,10 +102,11 @@ def get_builder_options(self):
class JenkinsGenericBuildStep(JenkinsBuildStep):
builder_cls = JenkinsGenericBuilder

def __init__(self, job_name, script, cluster, path='', workspace='',
def __init__(self, job_name, script, cluster, diff_cluster='', path='', workspace='',
**kwargs):
self.script = script
self.cluster = cluster
self.diff_cluster = diff_cluster
self.path = path
self.workspace = workspace
super(JenkinsGenericBuildStep, self).__init__(job_name=job_name, **kwargs)
Expand All @@ -117,5 +118,6 @@ def get_builder_options(self):
'cluster': self.cluster,
'path': self.path,
'workspace': self.workspace,
'diff_cluster': self.diff_cluster,
})
return options

0 comments on commit 5f55d4b

Please sign in to comment.