Page History

This page lists some useful Gerrit searches;.

Andreas Dilger: find open patches affecting a particular file (lustre_idl.h in this case):

https://review.whamcloud.com/#/q/project:fs/lustre-release+file:lustre_idl.h

Andreas Dilger: This query will return all find Lustre patches on master which have requested landing by the Gerrit Gatekeeperare (almost) ready to land, but are not yet merged, and have not changed in the last three daysseven days.  Note that this is the minimum requirement, there is actually a need for two reviewers for each patch but this cannot be expressed in the Gerrit query syntax.  However, this also provides an opportunity to find patches that are almost ready and can benefit from an additional reviewer:

http://review.whamcloud.com/#/#q,q/is:open+project:fs/lustre-release+Verified>=1+-Verified<=-1+CodeReview>Code-Review>=1+-CodeReview<Code-Review<=-1+branch:master+reviewer:gerrit+age:3d,n,z7d

Peter Jones: find Andreas Dilger: This query will return all patches on master which are ready for landing, but have not requested landing by the Gerrit gatekeeper:

http://review.whamcloud.com/#q,#/q/is:open+project:fs/lustre-release+branch:master+Code-Review>=1+Verified>=1+-Verified<=-1+CodeReview>=1+-CodeReview<-Code-Review<=-1+branch:master+-reviewer:gerrit,n,z

Chris Gearing: This query will return all find patches that are awaiting some action on them. ie. they need reviewing, building or testing, and have not changed in the last 7 days:

http://review.whamcloud.com/#/q/#q,(-verified<=-1+OR+(verified>=1+AND+-reviewer:maloo))+AND+status:open+AND+project:%255E.*lustre.*\fs/lustre-release+AND+age:7d,n,z

Chris Gearing: This query will return all the reviews find patches that did not fully complete the review, test, build process before being merged. It's not fool-proof, merges can slip through the net but we'll work on that:

http://review.whamcloud.com/#q,(project:fs/lustre-dev+OR+#/q/project:fs/lustre-release)+AND+(-verified<=-1+OR+(verified>=1+AND+-reviewer:maloo))+AND+status:merged,n,z

Andreas Dilger: find Lustre patches that you have not inspected yet:

http://review.whamcloud.com/#/q/is:open+reviewer:self+-owner:self+-label:Code-Review>=1,user=self+-label:Code-Review<=-1,user=self+label:verified

Charlie Olmstead: Find patches that have been successfully built but have not received a result from MalooPeter's Query 

http://review.whamcloud.com/#q,status:open#/q/(NOT+label:Verified%253D1%252Cuser%253DMaloo+AND+NOT+label:Verified%253D-1%252Cuser%253DMaloo+AND+project:fs/lustre-release+branchAND+label:masterVerified%253D1%252Cuser%253DJenkins+CodeReview%252B1AND+Verified%252B1is:open+-AND+Verified-1+-+CodeReview-1+-CodeReview-2,n,zNOT+comment:fortestonly+AND+-age:1week)