bot_update: Allow patch_oauth2 to work in kitchen

recipe_modules/bot_update/api.py

Index: recipe_modules/bot_update/api.py
diff --git a/recipe_modules/bot_update/api.py b/recipe_modules/bot_update/api.py
index 8b3a35b0b8486556235c5a0834b87291d5329a39..48127affb79c837565228a0ada95aec672553038 100644
--- a/recipe_modules/bot_update/api.py
+++ b/recipe_modules/bot_update/api.py
@@ -57,12 +57,14 @@ class BotUpdateApi(recipe_api.RecipeApi):
                       patch=True, update_presentation=True,
                       force=False, patch_root=None, no_shallow=False,
                       with_branch_heads=False, refs=None,
-                      patch_oauth2=False,
+                      patch_oauth2=False, use_site_config_creds=True,
                       output_manifest=True, clobber=False,
                       root_solution_revision=None, rietveld=None, issue=None,
                       patchset=None, gerrit_no_reset=False, **kwargs):
     """
     Args:
+      patch_oauth2_in_puppet: If the oauth2 credentials live in puppet, or in

[Vadim Sh., 2016/06/29 04:35:02]

update (or remove, since Args section seems to be missing majority of arguments
anyway)

[martiniss, 2016/06/29 04:42:26]

Updated. Trying to document stuff as we add it.

+        the buildbot site_config. See crbug.com/624212 for more information.
       gclient_config: The gclient configuration to use when running bot_update.
         If omitted, the current gclient configuration is used.
       rietveld: The rietveld server to use. If omitted, will infer from
@@ -119,10 +121,15 @@ class BotUpdateApi(recipe_api.RecipeApi):
     # Point to the oauth2 auth files if specified.
     # These paths are where the bots put their credential files.
     if patch_oauth2:
-      email_file = self.m.path['build'].join(
-          'site_config', '.rietveld_client_email')
-      key_file = self.m.path['build'].join(
-          'site_config', '.rietveld_secret_key')
+      # TODO(martiniss): remove this hack :(. crbug.com/624212
+      if use_site_config_creds:
+        email_file = self.m.path['build'].join(
+            'site_config', '.rietveld_client_email')
+        key_file = self.m.path['build'].join(
+            'site_config', '.rietveld_secret_key')
+      else: #pragma: no cover
+        email_file = '/creds/rietveld/client_email'

[Vadim Sh., 2016/06/29 04:35:02]

this won't work on Windows btw, its C:\Creds\ on windows. 

We can fix this later though. We probably needs m.path['creds'] config or
something like that.

[martiniss, 2016/06/29 04:42:26]

Ok, added TODO for myself :/

+        key_file = '/creds/rietveld/secret_key'
     else:
       email_file = key_file = None