On Fri, Sep 19, 2014 at 12:30:50PM +0200, Daniel Izquierdo wrote:
> Hi!
> 
> After having a more in depth review of data coming from Gerrit, some other
> issues appear when using the SSH Gerrit API.
> 
> Some of the dates when reviewing the code (Code-Review, Verified, etc) seem
> to be inconsistent with the data found in the comments section.
> 
> Example: issue https://review.openstack.org/#/c/98688/  (Heat project).
> 
> Let's take the PatchSet 44.
> 
> $ ssh -p 29418 review.openstack.org gerrit query 98688 --format=JSON
> --files --comments --patch-sets --all-approvals --commit-message
> --submit-records
> 
> After parsing a bit the JSON file...
> 
> Revision (patchset) 44 (cf7819ccdbf8999fc963e9c8d400c932ad780674)
>   Date: 2014-08-14 03:18:48
>   Code-Review: 1
>     Date: 2014-07-23 07:01:31
>   Code-Review: 1
>     Date: 2014-07-31 11:16:21
>   Code-Review: 1
>     Date: 2014-08-01 10:53:28
>   Code-Review: 2
>     Date: 2014-08-12 20:15:14
>   Code-Review: -1
>     Date: 2014-08-14 10:23:55
>   Workflow: -1
>     Date: 2014-08-18 00:36:32
>   Verified: 1
>     Date: 2014-09-01 12:39:05
> 
> 
> 
> As you can see, the date of some of the reviews are older than the upload
> date of the patchset 44 (Aug 14 3:18).

It looks like Patchset 44 is a trivial update of previous patchsets. When
gerrit detects this, the votes from previous patchsets are copied across
to the new patchset, and the time of those votes is preserved, since they
are not actual new votes. So in that way, you can get votes which are
older than the patchset itself.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to