On Sun, Oct 20, 2013 at 05:01:23AM +0000, Joshua Harlow wrote:
> I created some gerrit tools that I think others might find useful.
>
> https://github.com/harlowja/gerrit_view
>
> The neat one there is a curses based real time gerrit review receiver
> that uses a similar mechanism as the gerrit irc bot to sit on the
> gerrit event queue and receive events.
Actually, from my POV, the neat one there is the qgerrit script - I had
no idea you could query this info so easily. I've done some work on it
to allow you to filter based on project name, commit message string,
approval flags, and best of all, file path changed. I also improved the
date display to make it clearer how old patches are, which may help
people prioritize reviews for oldest stuff.
With this, I can now finally keep an eye on any change which impacts the
libvirt driver code:
eg to see all code touching 'nova/virt/libvirt', which has not been
-1'd by jenkins
$ qgerrit -f url -f subject:100 -f approvals -f lastUpdated -f createdOn -p
openstack/nova -a v1 nova/virt/libvirt
+------------------------------------+-----------------------------------------------------------------------+----------+----------+---------------------+
| URL | Subject
| Created | Updated | Approvals |
+------------------------------------+-----------------------------------------------------------------------+----------+----------+---------------------+
| https://review.openstack.org/33409 | Adding image multiple location support
| 127 days | 17 hours | v=1 c=-1,1 |
| https://review.openstack.org/35303 | Stop, Rescue, and Delete should give
guest a chance to shutdown | 112 days | 2 hours | v=1,1 c=-1 |
| https://review.openstack.org/35760 | Added monitor (e.g. CPU) to monitor and
collect data | 110 days | 18 hours | v=1,1 c=-1,-1 |
| https://review.openstack.org/39929 | Port to oslo.messaging
| 82 days | 7 hours | v=1,1 |
| https://review.openstack.org/43984 | Call baselineCPU for full feature list
| 56 days | 1 day | v=1,1 c=-1,1,1,1 |
| https://review.openstack.org/44359 | Wait for files to be accessible when
migrating | 54 days | 2 days | v=1 c=1,1,1 |
| https://review.openstack.org/45993 | Remove multipath mapping device
descriptor | 42 days | 4 hours | v=1,1 c=-1
|
| https://review.openstack.org/46055 | Remove dup of LibvirtISCSIVolumeDriver
in LibvirtISERVolumeDriver | 42 days | 18 hours | v=1,1 c=2 |
| https://review.openstack.org/48246 | Disconnect from iSCSI volume sessions
after live migration | 28 days | 5 days | v=1 |
| https://review.openstack.org/48362 | Fixing ephemeral disk creation.
| 27 days | 16 hours | v=1,1 c=2 |
| https://review.openstack.org/49329 | Add unsafe flag to libvirt live
migration call. | 21 days | 6 days | v=1,1
c=-1,-1,1,1,1 |
| https://review.openstack.org/50857 | Apply six for metaclass
| 13 days | 6 hours | v=1,1 |
| https://review.openstack.org/51193 | clean up numeric expressions with byte
constants | 12 days | 9 hours | v=1 |
| https://review.openstack.org/51282 | nova.exception does not have a
ProcessExecutionError | 11 days | 21 hours | v=1,1
|
| https://review.openstack.org/51287 | Remove vim header from from nova/virt
| 11 days | 2 days | v=1,1 c=-1,-1 |
| https://review.openstack.org/51718 | libvirt: Fix spurious backing file
existence check. | 8 days | 5 days | v=1 c=1 |
| https://review.openstack.org/52184 | Reply with a meaningful exception, when
libvirt connection is broken. | 6 days | 16 hours | v=1,1 c=2 |
| https://review.openstack.org/52363 | Remove unnecessary steps for cold
snapshots | 6 days | 45 mins | v=1,1 c=-1
|
| https://review.openstack.org/52401 | make libvirt driver get_connection
thread-safe | 5 days | 3 hours | v=1,1 |
| https://review.openstack.org/52581 | Add context as parameter for resume
| 5 days | 2 days | v=1,1,1 c=1,1 |
| https://review.openstack.org/52777 | Optimize libvirt live migration workflow
at source | 3 days | 1 day | v=1,1 |
| https://review.openstack.org/52807 | Create image again when resize revert a
VM with image type as LVM | 3 days | 3 days | v=1,1 |
| https://review.openstack.org/53069 | Fix lxc rootfs attached two devices in
some action | 1 day | 1 day | v=1,1 |
+------------------------------------+-----------------------------------------------------------------------+----------+----------+---------------------+
> So no longer will u web browser haters (and console lovers) have to refresh
> to see changes. Haha.
>
> Hopefully it works for u guys :)
>
> Pull requests welcome too :)
Incoming... :-)
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
