Closed Bug 1316117 Opened 8 years ago Closed 7 years ago

Remove "support" for try: -a by defaulting -b to none in buildbot

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: billm, Assigned: philor)

References

Details

(Keywords: trychooser)

Attachments

(1 file)

I had assumed that "try: -a" meant the same thing as "try: -b do -p all -u all -t <something>". However, it seems that it doesn't actually do anything for TaskCluster. I'm basing this on https://bugzilla.mozilla.org/show_bug.cgi?id=1272159#c8 and on the fact that I was backed out for Linux64 build bustage despite running try -a.

So at this point it seems that "try: -a" just runs a random collection of jobs. I think it should be removed so that other people don't get backed out for this reason. If you use "try: -a", you should get an error telling you to use a different syntax.
See bug 1303663 comment 1 - actual support for -a was removed in 2011, and since then it has been just the defaults for each of -b/-p/-u/-t, the same as "try: garbage". Then, incomprehensibly making different incomprehensible decisions than buildbot's incomprehensible decisions, taskcluster chose to have defaults for -p/-u/-t, but no default for -b.

And an error message is not an option, the only option is to get nothing at all run, ask why, be told that it was a result of using something that hasn't been supported since 2011.
Blocks: 1303663
Keywords: trychooser
Summary: Remove support for try: -a → Remove "support" for try: -a by defaulting -b to none in buildbot
Attached patch FixSplinter Review
Chances are this probably works.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #8816738 - Flags: review?(catlee)
Attachment #8816738 - Flags: review?(catlee) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: