Add bot-page summary with utilization stats

Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=631047
Committed: https://github.com/luci/luci-py/commit/5808042b94ab28b513bc3e1e7cab0a1c7b74881f

[kjlubick, 2016-09-30 19:29:20]

Description was changed from

==========
Add bot-page summary with utilization stats

split into two


add task stats


Add bulk stats


hang on


stub

BUG=
==========

to

==========
Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=
==========

[kjlubick, 2016-09-30 20:23:59]

Description was changed from

==========
Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=
==========

to

==========
Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=631047
==========

[kjlubick, 2016-09-30 20:23:59]

kjlubick@google.com changed reviewers:
+ jcgregorio@google.com, stephana@google.com

[kjlubick, 2016-09-30 20:25:28]

Live on 
https://2365-bd022ad-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com/newu...

Better mock data at
https://goto.google.com/exfpx

https://screenshot.googleplex.com/JKUOkZAVf5B

If there are a lot of individual tasks, we only show the first 15 by default:
https://screenshot.googleplex.com/Ovex3B3hNEH

This raises the question, should I show the totals for all that we have fetched
(as I do now) or just the ones that are shown?

[jcgregorio, 2016-10-03 15:15:28]

https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
File appengine/swarming/elements/third_party/common-strings.js (right):

https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
appengine/swarming/elements/third_party/common-strings.js:1: (function(){
What is the license for these libraries?

Why is this checked in? Why not do the cat as part of the build process?

[kjlubick, 2016-10-03 15:21:32]

On 2016/10/03 at 15:15:28, jcgregorio wrote:
>
https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
> File appengine/swarming/elements/third_party/common-strings.js (right):
> 
>
https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
> appengine/swarming/elements/third_party/common-strings.js:1: (function(){
> What is the license for these libraries?
> 
> Why is this checked in? Why not do the cat as part of the build process?

License is MIT.

The assembly isn't a straight-forward cat.  The libraries were originally made
to be loaded with require-js, which I would rather not use. 
A few lines of code needed to be deleted, e.g.

    module.exports=SuffixTrie

because module (defined by requirejs) isn't defined.

[kjlubick, 2016-10-03 17:59:10]

On 2016/10/03 at 15:21:32, kjlubick wrote:
> On 2016/10/03 at 15:15:28, jcgregorio wrote:
> >
https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
> > File appengine/swarming/elements/third_party/common-strings.js (right):
> > 
> >
https://codereview.chromium.org/2381853003/diff/80001/appengine/swarming/elem...
> > appengine/swarming/elements/third_party/common-strings.js:1: (function(){
> > What is the license for these libraries?
> > 
> > Why is this checked in? Why not do the cat as part of the build process?
> 
> License is MIT.
> 
> The assembly isn't a straight-forward cat.  The libraries were originally made
to be loaded with require-js, which I would rather not use. 
> A few lines of code needed to be deleted, e.g.
> 
>     module.exports=SuffixTrie
> 
> because module (defined by requirejs) isn't defined.

Margin fixed as well as the building of the common-strings.js

[jcgregorio, 2016-10-03 20:12:20]

lgtm

https://codereview.chromium.org/2381853003/diff/140001/appengine/swarming/ele...
File appengine/swarming/elements/res/imp/botpage/bot-page-summary.html (right):

https://codereview.chromium.org/2381853003/diff/140001/appengine/swarming/ele...
appengine/swarming/elements/res/imp/botpage/bot-page-summary.html:37:
paper-checkbox {
blank line between styles.

[kjlubick, 2016-10-03 20:14:44]

https://codereview.chromium.org/2381853003/diff/140001/appengine/swarming/ele...
File appengine/swarming/elements/res/imp/botpage/bot-page-summary.html (right):

https://codereview.chromium.org/2381853003/diff/140001/appengine/swarming/ele...
appengine/swarming/elements/res/imp/botpage/bot-page-summary.html:37:
paper-checkbox {
On 2016/10/03 at 20:12:20, jcgregorio wrote:
> blank line between styles.

Done.

[KevinL, 2016-10-03 20:15:46]

kjlubick@chromium.org changed reviewers:
+ kjlubick@chromium.org

[KevinL, 2016-10-03 20:15:47]

lgtm

[KevinL, 2016-10-03 20:15:48]

The CQ bit was checked by kjlubick@chromium.org

[KevinL, 2016-10-03 20:15:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
jcgregorio@google.com
Link to the patchset: https://codereview.chromium.org/2381853003/#ps160001
(title: "address spaces")

[commit-bot: I haz the power, 2016-10-03 20:16:00]

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, 2016-10-03 20:19:37]

Description was changed from

==========
Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=631047
==========

to

==========
Add bot-page summary with utilization stats

You can ignore most of the bot-page changes - they are
mostly whitespace changes.

BUG=631047

Committed:
https://github.com/luci/luci-py/commit/5808042b94ab28b513bc3e1e7cab0a1c7b74881f
==========

[commit-bot: I haz the power, 2016-10-03 20:19:38]

Committed patchset #9 (id:160001) as
https://github.com/luci/luci-py/commit/5808042b94ab28b513bc3e1e7cab0a1c7b74881f