commit ade27ce41bb1814faf0a2ce6ffd2dc09471068db
author Vadim Shtayura <vadimsh@chromium.org> Thu Aug 29 11:01:43 2013 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat Aug 31 00:08:55 2013 +0000
tree 62c19d9225bd3b9c02cb7e5f5fc80e2604309b00
parent bd39a32381a15b99fde47f4b633f821d2f31f485

Use email in tryjob ref name instead of username.

GoB doesn't have a concept of username so it can't restrict access only
to 'refs/tryjobs/${username}/*' like old Gerrit did.

GoB knows about user email though and at some point it will restrict
access to 'refs/tryjobs/${useremail}/*'.

TEST=ran 'cbuildbot --remote -p chromiumos/chromite --buildbot --debug amd64-generic-paladin' and verified try bot picked up my CL. 

Change-Id: I81588ef99cbb364a8a017bd5a38f658067e83fd1
Reviewed-on: https://chromium-review.googlesource.com/167480
Tested-by: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Chase Phillips <cmp@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: David James <davidjames@chromium.org>

buildbot/remote_try.py

diff --git a/buildbot/remote_try.py b/buildbot/remote_try.py
index e48b2b7..3e42973 100644
--- a/buildbot/remote_try.py
+++ b/buildbot/remote_try.py
@@ -136,7 +136,7 @@
     # TODO(rcui): convert to shallow clone when that's available.
     current_time = str(int(time.time()))
 
-    ref_base = os.path.join('refs/tryjobs', self.user, current_time)
+    ref_base = os.path.join('refs/tryjobs', self.user_email, current_time)
     for patch in self.local_patches:
       # Isolate the name; if it's a tag or a remote, let through.
       # Else if it's a branch, get the full branch name minus refs/heads.