swarming: Remove support for task without 'pool' dimension

swarming: Remove support for task without 'pool' dimension

Previously, a task with 'id' would still be allowed when authenticating as an
administrator. This change removes this one off.

There is only one exception remaining: terminate task are still allowed to use
'id' only dimensions.

R=vadimsh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2985753003
Committed: https://github.com/luci/luci-py/commit/3e13708cef8e22e52fbeb27935e762078a96917d

[M-A Ruel, 2017-07-24 23:51:43]

I've split out the change to enforce pool as a separate CL because it's a
significant behavior change.

[Vadim Sh., 2017-07-25 00:34:11]

lgtm

[M-A Ruel, 2017-07-25 11:50:52]

On 2017/07/25 00:34:11, Vadim Sh. wrote:
> lgtm

Improved the comments a bit in patchset 2.

[M-A Ruel, 2017-07-25 11:51:00]

The CQ bit was checked by maruel@chromium.org

[M-A Ruel, 2017-07-25 11:51:01]

The patchset sent to the CQ was uploaded after l-g-t-m from vadimsh@chromium.org
Link to the patchset: https://codereview.chromium.org/2985753003/#ps20001
(title: "Tweak comments")

[commit-bot: I haz the power, 2017-07-25 11:51:15]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-25 11:54:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1500983460835300,
"parent_rev": "1ebf96b6f9191e764964a080368b6b85140b3477", "commit_rev":
"3e13708cef8e22e52fbeb27935e762078a96917d"}

[commit-bot: I haz the power, 2017-07-25 11:54:46]

Description was changed from

==========
swarming: Remove support for task without 'pool' dimension

Previously, a task with 'id' would still be allowed when authenticating as an
administrator. This change removes this one off.

There is only one exception remaining: terminate task are still allowed to use
'id' only dimensions.

R=vadimsh@chromium.org
BUG=
==========

to

==========
swarming: Remove support for task without 'pool' dimension

Previously, a task with 'id' would still be allowed when authenticating as an
administrator. This change removes this one off.

There is only one exception remaining: terminate task are still allowed to use
'id' only dimensions.

R=vadimsh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2985753003
Committed:
https://github.com/luci/luci-py/commit/3e13708cef8e22e52fbeb27935e762078a96917d
==========

[commit-bot: I haz the power, 2017-07-25 11:54:47]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/3e13708cef8e22e52fbeb27935e762078a96917d