Add " (CQ)" to the author name in the console if the commit-queue was used.

buildbot/changes/svnpoller.py

Index: buildbot/changes/svnpoller.py
===================================================================
--- buildbot/changes/svnpoller.py	(revision 113419)
+++ buildbot/changes/svnpoller.py	(working copy)
@@ -202,7 +202,8 @@
 
     def get_logs(self, _):
         args = []
-        args.extend(["log", "--xml", "--verbose", "--non-interactive"])
+        args.extend(["log", "--xml", "--verbose", "--non-interactive",
+            "--with-all-revprops"])

[sadrul, 2012/01/26 17:57:49]

You suggested in the previous CL for this that this should be moved to
scripts/master/. I see that SVNPoller is subclassed in
scripts/master/try_job_svn.py. If I understand this right, I should do something
similar (in chromium_svnpoller.py?). I am not sure what else I should change to
make sure the subclass'ed SVNPoller is used. So for now, I have kept this change
here.

[M-A Ruel, 2012/01/26 18:53:47]

Yes, I want you to do the same thing that what is done with try_job_svn.py. Then
you can use the subclassed class in each master.cfg. This way, we can start with
master.chromium.fyi/master.cfg first and once it's determined to work, we can
enable all the other masters.

So in short;
In master.cfg, instead of "from buildbot.changes import svnpoller", use "from
master import svnpoller"

Then in scripts/master/svnpoller.py:

class SVNPoller(svnpoller.SVNPoller):
  def get_logs(self, _):
    args = ["log", "--xml", "--verbose", "--non-interactive",
"--with-all-revprops"]
(...)
  def create_changes(self, new_logentries):
(...)

         if self.svnuser:
             args.extend(["--username=%s" % self.svnuser])
         if self.svnpasswd:
@@ -289,6 +290,14 @@
     def create_changes(self, new_logentries):
         changes = []
 
+        def commit_bot_used(logentry):
+            revprops = logentry.getElementsByTagName('revprops')
+            if revprops is not None:
+                for revprop in revprops.getElementsByTagName('property'):
+                    if revprop.getAttribute("name") == "commit-bot":
+                        return True
+            return False
+
         for el in new_logentries:
             revision = str(el.getAttribute("revision"))
 
@@ -301,6 +310,7 @@
             log.msg("Adding change revision %s" % (revision,))
             author   = self._get_text(el, "author")
             comments = self._get_text(el, "msg")
+            used_cq  = commit_bot_used(el)
             # there is a "date" field, but it provides localtime in the
             # repository's timezone, whereas we care about buildmaster's
             # localtime (since this will get used to position the boxes on
@@ -346,6 +356,7 @@
                 else:
                     chdict = dict(
                             who=author,
+                            cq=used_cq,
                             files=files,
                             comments=comments,
                             revision=revision,