Add task-list summary

Add task-list summary

This cleans up bot-list-summary as well, given the two 
elements are related.

This removes the restriction server side that the end time 
must be given for a count - as that just makes queries a
bit more awkward to construct. The start time is definitely
required, as counting things from the beginning of time would
time out.

BUG=673774
Review-Url: https://codereview.chromium.org/2676513004
Committed: https://github.com/luci/luci-py/commit/eb1a0447e6c2cbc16c854ff2ea22f424bf8af0e0

[kjlubick, 2017-02-03 13:48:25]

Description was changed from

==========
Add task-list summary

BUG=673774
==========

to

==========
Add task-list summary

This cleans up bot-list-summary as well, given the two 
elements are related.

This removes the restriction server side that the end time 
must be given for a count - as that just makes queries a
bit more awkward to construct. The start time is definitely
required, as counting things from the beginning of time would
time out.

BUG=673774
==========

[kjlubick, 2017-02-03 13:49:17]

kjlubick@google.com changed reviewers:
+ maruel@chromium.org, stephana@chromium.org

[M-A Ruel, 2017-02-03 17:05:22]

python lgtm

[kjlubick, 2017-02-06 18:56:17]

kjlubick@google.com changed reviewers:
+ stephana@google.com
- stephana@chromium.org

[kjlubick, 2017-02-06 18:56:18]

After talking with M-A, I expanded what the summary included:
https://screenshot.googleplex.com/F8vj3EE8eBT

See it live at
https://2602-dacb7a1-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com/task...

[M-A Ruel, 2017-02-06 20:04:33]

https://codereview.chromium.org/2676513004/diff/60001/appengine/swarming/ui/r...
File appengine/swarming/ui/res/imp/tasklist/task-list-summary.html (right):

https://codereview.chromium.org/2676513004/diff/60001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-list-summary.html:242: var last2Days
= now - 2 * 24 * 60 * 60;
While this works great on the staging instance, this will totally time out on
prod. :/

[kjlubick, 2017-02-06 21:18:02]

https://codereview.chromium.org/2676513004/diff/60001/appengine/swarming/ui/r...
File appengine/swarming/ui/res/imp/tasklist/task-list-summary.html (right):

https://codereview.chromium.org/2676513004/diff/60001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-list-summary.html:242: var last2Days
= now - 2 * 24 * 60 * 60;
On 2017/02/06 at 20:04:32, M-A Ruel wrote:
> While this works great on the staging instance, this will totally time out on
prod. :/

Backed it off to 12 hours.  Testing on prod shows the ALL query takes the
longest, at about 15-20 seconds.

[stephana, 2017-02-07 19:26:18]

lgtm modulo a nit

https://codereview.chromium.org/2676513004/diff/100001/appengine/swarming/ui/...
File appengine/swarming/ui/res/imp/tasklist/task-list-summary.html (right):

https://codereview.chromium.org/2676513004/diff/100001/appengine/swarming/ui/...
appengine/swarming/ui/res/imp/tasklist/task-list-summary.html:219: _anyBusy:
function() {
nit: It would be useful if the methods had a short description of what they are
doing. Just a suggestion.

[kjlubick, 2017-02-07 19:31:12]

On 2017/02/07 at 19:26:18, stephana wrote:
> lgtm modulo a nit
> 
>
https://codereview.chromium.org/2676513004/diff/100001/appengine/swarming/ui/...
> File appengine/swarming/ui/res/imp/tasklist/task-list-summary.html (right):
> 
>
https://codereview.chromium.org/2676513004/diff/100001/appengine/swarming/ui/...
> appengine/swarming/ui/res/imp/tasklist/task-list-summary.html:219: _anyBusy:
function() {
> nit: It would be useful if the methods had a short description of what they
are doing. Just a suggestion.

done.

[kjlubick, 2017-02-07 19:31:14]

The CQ bit was checked by kjlubick@google.com

[kjlubick, 2017-02-07 19:31:14]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org,
stephana@google.com
Link to the patchset: https://codereview.chromium.org/2676513004/#ps120001
(title: "Add comments")

[commit-bot: I haz the power, 2017-02-07 19:31:16]

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-02-07 19:33:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-07 19:33:29]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3432e199f44aae10)

[kjlubick, 2017-02-07 19:40:35]

The CQ bit was checked by kjlubick@google.com

[kjlubick, 2017-02-07 19:40:35]

The patchset sent to the CQ was uploaded after l-g-t-m from stephana@google.com,
maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2676513004/#ps140001
(title: "rebase")

[commit-bot: I haz the power, 2017-02-07 19:40:39]

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-02-07 19:44:34]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1486496435133420,
"parent_rev": "7737fed2e2fc56766264e67192bb33f8ed454771", "commit_rev":
"eb1a0447e6c2cbc16c854ff2ea22f424bf8af0e0"}

[commit-bot: I haz the power, 2017-02-07 19:44:42]

Description was changed from

==========
Add task-list summary

This cleans up bot-list-summary as well, given the two 
elements are related.

This removes the restriction server side that the end time 
must be given for a count - as that just makes queries a
bit more awkward to construct. The start time is definitely
required, as counting things from the beginning of time would
time out.

BUG=673774
==========

to

==========
Add task-list summary

This cleans up bot-list-summary as well, given the two 
elements are related.

This removes the restriction server side that the end time 
must be given for a count - as that just makes queries a
bit more awkward to construct. The start time is definitely
required, as counting things from the beginning of time would
time out.

BUG=673774

Review-Url: https://codereview.chromium.org/2676513004
Committed:
https://github.com/luci/luci-py/commit/eb1a0447e6c2cbc16c854ff2ea22f424bf8af0e0
==========

[commit-bot: I haz the power, 2017-02-07 19:44:43]

Committed patchset #8 (id:140001) as
https://github.com/luci/luci-py/commit/eb1a0447e6c2cbc16c854ff2ea22f424bf8af0e0