:glob ✱ Assignee
	Description • 9 years ago

generate_bmo_data.pl should accept the admin email address as an optional argument.

this allows downstream consumers of this script, such as mozreview, to override the admin email address without needing to patch the file during docker deployment.

	:glob ✱ Assignee
	Comment 2 • 9 years ago

this is what mozreview currently has to do:

https://reviewboard-hg.mozilla.org/version-control-tools/file/tip/testing/docker/builder-bmoweb/bmodata.patch
https://reviewboard-hg.mozilla.org/version-control-tools/file/tip/testing/docker/builder-bmoweb/entrypoint.py#l56
https://reviewboard-hg.mozilla.org/version-control-tools/file/tip/testing/docker/builder-bmoweb/entrypoint.py#l82

it breaks any time that neck of the woods of generate_bmo_data.pl is touched.

	David Lawrence [:dkl]
	Comment 3 • 9 years ago

Comment on attachment 8688959 [details] [diff] [review]
1225831_1.patch

Review of attachment 8688959 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl

	:glob ✱ Assignee
	Comment 4 • 9 years ago

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   195a21e..94800e1  master -> master