support multiple builders for tasks stored in distgit. Fixes T925
ClosedPublic
Actions

Authored by tflink on Mar 7 2017, 9:39 PM.

Details

Reviewers

mkrizek
jskladan

Maniphest Tasks

T925: Support multiple builders in the buildbot trigger

Commits

Summary

support sending dist-git stored tasks off to a second builder so that buildbot can more easily

Test Plan

running in stg right now :)

Repository

rTRGR taskotron-trigger

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

tflink retitled this revision from to support multiple builders for tasks stored in distgit. Fixes T925.Mar 7 2017, 9:39 PM

tflink updated this object.

tflink edited the test plan for this revision. (Show Details)

Fine with me

This revision is now accepted and ready to land.Mar 8 2017, 1:58 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			jobtriggers/runners.py (7 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	2954	9fe81f1		Mar 7 2017, 9:39 PM	★	★
Diff 2	2964	f8a6a47	rTRGRfc540e69e6006f656ae4cb1c0f448eb0e5104e57	Mar 9 2017, 9:01 PM	★	★

Commit	Tree	Parents	Author	Summary	Date
f71b37e91c93	c9afc47618fa	9fe81f163474	Tim Flink	support multiple builders for tasks stored in distgit. Fixes T925 (Show More…)	Mar 7 2017, 9:33 PM

Show First 20 Lines • Show All 68 Lines • ▼ Show 20 Line(s)		68	except requests.exceptions.ConnectionError:
69	yield "error: connection to ExecDB failed; %r" % properties	69		yield "error: connection to ExecDB failed; %r" % properties
70		70
71	except RunnerException as e:	71		except RunnerException as e:
72	yield "error: %s; %r" % (e, properties)	72		yield "error: %s; %r" % (e, properties)
73		73
74	else:	74		else:
75	properties['uuid'] = uuid	75		properties['uuid'] = uuid
76		76
		77		if '/' in taskname:
		78		project = 'distgit_checks'
		79		else:
		80		project = 'rpmcheck'
		81
77	r = requests.post(config.buildbot_url,	82		r = requests.post(config.buildbot_url,
78	data={'author': 'taskotron',	83		data={'author': 'taskotron',
79	'project': 'rpmcheck',	84		'project': project,
80	'category': arch,	85		'category': arch,
81	'repository': '',	86		'repository': '',
82	'comments': 'build request from taskotron-trigger',	87		'comments': 'build request from taskotron-trigger',
83	'properties': json.dumps(properties)})	88		'properties': json.dumps(properties)})
84		89
85	yield "post-hook build request status: %s" % str(r.status_code)	90		yield "post-hook build request status: %s" % str(r.status_code)
86		91
87		92
88	_runners = {'StreamRunner': StreamRunner, 'BuildbotRunner': BuildbotRunner}	93		_runners = {'StreamRunner': StreamRunner, 'BuildbotRunner': BuildbotRunner}
89		94
90		95
91	def get_runner(runner_type):	96		def get_runner(runner_type):
92	return _runners[runner_type]()	97		return _runners[runner_type]()