cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Animesh Chaturvedi <animesh.chaturv...@citrix.com>
Subject RE: PLEASE READ: Patches in desperate need of reviews.
Date Fri, 14 Dec 2012 05:34:04 GMT
David

How do we track after  a review is  approved,  that it  is actually committed?

Animesh

> -----Original Message-----
> From: David Nalley [mailto:david@gnsa.us]
> Sent: Thursday, December 13, 2012 8:48 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: PLEASE READ: Patches in desperate need of reviews.
> 
> Folks:
> 
> I know we are all busy and have a ton of things to do. But we have a ton of
> reviews pending. (48) and many of those have had little or no feedback at all.
> The below patches have had no meaningful review, and some are as old as 2
> months. Please make some time in the next few days to review these (or
> other) patches.
> 
> On a slightly related note, there is now a CloudStack authorization group in
> reviewboard and any committer can now file a ticket with INFRA and request
> to be added to that group and gain the ability to close existing submissions.
> I've gone through and closed about a dozen previously submitted patches to
> clean things up a bit.
> 
> https://reviews.apache.org/r/8294/
> Store NetworkServiceProvider connectivity service configuration in database.
> 
> https://reviews.apache.org/r/7571/
> Encode URI during vmware cluster discovery
> 
> https://reviews.apache.org/r/7594/
> Snapshot errors with multiple secondary storage in one zone.
> 
> https://reviews.apache.org/r/7660/
> CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM,
> and they subsequently cannot be detached from a running VM
> 
> https://reviews.apache.org/r/7827/
> CLOUDSTACK-435: Vmware network labels are ignored when creating a Zone
> using basic networking
> 
> https://reviews.apache.org/r/7891/
> CLOUDSTACK-446: Host going to alert state, if you are adding already added
> host
> 
> https://reviews.apache.org/r/8013/
> removed some unused connection properties
> 
> https://reviews.apache.org/r/8037/
> Fix for CLOUDSTACK-284
> 
> https://reviews.apache.org/r/8082/
> Fixes marvin integration test for the copy template/iso between zones
> 
> https://reviews.apache.org/r/8289/
> The already deleted same hostname is not deleted from /etc/hosts of
> vRouter.
> 
> https://reviews.apache.org/r/8293/
> Default admin user account will not be created in clean setup
> 
> https://reviews.apache.org/r/8294/
> Store NetworkServiceProvider connectivity service configuration in database.
> 
> https://reviews.apache.org/r/8432/
> CLOUDSTACK-595 : Recreate root volume scenarios doesn't work in vmware
> 
> https://reviews.apache.org/r/8464/
> [AWSAPI] Property file fix for AWSAPI db(cloudbridge)

Mime
View raw message