Page MenuHomePhabricator

For new authors on C_Gerrit_Demo, provide a way to access the list of Gerrit patches of each new author
Closed, ResolvedPublic

Description

On https://wikimedia.biterg.io/app/kibana#/dashboard/C_Gerrit_Demo , the C_Gerrit_Demo_Table widget and the C_Gerrit_Demo_Last_Table widget list names of authors.
[Definitely for new contributors, maybe also for lost contributors,] I would like to be able to access a list of each author's contributions, similar to the URL column in the gerrit_backlog_table widget on https://wikimedia.biterg.io/app/kibana#/dashboard/Gerrit-Backlog .
Currently I know that we have new authors, but it's cumbersome to follow up if the contributions of these new authors receive reviews and feedback.

Quick and dirty workaround might be adding a link to each author's Gerrit profile to each list row?
I'm open for better ideas. :)

Event Timeline

Aklapper triaged this task as Low priority.
Aklapper updated the task description. (Show Details)
Aklapper moved this task from Backlog to Need discussion on the wikimedia.biterg.io board.
Aklapper moved this task from Backlog to March on the Developer-Advocacy (Jan-Mar-2018) board.

Ahem, that is way easier to achieve...

Aklapper raised the priority of this task from Low to Medium.Mar 1 2018, 5:07 PM

@srishakatux:
On https://wikimedia.biterg.io/app/kibana#/dashboard/Gerrit-Backlog we already have a widget/visualization called gerrit_backlog_table listing and linking unmerged patchsets in Gerrit.
And on https://wikimedia.biterg.io/app/kibana#/dashboard/Gerrit-Timing we already have a widget/visualization gerrit_all_changesets listing and linking all patchsets in Gerrit.

I now added both those widgets (for testing) on https://wikimedia.biterg.io/app/kibana#/dashboard/C_Gerrit_Demo .
Once you click on the name of an author (to apply a filter view), those new two widgets will list all unmerged patches resp. all patches by that author.
Could that work for us? (If your screen is smaller, you can change the size of a widget by dragging its corner.)

If this turns out to be useful and we want to keep this, I'd need to figure out how to not get my custom change overwritten when upstream deploys the next update.

(Note to myself / admins on wikimedia.biterg.io: To add such widgets to an existing dashboard, go to https://wikimedia.biterg.io/edit/app/kibana#/dashboard/C_Gerrit_Demo (in this case), click "Add" in the upper corner, enter the name of the visualization in the search dialog, select it, and click "Save" to overwrite the default version.)

Aklapper closed this task as Resolved.EditedMar 6 2018, 8:53 AM

"For new authors on C_Gerrit_Demo, provide a way to access the list of Gerrit patches of each new author" is resolved by the addition of the aforementioned widget(s) to the C_Gerrit_Demo dashboard.

Closing task as resolved.

(It could also be nice to have the CR-2/CR-1/CR0/CR+1 status displayed, but that could also be feature creep (we had that info on korma.wmflabs.org until Feb 2017 but we never used it as we had no related workflow), plus perceval/perceval/backends/core/gerrit.py does not support this currently - see https://github.com/chaoss/grimoirelab-perceval/issues/357 .)

Wohooooo <3 This is super awesome!! Yes, this would totally work for us.