Re: Review Request: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-03-20 Thread Nitin Mehta
m>>, Bharat Kumar mailto:bharat.ku...@citrix.com>>, "cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>" mailto:cloudstack-dev@incubator.apache.org>> Subject: Re: Review Request: when a template is deleted and then copied over again , it is

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread prasanna
On 20 March 2013 23:56, Vijayendra Bhamidipati wrote: > Hi Chip, Prasanna, > > Yes, the change is pretty straightforward, the reasoning is to make default > password encoding more secure because the SHA256salted authenticator recently > added by Hugo salts the passwords while the existing MD5 au

Re: Review Request: Mulitiple ip address per nic support for advacned shared networks

2013-03-20 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10040/ --- (Updated March 21, 2013, 3:58 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread Chip Childers
asses through all the authenticators available in the UserAuthenticator > >> list until one of them succeeds or all fail. > >> > >> Regards, > >> Vijay > > > > Does everyone believe that this is a valid change for 4.1? Or should we > > wait for

Re: Review Request: CLOUDSTACK-1664 and 1558 Fix ActionEvent interception and optimize DB annotation lookup with caching

2013-03-20 Thread Chip Childers
.org/r/10001/ > --- > > (Updated March 20, 2013, 9:57 p.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > Fix ActionEvent interception and optimize DB annotation lookup with caching > > > This addresses bugs CLOUDSTACK-155

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread David Nalley
gards, >> Vijay > > Does everyone believe that this is a valid change for 4.1? Or should we > wait for 4.2 or 4.1.1? > 4.2 Review request is for master Lets try an minimize change to 4.1 if at all possible. --David

Re: Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread Chip Childers
To reply, visit: > https://reviews.apache.org/r/10046/ > --- > > (Updated March 20, 2013, 6:37 p.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, and Noa Resare. > &g

Re: Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread ASF Subversion and Git Services
nerated e-mail. To reply, visit: > https://reviews.apache.org/r/10046/ > --- > > (Updated March 20, 2013, 6:37 p.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, an

Re: Review Request: BUG-ID: CLOUDSTACK-1713: EC2 REST API: AWS API Installation Problems

2013-03-20 Thread ASF Subversion and Git Services
generated e-mail. To reply, visit: > https://reviews.apache.org/r/10007/ > --- > > (Updated March 19, 2013, 10:36 a.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, an

Re: Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread ASF Subversion and Git Services
nerated e-mail. To reply, visit: > https://reviews.apache.org/r/10047/ > --- > > (Updated March 20, 2013, 6:41 p.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, an

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread Chip Childers
gt; -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Wednesday, March 20, 2013 11:17 AM > To: cloudstack-dev@incubator.apache.org > Cc: Vijayendra Bhamidipati > Subject: Re: Review Request: Make SHA256Salt the default password encoding >

Re: Review Request: CLOUDSTACK-1664 and 1558 Fix ActionEvent interception and optimize DB annotation lookup with caching

2013-03-20 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10001/ --- (Updated March 20, 2013, 9:57 p.m.) Review request for cloudstack and Chip

Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread Pradeep Soundararajan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10047/ --- Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den

Re: Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread Pradeep Soundararajan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10046/ --- (Updated March 20, 2013, 6:37 p.m.) Review request for cloudstack, Chip

RE: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread Vijayendra Bhamidipati
fail. Regards, Vijay -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Wednesday, March 20, 2013 11:17 AM To: cloudstack-dev@incubator.apache.org Cc: Vijayendra Bhamidipati Subject: Re: Review Request: Make SHA256Salt the default password encoding and authentic

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread Chip Childers
, Venkata Siva Vijayendra Bhamidipati > wrote: > > > > --- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/10039/ > > -------

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread prasanna
.org/r/10039/ > ------- > > Review request for cloudstack and Kelven Yang. > > > Description > --- > > Changing default password encoding mechanism from MD5 to SHA256Salted. > > > This addresses bug CS-1734.

Re: Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-20 Thread Animesh Chaturvedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10039/ --- (Updated March 20, 2013, 6 p.m.) Review request for cloudstack, Hugo Trippaers

Review Request: BUG-ID: CLOUDSTACK-1713, CLOUDSTACK-1719

2013-03-20 Thread Pradeep Soundararajan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10046/ --- Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den

Re: Review Request: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-03-20 Thread Animesh Chaturvedi
ews.apache.org/r/7018/ > --- > > (Updated Jan. 13, 2013, 5:14 p.m.) > > > Review request for cloudstack and Nitin Mehta. > > > Description > --- > > This bug fix (CLOUDSTACK-970) deals with the database aspects of template > creation, thi

Re: Review Request: review request for merging GSLB feature into master

2013-03-20 Thread Chip Childers
.org/r/10021/ > --- > > (Updated March 20, 2013, 1:43 a.m.) > > > Review request for cloudstack and Murali Reddy. > > > Description > --- > > Merge request for GSLB feature branch. > > >

Review Request: CLOUDSTACK-1714/CLOUDSTACK-1715: Updates to Install and Admin Guide documentation

2013-03-20 Thread David Grizzanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10045/ --- Review request for cloudstack. Description --- CLOUDSTACK-1714: Doc

Re: Review Request: StatsCollector was not running, fixed it

2013-03-20 Thread Chip Childers
.org/r/10043/ > --- > > (Updated March 20, 2013, 9:43 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > StatsCollector was not running because the corresponding class was not > getting initialized. Adding code for Stat

Re: Review Request: Included VpnUsers in Add state while listing. Added state to Vpn user response

2013-03-20 Thread Chip Childers
ch 20, 2013, 6:21 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > Included VpnUsers in Add state while listing. Added state to Vpn user response > > > This addresses bug CLOUDSTACK-198. > > &g

Re: Review Request: remove duplicated ExpungeTask on 4.0 branch

2013-03-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10042/ --- (Updated March 20, 2013, 3:39 p.m.) Review request for cloudstack and Joe

Re: Review Request: Two test cases from snapshots imporevement feature

2013-03-20 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9752/ --- (Updated March 20, 2013, 1:33 p.m.) Review request for cloudstack and Prasanna

Review Request: StatsCollector was not running, fixed it

2013-03-20 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10043/ --- Review request for cloudstack and Chip Childers. Description

Re: Review Request: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-20 Thread Devdeep Singh
pache.org/r/10017/ > --- > > (Updated March 19, 2013, 5:18 p.m.) > > > Review request for cloudstack, Devdeep Singh, Murali Reddy, Chiradeep Vittal, > and Ram Ganesh. > > > Description > --- > > Bug

Review Request: remove duplicated ExpungeTask on 4.0 branch

2013-03-20 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10042/ --- Review request for cloudstack. Description --- There are two ExpungeTask

Re: Review Request: Included VpnUsers in Add state while listing. Added state to Vpn user response

2013-03-19 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10010/ --- (Updated March 20, 2013, 6:21 a.m.) Review request for cloudstack and Chip

Re: Review Request: CLOUDSTACK-1708: Let cloudmonkey have multiple profiles

2013-03-19 Thread Rohit Yadav
- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10008/ > --- > > (Updated March 19, 2013, 12:33 p.m.) > > > Review request for cloudstack and Chip Childers. > &

Review Request: Mulitiple ip address per nic support for advacned shared networks

2013-03-19 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10040/ --- Review request for cloudstack. Description --- Changes added for

Re: Review Request: review request for merging GSLB feature into master

2013-03-19 Thread Murali Reddy
ady by the caller, where it checks if a >siteobject already exists. May be we can remove in the caller and use a >name like applySite here? > > > >server/src/com/cloud/api/ApiResponseHelper.java ><https://reviews.apache.org/r/10021/#comment38280> > >IMPORTANT: Account, Domain...not set! > >

Review Request: Make SHA256Salt the default password encoding and authentication mechanism for cloudstack

2013-03-19 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10039/ --- Review request for cloudstack and Kelven Yang. Description --- Changing

Review Request: CLOUDSTACK-301: Fix validation of VMware cluster associated with Nexus 1000v

2013-03-19 Thread Sateesh Chodapuneedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10038/ --- Review request for cloudstack, Murali Reddy and Kelven Yang. Description

Re: Review Request: review request for merging GSLB feature into master

2013-03-19 Thread Vijay Venkatachalam
ccount, Domain...not set! - Vijay Venkatachalam On March 20, 2013, 1:43 a.m., Murali Reddy wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10021/ > ------- > > (Updated Ma

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-03-19 Thread Chiradeep Vittal
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9871/ > --- > > (Updated March 15, 2013, 6:51 a.m.) >

Review Request: review request for merging GSLB feature into master

2013-03-19 Thread Murali Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10021/ --- Review request for cloudstack and Murali Reddy. Description --- Merge

Re: Review Request: CLOUDSTACK-1511 and CLOUDSTACK-1446: Instance NIC is missing network name and DomainRouterResponse has null vpcId.

2013-03-19 Thread Chip Childers
pdated March 20, 2013, 12:03 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > This patch fixed two bugs (CLOUDSTACK-1446 and CLOUDSTACK-1511), I combined > them into one patch since they will touch the same schema-40to

Re: Review Request: CLOUDSTACK-1511: Instance NIC details do not have network name.

2013-03-19 Thread ASF Subversion and Git Services
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10030/ > --- > > (Updated March 19, 2013, 10:33 p.m.) > > > Review request for cloudstack and Chip Childers. > >

Re: Review Request: CLOUDSTACK-1511 and CLOUDSTACK-1446: Instance NIC is missing network name and DomainRouterResponse has null vpcId.

2013-03-19 Thread ASF Subversion and Git Services
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10036/ > --- > > (Updated March 20, 2013, 12:03 a.m.) > > > Review request for cloudstack and Chip Childers. > >

Re: Review Request: CLOUDSTACK-1511: Instance NIC details do not have network name.

2013-03-19 Thread ASF Subversion and Git Services
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10030/ > --- > > (Updated March 19, 2013, 10:33 p.m.) > > > Review request for cloudstack and Chip Childers. > >

Re: Review Request: CLOUDSTACK-1511 and CLOUDSTACK-1446: Instance NIC is missing network name and DomainRouterResponse has null vpcId.

2013-03-19 Thread ASF Subversion and Git Services
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10036/ > --- > > (Updated March 20, 2013, 12:03 a.m.) > > > Review request for cloudstack and Chip Childers. > >

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier on master/4.1

2013-03-19 Thread Chip Childers
ted March 19, 2013, 8:25 p.m.) > > > Review request for cloudstack. > > > Description > --- > > Currently, allPossibleIps return the Ip lists which include the gateway, so > we need to remove gateway ip from this list. > > Now, for non-VPC network it w

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier

2013-03-19 Thread Chip Childers
ted March 14, 2013, 8:41 p.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Description > --- > > Currently, allPossibleIps return the Ip lists which include the gateway, so > we need to remove gateway ip from this list. > > Now, for non

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier

2013-03-19 Thread ASF Subversion and Git Services
e.org/r/9923/ > --- > > (Updated March 14, 2013, 8:41 p.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Description > --- > > Currently, allPossibleIps return the Ip lists which include the gateway, so > we need to remo

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier on master/4.1

2013-03-19 Thread ASF Subversion and Git Services
.org/r/10022/ > --- > > (Updated March 19, 2013, 8:25 p.m.) > > > Review request for cloudstack. > > > Description > --- > > Currently, allPossibleIps return the Ip lists which include the gateway, so > we need to remove gateway ip fro

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier on master/4.1

2013-03-19 Thread ASF Subversion and Git Services
-mail. To reply, visit: > https://reviews.apache.org/r/10022/ > --- > > (Updated March 19, 2013, 8:25 p.m.) > > > Review request for cloudstack. > > > Description > --- > > Currently, allPossibleI

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier

2013-03-19 Thread ASF Subversion and Git Services
-mail. To reply, visit: > https://reviews.apache.org/r/9923/ > --- > > (Updated March 14, 2013, 8:41 p.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Description > --- > > Cur

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier

2013-03-19 Thread ASF Subversion and Git Services
To reply, visit: > https://reviews.apache.org/r/9923/ > --- > > (Updated March 14, 2013, 8:41 p.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Description > --- > > Currently,

Re: Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier on master/4.1

2013-03-19 Thread ASF Subversion and Git Services
-mail. To reply, visit: > https://reviews.apache.org/r/10022/ > --- > > (Updated March 19, 2013, 8:25 p.m.) > > > Review request for cloudstack. > > > Description > --- > > Currently, allPossibleIps r

Review Request: CLOUDSTACK-1511 and CLOUDSTACK-1446: Instance NIC is missing network name and DomainRouterResponse has null vpcId.

2013-03-19 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10036/ --- Review request for cloudstack and Chip Childers. Description --- This

Review Request: CLOUDSTACK-1446: VPC Router type should be of type vpc and not system

2013-03-19 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10033/ --- Review request for cloudstack and Chip Childers. Description --- This

Review Request: CLOUDSTACK-1511: Instance NIC details do not have network name.

2013-03-19 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10030/ --- Review request for cloudstack and Chip Childers. Description --- This

Review Request: (CLOUDSTACK-1668) Fix IP conflict in VPC tier on master/4.1

2013-03-19 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10022/ --- Review request for cloudstack. Summary (updated

Re: Review Request: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-19 Thread Chiradeep Vittal
- > > (Updated March 19, 2013, 5:18 p.m.) > > > Review request for cloudstack, Devdeep Singh, Murali Reddy, Chiradeep Vittal, > and Ram Ganesh. > > > Description > --- > > Bug Title: Restart network with clean up set to true causes Autoscaled LB &g

Re: Review Request: Included VpnUsers in Add state while listing. Added state to Vpn user response

2013-03-19 Thread Chip Childers
2:58 p.m., Kishan Kavala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10010/ > --- > > (Updated March 19, 2013, 12:58 p.m.) >

Re: Review Request: CLOUDSTACK-1708: Let cloudmonkey have multiple profiles

2013-03-19 Thread Chip Childers
.org/r/10008/ > --- > > (Updated March 19, 2013, 12:33 p.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > Added command line opt parsing. Add -c that lets cloudmonkey to > use a custom cfg file and not t

Re: Review Request: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-19 Thread Vijay Venkatachalam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10017/ --- (Updated March 19, 2013, 5:18 p.m.) Review request for cloudstack, Devdeep

Re: Review Request: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-19 Thread Vijay Venkatachalam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10017/ --- (Updated March 19, 2013, 5:13 p.m.) Review request for cloudstack, Devdeep

Re: Review Request: Corrected RemoveRegion response. Removed /api in Local region end_point

2013-03-19 Thread Chip Childers
.org/r/10009/ > --- > > (Updated March 19, 2013, 11:38 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > Remove region response corrected to return removeregionresponse instead of > updateregionresponse. &g

Re: Review Request: BUG-ID: CLOUDSTACK-1713: EC2 REST API: AWS API Installation Problems

2013-03-19 Thread Chip Childers
ews.apache.org/r/10007/ > --- > > (Updated March 19, 2013, 10:36 a.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, and Noa Resare. > > > Description > --- > > BUG-ID: CLO

Re: Review Request: BUG-ID: CLOUDSTACK-1713: EC2 REST API: AWS API Installation Problems

2013-03-19 Thread ASF Subversion and Git Services
ply, visit: > https://reviews.apache.org/r/10007/ > --- > > (Updated March 19, 2013, 10:36 a.m.) > > > Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den > Hollander, and Noa Resare. > > >

Review Request: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-19 Thread Vijay Venkatachalam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10017/ --- Review request for cloudstack. Summary (updated) - Fix for

Re: Review Request: CLOUDSTACK-1720: Add upgrade path (db) from 4.0.x to 4.1.0

2013-03-19 Thread Chip Childers
ch 19, 2013, 6:56 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > Testing somehow missed upgrading from 4.0.1 to 4.1.0; it failed for me (see > CLOUDSTACK-1720). Patch adds upgrade path from 4.0.1 and 4.0.2

Re: Review Request: CLOUDSTACK-1720: Add upgrade path (db) from 4.0.x to 4.1.0

2013-03-19 Thread ASF Subversion and Git Services
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10003/ > --- > > (Updated March 19, 2013, 6:56 a.m.) > > > Review r

Re: Review Request: CLOUDSTACK-1664 and 1558 Fix ActionEvent interception and optimize DB annotation lookup with caching

2013-03-19 Thread Chip Childers
; This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10001/ > ----------- > > (Updated March 19, 2013, 5:01 p.m.) > > > Review request for cloudstack and Chip Childers. > > > Des

Re: Review Request: CLOUDSTACK-1664 and 1558 Fix ActionEvent interception and optimize DB annotation lookup with caching

2013-03-19 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10001/ --- (Updated March 19, 2013, 5:01 p.m.) Review request for cloudstack and Chip

Re: Review Request: fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data for master

2013-03-19 Thread Chip Childers
gt; (Updated March 16, 2013, 9:31 a.m.) > > > Review request for cloudstack, Sateesh Chodapuneedi, mice xia, and Kelven > Yang. > > > Description > --- > > fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data > The code works in s

Re: Review Request: fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data

2013-03-19 Thread Chip Childers
gt; (Updated March 16, 2013, 2:37 a.m.) > > > Review request for cloudstack, Sateesh Chodapuneedi, mice xia, and Kelven > Yang. > > > Description > --- > > fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data > The code wo

Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM

2013-03-19 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9908/ --- (Updated March 19, 2013, 2:26 p.m.) Review request for cloudstack, Kishan

Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM

2013-03-19 Thread Kishan Kavala
ews.apache.org/r/9908/ > --- > > (Updated March 15, 2013, 10:10 a.m.) > > > Review request for cloudstack, Kishan Kavala and edison su. > > > Description > --- > > add procedure VPCNetworkUsage and getVPC

Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM

2013-03-19 Thread Kishan Kavala
., Wei Zhou wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9908/ > --- > > (Updated March 15, 2013, 10:10

Re: Review Request: (CLOUDSTACK-1666) Fix NetworkUsage for VPC on KVM

2013-03-19 Thread ASF Subversion and Git Services
ttps://reviews.apache.org/r/9908/ > --- > > (Updated March 15, 2013, 10:10 a.m.) > > > Review request for cloudstack, Kishan Kavala and edison su. > > > Description > --- > > add procedure VPCNetworkUsage

Review Request: Included VpnUsers in Add state while listing. Added state to Vpn user response

2013-03-19 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10010/ --- Review request for cloudstack and Chip Childers. Description --- Included

Re: Review Request: Two test cases from snapshots imporevement feature

2013-03-19 Thread Prasanna Santhanam
--- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9752/ > --- > > (Updated March 19, 2013, 12:45 p.m.) > > > Review request for cloudstack and Prasanna Santhanam.

Re: Review Request: Two test cases from snapshots imporevement feature

2013-03-19 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9752/ --- (Updated March 19, 2013, 12:45 p.m.) Review request for cloudstack and Prasanna

Re: Review Request: CLOUDSTACK-1708: Let cloudmonkey have multiple profiles

2013-03-19 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10008/ --- (Updated March 19, 2013, 12:33 p.m.) Review request for cloudstack and Chip

RE: [Review Request] Standalone NIC

2013-03-19 Thread Simon Waterhouse
Thanks Prasanna - that would be a great help! -Original Message- From: prasanna [mailto:srivatsav.prasa...@gmail.com] On Behalf Of Prasanna Santhanam Sent: 19 March 2013 12:03 To: cloudstack-dev@incubator.apache.org Cc: Jayapal Reddy Uradi; Chiradeep Vittal Subject: Re: [Review Request

Re: [Review Request] Standalone NIC

2013-03-19 Thread Prasanna Santhanam
On Tue, Mar 19, 2013 at 11:56:09AM +, Simon Waterhouse wrote: > Hi, > I am working on the standalone NIC feature described in > https://cwiki.apache.org/confluence/display/CLOUDSTACK/AWS+Style+NIC+support > > The work is staged in > https://github.com/siwater/incubator-cloudstack (branch > sta

[Review Request] Standalone NIC

2013-03-19 Thread Simon Waterhouse
Hi, I am working on the standalone NIC feature described in https://cwiki.apache.org/confluence/display/CLOUDSTACK/AWS+Style+NIC+support The work is staged in https://github.com/siwater/incubator-cloudstack (branch standalone-nic) and the phase 1 development (see feature spec/appendix) is ready

Review Request: Corrected RemoveRegion response. Removed /api in Local region end_point

2013-03-19 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10009/ --- Review request for cloudstack and Chip Childers. Description --- Remove

Review Request: CLOUDSTACK-1708: Let cloudmonkey have multiple profiles

2013-03-19 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10008/ --- Review request for cloudstack and Chip Childers. Description --- Added

Re: Review Request: Feature Syslog Enhancements

2013-03-19 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9870/ --- (Updated March 19, 2013, 10:52 a.m.) Review request for cloudstack, Rajesh

Re: Review Request: Dashboard showing wrong alert type titles for alerts

2013-03-19 Thread Pranav Saxena
.org/r/10006/ > --- > > (Updated March 19, 2013, 9:24 a.m.) > > > Review request for cloudstack and Pranav Saxena. > > > Description > --- > > https://issues.apache.org/jira/browse/CLOUDSTACK-1723 > > In UI, in gener

Review Request: BUG-ID: CLOUDSTACK-1713: EC2 REST API: AWS API Installation Problems

2013-03-19 Thread Pradeep Soundararajan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10007/ --- Review request for cloudstack, Chip Childers, Hugo Trippaers, Wido den

Review Request: Dashboard showing wrong alert type titles for alerts

2013-03-19 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10006/ --- Review request for cloudstack and Pranav Saxena. Description --- https

Re: Review Request: CLOUDSTACK-1647: IP Reservation should not happen if the guest-vm cidr and network cidr is not same but their start ip and end ip are same.

2013-03-19 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10005/ --- (Updated March 19, 2013, 9:07 a.m.) Review request for cloudstack, Murali

Review Request: CLOUDSTACK-1647: IP Reservation should not happen if the guest-vm cidr and network cidr is not same but their start ip and end ip are same.

2013-03-19 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10005/ --- Review request for cloudstack, Murali Reddy and Sateesh Chodapuneedi

Review Request: CLOUDSTACK-1720: Add upgrade path (db) from 4.0.x to 4.1.0

2013-03-18 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10003/ --- Review request for cloudstack and Chip Childers. Description --- Testing

Re: Review Request: Feature Syslog Enhancements

2013-03-18 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9870/ --- (Updated March 19, 2013, 6:23 a.m.) Review request for cloudstack, Rajesh

Re: Review Request: MidoNet Networking Plugin [2/2]

2013-03-18 Thread joe mills
iew17849 --- On March 13, 2013, 9:32 a.m., Dave Cahill wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9898/ > -

Re: Review Request: CLOUDSTACK-1690: NPE from API server when starting mgmt server.

2013-03-18 Thread ASF Subversion and Git Services
- > > (Updated March 15, 2013, 11:36 p.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > --- > > This patch fixed the blocker bug CLOUDSTACK-1690, this bug is caused by > recent region commit b41e6e929d5f4f9

Re: Review Request: (CLOUDSTACK-1475) update keystore in SSVM and change download iso/template url after Update SSL Certificate

2013-03-18 Thread Wei Zhou
https://reviews.apache.org/r/9696/#review17961 --- On March 15, 2013, 9:54 a.m., Wei Zhou wrote: > > ------- > This is an automatically generated e-mail. To reply, visit: > htt

Re: Review Request: CLOUDSTACK-1201: fix for failed to create ssh key for cloud

2013-03-18 Thread ASF Subversion and Git Services
//reviews.apache.org/r/9424/ > --- > > (Updated Feb. 13, 2013, 1:06 p.m.) > > > Review request for cloudstack and Hugo Trippaers. > > > Description > --- > >packaging: rpm setting the homedir of user c

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts

2013-03-17 Thread Nitin Mehta
wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9541/ > --- > > (Updated March 13, 2013, 5:44 a.m.) > > > Review request for cloudstack, Devdeep Singh, Nitin

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts

2013-03-17 Thread Nitin Mehta
eviews.apache.org/r/9541/ > --- > > (Updated March 13, 2013, 5:44 a.m.) > > > Review request for cloudstack, Devdeep Singh, Nitin Mehta, Sateesh > Chodapuneedi, mice xia, and Min Chen. > > > Description > --- >

Re: Review Request: fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data for master

2013-03-16 Thread Hongtu Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9989/ --- (Updated March 16, 2013, 9:31 a.m.) Review request for cloudstack, Sateesh

Re: Review Request: MidoNet Networking Plugin [2/2]

2013-03-16 Thread Hiroaki Kawai
- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9898/ > ----------- > > (Updated March 13, 2013, 9:32 a.m.) > > > Review request

Re: Review Request: fix bug CLOUDSTACK-1531: vmware create volume from snapshot will missing data

2013-03-15 Thread Hongtu Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9772/ --- (Updated March 16, 2013, 2:37 a.m.) Review request for cloudstack, Sateesh

  1   2   3   4   5   6   7   8   9   10   >