[openstack-dev] Gerrit tools
Joshua Harlow
harlowja at yahoo-inc.com
Wed Oct 23 15:16:21 UTC 2013
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" <berrange at 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 at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
More information about the OpenStack-dev
mailing list