Add links to MP and resolve minor fixes

appengine/swarming/proto/config.proto

 // Copyright 2016 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 syntax = "proto2";
 
 
 // Schema for settings.cfg service config file in luci-config.
 message SettingsCfg {
   // id to inject into pages if applicable.
@@ skipping 20 matching lines @@
   // Emergency setting to disable bot task reaping. When set, all bots are
   // always put to sleep and are never granted task.
   optional bool force_bots_to_sleep_and_not_run_task = 8;
 
   // oauth client id for the ui. This is created in the developer's console
   // under Credentials.
   optional string ui_client_id = 9;
 
   // A mapping "dimension" => "who can posts tasks for it".
   optional DimensionACLs dimension_acls = 10;
+
+  // A partial url which a task id will be appended. The result should be a
+  // valid link to the milo output of a task (if any).
+  optional string milo_server_prefix = 11;
+
+  // A partial url which a Skia git revision will be appended. The result

[M-A Ruel, 2016/10/11 16:26:55]

I think we need a more generic way to define this since this doesn't scale.

One thing I could propose is to parse a tag that would include the link to the
source code itself. It's cool because then one can search for tasks with this
commit hash.

[kjlubick, 2016/10/11 17:00:03]

I agree.  I'll rework this to deal with a tag instead.

[kjlubick, 2016/10/12 13:59:51]

As it turns out, there was a sk_repo tag already being used, so I reworked the
code to use that instead.

+  // should be a valid link to the source code at that revision (if any).
+  optional string sk_revision_prefix = 12;
 }
 
 
 // Configuration for swarming-isolate integration.
 message IsolateSettings {
   // URL of the default isolate server to use if it is not specified in a
   // task. Must start with "https://" or "http://",
   // e.g. "https://isolateserver.appspot.com"
   optional string default_server = 1;
 
@@ skipping 65 matching lines @@
     //
     // This check always work in conjunction with 'swarming-users' group check:
     // in order to post a task that uses dimension "X:Y", the user must be both
     // in 'swarming-users' group and in "X:Y"'s 'usable_by' group.
     optional string usable_by = 2;
   }
 
   // Entries with ACLs for individual dimensions. Order is irrelevant.
   repeated Entry entry = 1;
 }