Build failed in Jenkins: build-master-slowbuild #391

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Build failed in Jenkins: build-master-slowbuild #392

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Build failed in Jenkins: build-master-slowbuild #393

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Jenkins build is back to normal : build-systemvm64-4.2 #196

[jenkins]

See 

Build failed in Jenkins: build-master-slowbuild #394

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Re: CLOUDSTACK-6096

[Tanner Danzey]

Hey! I'm extremely sorry, it looks like my Gmail filter settings buried
your message. No, I haven't been able to test because of some issues with
S3 and snapshots that made us go back to 4.2.1 for answers. Sorry sorry
sorry!


On Mon, Feb 17, 2014 at 9:25 AM, Francois Gaudreault <
fgaudrea...@cloudops.com> wrote:

> Tanner,
>
> Just curious, have you been able to test? If not let me know, Ill try to
> get some time to spin up an environment real quick at my office.
>
> Thanks again!
>
> FG
>
>
> On 2/13/2014, 8:57 AM, Francois Gaudreault wrote:
>
>> Thanks Tanner :) Keep us posted please :)
>>
>> FG
>>
>> On 2/13/2014, 8:54 AM, Tanner Danzey wrote:
>>
>>> Hi Francois,
>>>
>>> I can potentially test this tonight for you. We're deploying 4.3 to see
>>> if
>>> it alleviates some of our issues with S3, but thats a few hours from now
>>> in
>>> case someine wants to test it sooner.
>>>
>>> Regards,
>>>
>>> Tanner
>>> On Feb 13, 2014 7:50 AM, "Francois Gaudreault" >> >
>>> wrote:
>>>
>>>  Hi,

 https://issues.apache.org/jira/browse/CLOUDSTACK-6096
 I filed this today. This impact 4.2.1, but I would like to know if it
 also
 impact 4.3. Does anyone have an environment with 4.3 that would be
 able/willing to test?

 Thanks!

 --
 Francois Gaudreault
 Architecte de Solution Cloud | Cloud Solutions Architect
 fgaudrea...@cloudops.com
 514-629-6775
 - - -
 CloudOps
 420 rue Guy
 Montréal QC  H3J 1S6
 www.cloudops.com
 @CloudOps_



>>
>>
>
> --
> Francois Gaudreault
> Architecte de Solution Cloud | Cloud Solutions Architect
> fgaudrea...@cloudops.com
> 514-629-6775
> - - -
> CloudOps
> 420 rue Guy
> Montréal QC  H3J 1S6
> www.cloudops.com
> @CloudOps_
>
>


-- 
*Tanner Danzey*
Systems Engineer
Northstar Technology Group
arkan...@gmail.com / tanner.dan...@northstar-tg.com
(701) 237-9096 x7122

Build failed in Jenkins: build-master-slowbuild #395

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Re: Review Request 18759: HTTP support for console proxy and making it default

[John Kinsella]

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

Ship it!


Ship It!

- John Kinsella


On March 7, 2014, 12:32 a.m., Amogh Vasekar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18759/
> ---
> 
> (Updated March 7, 2014, 12:32 a.m.)
> 
> 
> Review request for cloudstack, Demetrius Tsitrelis and John Kinsella.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6204
> 
> https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6204
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Changes to support HTTP mode in Console Proxy, per 
> http://www.mail-archive.com/dev@cloudstack.apache.org/msg24151.html , as 
> realhostip may go away
> 
> More details here : 
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Realhost+IP+changes
> 
> 
> Diffs
> -
> 
>   core/src/com/cloud/info/ConsoleProxyInfo.java 
> 3439f3d3bfa2e262c48f1d7b1ea4f58522f3fcbe 
>   
> engine/storage/image/src/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
>  a649bb7212308de70c41e2d74de1d865949f1cb7 
>   
> plugins/storage/image/default/src/org/apache/cloudstack/storage/datastore/driver/CloudStackImageStoreDriverImpl.java
>  52cad3bc7af291e59eabc68e23e09248877e0a81 
>   server/src/com/cloud/configuration/Config.java 
> 473db96059a4367858f9487d901b7cb3a054654a 
>   server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 
> c8769d43349dbc5a3103a00c905f29b7edef0468 
>   server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 
> 611550e90832911fb182ad18d93a8a18333f3a35 
>   setup/db/db/schema-421to430.sql 39f58d43822ebbe469d9af433582846a80ce91a0 
>   systemvm/conf/consoleproxy.properties 
> bb452f5823cb2da2e12aa61d762de90e4349e9ee 
> 
> Diff: https://reviews.apache.org/r/18759/diff/
> 
> 
> Testing
> ---
> 
> Tested on local environment by 
> 1. Using HTTP based console proxy
> 2. HTTPS with realhostip domain
> 3. HTTPS with custom domain and self-signed cert
> 4. Secondary storage template download with custom domain and self-signed cert
> 
> 
> Thanks,
> 
> Amogh Vasekar
> 
>

[4.3][Cherry-pick] realhostip changes

[John Kinsella]

Animesh - please pick the commit below from 4.3-forward into 4.3. This is for 
CLOUDSTACK-6204.

2fe7aeea23ddef25224e3e248f0a91513a14811f

John

Re: [DISCUSS] realhostip.com going away

[John Kinsella]

Folks - just applied Amogh’s patch to 4.3-forward, and back ported that to 
master.

Two steps left on the code side:
 * Need to get this retirement into the 4.3 docs
 * Need to backport this to 4.2

John

On Feb 28, 2014, at 12:27 PM, John Kinsella 
mailto:j...@stratosec.co>> wrote:

Folks: Recently the PMC was informed that the 
realhostip.com DNS service that ACS currently uses by 
default as part of the console proxy will be disbanded this summer.

We’ve been informed the realhostip service will be shut down June 30th, 2014, 
so we have approximately 4 months to mitigate this.

Here’s my thoughts on how to proceed, in order of priority:

* Make the transition as smooth as possible for current ACS users. Need to 
create clear documentation in the wiki that we can point to on how to migrate 
an existing ACS installation from using realhostip.com 
to a user’s own certificate and resolver. I see section 16.4.2 in the 4.2 admin 
guide talks about this, but I think we can improve a bit. e.g. the current docs 
don’t make it clear that a wildcard cert is required. Once we have a transition 
guide in place, I intend to announce to users@ and announce@ along with the 
social media paths. This isn’t private, but I’d rather not announce until we 
have a clear, tested easy to follow transition guide to make this as painless 
as possible for folks. I’m working on this and will update after testing.
* If at all possible, I’d really like to get something big and visible into the 
4.3 documentation warning users about this.
* For 4.4, we should no longer be using SSL/realhostip for console proxy. We’re 
expecting some patches to address this, I’ll update this thread once they hit 
and/or a Jira issue is created.

Open to any thoughts/suggestions.

John

Stratosec - Compliance as a Service
o: 415.315.9385
@johnlkinsella

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

[Alex Ough]

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

(Updated March 9, 2014, 10:50 p.m.)


Review request for cloudstack.


Changes
---

Added a new feature that was requested internally, which is to provide an 
option to assign the same uuid when creating the same resources in other 
regions.
There is a new global parameter, 'multi.region.resource.uuid.match' whose 
default value is false.
If set to true, the uuid of the target resource will be assigned when the same 
resource is created in other regions.
If set to false, each region will use different uuids for same resources. 
Default value is false.


Repository: cloudstack-git


Description
---

Currently, under the environment of cloudstack with multiple regions, each 
region has its own management server running with a separate database, which 
will cause data discrepancies when users create/update/delete 
domain/account/user data independently in each management server. So to support 
multiple regions and provide one point of entry for each customer, this 
implementation duplicates domain/account/user information of customers in one 
region to all of the regions independently whenever there is any change.

https://issues.apache.org/jira/browse/CLOUDSTACK-4992
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Domain-Account-User+Sync+Up+Among+Multiple+Regions


Diffs (updated)
-

  api/src/com/cloud/domain/Domain.java c4755d7 
  api/src/com/cloud/event/EventTypes.java ec54ea1 
  api/src/com/cloud/user/Account.java 99ef954 
  api/src/com/cloud/user/AccountService.java a9be292 
  api/src/com/cloud/user/User.java 36e9028 
  api/src/com/cloud/user/UserAccount.java c5a0637 
  api/src/org/apache/cloudstack/api/ApiConstants.java 14df653 
  api/src/org/apache/cloudstack/api/BaseCmd.java e869ddf 
  api/src/org/apache/cloudstack/api/response/AccountResponse.java b7d30ca 
  api/src/org/apache/cloudstack/api/response/DomainResponse.java 0c0281e 
  api/src/org/apache/cloudstack/api/response/UserResponse.java 40e1561 
  api/src/org/apache/cloudstack/region/Region.java df64e44 
  
engine/schema/resources/META-INF/cloudstack/core/spring-engine-schema-core-daos-context.xml
 08efb83 
  engine/schema/src/com/cloud/domain/DomainVO.java f6494b3 
  engine/schema/src/com/cloud/rmap/RmapVO.java PRE-CREATION 
  engine/schema/src/com/cloud/rmap/dao/RmapDao.java PRE-CREATION 
  engine/schema/src/com/cloud/rmap/dao/RmapDaoImpl.java PRE-CREATION 
  engine/schema/src/com/cloud/user/AccountVO.java fb1b58a 
  engine/schema/src/com/cloud/user/UserAccountVO.java cef9239 
  engine/schema/src/com/cloud/user/UserVO.java 68879f6 
  engine/schema/src/org/apache/cloudstack/region/RegionVO.java 608bd2b 
  framework/db/src/com/cloud/utils/db/Attribute.java 82c2bdb 
  framework/db/src/com/cloud/utils/db/GenericDao.java cb401cd 
  framework/db/src/com/cloud/utils/db/GenericDaoBase.java cecea84 
  framework/db/src/com/cloud/utils/db/SqlGenerator.java befe34b 
  
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/AccountSubscriber.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/DomainSubscriber.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/MultiRegionEventBus.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/MultiRegionSubscriber.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/UserSubscriber.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/test/org/apache/cloudstack/mom/rabbitmq/AccountSubscriberTest.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/test/org/apache/cloudstack/mom/rabbitmq/DomainSubscriberTest.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/test/org/apache/cloudstack/mom/rabbitmq/MultiRegionSubscriberTest.java
 PRE-CREATION 
  
plugins/event-bus/rabbitmq/test/org/apache/cloudstack/mom/rabbitmq/UserSubscriberTest.java
 PRE-CREATION 
  
plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/MockAccountManager.java
 fa7be58 
  server/resources/META-INF/cloudstack/core/spring-event-bus-context.xml 
PRE-CREATION 
  server/src/com/cloud/api/ApiDispatcher.java 5bdefe7 
  server/src/com/cloud/api/ApiResponseHelper.java 81bfe21 
  server/src/com/cloud/api/query/dao/AccountJoinDaoImpl.java 5e087fd 
  server/src/com/cloud/api/query/dao/UserAccountJoinDaoImpl.java 923a238 
  server/src/com/cloud/api/query/vo/AccountJoinVO.java 8d642ed 
  server/src/com/cloud/api/query/vo/UserAccountJoinVO.java ed29284 
  server/src/com/cloud/event/ActionEventUtils.java 9724d99 
  server/src/com/cloud/projects/ProjectManagerImpl.java 5a0ed1c 
  server/src/com/cloud/region/api_interface/AccountInterface.java PRE-CREATION 
  server/src/com/cloud/region/api_interface/BaseInterface.java PRE-CREATION 
  server/src/

Build failed in Jenkins: build-master-slowbuild #396

[jenkins]

See 

Changes:

[jlk] CLOUDSTACK-6204: removing realhostip dependency

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main

Build failed in Jenkins: build-master-slowbuild #397

[jenkins]

See 

--
[...truncated 18117 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Re: Review Request 16385: Fix for CloudStack JIRA 4406

[Mandar Barve]

We surely need a way to make this generic since cleanString looks for
specific keywords to filter. I will take a look at this. Using @Parameter
may have its own limitations like running through the entire list of
parameters per API before deciding which ones to exclude. But let me take a
look.

I believe we can mark 4406 resolved.

Thanks,
Mandar


On Sat, Mar 8, 2014 at 3:46 AM, Daan Hoogland wrote:

> Mandar, you want to take it?
>
> On Fri, Mar 7, 2014 at 11:12 PM, Alena Prokharchyk
>  wrote:
> > And here is the Jira ticket:
> >
> > https://issues.apache.org/jira/browse/CLOUDSTACK-6213
> >
> > "Add new field to API @Parameter indicating if the param should be
> skipped
> > from logs"
> >
> > -Alena.
> >
> > On 3/7/14, 1:47 PM, "Daan Hoogland"  wrote:
> >
> >>no problem, glad we agree.
> >>
> >>On Fri, Mar 7, 2014 at 8:38 PM, Alena Prokharchyk
> >> wrote:
> >>> Ok, got it, somehow missed the "hardcoded" parameters part. In this
> case
> >>> true is fine as the parameter in @ApiCommand just triggers the
> >>>validation.
> >>> We only have to fix one part - instead of hardcoding the parameter(s)
> to
> >>> hide, we have to come up with the new parameter in @Parameter to
> trigger
> >>> the exclusion from the logs.
> >>>
> >>> Thank you,
> >>> Alena.
> >>>
> >>> On 3/7/14, 11:32 AM, "Daan Hoogland"  wrote:
> >>>
> Alena, I can see I am not being clear because what you say is the
> sensible way and apart from the parameter level exactly what happens.
> 
> The parameter thing is an enhancement that we can make on top of this.
> At the moment it only obfuscate a set of parameters with a fixed set
> of names. We will have to have a new discussion of what the desirable
> default is however. I say security first. but let's not have that
> discussion in this thread.
> 
> Hope this clarifies,
> Daan
> 
> On Fri, Mar 7, 2014 at 8:21 PM, Alena Prokharchyk
>  wrote:
> > Daan, if the default comes as true for the command, I assume that the
> >user
> > won¹t see the command logged at all? Unless he overrides it.
> > I assume sensitive=³true² means not ³analyze the command² but rather
> > ³don¹t log the command². That doesn¹t seem right to me.
> >
> > True would seem right to me if the parameter is defined on both
> > parameter/command level (which is not how it works today). Then
> >parameter
> > in @ApiCommand annotation will just trigger the analyze for sensitive
> > parameters, and the parameter in the @Parameter will tell whether to
> >log
> > the parameter itself.
> >
> >
> > -Alena.
> >
> > On 3/7/14, 10:51 AM, "Daan Hoogland" 
> wrote:
> >
> >>On Fri, Mar 7, 2014 at 7:31 PM, Alena Prokharchyk
> >> wrote:
> >>> And the defaults should be false,
> >>
> >>
> >>I don't agree, The true case does nothing if no fields are recognized
> >>as sensitive, but it the flase case skips sensitive data containing
> >>log messages. The only consquence of true as default is a performance
> >>penalty that we were paying in the old case anyhow.
> >>
> >>--
> >>Daan
> >
> 
> 
> 
> --
> Daan
> >>>
> >>
> >>
> >>
> >>--
> >>Daan
> >
>
>
>
> --
> Daan
>
>

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

[Mandar Barve]

Hi Mike,
 Did you get a chance to look at this?

Thanks,
Mandar


On Wed, Mar 5, 2014 at 10:12 AM, Mandar Barve wrote:

> I tested this with CS 4.3.
>
> Thanks,
> Mandar
>
>
> On Tue, Mar 4, 2014 at 9:09 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> Hi,
>>
>> Can you tell me what release you tested this with? I noticed the problem
>> while developing on CloudStack 4.3.
>>
>> Thanks!
>>
>>
>> On Tue, Mar 4, 2014 at 3:43 AM, Mandar Barve wrote:
>>
>>> Hi,
>>> I tried to reproduce the issue but couldn't get this to fail for
>>> insufficient space. I then injected an exception trying to list files from
>>> a non existent path (added this code in the "try" block). This landed me
>>> into the exception handling code. It raised correct exception saying "file
>>> not found" which was captured in the management server vmops log file. It
>>> was not displayed by the GUI. GUI just reported Error (Are we looking for
>>> GUI displaying error code?). The plugin code returns "0" immediately after
>>> the line of code that raises exception but I think this applies only for
>>> successful execution of the plugin code that reverts the snapshot.
>>>
>>>If any exception is raised (e.g. in the reported case here
>>> insufficient space) then the code should return appropriate error message
>>> to the caller as I found. In exception handling path return "0" wouldn't
>>> execute.
>>>
>>> I don't see any problem here. Let me know if I am missing anything.
>>>
>>> Thanks,
>>> Mandar
>>>
>>
>>
>>
>> --
>> *Mike Tutkowski*
>>  *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the 
>> cloud
>> *(tm)*
>>
>
>

[REMINDER]Review Request 18677: CLOUDSTACK-3272

[Sonal Ojha]

Kindly let me know your review comments.

-- Forwarded message --
From: Sonal Ojha 
Date: Tue, Mar 4, 2014 at 11:20 AM
Subject: Re: Review Request 18677: CLOUDSTACK-3272
To: Sonal Ojha , cloudstack <
dev@cloudstack.apache.org>


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

- Sonal Ojha

On March 3rd, 2014, 5:58 a.m. UTC, Sonal Ojha wrote:
  Review request for cloudstack.
By Sonal Ojha.

*Updated March 3, 2014, 5:58 a.m.*
 *Repository: * cloudstack-git
Description

At present, there are multiple event types that are generated by CS,
action events,usage events, resource state change events and alerts.
Current problem is
all the events gets published on the event bus when event bus is
enabled. Intent is to introduce global setting config parameters to
specify which category of events to be published or not be published
on the event bus.

  Testing

Successfully tested publishing / not publishing of various events
based on the global parameter publish.*.events.

  Diffs

   - 
plugins/event-bus/rabbitmq/src/org/apache/cloudstack/mom/rabbitmq/RabbitMQEventBus.java
   (478c8d7)
   - server/src/com/cloud/configuration/Config.java (2d55f46)
   - server/src/com/cloud/test/DatabaseConfig.java (9370218)

View Diff 



-- 

Regards,

___

*Sonal Ojha* ● Senior Engineer - Product Developement ● SunGard
Availability Services, India ● Mobile: +91 9922412645●  Email:
sonal.o...@sungard.com ● Website: http://www.sungardas.in/

8 Times Winner – BC Service Provider of the Year – 2011, 2010, 2009, 2006,
2005, 2002, 2000, 1999; Finalist – 2008, 2007, 2004, 2001 ● Excellence in
Infrastructure Management – 2010 ● Outstanding Excellence in Business
Continuity – 2008 ● Business Continuity Provider of the Year (BCM Service)
– 2013 BCI Global Awards ● Business Continuity Provider of the Year (BCM
Product) – 2013 BCI India Awards

*[image: AS_yt]* *[image:
AS_twitter]* *[image:
AS_in]**[image:
AS_gplus]* *[image:
AS_fb]**[image:
AS_ss]* 

Build failed in Jenkins: build-master-slowbuild #398

[jenkins]

See 

--
[...truncated 18097 lines...]

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi ---
[INFO] Cobertura 2.0.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/ec2/AmazonEC2SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/amazon/s3/AmazonS3SkeletonInterface.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SAcl.java.  Ensure this class was instrumented, and this 
data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SBucket.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/model/SHost.java.  Ensure this class was instrumented, and 
this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/BucketPolicyDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackAccountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java.  Ensure this 
class was instrumented, and this data file contains the instrumentation 
information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java.  Ensure this class 
was instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/CloudStackUserDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MHostMountDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartPartsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultiPartUploadsDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does not contain instrumentation information for the file 
com/cloud/bridge/persist/dao/MultipartMetaDao.java.  Ensure this class was 
instrumented, and this data file contains the instrumentation information.
[cobertura] INFO  [main] net.sourceforge.cobertura.reporting.html.HTMLReport - 
Data file does 

Re: Just checked in a minor change to the DB

[Rajani Karuturi]

I think a local rule in the mail client on the git commit mails which filters 
all the mails from comm...@cloudstack.apache.org and which has 
setup/db/db/schema-*.sql in the content should work 



~Rajani



On 09-Mar-2014, at 12:32 am, Daan Hoogland  wrote:

> in theory, we will have to set up a git (post or pre)  commit hook
> that parses the contents and amend the comment if a .sql is found. I
> do not know if we need extra rights from infra or whether our commit
> bit is enough. I won't be having a look for that this weekend. hope to
> get time next week.
> 
> 
> On Sat, Mar 8, 2014 at 7:55 PM, Mike Tutkowski
>  wrote:
>> Sounds good - Who knows how to set that up?
>> 
>> 
>> On Sat, Mar 8, 2014 at 6:55 AM, Rajesh Battala 
>> wrote:
>> 
>>> Yes.. there might be  chance to miss out to inform if we need to do
>>> manually. If we automate it with git it would be awesome..
>>> 
>>> -Original Message-
>>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>>> Sent: Saturday, March 8, 2014 2:21 PM
>>> To: dev
>>> Subject: Re: Just checked in a minor change to the DB
>>> 
>>> We have git spam so we should just use that I think. we could add to it by
>>> automatically let git add the tag when any sql is touched... I like the
>>> automated approach of tagging/spamming. It would remove the need to
>>> manually add the ..-CHANGE tags. I not very much for more automated mails
>>> to dev, the ones to commits and issues should be enough.
>>> 
>>> On Sat, Mar 8, 2014 at 7:07 AM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
 We could, but some schema changes won't impact others.
 
 For example, if I removed a column and took out the corresponding Java
 code and you updated your code, the new code wouldn't fail against
 your old schema.
 
 People might also update the schema and check it in, but not update
 the Java code until later.
 
 Perhaps these are corner cases and it would be fine to have Git notify
>>> us?
 
 What do people think?
 
 
 On Fri, Mar 7, 2014 at 10:51 PM, Rajesh Battala
 wrote:
 
> Just a thought, instead of sending mail manually, can't we have a git
> hook to send out mail to dev list when there is a commit to schema
>>> files.
> So that git will take care of informing the dev about new schema
>>> changes.!
> 
> Thanks
> Rajesh Battala
> 
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Saturday, March 8, 2014 3:21 AM
> To: dev
> Subject: Re: Just checked in a minor change to the DB
> 
> saw, good, will do some more if I think of them as well, as well as
> add to the descriptions
> 
> On Fri, Mar 7, 2014 at 9:29 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>> I went ahead and added to the Wiki page you created a few more tags
>> that I've seen us use from time to time.
>> 
>> 
>> On Fri, Mar 7, 2014 at 12:37 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>> 
>>> Great! Thanks, Daan
>>> 
>>> I also agree that it would be useful info to have in commit messages.
>>> 
>>> 
>>> On Fri, Mar 7, 2014 at 12:14 PM, Daan Hoogland
>>> > wrote:
>>> 
 I created
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/mail+tags+
 to+
 use+to+help+each+other
 below the path
 Apache Cloudstack / Home / Community / Getting Help
 
 
 On Fri, Mar 7, 2014 at 8:03 PM, Daan Hoogland
 
 wrote:
> I had a quick look. Nothing found but a lot of hits on tags of
> course.
> I'll try to find a place to create the page and start chapters
> on Dev and Users
> 
> 
> On Fri, Mar 7, 2014 at 7:54 PM, Daan Hoogland
> 
 wrote:
>> No, but I do know that it might be useful to use them in
>> commit comments as well. If you so feel inclined you can refer
>> to that as well
>> 
>> On Fri, Mar 7, 2014 at 7:41 PM, Mike Tutkowski
>>  wrote:
>>> Does anyone know if our e-mail tags are documented on the Wiki?
>>> 
>>> 
>>> On Fri, Mar 7, 2014 at 11:36 AM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
>>> 
 That sounds good, Daan.
 
 Let's use both tags.
 
 Thanks!
 
 
 On Fri, Mar 7, 2014 at 9:33 AM, Daan Hoogland <
 daan.hoogl...@gmail.com>wrote:
 
> Absolutaly,
> 
> But I meant for us to use both tags. I checked in a commit
> (that I didn't give a head up on) for a colleague in which
> a CloudRuntimeException was removed to be more consistent
> with the
 rest
> of the code. It was onl

Re: Review Request 18016: CLOUDSTACK-5626: Simplifying VM Migrate code

[ASF Subversion and Git Services]

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


Commit 3b3ae024591fae4db9a6ca991d4838d98b96154e in cloudstack's branch 
refs/heads/master from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3b3ae02 ]

CLOUDSTACK-5626: Simplifying VM Migrate code

Signed-off-by: SrikanteswaraRao Talluri 


- ASF Subversion and Git Services


On March 7, 2014, 9:36 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18016/
> ---
> 
> (Updated March 7, 2014, 9:36 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-5626
> https://issues.apache.org/jira/browse/CLOUDSTACK-5626
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Removing different ways of obtaining suitable hosts. Used built in API 
> "findHostsForMigration" and skipped the test if suitable host is not found 
> instead of throwing assertion error.
> Removed unnecessary common function.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_cpu_domain_limits.py c427e4f 
>   test/integration/component/test_cpu_limits.py bdf2869 
>   test/integration/component/test_cpu_project_limits.py a8a1b3c 
>   test/integration/component/test_memory_limits.py 7921e4b 
>   test/integration/component/test_mm_domain_limits.py 68660c1 
>   test/integration/component/test_mm_project_limits.py c314011 
>   test/integration/component/test_vpc_vm_life_cycle.py 01373ac 
>   tools/marvin/marvin/codes.py 3882f0d 
>   tools/marvin/marvin/integration/lib/common.py b2da3ff 
> 
> Diff: https://reviews.apache.org/r/18016/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: Review Request 18623: CLOUDSTACK-6147: Adding first set of test cases for Dynamic Compute Offering feature

[ASF Subversion and Git Services]

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


Commit 3f7b628c707cd9c30542efd670416be5fbb473f0 in cloudstack's branch 
refs/heads/master from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3f7b628 ]

CLOUDSTACK-6147: Adding first set of test caess for Dynamic Compute Offering 
feature

Signed-off-by: SrikanteswaraRao Talluri 


- ASF Subversion and Git Services


On March 7, 2014, 9:08 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18623/
> ---
> 
> (Updated March 7, 2014, 9:08 a.m.)
> 
> 
> Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-6147
> https://issues.apache.org/jira/browse/CLOUDSTACK-6147
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Adding first set of automation test cases for Dynamic Compute Offering 
> Feature.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_dynamic_compute_offering.py PRE-CREATION 
>   tools/marvin/marvin/integration/lib/base.py 0a7ad94 
>   tools/marvin/marvin/integration/lib/common.py b2da3ff 
> 
> Diff: https://reviews.apache.org/r/18623/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> Log:
> Deploy VM with dynamic service offering and check VM stats ... ok
> Deploy VM with dynamic service offering and check VM stats ... ok
> Create dynamic compute offering with cpunumber, cpuspeed and memory ... ok
> Create dynamic compute offering with only cpunumber unspecified ... ok
> Create dynamic compute offering with only cpuspeed unspecified ... ok
> Create dynamic compute offering with only memory unspecified ... ok
> Create normal compute offering with non zero values for cpu, ... ok
> Test deploy VM with dynamic compute offering ... ok
> Test deploy VM with dynamic compute offering ... ok
> Test deploy VM with static offering ... ok
> Test deploy VM with static offering ... ok
> --
> Ran 11 tests in 794.431s
> 
> OK
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

RE: Just checked in a minor change to the DB

[Rajesh Battala]

I prefer server side hook because ppl use mail clients in mobiles/web etc and 
applying client side rules is not effective.
And moreover we need to keep exporting this client side rule and associated 
mail client support is required for the same.
Best option is to have a server side hook . 

Thanks
Rajesh Battala

-Original Message-
From: Rajani Karuturi [mailto:rajani.karut...@citrix.com] 
Sent: Monday, March 10, 2014 10:46 AM
To: dev@cloudstack.apache.org
Subject: Re: Just checked in a minor change to the DB


I think a local rule in the mail client on the git commit mails which filters 
all the mails from comm...@cloudstack.apache.org and which has 
setup/db/db/schema-*.sql in the content should work 



~Rajani



On 09-Mar-2014, at 12:32 am, Daan Hoogland  wrote:

> in theory, we will have to set up a git (post or pre)  commit hook 
> that parses the contents and amend the comment if a .sql is found. I 
> do not know if we need extra rights from infra or whether our commit 
> bit is enough. I won't be having a look for that this weekend. hope to 
> get time next week.
> 
> 
> On Sat, Mar 8, 2014 at 7:55 PM, Mike Tutkowski 
>  wrote:
>> Sounds good - Who knows how to set that up?
>> 
>> 
>> On Sat, Mar 8, 2014 at 6:55 AM, Rajesh Battala 
>> wrote:
>> 
>>> Yes.. there might be  chance to miss out to inform if we need to do 
>>> manually. If we automate it with git it would be awesome..
>>> 
>>> -Original Message-
>>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>>> Sent: Saturday, March 8, 2014 2:21 PM
>>> To: dev
>>> Subject: Re: Just checked in a minor change to the DB
>>> 
>>> We have git spam so we should just use that I think. we could add to 
>>> it by automatically let git add the tag when any sql is touched... I 
>>> like the automated approach of tagging/spamming. It would remove the 
>>> need to manually add the ..-CHANGE tags. I not very much for more 
>>> automated mails to dev, the ones to commits and issues should be enough.
>>> 
>>> On Sat, Mar 8, 2014 at 7:07 AM, Mike Tutkowski < 
>>> mike.tutkow...@solidfire.com> wrote:
 We could, but some schema changes won't impact others.
 
 For example, if I removed a column and took out the corresponding 
 Java code and you updated your code, the new code wouldn't fail 
 against your old schema.
 
 People might also update the schema and check it in, but not update 
 the Java code until later.
 
 Perhaps these are corner cases and it would be fine to have Git 
 notify
>>> us?
 
 What do people think?
 
 
 On Fri, Mar 7, 2014 at 10:51 PM, Rajesh Battala
 wrote:
 
> Just a thought, instead of sending mail manually, can't we have a 
> git hook to send out mail to dev list when there is a commit to 
> schema
>>> files.
> So that git will take care of informing the dev about new schema
>>> changes.!
> 
> Thanks
> Rajesh Battala
> 
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Saturday, March 8, 2014 3:21 AM
> To: dev
> Subject: Re: Just checked in a minor change to the DB
> 
> saw, good, will do some more if I think of them as well, as well 
> as add to the descriptions
> 
> On Fri, Mar 7, 2014 at 9:29 PM, Mike Tutkowski < 
> mike.tutkow...@solidfire.com> wrote:
>> I went ahead and added to the Wiki page you created a few more 
>> tags that I've seen us use from time to time.
>> 
>> 
>> On Fri, Mar 7, 2014 at 12:37 PM, Mike Tutkowski < 
>> mike.tutkow...@solidfire.com> wrote:
>> 
>>> Great! Thanks, Daan
>>> 
>>> I also agree that it would be useful info to have in commit messages.
>>> 
>>> 
>>> On Fri, Mar 7, 2014 at 12:14 PM, Daan Hoogland 
>>> > wrote:
>>> 
 I created
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/mail+tag
 s+
 to+
 use+to+help+each+other
 below the path
 Apache Cloudstack / Home / Community / Getting Help
 
 
 On Fri, Mar 7, 2014 at 8:03 PM, Daan Hoogland 
 
 wrote:
> I had a quick look. Nothing found but a lot of hits on tags of
> course.
> I'll try to find a place to create the page and start chapters 
> on Dev and Users
> 
> 
> On Fri, Mar 7, 2014 at 7:54 PM, Daan Hoogland 
> 
 wrote:
>> No, but I do know that it might be useful to use them in 
>> commit comments as well. If you so feel inclined you can 
>> refer to that as well
>> 
>> On Fri, Mar 7, 2014 at 7:41 PM, Mike Tutkowski 
>>  wrote:
>>> Does anyone know if our e-mail tags are documented on the Wiki?
>>> 
>>> 
>>> On Fri, Mar 7, 2014 at 11:36 AM, Mike Tutkowski < 
>>> mike.tutkow...@solidfire.com> wrote:
>>> 
 That sou

Re: Review Request 16385: Fix for CloudStack JIRA 4406

[Mandar Barve]

Guess in addition to the command level flag that we have Parameter walk
will need to be done only for the already identified "sensitive" responses
as discussed on the thread so this may be fine.

Thanks,
Mandar


On Mon, Mar 10, 2014 at 9:34 AM, Mandar Barve wrote:

> We surely need a way to make this generic since cleanString looks for
> specific keywords to filter. I will take a look at this. Using @Parameter
> may have its own limitations like running through the entire list of
> parameters per API before deciding which ones to exclude. But let me take a
> look.
>
> I believe we can mark 4406 resolved.
>
> Thanks,
> Mandar
>
>
> On Sat, Mar 8, 2014 at 3:46 AM, Daan Hoogland wrote:
>
>> Mandar, you want to take it?
>>
>> On Fri, Mar 7, 2014 at 11:12 PM, Alena Prokharchyk
>>  wrote:
>> > And here is the Jira ticket:
>> >
>> > https://issues.apache.org/jira/browse/CLOUDSTACK-6213
>> >
>> > "Add new field to API @Parameter indicating if the param should be
>> skipped
>> > from logs"
>> >
>> > -Alena.
>> >
>> > On 3/7/14, 1:47 PM, "Daan Hoogland"  wrote:
>> >
>> >>no problem, glad we agree.
>> >>
>> >>On Fri, Mar 7, 2014 at 8:38 PM, Alena Prokharchyk
>> >> wrote:
>> >>> Ok, got it, somehow missed the "hardcoded" parameters part. In this
>> case
>> >>> true is fine as the parameter in @ApiCommand just triggers the
>> >>>validation.
>> >>> We only have to fix one part - instead of hardcoding the parameter(s)
>> to
>> >>> hide, we have to come up with the new parameter in @Parameter to
>> trigger
>> >>> the exclusion from the logs.
>> >>>
>> >>> Thank you,
>> >>> Alena.
>> >>>
>> >>> On 3/7/14, 11:32 AM, "Daan Hoogland"  wrote:
>> >>>
>> Alena, I can see I am not being clear because what you say is the
>> sensible way and apart from the parameter level exactly what happens.
>> 
>> The parameter thing is an enhancement that we can make on top of this.
>> At the moment it only obfuscate a set of parameters with a fixed set
>> of names. We will have to have a new discussion of what the desirable
>> default is however. I say security first. but let's not have that
>> discussion in this thread.
>> 
>> Hope this clarifies,
>> Daan
>> 
>> On Fri, Mar 7, 2014 at 8:21 PM, Alena Prokharchyk
>>  wrote:
>> > Daan, if the default comes as true for the command, I assume that
>> the
>> >user
>> > won¹t see the command logged at all? Unless he overrides it.
>> > I assume sensitive=³true² means not ³analyze the command² but rather
>> > ³don¹t log the command². That doesn¹t seem right to me.
>> >
>> > True would seem right to me if the parameter is defined on both
>> > parameter/command level (which is not how it works today). Then
>> >parameter
>> > in @ApiCommand annotation will just trigger the analyze for
>> sensitive
>> > parameters, and the parameter in the @Parameter will tell whether to
>> >log
>> > the parameter itself.
>> >
>> >
>> > -Alena.
>> >
>> > On 3/7/14, 10:51 AM, "Daan Hoogland" 
>> wrote:
>> >
>> >>On Fri, Mar 7, 2014 at 7:31 PM, Alena Prokharchyk
>> >> wrote:
>> >>> And the defaults should be false,
>> >>
>> >>
>> >>I don't agree, The true case does nothing if no fields are
>> recognized
>> >>as sensitive, but it the flase case skips sensitive data containing
>> >>log messages. The only consquence of true as default is a
>> performance
>> >>penalty that we were paying in the old case anyhow.
>> >>
>> >>--
>> >>Daan
>> >
>> 
>> 
>> 
>> --
>> Daan
>> >>>
>> >>
>> >>
>> >>
>> >>--
>> >>Daan
>> >
>>
>>
>>
>> --
>> Daan
>>
>>
>

Regarding CLOUDSTACK-5825

[Namita Chaudhari]

Hi Chris,

I tried to create volume snapshot on 4.3 version. When the volume snapshot
creation failed, the API threw an exception "Failed to create the snapshot
due to an internal error for volume x".
I have attached the screenshot and log file for the same in the jira.

Please let me know your comments.

Thanks and Regards,

-- 

*Namita Chaudhari* ● Associate Engineer- Product Development ● SunGard
Availability Services, India ● Mobile: +91 9762922138 ● Email:
namita.chaudh...@sungard.com

*[image: AS_yt]* *[image:
AS_twitter]* *[image:
AS_in]**[image:
AS_gplus]* *[image:
AS_fb]**[image:
AS_ss]*