ffxbld pushes to central and aurora and beta fail gecko-decision with "HTTPError: 400 Client Error: Bad Request for url: http://taskcluster/queue/v1/task/..."

RESOLVED FIXED in mozilla52

Status

defect
--
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: philor, Assigned: dustin)

Tracking

unspecified
mozilla52

Details

Attachments

(1 attachment)

ffxbld isn't quite unique (my vague memory is that it came up in some bug recently that there are four such Mercurial accounts), but quite rare in that his Mercurial account is exactly that, "ffxbld", not an email address.

He does two sorts of pushes, pushing release tags to mozilla-esr45, mozilla-release, and mozilla-beta, and pushing updates to the blocklist, HSTS and HPKP lists to mozilla-esr45, mozilla-aurora and mozilla-central.

gecko-decision on his pushes to esr45/release/beta still works, but somewhere between July 30th and September 3rd on aurora (https://treeherder.mozilla.org/#/jobs?repo=mozilla-aurora&author=ffxbld&filter-searchStr=8b92d81cffc3f97b03bcb1cb01220b58b731c962&fromchange=fb90485f11a590834aaa80481d61f8a4c11368e2) and between June 4th and August 27th on central (https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&author=ffxbld&filter-searchStr=6751f6b4d53bef7733d3063aa3f72b0832dbde74&fromchange=3e8ee3599a67edd971770af4982ad4b0fe77f073) they started failing, "HTTPError: 400 Client Error: Bad Request for url: http://taskcluster/queue/v1/task/..."

On aurora, that failure apparently comes after dependencies are triggered, so we still run builds and tests on the push, but on central, we don't, so we have a security-related automated push carefully timed to happen when nobody will be around to watch it, pushed directly to mozilla-central, which doesn't run multiple tier-1 builds and their tests.
Summary: ffxbld pushes to central and aurora fail gecko-decision with "HTTPError: 400 Client Error: Bad Request for url: http://taskcluster/queue/v1/task/..." → ffxbld pushes to central and aurora and beta fail gecko-decision with "HTTPError: 400 Client Error: Bad Request for url: http://taskcluster/queue/v1/task/..."
This seems like a pretty critical issue to me.
Severity: normal → critical
Flags: needinfo?(catlee)
Apparently my guess that it was his lack of email address was wrong (or, that's been broken in-tree after Gecko 49), since seabld's pushes to mozilla-release don't burn the gecko-decision task.
[task 2016-10-01T12:55:42.304659Z] * data.metadata.owner should match format "email" 

so, your guess is correct.

I'm assuming it's hard to change that user to have the same form as EVERY OTHER USERNAME.  Jonas, how would you feel about relaxing the format requirement for the owner?  The other option is to check the format of `owner` in the decision task and substitute nobody@mozilla.org if it doesn't match.
Flags: needinfo?(catlee) → needinfo?(jopsen)
Assignee: nobody → dustin
@dustin,
We should just create a dummy email for bots like ffxbld
(I assume ffxbld is a bot).

For github we already do this, and github advices a pattern like:
  <username>@users.noreply.github.com

I'm sure you can come up with a different email address pattern for bots.
We loose a lot of value if we just allow random strings as owner.
Flags: needinfo?(jopsen)
Comment on attachment 8797296 [details]
Bug 1303556: ensure owner is always an email address;

https://reviewboard.mozilla.org/r/82896/#review81784

Looks like a decent solution to me.
Attachment #8797296 - Flags: review?(jopsen) → review+
Pushed by dmitchell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8a271c72525
ensure owner is always an email address; r=jonasfj
https://hg.mozilla.org/mozilla-central/rev/d8a271c72525
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.