Thanks, did some cleanup and pushed a newer version. https://pypi.python.org/pypi/gerrit-view/0.1.1
-Josh On 10/23/13 8:16 AM, "Joshua Harlow" <harlo...@yahoo-inc.com> wrote: >Wow, awesomeness! > >I'll put out a 0.2 on pypi when u are ready with that, very cool. > >Sent from my really tiny device... > >> On Oct 23, 2013, at 7:12 AM, "Daniel P. Berrange" <berra...@redhat.com> >>wrote: >> >>> 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 > >_______________________________________________ >OpenStack-dev mailing list >OpenStack-dev@lists.openstack.org >http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
