Lukas Blakk [:lsblakk] use ?needinfo Assignee
	Description • 14 years ago

Right now we can work around by inputting the 'who' and 'revision' properties, but it would be great to do it from the buildbot page.

	Chris AtLee [:catlee]
	Comment 1 • 14 years ago

Actually we can't work around it right now.  We need to update the code to check for a pre-existing 'who' property for this to work.

	Chris AtLee [:catlee]
	Comment 3 • 14 years ago

So this _does_ work on a standalone master (builders and schedulers in the same instance).  Maybe it's fixed now?  Or maybe unique to running builders separate from schedulers?

	Lukas Blakk [:lsblakk] use ?needinfo Assignee
	Comment 4 • 14 years ago

I'll do some tests to find out.

	Lukas Blakk [:lsblakk] use ?needinfo Assignee
	Comment 5 • 14 years ago

It does work both in the self-serve via db and in the try waterfall on force rebuild now that we don't set who via parse_email, in fact that little function can be removed, and we get who from build.source.changes[0].who

We can resolve this fixed once the parse_email patch lands just for cleanliness.

	Lukas Blakk [:lsblakk] use ?needinfo Assignee
	Comment 6 • 14 years ago

Comment on attachment 511911 [details] [diff] [review]
remove parse_email from process/factory.py

landed: http://hg.mozilla.org/build/buildbotcustom/rev/1946643325e2