Move builder_spec, [dm|nanobench]_flags, builder_name_schema to recipes

infra/bots/recipe_modules/builder_name_schema/api.py

Index: infra/bots/recipe_modules/builder_name_schema/api.py
diff --git a/infra/bots/recipe_modules/builder_name_schema/api.py b/infra/bots/recipe_modules/builder_name_schema/api.py
new file mode 100644
index 0000000000000000000000000000000000000000..09228aa4b70c8508b50c73b6025878c26146db71
--- /dev/null
+++ b/infra/bots/recipe_modules/builder_name_schema/api.py
@@ -0,0 +1,39 @@
+# Copyright 2016 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.
+
+
+# pylint: disable=W0201
+
+
+from recipe_engine import recipe_api
+
+from . import builder_name_schema
+
+
+class BuilderNameSchemaApi(recipe_api.RecipeApi):
+  def __init__(self, *args, **kwargs):
+    super(BuilderNameSchemaApi, self).__init__(*args, **kwargs)
+
+    # See builder_name_schema.py for documentation.
+    self.BUILDER_NAME_SCHEMA = builder_name_schema.BUILDER_NAME_SCHEMA
+    self.BUILDER_NAME_SEP = builder_name_schema.BUILDER_NAME_SEP
+
+    self.BUILDER_ROLE_CANARY = builder_name_schema.BUILDER_ROLE_CANARY
+    self.BUILDER_ROLE_BUILD = builder_name_schema.BUILDER_ROLE_BUILD
+    self.BUILDER_ROLE_HOUSEKEEPER = builder_name_schema.BUILDER_ROLE_HOUSEKEEPER
+    self.BUILDER_ROLE_INFRA = builder_name_schema.BUILDER_ROLE_INFRA
+    self.BUILDER_ROLE_PERF = builder_name_schema.BUILDER_ROLE_PERF
+    self.BUILDER_ROLE_TEST = builder_name_schema.BUILDER_ROLE_TEST
+    self.BUILDER_ROLES = builder_name_schema.BUILDER_ROLES
+
+    self.TRYBOT_NAME_SUFFIX = builder_name_schema.TRYBOT_NAME_SUFFIX

[rmistry, 2016/08/04 17:15:46]

Not sure these variables need to be in CAPS but if it helped minimize diffs then
go ahead and leave as is.

[borenet, 2016/08/04 17:23:40]

I just wanted everything to be pretty much the same.  They're "constants" in
builder_name_schema.py so it makes sense there, and since they can't be accessed
that way through recipes depending on this module, IMO it makes sense to have
them in CAPS here.

+
+  def MakeBuilderName(self, *args, **kwargs):
+    return builder_name_schema.MakeBuilderName(*args, **kwargs)
+
+  def IsTrybot(self, *args, **kwargs):
+    return builder_name_schema.IsTrybot(*args, **kwargs)
+
+  def DictForBuilderName(self, *args, **kwargs):
+    return builder_name_schema.DictForBuilderName(*args, **kwargs)