Add Machine Provider ui parts

Add Machine Provider ui parts

BUG=

Committed: https://github.com/luci/luci-py/commit/26703045663c26ed79bf0ce7e9ed8386002a8074

[kjlubick, 2016-10-07 18:36:06]

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

[kjlubick, 2016-10-07 18:36:07]

https://screenshot.googleplex.com/izHWMLVwfof

Ready for when https://codereview.chromium.org/2402543002/ lands

[jcgregorio, 2016-10-07 19:06:20]

lgtm

[KevinL, 2016-10-07 19:14:44]

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

[KevinL, 2016-10-07 19:14:44]

lgtm

[KevinL, 2016-10-07 19:14:46]

The CQ bit was checked by kjlubick@chromium.org

[commit-bot: I haz the power, 2016-10-07 19:14:53]

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-07 19:18:11]

Description was changed from

==========
Add Machine Provider ui parts

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

to

==========
Add Machine Provider ui parts

BUG=

Committed:
https://github.com/luci/luci-py/commit/26703045663c26ed79bf0ce7e9ed8386002a8074
==========

[commit-bot: I haz the power, 2016-10-07 19:18:12]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-py/commit/26703045663c26ed79bf0ce7e9ed8386002a8074

[smut, 2016-10-07 21:43:07]

smut@google.com changed reviewers:
+ smut@google.com

[smut, 2016-10-07 21:43:08]

https://codereview.chromium.org/2403543002/diff/1/appengine/swarming/elements...
File appengine/swarming/elements/res/imp/botpage/bot-page.html (right):

https://codereview.chromium.org/2403543002/diff/1/appengine/swarming/elements...
appengine/swarming/elements/res/imp/botpage/bot-page.html:237: <td
colspan=2>[[_bot.lease_id]]</td>
Can you linkify this or do I need to include that info in the proto?

Basically the MP URL base needs to be extracted from luci-config (we already do
that and store it in the datastore) and joined with this lease_id to make a URL
to anchor here.

[kjlubick, 2016-10-08 12:27:09]

On 2016/10/07 at 21:43:08, smut wrote:
>
https://codereview.chromium.org/2403543002/diff/1/appengine/swarming/elements...
> File appengine/swarming/elements/res/imp/botpage/bot-page.html (right):
> 
>
https://codereview.chromium.org/2403543002/diff/1/appengine/swarming/elements...
> appengine/swarming/elements/res/imp/botpage/bot-page.html:237: <td
colspan=2>[[_bot.lease_id]]</td>
> Can you linkify this or do I need to include that info in the proto?
> 
> Basically the MP URL base needs to be extracted from luci-config (we already
do that and store it in the datastore) and joined with this lease_id to make a
URL to anchor here.
Yes I can link it, but you bring up a good point.

I have a few other places where I just hardcoded something to be used for links
to get it off the ground.

https://github.com/luci/luci-py/blob/d40718cea5e0337cb4bc4fd242e2690accd56ab7...
https://github.com/luci/luci-py/blob/c3c1c43710c301aab00584dd49a150f56dbfd25c...

Now would be a good time to add both of those things to the config proto (I
found the MP URL you mentioned:
https://github.com/luci/luci-py/blob/d40718cea5e0337cb4bc4fd242e2690accd56ab7...)
and then pipe those through an endpoint, somehow.  

Perhaps in the server details API, as that's already called on every page and
then any page could use the links provided there.