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