<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 20, 2015 at 12:09 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On 10/20/2015 12:43 PM, Dolph Mathews wrote:<br>
> This is actually something I've thought a lot about (focusing the<br>
> community's review efforts), and have experimented with various<br>
> solutions in the keystone community. I've built external solutions that<br>
> have worked fairly well, but my current preference is to take advantage<br>
> of what's already built into gerrit: starred reviews and dashboards.<br>
><br>
> For example, here is a dashboard of reviews in the keystone ecosystem<br>
> starred by any member of keystone-core that *you* have not reviewed<br>
> yourself (so you'll need to be logged into gerrit for this link to work):<br>
><br>
>   <a href="http://bit.ly/1GnOuqw" rel="noreferrer" target="_blank">http://bit.ly/1GnOuqw</a> [1]<br>
><br>
> In other words, it's a personalized review queue of things keystone-core<br>
> deems important.<br>
><br>
> The advantage I see to this approach over a new label is that the star<br>
> feature is already in the gerrit UI and so people already use it. But<br>
> broadly speaking, I'm not aware of anyone utilizing that data today as a<br>
> crowd sourced data point.<br>
<br>
</span>The rally team uses it as part of their dashboard -<br>
<a href="https://github.com/stackforge/gerrit-dash-creator/blob/4d597f3b9c62f4422e3f8cdcd3b9c96921faa155/dashboards/rally.dash#L6-L7" rel="noreferrer" target="_blank">https://github.com/stackforge/gerrit-dash-creator/blob/4d597f3b9c62f4422e3f8cdcd3b9c96921faa155/dashboards/rally.dash#L6-L7</a><br>
<span class=""><br>
> If you'd like to create your own version of this dashboard, here's the<br>
> gerrit-dash-creator config file for this dashboard:<br>
><br>
>   <a href="https://github.com/dolph/dotfiles/blob/master/gerrit-dashboard-keystone" rel="noreferrer" target="_blank">https://github.com/dolph/dotfiles/blob/master/gerrit-dashboard-keystone</a><br>
><br>
> To customize it for yourself, you'd pretty much only need to edit the<br>
> [dashboard] section, and specifically the "foreach" value to reflect the<br>
> right collection of projects and group of reviewers. After that it's a<br>
> matter of using gerrit-dash-creator to produce the link:<br>
><br>
>   <a href="https://github.com/openstack/gerrit-dash-creator" rel="noreferrer" target="_blank">https://github.com/openstack/gerrit-dash-creator</a><br>
><br>
> I'd love to take this a couple steps further if people find the approach<br>
> valuable. I'd like to regularly recreate these links for every project<br>
> based on the latest *-core membership, and the latest set of projects in<br>
> a given community, publish them to permalinks, and share those<br>
> permalinks with code reviewers.<br>
<br>
</span>One of the challenges you run into is url length limits in browsers,<br>
given that we're encoding the whole thing in the url.<br></blockquote><div><br></div><div>I haven't run into any issues yet myself, but yes, definitely. One of the easiest changes I'd make to "compress" the URL is to use user IDs instead of emails like I did here, e.g.:</div><div><br></div><div>  <a href="https://review.openstack.org/#/q/status:open+starredby:4,n,z">https://review.openstack.org/#/q/status:open+starredby:4,n,z</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
I do think the overall approach has a lot of merrit, though it would<br>
still be awesome if there was a more baked in tagging in gerrit.<br>
<span class=""><font color="#888888"><br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</a><br>
</font></span><div class=""><div class="h5"><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>