David How do we track after a review is approved, that it is actually committed?
Animesh > -----Original Message----- > From: David Nalley [mailto:[email protected]] > Sent: Thursday, December 13, 2012 8:48 PM > To: [email protected] > 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)
