convert most webrtc builders to remote_run

masters/master.client.webrtc/master_mac_cfg.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from buildbot.schedulers.basic import SingleBranchScheduler
 
-from master.factory import annotator_factory
+from master.factory import remote_run_factory
 
-m_annotator = annotator_factory.AnnotatorFactory()
+import master_site_config
+ActiveMaster = master_site_config.WebRTC
+
+
+def m_remote_run(recipe, **kwargs):
+  return remote_run_factory.RemoteRunFactory(
+      active_master=ActiveMaster,
+      repository='https://chromium.googlesource.com/chromium/tools/build.git',

[kjellander_chromium, 2016/08/09 14:16:21]

This URL is duplicated a lot. Is that a good long-term solution for all masters?

[Paweł Hajdan Jr., 2016/08/09 15:04:52]

The long term solution is to get rid of buildbot.

Current way master configs are done leads to a lot of duplication like this.

+      recipe=recipe,
+      factory_properties={'path_config': 'kitchen'},
+      **kwargs)
+
 
 def Update(c):
   c['schedulers'].extend([
       SingleBranchScheduler(name='webrtc_mac_scheduler',
                             branch='master',
                             treeStableTimer=30,
                             builderNames=[
           'Mac64 Debug',
           'Mac64 Release',
           'Mac64 Release [large tests]',
@@ skipping 68 matching lines @@
     {
       'name': 'iOS API Framework Builder',
       'slavebuilddir': 'mac64',
       'recipe': 'webrtc/ios_api_framework',
     },
   ]
 
   c['builders'].extend([
       {
         'name': spec['name'],
-        'factory': m_annotator.BaseFactory(spec.get('recipe',
-                                                    'webrtc/standalone')),
+        'factory': m_remote_run(spec.get('recipe', 'webrtc/standalone')),
         'notify_on_missing': True,
         'category': spec.get('category', 'compile|testers'),
         'slavebuilddir': spec['slavebuilddir'],
       } for spec in specs
   ])