Last modified: 2014-09-01 07:53:08 UTC
Created attachment 15559 [details] OpenStack's Jenkins Comments [I assume this would be fixed in Zuul/Jenkins not in Gerrit (Zuul/Jenkins is leaving the comment).]
(Gah! Mis-targetted enter key!) Compare the screenshot of OpenStack's Jenkins comments in Gerrit and ours. One is easily readable, the other isn't. Let's have easily readable.
Created attachment 15560 [details] WMF Jenkins Comments
We could do it. Zuul need to prepend the job name in the reported URL which is a single configuration change in zuul.conf: https://review.openstack.org/#/c/42495/2/modules/zuul/templates/zuul.conf.erb Then in Gerrit configuration add a [commentlink] that for a matched regex replace it with some HTML: https://review.openstack.org/#/c/42495/2/modules/openstack_project/manifests/gerrit.pp,unified [commentlink "testresult"] match = '<li>([^ ]+) <a href=\"[^\"]+\">([^<]+)</a> : ([^ ]+)([^<]*)</li>', html = '<li><span class=\"comment_test_name\"><a href=\"$2\">$1</a></span> <span class=\"c omment_test_result\"><span class=\"result_$3\">$3</span>$4</span></li>', We can give it a try on integration-dev.eqiad.wmflabs. I don't have much time to look at it right now though.
Change 154524 had a related patch set uploaded by Hashar: gerrit: prettify Zuul build results https://gerrit.wikimedia.org/r/154524
I gave it a try and the above puppet change tweak Gerrit and Zuul config to make the build result a bit nicer. An example can be shown on a labs instance http://integration.wmflabs.org/gerrit/#/c/12/ (the link will eventually disappear one day).
+1
Change 154524 merged by Dzahn: gerrit: prettify Zuul build results https://gerrit.wikimedia.org/r/154524
Created attachment 16250 [details] Pretty comment on Wikimedia setup Attached is the result of the crazy regex on Wikimedia Gerrit installation. Much nicer.
The original regex comes from James E. Blair (OpenStack). Christian Aistleitner kindly reviewed and further improved the regex (thanks!). Daniel Zahn deployed the change on spot. \O/