Re: Review Request 16466: CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored

2013-12-26 Thread Min Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16466/#review30880
---

Ship it!


Ship It!

- Min Chen


On Dec. 26, 2013, 10:09 a.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16466/
> ---
> 
> (Updated Dec. 26, 2013, 10:09 a.m.)
> 
> 
> Review request for cloudstack and Min Chen.
> 
> 
> Bugs: CLOUDSTACK-5530
> https://issues.apache.org/jira/browse/CLOUDSTACK-5530
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/StorageManagerImpl.java a96a4ee 
> 
> Diff: https://reviews.apache.org/r/16466/diff/
> 
> 
> Testing
> ---
> 
> Tested 4.3
> 
> 
> Thanks,
> 
> bharat kumar
> 
>



Re: Review Request 16466: CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored

2013-12-26 Thread ASF Subversion and Git Services

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16466/#review30881
---


Commit 69fb1d4ed04156b61ca7993516d6ca02e72d45a2 in branch refs/heads/4.3 from 
Bharat Kumar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=69fb1d4 ]

CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored


- ASF Subversion and Git Services


On Dec. 26, 2013, 10:09 a.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16466/
> ---
> 
> (Updated Dec. 26, 2013, 10:09 a.m.)
> 
> 
> Review request for cloudstack and Min Chen.
> 
> 
> Bugs: CLOUDSTACK-5530
> https://issues.apache.org/jira/browse/CLOUDSTACK-5530
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/StorageManagerImpl.java a96a4ee 
> 
> Diff: https://reviews.apache.org/r/16466/diff/
> 
> 
> Testing
> ---
> 
> Tested 4.3
> 
> 
> Thanks,
> 
> bharat kumar
> 
>



Re: Review Request 16466: CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored

2013-12-26 Thread ASF Subversion and Git Services

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16466/#review30887
---


Commit fead6706ff25f42b6185aa2c41856778f639cc76 in branch refs/heads/master 
from Bharat Kumar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=fead670 ]

CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored


- ASF Subversion and Git Services


On Dec. 26, 2013, 10:09 a.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16466/
> ---
> 
> (Updated Dec. 26, 2013, 10:09 a.m.)
> 
> 
> Review request for cloudstack and Min Chen.
> 
> 
> Bugs: CLOUDSTACK-5530
> https://issues.apache.org/jira/browse/CLOUDSTACK-5530
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-5530 attempt to add secondary storage with the same name is ignored
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/StorageManagerImpl.java a96a4ee 
> 
> Diff: https://reviews.apache.org/r/16466/diff/
> 
> 
> Testing
> ---
> 
> Tested 4.3
> 
> 
> Thanks,
> 
> bharat kumar
> 
>