Re: Review Request 21131: Remove hard coding of guest OS mapping for KVM

2014-05-06 Thread Amogh Vasekar

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

(Updated May 7, 2014, 12:26 a.m.)


Review request for cloudstack and Nitin Mehta.


Changes
---

Cleaning up code


Bugs: 
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358

https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358


Repository: cloudstack-git


Description
---

As a part of supporting dynamic guest OS defined by user, removing the 
hard-coded dependencies.
This patch is for KVM
This patch is to be applied after https://reviews.apache.org/r/21087/


Diffs (updated)
-

  engine/schema/src/com/cloud/storage/dao/GuestOSHypervisorDaoImpl.java 
3b05120f5b9d5bcf4bab6ef25cd1dab2a6043c0d 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 345a8eea0cf3033eef10accdd94639bf014b0fb3 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
4032305b03268b4b8b93a3f41038dce83bcaaf19 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/VifDriverBase.java
 d89d71a1b2a93959ebcd4f0df529901e12445617 
  server/src/com/cloud/hypervisor/KVMGuru.java 
174b32db578f66da83ed86e942f7f5b316512a55 

Diff: https://reviews.apache.org/r/21131/diff/


Testing
---

1. Local testing on KVM
2. Successfully got up system VMs
3. Successfully created a CentOS VM
4. Snapshots are not supported for KVM


Thanks,

Amogh Vasekar



[ACS44] Cherry pick for CLOUDSTACK-6585

2014-05-06 Thread Alena Prokharchyk
Daan,

Can you please cherry pick the commit 8ffb2c11485c178a97dd30f620458684c79d186a, 
to 4.4 branch?

Thanks,
Alena.


Re: Review Request 21131: Remove hard coding of guest OS mapping for KVM

2014-05-06 Thread Amogh Vasekar

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

(Updated May 7, 2014, 12:11 a.m.)


Review request for cloudstack and Nitin Mehta.


Changes
---

Add missing file


Bugs: 
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358

https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358


Repository: cloudstack-git


Description
---

As a part of supporting dynamic guest OS defined by user, removing the 
hard-coded dependencies.
This patch is for KVM
This patch is to be applied after https://reviews.apache.org/r/21087/


Diffs (updated)
-

  engine/schema/src/com/cloud/storage/dao/GuestOSHypervisorDaoImpl.java 
3b05120f5b9d5bcf4bab6ef25cd1dab2a6043c0d 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 345a8eea0cf3033eef10accdd94639bf014b0fb3 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
4032305b03268b4b8b93a3f41038dce83bcaaf19 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/VifDriverBase.java
 d89d71a1b2a93959ebcd4f0df529901e12445617 
  server/src/com/cloud/hypervisor/KVMGuru.java 
174b32db578f66da83ed86e942f7f5b316512a55 

Diff: https://reviews.apache.org/r/21131/diff/


Testing
---

1. Local testing on KVM
2. Successfully got up system VMs
3. Successfully created a CentOS VM
4. Snapshots are not supported for KVM


Thanks,

Amogh Vasekar



[PROPOSAL] Using continuous integration to maintain our code quality...

2014-05-06 Thread Alex Huang
Hi All,

This is something I brought up a long time ago but really didn't have the 
resources to get it all up and running until now.  Throughout the past year, 
Edison, Prasanna, Amogh, Bharat, Koushik, Talluri, and others have been 
chipping away at it.  At this point, we finally pull together a continuous 
integration setup that we can use to make sure that CloudStack master and the 
currently release branch are always stable.  This is getting pretty close to be 
completed and we like to share it with the community in hopes that we can 
reduce/eliminate that problems we've seen with our recent releases.  Currently, 
the physical hardware are hosted by Citrix but we'll be more than willing to 
donate the work to infra when that's all settled.  

This does require effort from the community to make a change in their 
development process.  These steps are detailed at [1].  I like to get feedback 
on what everyone think about this.  

What have we done:
  - We replaced a large selection of the BVT tests to run with the simulator 
instead of actual hardware.  This shortens the duration of each BVT run.  
Today, a BVT that runs tests for XenServer and KVM completes in 30-40 minutes.
  - We will run the new BVT on master and the current release branch on a 
continuous basis.
  - Developers can use Jenkins to ask BVT to be run on their branch so they can 
know it won't break the continuous integration before they merge to master and 
the current release branch.

Please have a read and let me know what you think.

--Alex

[1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Development+Process


[ACS44] cherry pick

2014-05-06 Thread Kelven Yang

Hi Daan,
Could you cherry-pick following fix in 4.4-forward to 4.4 branch.

commit be2b5918e6b09a488d75966c49d34bdeb803fdd8
Author: Kelven Yang 
Date:   Mon May 5 15:21:59 2014 -0700

CLOUDSTACK-6586: Move EventBus hookup on job framework to ApiServer to
decouple job framework away from business logic related hookups.

Thanks

Kelven



Re: Congrats on 4.3 release!

2014-05-06 Thread Amogh Vasekar
Hi,

Can someone please commit the missing file wrt below for me? It has been
lingering for long.
The review request is at https://reviews.apache.org/r/18759/ , and the
changed file (ConsoleProxyInfo.java) that is responsible for handing out
the correct URL for console is missing in the commit for ACS 4.3 *only*

Thanks,
Amogh

On 3/27/14 1:00 PM, "Amogh Vasekar"  wrote:

>Hi Remi,
>
>Thanks for bringing this up.
>
>John, 
>I looked up the commit made and seems like one of the files -
>ConsoleProxyInfo.java, is missing from the commit in 4.3. It made it to
>the
>master though.
>4.3 : https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=ec4db7b
>Master : 
>https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=09c3753
>
>
>Can you please commit the file so that it makes it to the maintenance
>releases?
>The current work around is to remove the "*" when upgrading, from config :
>consoleproxy.url.domain
>I will update the bug accordingly.
>
>Thanks,
>Amogh
>
>On 3/27/14 10:47 AM, "Remi Bergsma"  wrote:
>
>>Hi all,
>>
>>Just a quick note we upgraded one of our cloud environments (~600 vms)
>>today to 4.3 and it all went smooth. We had a little issue
>>(CLOUDSTACK-6293) with the custom SSL cert on the console proxy you might
>>want to have a look at.
>>
>>The GUI looks great and "feels" faster. The little "feature" that
>>re-creates router vms on reboot REALLY saved us a lot of time. Cool!
>>
>>Thanks for your hard work and congratulations on what looks like a great
>>release! :-)
>>
>>
>>Kind regards,
>>
>>Remi Bergsma
>>



Re: Review Request 21131: Remove hard coding of guest OS mapping for KVM

2014-05-06 Thread Amogh Vasekar

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

(Updated May 6, 2014, 10:51 p.m.)


Review request for cloudstack and Nitin Mehta.


Bugs: 
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358

https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358


Repository: cloudstack-git


Description
---

As a part of supporting dynamic guest OS defined by user, removing the 
hard-coded dependencies.
This patch is for KVM
This patch is to be applied after https://reviews.apache.org/r/21087/


Diffs
-

  engine/schema/src/com/cloud/storage/dao/GuestOSHypervisorDaoImpl.java 
3b05120f5b9d5bcf4bab6ef25cd1dab2a6043c0d 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 345a8eea0cf3033eef10accdd94639bf014b0fb3 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
4032305b03268b4b8b93a3f41038dce83bcaaf19 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/VifDriverBase.java
 d89d71a1b2a93959ebcd4f0df529901e12445617 

Diff: https://reviews.apache.org/r/21131/diff/


Testing
---

1. Local testing on KVM
2. Successfully got up system VMs
3. Successfully created a CentOS VM
4. Snapshots are not supported for KVM


Thanks,

Amogh Vasekar



Review Request 21131: Remove hard coding of guest OS mapping for KVM

2014-05-06 Thread Amogh Vasekar

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

Review request for cloudstack and Nitin Mehta.


Bugs: 
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358

https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358


Repository: cloudstack-git


Description
---

As a part of supporting dynamic guest OS defined by user, removing the 
hard-coded dependencies.
This patch is for KVM
This patch is to be applied after https://reviews.apache.org/r/21087/


Diffs
-

  engine/schema/src/com/cloud/storage/dao/GuestOSHypervisorDaoImpl.java 
3b05120f5b9d5bcf4bab6ef25cd1dab2a6043c0d 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 345a8eea0cf3033eef10accdd94639bf014b0fb3 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
4032305b03268b4b8b93a3f41038dce83bcaaf19 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/VifDriverBase.java
 d89d71a1b2a93959ebcd4f0df529901e12445617 

Diff: https://reviews.apache.org/r/21131/diff/


Testing
---

1. Local testing on KVM
2. Successfully got up system VMs
3. Successfully created a CentOS VM
4. Snapshots are not supported for KVM


Thanks,

Amogh Vasekar



Re: Review Request 21087: Remove hard-coding for platform emulators in Xen Server

2014-05-06 Thread Amogh Vasekar

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

(Updated May 6, 2014, 10:45 p.m.)


Review request for cloudstack and Nitin Mehta.


Changes
---

Cleanly applying patch


Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6358

https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358


Repository: cloudstack-git


Description
---

As a part of supporting dynamic guest OS defined by user, removing the 
hard-coded dependencies.
This patch is for XenServer


Diffs (updated)
-

  api/src/com/cloud/agent/api/to/VirtualMachineTO.java 
bbd83852abf840ab5d93f53ad0c71e61491b2dda 
  core/src/com/cloud/agent/api/VMSnapshotBaseCommand.java 
d2abd54e0407ab597636d9c9d5a596e55f987108 
  
engine/storage/snapshot/src/org/apache/cloudstack/storage/vmsnapshot/DefaultVMSnapshotStrategy.java
 e86bd23b0500f25268a603a7c41e8e00d18957d2 
  engine/storage/snapshot/test/src/VMSnapshotStrategyTest.java 
9eaa7e19c134caf0633bb9099c5f858075fddba7 
  
engine/storage/src/org/apache/cloudstack/storage/helper/HypervisorHelperImpl.java
 b41c1fa14ac77cffa53ce5ae2aec6d9bea9cd649 
  plugins/hypervisors/xen/src/com/cloud/hypervisor/XenServerGuru.java 
89e4ab5d94b61cc6acdefc8102da4a33dbbf23b5 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 ad3797306c92d80d929a1e09b46b471231baa6c0 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpOssResource.java
 4a69d3aececd718cdb7257d3807225c429b73823 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpServer16Resource.java
 8cb7997305f1344429c5b256bea35878137bb5a3 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpServerResource.java
 cc623bfab61f732a4310e3c6c93a51bc36a463b9 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java
 3dc527e93d82733823f4e8fddb242d698c21629f 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56Resource.java
 a78ed819bdeb4878850989915a7280b0639e3ab6 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56SP2Resource.java
 9e7d5dd09d0c1a1d18c6c348ce098218933d18ed 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer600Resource.java
 021b9cbd4dc7ed4a5041c69aa55f29daa732f52b 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer602Resource.java
 5c9bfbb5952008b332900cd7af21671f7dc465a5 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer610Resource.java
 0c3e1e9660abdbfee1dcfe58c62145cbe0e06c9b 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
 c10dc1e90cbce2ff41cc62e2120f5e23931a4207 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620SP1Resource.java
 4f04961e35742d7e846cec47725551d67b704bfc 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java
 40143646f7744e05e2ca009d99d53973ceaf58ee 

Diff: https://reviews.apache.org/r/21087/diff/


Testing
---

1. Local testing on XenServer 6.0.2
2. Successfully got up system VMs
3. Successfully created a CentOS VM
4. Created VM snapshot and reverted to snapshot
5. Manipulated guest_os_hypervisor table for mapping, and checked for correct 
propagation of platform emulator.


Thanks,

Amogh Vasekar



Re: 44 API breakage in UpdateUser

2014-05-06 Thread Daan Hoogland
Ove,

Can you create a patch with "git formaat-patch" from what you entered
in the ticket, please?

On Tue, May 6, 2014 at 8:02 PM, Ove Ewerlid  wrote:
> On 05/06/2014 08:45 AM, Antonio Fornié Casarrubios wrote:
>>
>> Hi Ove,
>>
>> Sorry but I think the only thing I did here was to put in a constant a
>> hardcoded value, but you should find who put the hardcoded value in the
>> first place.
>
>
> yes, 'git blame' is lacking in conveying specifics.
> 'git log -p' provides additional detail;
>  git log -p --grep 'Corrects problems from prev'
> api/src/org/apache/cloudstack/api/ApiConstants.java
> showing a commit of yours that modifies the value of API_KEY. Is this
> interpretation incorrect?
>
> Irregardless of the source of the change, the value of API_KEY is exposed in
> UpdateUserCmd.java as an API parameter hence changing this value changes the
> behavior of the API.
> The same commit introduces USER_API_KEY retaining the old value of API_KEY.
> Perhaps this new constant should have been introduced in more places.
>
> Ove
>
>
>
> commit c211f0bbbe940ed3b9a6e9ef4b5a29be16062e85
> Author: Antonio Fornie 
> Date:   Fri Mar 7 09:57:31 2014 -0600
>
> Dispatcher corrections, refactoring and tests
>
> Corrects problems from previous attempt. Fixes based on help comments
> from
> the community and conflict resolution
>
> Signed-off-by: Daan Hoogland 
>
> diff --git a/api/src/org/apache/cloudstack/api/ApiConstants.java
> b/api/src/org/apache/cloudstack/api/ApiConstants.java
> index 14df653..f0de48e 100755
> --- a/api/src/org/apache/cloudstack/api/ApiConstants.java
> +++ b/api/src/org/apache/cloudstack/api/ApiConstants.java
> @@ -23,7 +23,8 @@ public class ApiConstants {
>  public static final String ACCOUNT_ID = "accountid";
>  public static final String ALGORITHM = "algorithm";
>  public static final String ALLOCATED_ONLY = "allocatedonly";
> -public static final String API_KEY = "userapikey";
>
> +public static final String API_KEY = "apikey";
> +public static final String USER_API_KEY = "userapikey";
>  public static final String APPLIED = "applied";
>  public static final String AVAILABLE = "available";
>  public static final String BITS = "bits";
>
>
>
>
>> Cheers
>> Antonio
>>
>>
>>
>> 2014-05-06 0:44 GMT+02:00 Ove Ewerlid :
>>
>>> On 05/04/2014 10:19 PM, Ove Ewerlid wrote:
>>>
 See https://issues.apache.org/jira/browse/CLOUDSTACK-6570
 Suggested patch in comment.
 /Ove


>>> Looping in the person that 'git blame' thinks changed the value of
>>> API_KEY
>>> in ApiConstants the last time.
>>>
>>> Antonio, was USER_API_KEY introduced for cases where the "userapikey"
>>> value needed to be preserved, such as  in the UpdateUserCmd.java API
>>> call,
>>> and that this update was not done for this API?
>>>
>>> /Ove
>>>
>>> NB; "git blame" may not be accurate hence I may be looping in someone
>>> that
>>> has nothing to do with this. Sorry if that is the case.
>>>
>>>
>>> [oewerlid@amd-a cloudstack]$ git blame
>>> api/src/org/apache/cloudstack/api/ApiConstants.java
>>> | grep "API_KEY"
>>> c211f0bb api/src/org/apache/cloudstack/api/ApiConstants.java (Antonio
>>> Fornie2014-03-07 09:57:31 -0600  26) public static final
>>> String API_KEY = "apikey";
>>> c211f0bb api/src/org/apache/cloudstack/api/ApiConstants.java (Antonio
>>> Fornie2014-03-07 09:57:31 -0600  27) public static final
>>> String USER_API_KEY = "userapikey";
>>> [oewerlid@amd-a cloudstack]$
>>>
>>>
>>>
>>>
>>> --
>>> Ove Everlid
>>> System Administrator / Architect / SDN- & Automation- & Linux-hacker
>>> Mobile: +46706662363 (dedicated work mobile)
>>> Country: Sweden, timezone; Middle Europan Time (MET or GMT+1)
>>>
>>
>
>
> --
> Ove Everlid
> System Administrator / Architect / SDN- & Automation- & Linux-hacker
> Mobile: +46706662363 (dedicated work mobile)
> Country: Sweden, timezone; Middle Europan Time (MET or GMT+1)



-- 
Daan


RE: [ACS44] 4.4.0 blokker issues

2014-05-06 Thread Santhosh Edukulla
Hi Daan,

Looking at this file ./setup/db/db/schema-430to440-cleanup.sql, it seems we 
altered to remove cidr column from network_acl_item, from 4.3 to 4.4 but 
corresponding references are available in code, and its failing networkacl cmd. 

Santhosh

From: Daan Hoogland [daan.hoogl...@gmail.com]
Sent: Tuesday, May 06, 2014 11:27 AM
To: dev
Subject: Re: [ACS44] 4.4.0 blokker issues

I will look at your patch, I created one myself that conflict so I
will see if I can merge.

On Tue, May 6, 2014 at 4:37 PM, Jayapal Reddy Uradi
 wrote:
> Hi Daan,
>
> While working on 
> CLOUDSTACK-6084 fixed 
> few issues for CLOUDSTACK-6582 and uploaded the patch in the bug.
> There are still few issues in network acl cidr changes.
>
> CLOUDSTACK-6582 and CLOUDSTACK-6485 fixes created regression for vpc create 
> private gateway CLOUDSTACK-6084.
>
> Thanks,
> Jayapal
>
>
> On 06-May-2014, at 7:25 PM, Rajesh Battala 
> mailto:rajesh.batt...@citrix.com>>
>  wrote:
>
> CLOUDSTACK-6295,  I have commented on it and closed it.
>
> Thanks
> Rajesh Battala
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Tuesday, May 6, 2014 6:24 PM
> To: dev
> Subject: [ACS44] 4.4.0 blokker issues
>
> At the moment we have 14 blokker issues in jira. Can all reporters and 
> assignees please have a look to see if these are really blokkers and if they 
> can be resolved before long?
>
> Key:Summary:Reporter:Assignee
> CLOUDSTACK-6130
> [Automation] listPublicAdress is failing Srikanteswararao Talluri Hugo 
> Trippaers
>
> CLOUDSTACK-6245
> Security group rules on hypervisor host are lagging behind rules in DB edison 
> su edison su
>
> CLOUDSTACK-6295
> Management server cannot start because Java 7 missing Paul Angus Unassigned
>
> CLOUDSTACK-6444
> [Automation] test_01_primary_storage_iscsi failed on 
> "test_primary_storage.py" - Wrong iscsi path format - it should be 
> /targetIQN/LUN Chandan Purushothama Chandan Purushothama
>
> CLOUDSTACK-6453
> [GPU] Windows 2012 Server instance created with vGPU offering is not coming 
> up after installing PV drivers Sailaja Mada Sanjay Tripathi
>
> CLOUDSTACK-6544
> [Automation] Failed to create template for ROOT volume in Xen, with
> Exception: callHostPlugin failed
> Rayees Namathponnan Harikrishna Patnala
>
> CLOUDSTACK-6547
> [Automation] Failed to download volume in Xen with error "Failed to copy the 
> volume from pri stor pool to sec stor pool"
> Rayees Namathponnan Nitin Mehta
>
> CLOUDSTACK-6548
> [Automation] createPrivateNetwork API failed with NullPointer exception 
> Rayees Namathponnan Santhosh Kumar Edukulla
>
> CLOUDSTACK-6551
> [Automation] Failed to revert vm snapshot in xen Rayees Namathponnan 
> Harikrishna Patnala
>
> CLOUDSTACK-6553
> [Automation] Resize volume test case failing with error "value=invalid id due 
> to incorrect long value format"
> Rayees Namathponnan Rajani Karuturi
>
> CLOUDSTACK-6570
> API breakage of the UpdateUser API call
> Ove Ewerlid Unassigned
>
> CLOUDSTACK-6572
> [Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic Sowmya 
> Krishnan Unassigned
>
> CLOUDSTACK-6573
> Multiple exceptions after upgrading from 4.3 to 4.4 manasaveloori Unassigned
>
> CLOUDSTACK-6582
> Listing acl list rules are failed with exception Unknown column 
> 'network_acl_item.cidr'
> Jayapal Reddy Unassigned
>
> I am taking the last one.
> --
> Daan
>



--
Daan

Jenkins build is back to normal : build-master-slowbuild #776

2014-05-06 Thread jenkins
See 



Re: 44 API breakage in UpdateUser

2014-05-06 Thread Ove Ewerlid

On 05/06/2014 08:45 AM, Antonio Fornié Casarrubios wrote:

Hi Ove,

Sorry but I think the only thing I did here was to put in a constant a
hardcoded value, but you should find who put the hardcoded value in the
first place.


yes, 'git blame' is lacking in conveying specifics.
'git log -p' provides additional detail;
 git log -p --grep 'Corrects problems from prev' 
api/src/org/apache/cloudstack/api/ApiConstants.java
showing a commit of yours that modifies the value of API_KEY. Is this 
interpretation incorrect?


Irregardless of the source of the change, the value of API_KEY is 
exposed in UpdateUserCmd.java as an API parameter hence changing this 
value changes the behavior of the API.
The same commit introduces USER_API_KEY retaining the old value of 
API_KEY. Perhaps this new constant should have been introduced in more 
places.


Ove



commit c211f0bbbe940ed3b9a6e9ef4b5a29be16062e85
Author: Antonio Fornie 
Date:   Fri Mar 7 09:57:31 2014 -0600

Dispatcher corrections, refactoring and tests

Corrects problems from previous attempt. Fixes based on help 
comments from

the community and conflict resolution

Signed-off-by: Daan Hoogland 

diff --git a/api/src/org/apache/cloudstack/api/ApiConstants.java 
b/api/src/org/apache/cloudstack/api/ApiConstants.java

index 14df653..f0de48e 100755
--- a/api/src/org/apache/cloudstack/api/ApiConstants.java
+++ b/api/src/org/apache/cloudstack/api/ApiConstants.java
@@ -23,7 +23,8 @@ public class ApiConstants {
 public static final String ACCOUNT_ID = "accountid";
 public static final String ALGORITHM = "algorithm";
 public static final String ALLOCATED_ONLY = "allocatedonly";
-public static final String API_KEY = "userapikey";
+public static final String API_KEY = "apikey";
+public static final String USER_API_KEY = "userapikey";
 public static final String APPLIED = "applied";
 public static final String AVAILABLE = "available";
 public static final String BITS = "bits";




Cheers
Antonio



2014-05-06 0:44 GMT+02:00 Ove Ewerlid :


On 05/04/2014 10:19 PM, Ove Ewerlid wrote:


See https://issues.apache.org/jira/browse/CLOUDSTACK-6570
Suggested patch in comment.
/Ove



Looping in the person that 'git blame' thinks changed the value of API_KEY
in ApiConstants the last time.

Antonio, was USER_API_KEY introduced for cases where the "userapikey"
value needed to be preserved, such as  in the UpdateUserCmd.java API call,
and that this update was not done for this API?

/Ove

NB; "git blame" may not be accurate hence I may be looping in someone that
has nothing to do with this. Sorry if that is the case.


[oewerlid@amd-a cloudstack]$ git blame 
api/src/org/apache/cloudstack/api/ApiConstants.java
| grep "API_KEY"
c211f0bb api/src/org/apache/cloudstack/api/ApiConstants.java (Antonio
Fornie2014-03-07 09:57:31 -0600  26) public static final
String API_KEY = "apikey";
c211f0bb api/src/org/apache/cloudstack/api/ApiConstants.java (Antonio
Fornie2014-03-07 09:57:31 -0600  27) public static final
String USER_API_KEY = "userapikey";
[oewerlid@amd-a cloudstack]$




--
Ove Everlid
System Administrator / Architect / SDN- & Automation- & Linux-hacker
Mobile: +46706662363 (dedicated work mobile)
Country: Sweden, timezone; Middle Europan Time (MET or GMT+1)






--
Ove Everlid
System Administrator / Architect / SDN- & Automation- & Linux-hacker
Mobile: +46706662363 (dedicated work mobile)
Country: Sweden, timezone; Middle Europan Time (MET or GMT+1)


Re: Heart Bleed - Apache Cloud Stack - open ssl

2014-05-06 Thread David Bierce
That blog posted mentions new templates being created for previous versions of 
Cloudstack.  The post implies a few days and it has now been a month.  Have new 
templates been created and the blog post hasn’t been updated or are those still 
pending release?




On May 6, 2014, at 8:18 AM, David Nalley  wrote:

> Hi Prakash:
> 
> You might want to check out:
> https://blogs.apache.org/cloudstack/entry/how_to_mitigate_openssl_heartbleed
> 
> 
> On Mon, May 5, 2014 at 1:48 PM, Prakash Rao Banuka > wrote:
> 
>> Hi:
>> 
>> It is learnt that openssl library is being called when console of system
>> vms (secondarystorage , console proxy vm and router vm) is launched.
>> 
>> Is it mandatory to update openssl library in Apache Cloud Stack? If yes,
>> is there any official document.
>> 
>> Thank you
>> Prakash
>> 
>> 
>> 



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

2014-05-06 Thread jenkins
See 

--
Started by timer
[EnvInject] - Loading node environment variables.
Building remotely on cloudstack-buildslave-centos6-f9c 
(cloudstack-buildslave-centos6) in workspace 

Fetching changes from the remote Git repository
Fetching upstream changes from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
ERROR: Timeout after 400 minutes
FATAL: Failed to fetch from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
hudson.plugins.git.GitException: Failed to fetch from 
https://git-wip-us.apache.org/repos/asf/cloudstack.git
at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:625)
at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:847)
at hudson.plugins.git.GitSCM.checkout(GitSCM.java:872)
at hudson.model.AbstractProject.checkout(AbstractProject.java:1320)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:609)
at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:88)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:518)
at hudson.model.Run.execute(Run.java:1688)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:88)
at hudson.model.Executor.run(Executor.java:231)
Caused by: hudson.plugins.git.GitException: Command "git fetch --tags 
--progress https://git-wip-us.apache.org/repos/asf/cloudstack.git 
+refs/heads/*:refs/remotes/origin/*" returned status code 128:
stdout: 
stderr: error: RPC failed; result=52, HTTP code = 100
error: fetch-pack died of signal 15

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1086)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandWithCredentials(CliGitAPIImpl.java:968)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.access$200(CliGitAPIImpl.java:71)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl$1.execute(CliGitAPIImpl.java:197)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$1.call(RemoteGitImpl.java:152)
at 
org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$1.call(RemoteGitImpl.java:145)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:328)
at 
hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:72)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
at java.util.concurrent.FutureTask.run(FutureTask.java:166)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:701)


Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-06 Thread Alena Prokharchyk


> On May 5, 2014, 6:28 p.m., Alena Prokharchyk wrote:
> > Alex,
> > 
> > Move all new API commands/responses you've added 
> > (ListSyncAccountsCmdByAdmin.java, SyncAccountResponse.java), to your 
> > plugin's "multiregion" folder. Look how its done for other CS plugins - 
> > internallb for example
> 
> Alex Ough wrote:
> I checked the 'internallb' and it looks like the cmd class files are 
> under "api/src/org/apache/cloudstack/api/command/admin/internallb", where all 
> the command classes are located. Please let me know if I'm missing any.

Sorry, internallb plugin doesn't quite follow the CS packaging convention; we 
will have to change it eventually. Refer to cloud-plugin-network-vns instead. 
Under the plugin folder, there is an "api" folder. The folder contains 2 
subfolders - commands (where Cmd classes are kept) and response (for API 
responses). Thats the right way to ship API commands/responses when they are a 
part of the plugin. 

Alex, you can also refer to this doc for more details if needed 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+Coding+Guidelines.
 It explains CS coding guidelines when it comes to API development/placement.


- Alena


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


On May 4, 2014, 9:18 p.m., Alex Ough wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17790/
> ---
> 
> (Updated May 4, 2014, 9:18 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> 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
> -
> 
>   
> api/src/org/apache/cloudstack/api/command/admin/account/ListSyncAccountsCmdByAdmin.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/domain/ListSyncDomainChildrenCmd.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/domain/ListSyncDomainsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/command/admin/user/ListSyncUsersCmd.java 
> PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/user/account/ListSyncAccountsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncAccountResponse.java 
> PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncDomainResponse.java 
> PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncUserResponse.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/RmapVO.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/RsyncVO.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RmapDao.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RmapDaoImpl.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RsyncDao.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RsyncDaoImpl.java 
> PRE-CREATION 
>   plugins/event-bus/multiregion/pom.xml PRE-CREATION 
>   
> plugins/event-bus/multiregion/resources/META-INF/cloudstack/spring-mom-multiregion-daos-context.xml
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/resources/META-INF/cloudstack/system/spring-plugin-multiregion-system-context.xml
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/FullSyncer.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/InjectedCollection.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/MultiRegionEventBus.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/StringManipulator.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/AccountCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/BaseCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudsta

Re: [ACS44] cherry pick

2014-05-06 Thread Daan Hoogland
3 commits pulled in

On Tue, May 6, 2014 at 4:18 PM, Jayapal Reddy Uradi
 wrote:
> Hi Daan,
>
> Can you please cherry-pick the following commits  from 4.4-forward.
>
> commit a708d5c4982595666cfe8fe03510517cfefe1326
> Author: Jayapal 
> Date:   Mon May 5 13:45:51 2014 +0530
>
> CLOUDSTACK-6577: Disable service monitoring in RVR
>
> commit 758f7f2f16d361c40bf61db1e7fd799efe9827db
> Author: Jayapal 
> Date:   Mon May 5 13:56:59 2014 +0530
>
> CLOUDSTACK-6578: Fixed issue in delete remote access vpn command
>
> commit 645516ee78a8117dfa221caa0fc8d4dfe521af2b
> Author: Rajani Karuturi 
> Date:   Mon May 5 15:31:35 2014 +0530
>
> CLOUDSTACK-6531: stopping the router in case of command failures. Also 
> added alerts for failures.
>
> Signed-off-by: Jayapal 
>
> Thanks,
> Jayapal



-- 
Daan


Re: Review Request 20652: CLOUDSTACK-6531: VR alerts for command failures during start and stop the router incase of failures

2014-05-06 Thread ASF Subversion and Git Services

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


Commit c081cf7b1ad7e45a228e7ad1fc62a428cff34d57 in cloudstack's branch 
refs/heads/4.4 from Rajani Karuturi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c081cf7 ]

CLOUDSTACK-6531: stopping the router in case of command failures. Also added 
alerts for failures.

Signed-off-by: Jayapal 


- ASF Subversion and Git Services


On May 5, 2014, 10:02 a.m., Rajani Karuturi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20652/
> ---
> 
> (Updated May 5, 2014, 10:02 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Jayapal Reddy, and Murali 
> Reddy.
> 
> 
> Bugs: CLOUDSTACK-6531
> https://issues.apache.org/jira/browse/CLOUDSTACK-6531
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> added a new configuration router.start.strict to optionally stop the router 
> in case of command failures. Also added alerts for failures.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
> 80ca1cf 
> 
> Diff: https://reviews.apache.org/r/20652/diff/
> 
> 
> Testing
> ---
> 
> Manually tested.
> 
> 
> Thanks,
> 
> Rajani Karuturi
> 
>



Re: [ACS44] Cherry pick for cloudstack-6472

2014-05-06 Thread Daan Hoogland
both pulled

On Tue, May 6, 2014 at 3:01 PM, Sebastien Goasguen  wrote:
> Daan,
>
> Can you cherry pick from 4.4-forward:
>
> f328f6157b682abc5bce007400f1cdf476097745
>
> and
>
> fe84b674c14b6494770b489100e92a116b3b1f49
>
> thanks
>
> -sebastien



-- 
Daan


Re: Review Request 21015: CLOUDSTACK-6472 listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 3a3457e7132d22f52aa38179d40a6eb9b0b29677 in cloudstack's branch 
refs/heads/4.4 from Sam Schmit
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3a3457e ]

CLOUDSTACK-6472 listUsageRecords: Pull information from removed items as well, 
fixing NPEs/Null UUIDs with usage API calls.


- ASF Subversion and Git Services


On May 2, 2014, 6:15 p.m., Sam Schmit wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21015/
> ---
> 
> (Updated May 2, 2014, 6:15 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6472
> https://issues.apache.org/jira/browse/CLOUDSTACK-6472
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Running "listUsageRecords" when there were any destroyed/expunged Virtual 
> Machines would return errors (on 4.3) or return null UUIDs (on master).  The 
> UUID is important for correlating records.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiResponseHelper.java 250f5a9 
> 
> Diff: https://reviews.apache.org/r/21015/diff/
> 
> 
> Testing
> ---
> 
> 1)  Created virtual machine
> 2)  Ran usage
> 3)  ran "listUsageRecords" API call, successful data returned
> 4)  Expunged VM
> 5)  Ran usage
> 6)  ran "listUsageRecords" API call, successful data returned (this would 
> previously return an error)
> 
> 
> Thanks,
> 
> Sam Schmit
> 
>



Re: [ACS44] 4.4.0 blokker issues

2014-05-06 Thread Daan Hoogland
I will look at your patch, I created one myself that conflict so I
will see if I can merge.

On Tue, May 6, 2014 at 4:37 PM, Jayapal Reddy Uradi
 wrote:
> Hi Daan,
>
> While working on 
> CLOUDSTACK-6084 fixed 
> few issues for CLOUDSTACK-6582 and uploaded the patch in the bug.
> There are still few issues in network acl cidr changes.
>
> CLOUDSTACK-6582 and CLOUDSTACK-6485 fixes created regression for vpc create 
> private gateway CLOUDSTACK-6084.
>
> Thanks,
> Jayapal
>
>
> On 06-May-2014, at 7:25 PM, Rajesh Battala 
> mailto:rajesh.batt...@citrix.com>>
>  wrote:
>
> CLOUDSTACK-6295,  I have commented on it and closed it.
>
> Thanks
> Rajesh Battala
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Tuesday, May 6, 2014 6:24 PM
> To: dev
> Subject: [ACS44] 4.4.0 blokker issues
>
> At the moment we have 14 blokker issues in jira. Can all reporters and 
> assignees please have a look to see if these are really blokkers and if they 
> can be resolved before long?
>
> Key:Summary:Reporter:Assignee
> CLOUDSTACK-6130
> [Automation] listPublicAdress is failing Srikanteswararao Talluri Hugo 
> Trippaers
>
> CLOUDSTACK-6245
> Security group rules on hypervisor host are lagging behind rules in DB edison 
> su edison su
>
> CLOUDSTACK-6295
> Management server cannot start because Java 7 missing Paul Angus Unassigned
>
> CLOUDSTACK-6444
> [Automation] test_01_primary_storage_iscsi failed on 
> "test_primary_storage.py" - Wrong iscsi path format - it should be 
> /targetIQN/LUN Chandan Purushothama Chandan Purushothama
>
> CLOUDSTACK-6453
> [GPU] Windows 2012 Server instance created with vGPU offering is not coming 
> up after installing PV drivers Sailaja Mada Sanjay Tripathi
>
> CLOUDSTACK-6544
> [Automation] Failed to create template for ROOT volume in Xen, with
> Exception: callHostPlugin failed
> Rayees Namathponnan Harikrishna Patnala
>
> CLOUDSTACK-6547
> [Automation] Failed to download volume in Xen with error "Failed to copy the 
> volume from pri stor pool to sec stor pool"
> Rayees Namathponnan Nitin Mehta
>
> CLOUDSTACK-6548
> [Automation] createPrivateNetwork API failed with NullPointer exception 
> Rayees Namathponnan Santhosh Kumar Edukulla
>
> CLOUDSTACK-6551
> [Automation] Failed to revert vm snapshot in xen Rayees Namathponnan 
> Harikrishna Patnala
>
> CLOUDSTACK-6553
> [Automation] Resize volume test case failing with error "value=invalid id due 
> to incorrect long value format"
> Rayees Namathponnan Rajani Karuturi
>
> CLOUDSTACK-6570
> API breakage of the UpdateUser API call
> Ove Ewerlid Unassigned
>
> CLOUDSTACK-6572
> [Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic Sowmya 
> Krishnan Unassigned
>
> CLOUDSTACK-6573
> Multiple exceptions after upgrading from 4.3 to 4.4 manasaveloori Unassigned
>
> CLOUDSTACK-6582
> Listing acl list rules are failed with exception Unknown column 
> 'network_acl_item.cidr'
> Jayapal Reddy Unassigned
>
> I am taking the last one.
> --
> Daan
>



-- 
Daan


Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-06 Thread Alex Ough


> On May 5, 2014, 6:28 p.m., Alena Prokharchyk wrote:
> > Alex,
> > 
> > Move all new API commands/responses you've added 
> > (ListSyncAccountsCmdByAdmin.java, SyncAccountResponse.java), to your 
> > plugin's "multiregion" folder. Look how its done for other CS plugins - 
> > internallb for example

I checked the 'internallb' and it looks like the cmd class files are under 
"api/src/org/apache/cloudstack/api/command/admin/internallb", where all the 
command classes are located. Please let me know if I'm missing any.


- Alex


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


On May 4, 2014, 9:18 p.m., Alex Ough wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17790/
> ---
> 
> (Updated May 4, 2014, 9:18 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> 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
> -
> 
>   
> api/src/org/apache/cloudstack/api/command/admin/account/ListSyncAccountsCmdByAdmin.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/domain/ListSyncDomainChildrenCmd.java
>  PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/admin/domain/ListSyncDomainsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/command/admin/user/ListSyncUsersCmd.java 
> PRE-CREATION 
>   
> api/src/org/apache/cloudstack/api/command/user/account/ListSyncAccountsCmd.java
>  PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncAccountResponse.java 
> PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncDomainResponse.java 
> PRE-CREATION 
>   api/src/org/apache/cloudstack/api/response/SyncUserResponse.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/RmapVO.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/RsyncVO.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RmapDao.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RmapDaoImpl.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RsyncDao.java 
> PRE-CREATION 
>   engine/schema/src/org/apache/cloudstack/multiregion/dao/RsyncDaoImpl.java 
> PRE-CREATION 
>   plugins/event-bus/multiregion/pom.xml PRE-CREATION 
>   
> plugins/event-bus/multiregion/resources/META-INF/cloudstack/spring-mom-multiregion-daos-context.xml
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/resources/META-INF/cloudstack/system/spring-plugin-multiregion-system-context.xml
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/FullSyncer.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/InjectedCollection.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/MultiRegionEventBus.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/StringManipulator.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/AccountCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/BaseCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/DomainCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/api/UserCaller.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/exception/APIFailureException.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/service/AccountFullSyncProcessor.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/service/AccountService.java
>  PRE-CREATION 
>   
> plugins/event-bus/multiregion/src/org/apache/cloudstack/mom/multiregion/service/BaseService.java
>  PRE

Re: Review Request 20652: CLOUDSTACK-6531: VR alerts for command failures during start and stop the router incase of failures

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 59bf3559196a9452a1a048849f4f9971753d373b in cloudstack's branch 
refs/heads/master from Rajani Karuturi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=59bf355 ]

CLOUDSTACK-6531: stopping the router in case of command failures. Also added 
alerts for failures.

Signed-off-by: Jayapal 


- ASF Subversion and Git Services


On May 5, 2014, 10:02 a.m., Rajani Karuturi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20652/
> ---
> 
> (Updated May 5, 2014, 10:02 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Jayapal Reddy, and Murali 
> Reddy.
> 
> 
> Bugs: CLOUDSTACK-6531
> https://issues.apache.org/jira/browse/CLOUDSTACK-6531
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> added a new configuration router.start.strict to optionally stop the router 
> in case of command failures. Also added alerts for failures.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
> 80ca1cf 
> 
> Diff: https://reviews.apache.org/r/20652/diff/
> 
> 
> Testing
> ---
> 
> Manually tested.
> 
> 
> Thanks,
> 
> Rajani Karuturi
> 
>



Re: [ACS44] 4.4.0 blokker issues

2014-05-06 Thread Jayapal Reddy Uradi
Hi Daan,

While working on 
CLOUDSTACK-6084 fixed 
few issues for CLOUDSTACK-6582 and uploaded the patch in the bug.
There are still few issues in network acl cidr changes.

CLOUDSTACK-6582 and CLOUDSTACK-6485 fixes created regression for vpc create 
private gateway CLOUDSTACK-6084.

Thanks,
Jayapal


On 06-May-2014, at 7:25 PM, Rajesh Battala 
mailto:rajesh.batt...@citrix.com>>
 wrote:

CLOUDSTACK-6295,  I have commented on it and closed it.

Thanks
Rajesh Battala

-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: Tuesday, May 6, 2014 6:24 PM
To: dev
Subject: [ACS44] 4.4.0 blokker issues

At the moment we have 14 blokker issues in jira. Can all reporters and 
assignees please have a look to see if these are really blokkers and if they 
can be resolved before long?

Key:Summary:Reporter:Assignee
CLOUDSTACK-6130
[Automation] listPublicAdress is failing Srikanteswararao Talluri Hugo Trippaers

CLOUDSTACK-6245
Security group rules on hypervisor host are lagging behind rules in DB edison 
su edison su

CLOUDSTACK-6295
Management server cannot start because Java 7 missing Paul Angus Unassigned

CLOUDSTACK-6444
[Automation] test_01_primary_storage_iscsi failed on "test_primary_storage.py" 
- Wrong iscsi path format - it should be /targetIQN/LUN Chandan Purushothama 
Chandan Purushothama

CLOUDSTACK-6453
[GPU] Windows 2012 Server instance created with vGPU offering is not coming up 
after installing PV drivers Sailaja Mada Sanjay Tripathi

CLOUDSTACK-6544
[Automation] Failed to create template for ROOT volume in Xen, with
Exception: callHostPlugin failed
Rayees Namathponnan Harikrishna Patnala

CLOUDSTACK-6547
[Automation] Failed to download volume in Xen with error "Failed to copy the 
volume from pri stor pool to sec stor pool"
Rayees Namathponnan Nitin Mehta

CLOUDSTACK-6548
[Automation] createPrivateNetwork API failed with NullPointer exception Rayees 
Namathponnan Santhosh Kumar Edukulla

CLOUDSTACK-6551
[Automation] Failed to revert vm snapshot in xen Rayees Namathponnan 
Harikrishna Patnala

CLOUDSTACK-6553
[Automation] Resize volume test case failing with error "value=invalid id due 
to incorrect long value format"
Rayees Namathponnan Rajani Karuturi

CLOUDSTACK-6570
API breakage of the UpdateUser API call
Ove Ewerlid Unassigned

CLOUDSTACK-6572
[Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic Sowmya 
Krishnan Unassigned

CLOUDSTACK-6573
Multiple exceptions after upgrading from 4.3 to 4.4 manasaveloori Unassigned

CLOUDSTACK-6582
Listing acl list rules are failed with exception Unknown column 
'network_acl_item.cidr'
Jayapal Reddy Unassigned

I am taking the last one.
--
Daan



[ACS44] cherry pick

2014-05-06 Thread Jayapal Reddy Uradi
Hi Daan,

Can you please cherry-pick the following commits  from 4.4-forward.

commit a708d5c4982595666cfe8fe03510517cfefe1326
Author: Jayapal 
Date:   Mon May 5 13:45:51 2014 +0530

CLOUDSTACK-6577: Disable service monitoring in RVR

commit 758f7f2f16d361c40bf61db1e7fd799efe9827db
Author: Jayapal 
Date:   Mon May 5 13:56:59 2014 +0530

CLOUDSTACK-6578: Fixed issue in delete remote access vpn command

commit 645516ee78a8117dfa221caa0fc8d4dfe521af2b
Author: Rajani Karuturi 
Date:   Mon May 5 15:31:35 2014 +0530

CLOUDSTACK-6531: stopping the router in case of command failures. Also 
added alerts for failures.

Signed-off-by: Jayapal 

Thanks,
Jayapal

Re: [ACS44] 4.4.0 blokker issues

2014-05-06 Thread Daan Hoogland
thanks

On Tue, May 6, 2014 at 3:55 PM, Rajesh Battala
 wrote:
> CLOUDSTACK-6295,  I have commented on it and closed it.
>
> Thanks
> Rajesh Battala
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Tuesday, May 6, 2014 6:24 PM
> To: dev
> Subject: [ACS44] 4.4.0 blokker issues
>
> At the moment we have 14 blokker issues in jira. Can all reporters and 
> assignees please have a look to see if these are really blokkers and if they 
> can be resolved before long?
>
> Key:Summary:Reporter:Assignee
> CLOUDSTACK-6130
> [Automation] listPublicAdress is failing Srikanteswararao Talluri Hugo 
> Trippaers
>
> CLOUDSTACK-6245
> Security group rules on hypervisor host are lagging behind rules in DB edison 
> su edison su
>
> CLOUDSTACK-6295
> Management server cannot start because Java 7 missing Paul Angus Unassigned
>
> CLOUDSTACK-6444
> [Automation] test_01_primary_storage_iscsi failed on 
> "test_primary_storage.py" - Wrong iscsi path format - it should be 
> /targetIQN/LUN Chandan Purushothama Chandan Purushothama
>
> CLOUDSTACK-6453
> [GPU] Windows 2012 Server instance created with vGPU offering is not coming 
> up after installing PV drivers Sailaja Mada Sanjay Tripathi
>
> CLOUDSTACK-6544
> [Automation] Failed to create template for ROOT volume in Xen, with
> Exception: callHostPlugin failed
> Rayees Namathponnan Harikrishna Patnala
>
> CLOUDSTACK-6547
> [Automation] Failed to download volume in Xen with error "Failed to copy the 
> volume from pri stor pool to sec stor pool"
> Rayees Namathponnan Nitin Mehta
>
> CLOUDSTACK-6548
> [Automation] createPrivateNetwork API failed with NullPointer exception 
> Rayees Namathponnan Santhosh Kumar Edukulla
>
> CLOUDSTACK-6551
> [Automation] Failed to revert vm snapshot in xen Rayees Namathponnan 
> Harikrishna Patnala
>
> CLOUDSTACK-6553
> [Automation] Resize volume test case failing with error "value=invalid id due 
> to incorrect long value format"
> Rayees Namathponnan Rajani Karuturi
>
> CLOUDSTACK-6570
> API breakage of the UpdateUser API call
> Ove Ewerlid Unassigned
>
> CLOUDSTACK-6572
> [Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic Sowmya 
> Krishnan Unassigned
>
> CLOUDSTACK-6573
> Multiple exceptions after upgrading from 4.3 to 4.4 manasaveloori Unassigned
>
> CLOUDSTACK-6582
> Listing acl list rules are failed with exception Unknown column 
> 'network_acl_item.cidr'
> Jayapal Reddy Unassigned
>
> I am taking the last one.
> --
> Daan



-- 
Daan


Re: Review Request 20652: CLOUDSTACK-6531: VR alerts for command failures during start and stop the router incase of failures

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 645516ee78a8117dfa221caa0fc8d4dfe521af2b in cloudstack's branch 
refs/heads/4.4-forward from Rajani Karuturi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=645516e ]

CLOUDSTACK-6531: stopping the router in case of command failures. Also added 
alerts for failures.

Signed-off-by: Jayapal 


- ASF Subversion and Git Services


On May 5, 2014, 10:02 a.m., Rajani Karuturi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20652/
> ---
> 
> (Updated May 5, 2014, 10:02 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Jayapal Reddy, and Murali 
> Reddy.
> 
> 
> Bugs: CLOUDSTACK-6531
> https://issues.apache.org/jira/browse/CLOUDSTACK-6531
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> added a new configuration router.start.strict to optionally stop the router 
> in case of command failures. Also added alerts for failures.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
> 80ca1cf 
> 
> Diff: https://reviews.apache.org/r/20652/diff/
> 
> 
> Testing
> ---
> 
> Manually tested.
> 
> 
> Thanks,
> 
> Rajani Karuturi
> 
>



RE: [ACS44] 4.4.0 blokker issues

2014-05-06 Thread Rajesh Battala
CLOUDSTACK-6295,  I have commented on it and closed it. 

Thanks
Rajesh Battala

-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] 
Sent: Tuesday, May 6, 2014 6:24 PM
To: dev
Subject: [ACS44] 4.4.0 blokker issues

At the moment we have 14 blokker issues in jira. Can all reporters and 
assignees please have a look to see if these are really blokkers and if they 
can be resolved before long?

Key:Summary:Reporter:Assignee
CLOUDSTACK-6130
[Automation] listPublicAdress is failing Srikanteswararao Talluri Hugo Trippaers

CLOUDSTACK-6245
Security group rules on hypervisor host are lagging behind rules in DB edison 
su edison su

CLOUDSTACK-6295
Management server cannot start because Java 7 missing Paul Angus Unassigned

CLOUDSTACK-6444
[Automation] test_01_primary_storage_iscsi failed on "test_primary_storage.py" 
- Wrong iscsi path format - it should be /targetIQN/LUN Chandan Purushothama 
Chandan Purushothama

CLOUDSTACK-6453
[GPU] Windows 2012 Server instance created with vGPU offering is not coming up 
after installing PV drivers Sailaja Mada Sanjay Tripathi

CLOUDSTACK-6544
[Automation] Failed to create template for ROOT volume in Xen, with
Exception: callHostPlugin failed
Rayees Namathponnan Harikrishna Patnala

CLOUDSTACK-6547
[Automation] Failed to download volume in Xen with error "Failed to copy the 
volume from pri stor pool to sec stor pool"
Rayees Namathponnan Nitin Mehta

CLOUDSTACK-6548
[Automation] createPrivateNetwork API failed with NullPointer exception Rayees 
Namathponnan Santhosh Kumar Edukulla

CLOUDSTACK-6551
[Automation] Failed to revert vm snapshot in xen Rayees Namathponnan 
Harikrishna Patnala

CLOUDSTACK-6553
[Automation] Resize volume test case failing with error "value=invalid id due 
to incorrect long value format"
Rayees Namathponnan Rajani Karuturi

CLOUDSTACK-6570
API breakage of the UpdateUser API call
Ove Ewerlid Unassigned

CLOUDSTACK-6572
[Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic Sowmya 
Krishnan Unassigned

CLOUDSTACK-6573
Multiple exceptions after upgrading from 4.3 to 4.4 manasaveloori Unassigned

CLOUDSTACK-6582
Listing acl list rules are failed with exception Unknown column 
'network_acl_item.cidr'
Jayapal Reddy Unassigned

I am taking the last one.
--
Daan


[GitHub] cloudstack-docs-install pull request: doc for enabling CSP support...

2014-05-06 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack-docs-install/pull/9


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Jenkins build is back to normal : cloudstack-4.4-maven-build-noredist #199

2014-05-06 Thread jenkins
See 




Re: Heart Bleed - Apache Cloud Stack - open ssl

2014-05-06 Thread David Nalley
Hi Prakash:

You might want to check out:
https://blogs.apache.org/cloudstack/entry/how_to_mitigate_openssl_heartbleed


On Mon, May 5, 2014 at 1:48 PM, Prakash Rao Banuka  wrote:

> Hi:
>
> It is learnt that openssl library is being called when console of system
> vms (secondarystorage , console proxy vm and router vm) is launched.
>
> Is it mandatory to update openssl library in Apache Cloud Stack? If yes,
> is there any official document.
>
> Thank you
> Prakash
>
>
>


Re: 4.4 release-notes

2014-05-06 Thread David Nalley
Animesh had to be granted karma on Jira to share filters/dashboards.

--David


On Tue, May 6, 2014 at 9:10 AM, Sebastien Goasguen  wrote:

>
> On May 6, 2014, at 4:34 AM, Daan Hoogland  wrote:
>
> > I think Pierre-Luc's idea is good and infra will surely pick up the
> > ticket one of these days. The dashboard is Animesh's work. I think he
> > used private filters.
>
> I agree, I built the RN for 4.3 based on animesh filters.
>
> I am just wondering why this needs special karma or that the filters are
> private.
>
> >
> > On Tue, May 6, 2014 at 10:19 AM, Sebastien Goasguen 
> wrote:
> >> Maybe we can ask animesh how he did it for 4.3 ...
> >>
> >> -sebastien
> >>
> >>
> >> On Mon, May 5, 2014 at 4:35 PM, Pierre-Luc Dion 
> wrote:
> >>
> >>> Request Created under ASF infrastructure team; INFRA project.
> >>>
> >>> Thanks David,
> >>>
> >>>
> >>> Pierre-Luc Dion
> >>> Architecte de Solution Cloud | Cloud Solutions Architect
> >>> 855-OK-CLOUD (855-652-5683) x1101
> >>> - - -
> >>>
> >>> *CloudOps*420 rue Guy
> >>> Montréal QC  H3J 1S6
> >>> www.cloudops.com
> >>> @CloudOps_
> >>>
> >>>
> >>> On Mon, May 5, 2014 at 4:17 PM, David Nalley  wrote:
> >>>
>  You'll need to file a ticket with infra.
> 
>  On Mon, May 5, 2014 at 2:52 PM, Daan Hoogland <
> daan.hoogl...@gmail.com>
>  wrote:
> > Animesh,
> >
> > Can you (give me rights to) add these filters to the release
> dashboard?
> >
> > On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion 
>  wrote:
> >> 4.4.0 Fixed Issues
> >> project = CLOUDSTACK AND type = Bug AND affectedVersion in (4.2.0,
>  4.2.1,
> >> 4.3.0) AND fixVersion = 4.4.0 AND resolution != "\"Unresolved\""
> ORDER
>  BY
> >> created DESC, priority DESC, key ASC
> >>
> >> 4.4.0 Known Issues
> >> project = CLOUDSTACK AND type = Bug AND (affectedVersion = 4.4.0 OR
> >> fixVersion = 4.4.0) AND resolution is EMPTY AND level = "Public"
> ORDER
>  BY
> >> priority DESC, key ASC
> >>
> >
> >
> >
> > --
> > Daan
> 
> >>>
> >>
> >>
> >>
> >> --
> >> ---
> >> Sebastien Goasguen
> >> Open Source Cloud Evangelist, Cloudstack
> >> http://sites.google.com/site/runseb/
> >
> >
> >
> > --
> > Daan
>
>


Re: 4.4 release-notes

2014-05-06 Thread Sebastien Goasguen

On May 6, 2014, at 4:34 AM, Daan Hoogland  wrote:

> I think Pierre-Luc's idea is good and infra will surely pick up the
> ticket one of these days. The dashboard is Animesh's work. I think he
> used private filters.

I agree, I built the RN for 4.3 based on animesh filters.

I am just wondering why this needs special karma or that the filters are 
private.

> 
> On Tue, May 6, 2014 at 10:19 AM, Sebastien Goasguen  wrote:
>> Maybe we can ask animesh how he did it for 4.3 ...
>> 
>> -sebastien
>> 
>> 
>> On Mon, May 5, 2014 at 4:35 PM, Pierre-Luc Dion  wrote:
>> 
>>> Request Created under ASF infrastructure team; INFRA project.
>>> 
>>> Thanks David,
>>> 
>>> 
>>> Pierre-Luc Dion
>>> Architecte de Solution Cloud | Cloud Solutions Architect
>>> 855-OK-CLOUD (855-652-5683) x1101
>>> - - -
>>> 
>>> *CloudOps*420 rue Guy
>>> Montréal QC  H3J 1S6
>>> www.cloudops.com
>>> @CloudOps_
>>> 
>>> 
>>> On Mon, May 5, 2014 at 4:17 PM, David Nalley  wrote:
>>> 
 You'll need to file a ticket with infra.
 
 On Mon, May 5, 2014 at 2:52 PM, Daan Hoogland 
 wrote:
> Animesh,
> 
> Can you (give me rights to) add these filters to the release dashboard?
> 
> On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion 
 wrote:
>> 4.4.0 Fixed Issues
>> project = CLOUDSTACK AND type = Bug AND affectedVersion in (4.2.0,
 4.2.1,
>> 4.3.0) AND fixVersion = 4.4.0 AND resolution != "\"Unresolved\"" ORDER
 BY
>> created DESC, priority DESC, key ASC
>> 
>> 4.4.0 Known Issues
>> project = CLOUDSTACK AND type = Bug AND (affectedVersion = 4.4.0 OR
>> fixVersion = 4.4.0) AND resolution is EMPTY AND level = "Public" ORDER
 BY
>> priority DESC, key ASC
>> 
> 
> 
> 
> --
> Daan
 
>>> 
>> 
>> 
>> 
>> --
>> ---
>> Sebastien Goasguen
>> Open Source Cloud Evangelist, Cloudstack
>> http://sites.google.com/site/runseb/
> 
> 
> 
> -- 
> Daan



[ACS44] Cherry pick for cloudstack-6472

2014-05-06 Thread Sebastien Goasguen
Daan,

Can you cherry pick from 4.4-forward:

f328f6157b682abc5bce007400f1cdf476097745

and

fe84b674c14b6494770b489100e92a116b3b1f49

thanks

-sebastien


Re: Review Request 21088: CLOUDSTACK-6472 (4.3-specific) listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-06 Thread Sebastien Goasguen

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

Ship it!


thanks for the patch, you can close the review as submitted

- Sebastien Goasguen


On May 5, 2014, 9:44 p.m., Sam Schmit wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21088/
> ---
> 
> (Updated May 5, 2014, 9:44 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6472
> https://issues.apache.org/jira/browse/CLOUDSTACK-6472
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Running "listUsageRecords" when there were any destroyed/expunged Virtual 
> Machines would return errors (on 4.3) or return null UUIDs (on master).  The 
> UUID is important for correlating records.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiResponseHelper.java a8bef47 
>   utils/src/com/cloud/utils/db/EntityManager.java d613d7d 
> 
> Diff: https://reviews.apache.org/r/21088/diff/
> 
> 
> Testing
> ---
> 
> 1)  Created virtual machine
> 2)  Ran usage
> 3)  ran "listUsageRecords" API call, successful data returned
> 4)  Expunged VM
> 5)  Ran usage
> 6)  ran "listUsageRecords" API call, successful data returned (this would 
> previously return an error)
> 
> 
> Thanks,
> 
> Sam Schmit
> 
>



Re: Review Request 21088: CLOUDSTACK-6472 (4.3-specific) listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-06 Thread ASF Subversion and Git Services

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


Commit e8047e11db7ef2bdc44bbedfdc47e63c55f080c5 in cloudstack's branch 
refs/heads/4.3 from Sam Schmit
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e8047e1 ]

CLOUDSTACK-6472 (4.3 specific) listUsageRecords: Pull information from removed 
items as well, fixing NPEs/Null UUIDs with usage API calls.

Signed-off-by: Sebastien Goasguen 


- ASF Subversion and Git Services


On May 5, 2014, 9:44 p.m., Sam Schmit wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21088/
> ---
> 
> (Updated May 5, 2014, 9:44 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6472
> https://issues.apache.org/jira/browse/CLOUDSTACK-6472
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Running "listUsageRecords" when there were any destroyed/expunged Virtual 
> Machines would return errors (on 4.3) or return null UUIDs (on master).  The 
> UUID is important for correlating records.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiResponseHelper.java a8bef47 
>   utils/src/com/cloud/utils/db/EntityManager.java d613d7d 
> 
> Diff: https://reviews.apache.org/r/21088/diff/
> 
> 
> Testing
> ---
> 
> 1)  Created virtual machine
> 2)  Ran usage
> 3)  ran "listUsageRecords" API call, successful data returned
> 4)  Expunged VM
> 5)  Ran usage
> 6)  ran "listUsageRecords" API call, successful data returned (this would 
> previously return an error)
> 
> 
> Thanks,
> 
> Sam Schmit
> 
>



RE: Cleanly remove management server

2014-05-06 Thread Michael Phillips
All I have done at this point is to create a JIRA request for this...

> From: daan.hoogl...@gmail.com
> Date: Mon, 5 May 2014 16:25:19 +0200
> Subject: Re: Cleanly remove management server
> To: dev@cloudstack.apache.org
> 
> H Michael,
> 
> Did you investigate this further? still planning to implement?
> 
> 
> On Fri, Apr 18, 2014 at 3:33 PM, Michael Phillips
>  wrote:
> > Last week we talked about a GUI option to cleanly remove a management 
> > server. I created a feature request for this as well. Hopefully this will 
> > get some traction as well.
> > https://issues.apache.org/jira/browse/CLOUDSTACK-6455
> > I think I may dust up on my coding skills so I can contribute to the dev 
> > cause.
> 
> 
> 
> -- 
> Daan
  

[ACS44] 4.4.0 blokker issues

2014-05-06 Thread Daan Hoogland
At the moment we have 14 blokker issues in jira. Can all reporters and
assignees please have a look to see if these are really blokkers and
if they can be resolved before long?

Key:Summary:Reporter:Assignee
CLOUDSTACK-6130
[Automation] listPublicAdress is failing
Srikanteswararao Talluri Hugo Trippaers

CLOUDSTACK-6245
Security group rules on hypervisor host are lagging behind rules in DB
edison su edison su

CLOUDSTACK-6295
Management server cannot start because Java 7 missing
Paul Angus Unassigned

CLOUDSTACK-6444
[Automation] test_01_primary_storage_iscsi failed on
"test_primary_storage.py" - Wrong iscsi path format - it should be
/targetIQN/LUN
Chandan Purushothama Chandan Purushothama

CLOUDSTACK-6453
[GPU] Windows 2012 Server instance created with vGPU offering is not
coming up after installing PV drivers
Sailaja Mada Sanjay Tripathi

CLOUDSTACK-6544
[Automation] Failed to create template for ROOT volume in Xen, with
Exception: callHostPlugin failed
Rayees Namathponnan Harikrishna Patnala

CLOUDSTACK-6547
[Automation] Failed to download volume in Xen with error "Failed to
copy the volume from pri stor pool to sec stor pool"
Rayees Namathponnan Nitin Mehta

CLOUDSTACK-6548
[Automation] createPrivateNetwork API failed with NullPointer exception
Rayees Namathponnan Santhosh Kumar Edukulla

CLOUDSTACK-6551
[Automation] Failed to revert vm snapshot in xen
Rayees Namathponnan Harikrishna Patnala

CLOUDSTACK-6553
[Automation] Resize volume test case failing with error "value=invalid
id due to incorrect long value format"
Rayees Namathponnan Rajani Karuturi

CLOUDSTACK-6570
API breakage of the UpdateUser API call
Ove Ewerlid Unassigned

CLOUDSTACK-6572
[Hyper-V] Deploy VM inside VPC tier fails due to VR unable to find nic
Sowmya Krishnan Unassigned

CLOUDSTACK-6573
Multiple exceptions after upgrading from 4.3 to 4.4
manasaveloori Unassigned

CLOUDSTACK-6582
Listing acl list rules are failed with exception Unknown column
'network_acl_item.cidr'
Jayapal Reddy Unassigned

I am taking the last one.
-- 
Daan


RE: StretchedL2Subnet

2014-05-06 Thread Ian Southam
Thanks!

-Original Message-
From: murali reddy [mailto:muralimmre...@gmail.com] 
Sent: dinsdag 6 mei 2014 14:11
To: dev@cloudstack.apache.org
Subject: Re: StretchedL2Subnet

On Tue, May 6, 2014 at 5:30 PM, Ian Southam wrote:

> Anybody know what this is?
>
> I see our Integration testing (for Nicira) has started failing of late.
>  We are not requesting such a network so I presume a default has 
> changed somewhere?
>

I added this as capability to 'Connectivity' service providers. This should not 
be affecting default behaviour. Let me check on this,


>
> 
>
> ERROR:
> test-create-nicira-network-offerings.testNetworkOffering.testCreateNet
> workOfferingNiciraL2
> --
> Traceback (most recent call last):
>   File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 
> 197, in runTest
> self.test(*self.arg)
>   File "/home/jenkins/slave/workspace/team_tkt/Cloudstack Integration 
> Testing/build-cloud/scripts/test-create-nicira-network-offerings.py", 
> line 80, in testCreateNetworkOfferingNiciraL2
> resp = conn.marvinRequest(createNetOff)
>   File
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py
> ",
> line 279, in marvinRequest
> response = jsonHelper.getResultObj(response.json(), response_type)
>   File "/usr/local/lib/python2.7/site-packages/marvin/jsonHelper.py", 
> line 148, in getResultObj
> raise cloudstackException.cloudstackAPIException(respname, errMsg)
> cloudstackAPIException: Execute cmd: createnetworkoffering failed, due to:
> errorCode: 431, errorText:Provider: NiciraNvp does not support 
> StretchedL2Subnet
>  >> begin captured stdout << -
> DEBUG: sending GET request: createNetworkOffering {'specifyipranges':
> 'false', 'serviceproviderlist[2].provider': 'VirtualRouter',
> 'serviceproviderlist[5].service': 'Lb', 'displaytext': 
> 'NiciraNvpL2SNAT',
> 'serviceproviderlist[0].service': 'UserData',
> 'serviceproviderlist[4].provider': 'VirtualRouter',
> 'serviceproviderlist[1].provider': 'VirtualRouter', 'availability':
> 'Optional', 'conservemode': 'true', 'serviceproviderlist[3].service':
> 'StaticNat', 'serviceproviderlist[5].provider': 'VirtualRouter',
> 'serviceproviderlist[0].provider': 'VirtualRouter', 'specifyvlan': 
> 'false',
> 'serviceproviderlist[8].provider': 'VirtualRouter',
> 'serviceproviderlist[9].provider': 'NiciraNvp',
> 'serviceproviderlist[1].service': 'Vpn', 'serviceproviderlist[4].service':
> 'PortForwarding', 'supportedservices':
> 'UserData,Vpn,Dhcp,StaticNat,PortForwarding,Lb,Firewall,Dns,SourceNat,
> Connectivity',
> 'traffictype': 'GUEST', 'serviceproviderlist[6].provider': 
> 'VirtualRouter',
> 'serviceproviderlist[8].service': 'SourceNat',
> 'serviceproviderlist[3].provider': 'VirtualRouter', 'name':
> 'NiciraNvpL2SNAT', 'serviceproviderlist[9].service': 'Connectivity',
> 'guestiptype': 'Isolated', 'serviceproviderlist[7].provider':
> 'VirtualRouter', 'serviceproviderlist[7].service': 'Dns',
> 'serviceproviderlist[2].service': 'Dhcp', 'serviceproviderlist[6].service':
> 'Firewall'}
> DEBUG: Computed Signature by Marvin: uwA97AMbhXAPNjW+vzx90pAZv1M=
> DEBUG: Request:
> http://cittapp232:8080/client/api?specifyipranges=false&serviceproviderlist%5B2%5D.provider=VirtualRouter&serviceproviderlist%5B5%5D.service=Lb&apiKey=ZJi1_VqmRIhto3bj9zGlPq9y9ywI15bskwdvIGooKjfppdKk5lQhwVSMPyhs2wDN8o_EK51ePHN8ibzPxXXSTg&displaytext=NiciraNvpL2SNAT&serviceproviderlist%5B0%5D.service=UserData&serviceproviderlist%5B4%5D.provider=VirtualRouter&serviceproviderlist%5B1%5D.provider=VirtualRouter&availability=Optional&conservemode=true&serviceproviderlist%5B3%5D.service=StaticNat&serviceproviderlist%5B5%5D.provider=VirtualRouter&response=json&serviceproviderlist%5B0%5D.provider=VirtualRouter&specifyvlan=false&serviceproviderlist%5B8%5D.provider=VirtualRouter&serviceproviderlist%5B9%5D.provider=NiciraNvp&serviceproviderlist%5B1%5D.service=Vpn&serviceproviderlist%5B4%5D.service=PortForwarding&supportedservices=UserData%2CVpn%2CDhcp%2CStaticNat%2CPortForwarding%2CLb%2CFirewall%2CDns%2CSourceNat%2CConnectivity&traffictype=GUEST&serviceproviderlist%5B6%5D.provider=VirtualRouter&serviceproviderlist%5B8%5D.service=SourceNat&serviceproviderlist%5B3%5D.provider=VirtualRouter&name=NiciraNvpL2SNAT&serviceproviderlist%5B9%5D.service=Connectivity&guestiptype=Isolated&serviceproviderlist%5B7%5D.provider=VirtualRouter&serviceproviderlist%5B7%5D.service=Dns&serviceproviderlist%5B2%5D.service=Dhcp&serviceproviderlist%5B6%5D.service=Firewall&command=createNetworkOffering&signature=uwA97AMbhXAPNjW%2Bvzx90pAZv1M%3DResponse:
>  { "createnetworkofferingresponse" :
> {"uuidList":[],"errorcode":431,"cserrorcode":4350,"errortext":"Provider:
> NiciraNvp does not support StretchedL2Subnet"} }
>
> 
>
> --
> TIA
> Ian
>


Re: StretchedL2Subnet

2014-05-06 Thread Daan Hoogland
Murali,

Can you have a look to see what defaults where introduced to make
these tests fail? I see StretchedL2 was introduced in commit
d2d54a946310d25a8edd4cab5abf8588a9e1b4df, committed on the 20th of
march.

thanks,
Daan

On Tue, May 6, 2014 at 2:00 PM, Ian Southam  wrote:
> Anybody know what this is?
>
> I see our Integration testing (for Nicira) has started failing of late.  We 
> are not requesting such a network so I presume a default has changed 
> somewhere?
>
> 
>
> ERROR: 
> test-create-nicira-network-offerings.testNetworkOffering.testCreateNetworkOfferingNiciraL2
> --
> Traceback (most recent call last):
>   File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in 
> runTest
> self.test(*self.arg)
>   File "/home/jenkins/slave/workspace/team_tkt/Cloudstack Integration 
> Testing/build-cloud/scripts/test-create-nicira-network-offerings.py", line 
> 80, in testCreateNetworkOfferingNiciraL2
> resp = conn.marvinRequest(createNetOff)
>   File 
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 
> 279, in marvinRequest
> response = jsonHelper.getResultObj(response.json(), response_type)
>   File "/usr/local/lib/python2.7/site-packages/marvin/jsonHelper.py", line 
> 148, in getResultObj
> raise cloudstackException.cloudstackAPIException(respname, errMsg)
> cloudstackAPIException: Execute cmd: createnetworkoffering failed, due to: 
> errorCode: 431, errorText:Provider: NiciraNvp does not support 
> StretchedL2Subnet
>  >> begin captured stdout << -
> DEBUG: sending GET request: createNetworkOffering {'specifyipranges': 
> 'false', 'serviceproviderlist[2].provider': 'VirtualRouter', 
> 'serviceproviderlist[5].service': 'Lb', 'displaytext': 'NiciraNvpL2SNAT', 
> 'serviceproviderlist[0].service': 'UserData', 
> 'serviceproviderlist[4].provider': 'VirtualRouter', 
> 'serviceproviderlist[1].provider': 'VirtualRouter', 'availability': 
> 'Optional', 'conservemode': 'true', 'serviceproviderlist[3].service': 
> 'StaticNat', 'serviceproviderlist[5].provider': 'VirtualRouter', 
> 'serviceproviderlist[0].provider': 'VirtualRouter', 'specifyvlan': 'false', 
> 'serviceproviderlist[8].provider': 'VirtualRouter', 
> 'serviceproviderlist[9].provider': 'NiciraNvp', 
> 'serviceproviderlist[1].service': 'Vpn', 'serviceproviderlist[4].service': 
> 'PortForwarding', 'supportedservices': 
> 'UserData,Vpn,Dhcp,StaticNat,PortForwarding,Lb,Firewall,Dns,SourceNat,Connectivity',
>  'traffictype': 'GUEST', 'serviceproviderlist[6].provider': 'VirtualRouter', 
> 'serviceproviderlist[8].service': 'SourceNat', 
> 'serviceproviderlist[3].provider': 'VirtualRouter', 'name': 
> 'NiciraNvpL2SNAT', 'serviceproviderlist[9].service': 'Connectivity', 
> 'guestiptype': 'Isolated', 'serviceproviderlist[7].provider': 
> 'VirtualRouter', 'serviceproviderlist[7].service': 'Dns', 
> 'serviceproviderlist[2].service': 'Dhcp', 'serviceproviderlist[6].service': 
> 'Firewall'}
> DEBUG: Computed Signature by Marvin: uwA97AMbhXAPNjW+vzx90pAZv1M=
> DEBUG: Request: 
> http://cittapp232:8080/client/api?specifyipranges=false&serviceproviderlist%5B2%5D.provider=VirtualRouter&serviceproviderlist%5B5%5D.service=Lb&apiKey=ZJi1_VqmRIhto3bj9zGlPq9y9ywI15bskwdvIGooKjfppdKk5lQhwVSMPyhs2wDN8o_EK51ePHN8ibzPxXXSTg&displaytext=NiciraNvpL2SNAT&serviceproviderlist%5B0%5D.service=UserData&serviceproviderlist%5B4%5D.provider=VirtualRouter&serviceproviderlist%5B1%5D.provider=VirtualRouter&availability=Optional&conservemode=true&serviceproviderlist%5B3%5D.service=StaticNat&serviceproviderlist%5B5%5D.provider=VirtualRouter&response=json&serviceproviderlist%5B0%5D.provider=VirtualRouter&specifyvlan=false&serviceproviderlist%5B8%5D.provider=VirtualRouter&serviceproviderlist%5B9%5D.provider=NiciraNvp&serviceproviderlist%5B1%5D.service=Vpn&serviceproviderlist%5B4%5D.service=PortForwarding&supportedservices=UserData%2CVpn%2CDhcp%2CStaticNat%2CPortForwarding%2CLb%2CFirewall%2CDns%2CSourceNat%2CConnectivity&traffictype=GUEST&serviceproviderlist%5B6%5D.provider=VirtualRouter&serviceproviderlist%5B8%5D.service=SourceNat&serviceproviderlist%5B3%5D.provider=VirtualRouter&name=NiciraNvpL2SNAT&serviceproviderlist%5B9%5D.service=Connectivity&guestiptype=Isolated&serviceproviderlist%5B7%5D.provider=VirtualRouter&serviceproviderlist%5B7%5D.service=Dns&serviceproviderlist%5B2%5D.service=Dhcp&serviceproviderlist%5B6%5D.service=Firewall&command=createNetworkOffering&signature=uwA97AMbhXAPNjW%2Bvzx90pAZv1M%3D
>  Response: { "createnetworkofferingresponse" : 
> {"uuidList":[],"errorcode":431,"cserrorcode":4350,"errortext":"Provider: 
> NiciraNvp does not support StretchedL2Subnet"} }
>
> 
>
> --
> TIA
> Ian



-- 
Daan


Re: StretchedL2Subnet

2014-05-06 Thread murali reddy
On Tue, May 6, 2014 at 5:30 PM, Ian Southam wrote:

> Anybody know what this is?
>
> I see our Integration testing (for Nicira) has started failing of late.
>  We are not requesting such a network so I presume a default has changed
> somewhere?
>

I added this as capability to 'Connectivity' service providers. This should
not be affecting default behaviour. Let me check on this,


>
> 
>
> ERROR:
> test-create-nicira-network-offerings.testNetworkOffering.testCreateNetworkOfferingNiciraL2
> --
> Traceback (most recent call last):
>   File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in
> runTest
> self.test(*self.arg)
>   File "/home/jenkins/slave/workspace/team_tkt/Cloudstack Integration
> Testing/build-cloud/scripts/test-create-nicira-network-offerings.py", line
> 80, in testCreateNetworkOfferingNiciraL2
> resp = conn.marvinRequest(createNetOff)
>   File
> "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py",
> line 279, in marvinRequest
> response = jsonHelper.getResultObj(response.json(), response_type)
>   File "/usr/local/lib/python2.7/site-packages/marvin/jsonHelper.py", line
> 148, in getResultObj
> raise cloudstackException.cloudstackAPIException(respname, errMsg)
> cloudstackAPIException: Execute cmd: createnetworkoffering failed, due to:
> errorCode: 431, errorText:Provider: NiciraNvp does not support
> StretchedL2Subnet
>  >> begin captured stdout << -
> DEBUG: sending GET request: createNetworkOffering {'specifyipranges':
> 'false', 'serviceproviderlist[2].provider': 'VirtualRouter',
> 'serviceproviderlist[5].service': 'Lb', 'displaytext': 'NiciraNvpL2SNAT',
> 'serviceproviderlist[0].service': 'UserData',
> 'serviceproviderlist[4].provider': 'VirtualRouter',
> 'serviceproviderlist[1].provider': 'VirtualRouter', 'availability':
> 'Optional', 'conservemode': 'true', 'serviceproviderlist[3].service':
> 'StaticNat', 'serviceproviderlist[5].provider': 'VirtualRouter',
> 'serviceproviderlist[0].provider': 'VirtualRouter', 'specifyvlan': 'false',
> 'serviceproviderlist[8].provider': 'VirtualRouter',
> 'serviceproviderlist[9].provider': 'NiciraNvp',
> 'serviceproviderlist[1].service': 'Vpn', 'serviceproviderlist[4].service':
> 'PortForwarding', 'supportedservices':
> 'UserData,Vpn,Dhcp,StaticNat,PortForwarding,Lb,Firewall,Dns,SourceNat,Connectivity',
> 'traffictype': 'GUEST', 'serviceproviderlist[6].provider': 'VirtualRouter',
> 'serviceproviderlist[8].service': 'SourceNat',
> 'serviceproviderlist[3].provider': 'VirtualRouter', 'name':
> 'NiciraNvpL2SNAT', 'serviceproviderlist[9].service': 'Connectivity',
> 'guestiptype': 'Isolated', 'serviceproviderlist[7].provider':
> 'VirtualRouter', 'serviceproviderlist[7].service': 'Dns',
> 'serviceproviderlist[2].service': 'Dhcp', 'serviceproviderlist[6].service':
> 'Firewall'}
> DEBUG: Computed Signature by Marvin: uwA97AMbhXAPNjW+vzx90pAZv1M=
> DEBUG: Request:
> http://cittapp232:8080/client/api?specifyipranges=false&serviceproviderlist%5B2%5D.provider=VirtualRouter&serviceproviderlist%5B5%5D.service=Lb&apiKey=ZJi1_VqmRIhto3bj9zGlPq9y9ywI15bskwdvIGooKjfppdKk5lQhwVSMPyhs2wDN8o_EK51ePHN8ibzPxXXSTg&displaytext=NiciraNvpL2SNAT&serviceproviderlist%5B0%5D.service=UserData&serviceproviderlist%5B4%5D.provider=VirtualRouter&serviceproviderlist%5B1%5D.provider=VirtualRouter&availability=Optional&conservemode=true&serviceproviderlist%5B3%5D.service=StaticNat&serviceproviderlist%5B5%5D.provider=VirtualRouter&response=json&serviceproviderlist%5B0%5D.provider=VirtualRouter&specifyvlan=false&serviceproviderlist%5B8%5D.provider=VirtualRouter&serviceproviderlist%5B9%5D.provider=NiciraNvp&serviceproviderlist%5B1%5D.service=Vpn&serviceproviderlist%5B4%5D.service=PortForwarding&supportedservices=UserData%2CVpn%2CDhcp%2CStaticNat%2CPortForwarding%2CLb%2CFirewall%2CDns%2CSourceNat%2CConnectivity&traffictype=GUEST&serviceproviderlist%5B6%5D.provider=VirtualRouter&serviceproviderlist%5B8%5D.service=SourceNat&serviceproviderlist%5B3%5D.provider=VirtualRouter&name=NiciraNvpL2SNAT&serviceproviderlist%5B9%5D.service=Connectivity&guestiptype=Isolated&serviceproviderlist%5B7%5D.provider=VirtualRouter&serviceproviderlist%5B7%5D.service=Dns&serviceproviderlist%5B2%5D.service=Dhcp&serviceproviderlist%5B6%5D.service=Firewall&command=createNetworkOffering&signature=uwA97AMbhXAPNjW%2Bvzx90pAZv1M%3DResponse:
>  { "createnetworkofferingresponse" :
> {"uuidList":[],"errorcode":431,"cserrorcode":4350,"errortext":"Provider:
> NiciraNvp does not support StretchedL2Subnet"} }
>
> 
>
> --
> TIA
> Ian
>


[ACS44] INSTALL.md update

2014-05-06 Thread Daan Hoogland
Can someone (or actually everybody!) please have a look at the
INSTALL.md file to decide if we need to update some of the package
names and version numbers in there?

thanks
-- 
Daan


StretchedL2Subnet

2014-05-06 Thread Ian Southam
Anybody know what this is?

I see our Integration testing (for Nicira) has started failing of late.  We are 
not requesting such a network so I presume a default has changed somewhere?



ERROR: 
test-create-nicira-network-offerings.testNetworkOffering.testCreateNetworkOfferingNiciraL2
--
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in 
runTest
self.test(*self.arg)
  File "/home/jenkins/slave/workspace/team_tkt/Cloudstack Integration 
Testing/build-cloud/scripts/test-create-nicira-network-offerings.py", line 80, 
in testCreateNetworkOfferingNiciraL2
resp = conn.marvinRequest(createNetOff)
  File "/usr/local/lib/python2.7/site-packages/marvin/cloudstackConnection.py", 
line 279, in marvinRequest
response = jsonHelper.getResultObj(response.json(), response_type)
  File "/usr/local/lib/python2.7/site-packages/marvin/jsonHelper.py", line 148, 
in getResultObj
raise cloudstackException.cloudstackAPIException(respname, errMsg)
cloudstackAPIException: Execute cmd: createnetworkoffering failed, due to: 
errorCode: 431, errorText:Provider: NiciraNvp does not support StretchedL2Subnet
 >> begin captured stdout << -
DEBUG: sending GET request: createNetworkOffering {'specifyipranges': 'false', 
'serviceproviderlist[2].provider': 'VirtualRouter', 
'serviceproviderlist[5].service': 'Lb', 'displaytext': 'NiciraNvpL2SNAT', 
'serviceproviderlist[0].service': 'UserData', 
'serviceproviderlist[4].provider': 'VirtualRouter', 
'serviceproviderlist[1].provider': 'VirtualRouter', 'availability': 'Optional', 
'conservemode': 'true', 'serviceproviderlist[3].service': 'StaticNat', 
'serviceproviderlist[5].provider': 'VirtualRouter', 
'serviceproviderlist[0].provider': 'VirtualRouter', 'specifyvlan': 'false', 
'serviceproviderlist[8].provider': 'VirtualRouter', 
'serviceproviderlist[9].provider': 'NiciraNvp', 
'serviceproviderlist[1].service': 'Vpn', 'serviceproviderlist[4].service': 
'PortForwarding', 'supportedservices': 
'UserData,Vpn,Dhcp,StaticNat,PortForwarding,Lb,Firewall,Dns,SourceNat,Connectivity',
 'traffictype': 'GUEST', 'serviceproviderlist[6].provider': 'VirtualRouter', 
'serviceproviderlist[8].service': 'SourceNat', 
'serviceproviderlist[3].provider': 'VirtualRouter', 'name': 'NiciraNvpL2SNAT', 
'serviceproviderlist[9].service': 'Connectivity', 'guestiptype': 'Isolated', 
'serviceproviderlist[7].provider': 'VirtualRouter', 
'serviceproviderlist[7].service': 'Dns', 'serviceproviderlist[2].service': 
'Dhcp', 'serviceproviderlist[6].service': 'Firewall'}
DEBUG: Computed Signature by Marvin: uwA97AMbhXAPNjW+vzx90pAZv1M=
DEBUG: Request: 
http://cittapp232:8080/client/api?specifyipranges=false&serviceproviderlist%5B2%5D.provider=VirtualRouter&serviceproviderlist%5B5%5D.service=Lb&apiKey=ZJi1_VqmRIhto3bj9zGlPq9y9ywI15bskwdvIGooKjfppdKk5lQhwVSMPyhs2wDN8o_EK51ePHN8ibzPxXXSTg&displaytext=NiciraNvpL2SNAT&serviceproviderlist%5B0%5D.service=UserData&serviceproviderlist%5B4%5D.provider=VirtualRouter&serviceproviderlist%5B1%5D.provider=VirtualRouter&availability=Optional&conservemode=true&serviceproviderlist%5B3%5D.service=StaticNat&serviceproviderlist%5B5%5D.provider=VirtualRouter&response=json&serviceproviderlist%5B0%5D.provider=VirtualRouter&specifyvlan=false&serviceproviderlist%5B8%5D.provider=VirtualRouter&serviceproviderlist%5B9%5D.provider=NiciraNvp&serviceproviderlist%5B1%5D.service=Vpn&serviceproviderlist%5B4%5D.service=PortForwarding&supportedservices=UserData%2CVpn%2CDhcp%2CStaticNat%2CPortForwarding%2CLb%2CFirewall%2CDns%2CSourceNat%2CConnectivity&traffictype=GUEST&serviceproviderlist%5B6%5D.provider=VirtualRouter&serviceproviderlist%5B8%5D.service=SourceNat&serviceproviderlist%5B3%5D.provider=VirtualRouter&name=NiciraNvpL2SNAT&serviceproviderlist%5B9%5D.service=Connectivity&guestiptype=Isolated&serviceproviderlist%5B7%5D.provider=VirtualRouter&serviceproviderlist%5B7%5D.service=Dns&serviceproviderlist%5B2%5D.service=Dhcp&serviceproviderlist%5B6%5D.service=Firewall&command=createNetworkOffering&signature=uwA97AMbhXAPNjW%2Bvzx90pAZv1M%3D
 Response: { "createnetworkofferingresponse" : 
{"uuidList":[],"errorcode":431,"cserrorcode":4350,"errortext":"Provider: 
NiciraNvp does not support StretchedL2Subnet"} }



--
TIA
Ian


RE: Apache Cloudstack Error!!!

2014-05-06 Thread Rajesh Battala
Wait till all the components got loaded and try

-Original Message-
From: Prakash Rao Banuka [mailto:prakas...@simplifydc.com] 
Sent: Tuesday, May 6, 2014 2:30 AM
To: dev@cloudstack.apache.org
Subject: Apache Cloudstack Error!!!

Hi:

When I issue url http://ip:8080/client in the browser, it says

HTTP Status 404 -

type Status report
message
description The requested resource () is not available.

Apache Tomcat/6.0.24
Any Ideas??

Thank you
Prakash



Re: Review Request 19399: CLOUDSTACK-6257: Resolving issue in template creation from user VM volume, added method to check the state of VM before template creation

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 6598167bedb1f00e9d1efc12b82999f8e822d5d6 in cloudstack's branch 
refs/heads/4.4-forward from Gaurav Aradhye
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6598167 ]

CLOUDSTACK-6257: Adding function to check the VM state


- ASF Subversion and Git Services


On April 28, 2014, 10:41 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19399/
> ---
> 
> (Updated April 28, 2014, 10:41 a.m.)
> 
> 
> Review request for cloudstack and Santhosh Edukulla.
> 
> 
> Bugs: CLOUDSTACK-6257
> https://issues.apache.org/jira/browse/CLOUDSTACK-6257
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> As suggested, added getState method for VirtualMachine class, and it is 
> called before template creation. If VM fails to go into desired state, it 
> returns failure.
> 
> 
> Diffs
> -
> 
>   tools/marvin/marvin/codes.py 28e907c 
>   tools/marvin/marvin/lib/base.py d753098 
> 
> Diff: https://reviews.apache.org/r/19399/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>



Re: Apache Cloudstack Error!!!

2014-05-06 Thread Wei ZHOU
stop tomcat at first, then restart cloudstack-management


Review Request 21109: [UI] New Zones tab for Templates and ISOs

2014-05-06 Thread Gabor Apati-Nagy

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

Review request for cloudstack, Brian Federle and Jessica Wang.


Bugs: CLOUDSTACK-6565
https://issues.apache.org/jira/browse/CLOUDSTACK-6565


Repository: cloudstack-git


Description
---

[UI] New Zones tab for Templates and ISOs


Diffs
-

  client/WEB-INF/classes/resources/messages.properties eff2c3e 
  ui/css/cloudstack3.css cb9fa35 
  ui/dictionary.jsp 9cc030a 
  ui/scripts/templates.js 67cc2fb 

Diff: https://reviews.apache.org/r/21109/diff/


Testing
---


Thanks,

Gabor Apati-Nagy



RE: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-06 Thread Donal Lafferty
You can derive the exact API call sequence from the log file using grep & sed.  

See http://dlafferty.blogspot.co.uk/2013/07/using-cloudmonkey-to-automate.html 

DL


> -Original Message-
> From: ilya musayev [mailto:ilya.mailing.li...@gmail.com]
> Sent: 06 May 2014 00:56
> To: dev@cloudstack.apache.org
> Subject: [API][CLOUDMONKEY] - looking for example on creating entire zone
> with everything else
> 
> For the CloudStack Automation testing project, i'm looking for entire flow of
> API commands or perhaps CloudMokey CLI equivalent commands to mimic
> what Create a Zone Wizard does. VMware example would be ideal, but i'm
> certain i can figure out difference if its not.
> 
> I know i can probably get the APIs from Chrome Debugger or Firefly in
> mozilla, but it seems a bit excessive on the output.
> 
> Would anyone have it handy?
> 
> Thanks
> ilya


RE: Apache Cloudstack Error!!!

2014-05-06 Thread Donal Lafferty
Check the logs.

There's a lot there, e.g. 
http://dlafferty.blogspot.co.uk/2013/08/using-cloudstacks-log-files-xenserver.html
 

DL


> -Original Message-
> From: Prakash Rao Banuka [mailto:prakas...@simplifydc.com]
> Sent: 05 May 2014 22:00
> To: dev@cloudstack.apache.org
> Subject: Apache Cloudstack Error!!!
> 
> Hi:
> 
> When I issue url http://ip:8080/client in the browser, it says
> 
> HTTP Status 404 -
> 
> type Status report
> message
> description The requested resource () is not available.
> 
> Apache Tomcat/6.0.24
> Any Ideas??
> 
> Thank you
> Prakash



Build failed in Jenkins: cloudstack-4.4-maven-build-noredist #198

2014-05-06 Thread jenkins
See 


Changes:

[Daan Hoogland] CLOUDSTACK-6552 Cloudstack-Management install package creates 
log

--
[...truncated 14946 lines...]
1497/1875 KB   
1501/1875 KB   
1505/1875 KB   
1507/1875 KB   
1509/1875 KB   
1513/1875 KB   
1516/1875 KB   
1519/1875 KB   
1520/1875 KB   
1524/1875 KB   
1525/1875 KB   
1527/1875 KB   
1530/1875 KB   
1532/1875 KB   
1533/1875 KB   
1536/1875 KB   
1539/1875 KB   
1542/1875 KB   
1545/1875 KB   
1549/1875 KB   
1550/1875 KB   
1553/1875 KB   
1557/1875 KB   
1559/1875 KB   
1562/1875 KB   
1564/1875 KB   
1567/1875 KB   
1570/1875 KB   
1572/1875 KB   
1573/1875 KB   
1576/1875 KB   
1579/1875 KB   
1582/1875 KB   
1586/1875 KB   
1587/1875 KB   
1590/1875 KB   
1593/1875 KB   
1596/1875 KB   
1599/1875 KB   
1602/1875 KB   
1604/1875 KB   
1607/1875 KB   
1610/1875 KB   
1613/1875 KB   
1616/1875 KB   
1619/1875 KB   
1623/1875 KB   
1624/1875 KB   
1627/1875 KB   
1629/1875 KB   
1630/1875 KB   
1633/1875 KB   
1636/1875 KB   
1639/1875 KB   
1641/1875 KB   
1644/1875 KB   
1647/1875 KB   
1650/1875 KB   
1653/1875 KB   
1654/1875 KB   
1656/1875 KB   
1660/1875 KB   
1661/1875 KB   
1664/1875 KB   
1667/1875 KB   
1670/1875 KB   
1673/1875 KB   
1674/1875 KB   
1678/1875 KB   
1679/1875 KB   
1681/1875 KB   
1684/1875 KB   
1686/1875 KB   
1687/1875 KB   
1689/1875 KB   
1690/1875 KB   
1691/1875 KB   
1695/1875 KB   
1699/1875 KB   
1701/1875 KB   
1704/1875 KB   
1707/1875 KB   
1710/1875 KB   
1714/1875 KB   
1718/1875 KB   
1718/1875 KB   
1722/1875 KB   
1726/1875 KB   
1727/1875 KB   
1730/1875 KB   
1734/1875 KB   
1738/1875 KB   
1742/1875 KB   
1746/1875 KB   
1747/1875 KB   
1750/1875 KB   
1754/1875 KB   
1756/1875 KB   
1760/1875 KB   
1764/1875 KB   
1768/1875 KB   
1772/1875 KB   
1776/1875 KB   
1780/1875 KB   
1784/1875 KB   
1788/1875 KB   
1792/1875 KB   
1796/1875 KB   
1800/1875 KB   
1804/1875 KB   
1808/1875 KB   
1812/1875 KB   
1816/1875 KB   
1820/1875 KB   
1824/1875 KB   
1828/1875 KB   
1832/1875 KB   
1836/1875 KB   
1840/1875 KB   
1844/1875 KB   
1848/1875 KB   
1852/1875 KB   
1856/1875 KB   
1860/1875 KB   
1864/1875 KB   
1868/1875 KB   
1872/1875 KB   
1875/1875 KB   
   
Downloaded: 
http://cs-maven.midokura.com/releases/org/midonet/midonet-client/1.1.0/midonet-client-1.1.0.jar
 (1875 KB at 611.5 KB/sec)
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS 
[1.623s]
[INFO] Apache CloudStack . SUCCESS [3.577s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.771s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [2.772s]
[INFO] Apache CloudStack Utils ... SUCCESS [12.842s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.039s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [6.851s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [3.888s]
[INFO] Apache CloudStack API . SUCCESS [14.859s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [2.823s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [5.536s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.043s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [4.262s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [2.075s]
[INFO] Apache CloudStack Core  SUCCESS [9.820s]
[INFO] Apache CloudStack Agents .. SUCCESS [7.229s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [3.044s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [3.090s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [12.505s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [2.075s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [2.898s]
[INFO] Apache CloudStack Server .. SUCCESS [39.322s]
[INFO] Apache CloudStack Usage Server  SUCCESS [6.149s]
[INFO] Apache XenSource XAPI . SUCCESS [6.738s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS [8.440s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.039s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.271s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [4.934s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [6.841s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [4.499s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [2.887s]
[INFO] Apache CloudStack Engine Storage Data Motio

Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread daan Hoogland


> On May 6, 2014, 9:17 a.m., daan Hoogland wrote:
> > Can you commit to a branch locally and then in that branch generate with
> > $ git format-patch HEAD^1
> > 
> > thanks

never mind. I had a closer look and just applied


- daan


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


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread Nux!

On 06.05.2014 10:14, Daan Hoogland wrote:

hm, I'm not sure if you want to go that way then. You'll need to run
validation builds and do magic with git. I'll apply it. In the mean
while first get very proficient with git and maven.


Thanks!


--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread daan Hoogland

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


Can you commit to a branch locally and then in that branch generate with
$ git format-patch HEAD^1

thanks

- daan Hoogland


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 5e067137617f28293b169d246c355cc268334c81 in cloudstack's branch 
refs/heads/4.4-forward from David Bierce
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5e06713 ]

CLOUDSTACK-6552 Cloudstack-Management install package creates log
directory that is never used


- ASF Subversion and Git Services


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread daan Hoogland

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

Ship it!


efccef485cb3e3f0a7d9e290653909750617f921 on 4.4
5e067137617f28293b169d246c355cc268334c81 on 4.4-forward
59eed9ef77a041a6f30fc7522700a3d4c18b5637 on master

- daan Hoogland


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread ASF Subversion and Git Services

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


Commit 59eed9ef77a041a6f30fc7522700a3d4c18b5637 in cloudstack's branch 
refs/heads/master from David Bierce
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=59eed9e ]

CLOUDSTACK-6552 Cloudstack-Management install package creates log
directory that is never used


- ASF Subversion and Git Services


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread ASF Subversion and Git Services

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


Commit efccef485cb3e3f0a7d9e290653909750617f921 in cloudstack's branch 
refs/heads/4.4 from David Bierce
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=efccef4 ]

CLOUDSTACK-6552 Cloudstack-Management install package creates log
directory that is never used

- ASF Subversion and Git Services


On April 30, 2014, 10:24 p.m., David Bierce wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/
> ---
> 
> (Updated April 30, 2014, 10:24 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6552
> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The RPM build cloud.spec creates a directory that is never used by cloudstack 
> management and is just left as an empty directory name cloudstack-management 
> in /var/log.
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 83c598b 
> 
> Diff: https://reviews.apache.org/r/20921/diff/
> 
> 
> Testing
> ---
> 
> Build RPMs on CentOS 6.4 and 6.5 using master
> 
> mvn clean install -Dnonoss -Dnoredist
> package.sh -p noredist
> 
> Resulting packages installed on Centos 6.5 without the unused directory. 
> 
> 
> Thanks,
> 
> David Bierce
> 
>



Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread Daan Hoogland
hm, I'm not sure if you want to go that way then. You'll need to run
validation builds and do magic with git. I'll apply it. In the mean
while first get very proficient with git and maven.

On Tue, May 6, 2014 at 11:12 AM, Nux!  wrote:
> On 06.05.2014 09:45, Daan Hoogland wrote:
>>
>> Lucian,
>>
>> Did you apply it? Or just approved?
>
>
> Approved, I'm yet to learn how to actually apply it. :)
> BTW any docs that can put me in the right direction? I'm afraid I'm not
> versed in dev stuff.
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro



-- 
Daan


Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread Nux!

On 06.05.2014 09:45, Daan Hoogland wrote:

Lucian,

Did you apply it? Or just approved?


Approved, I'm yet to learn how to actually apply it. :)
BTW any docs that can put me in the right direction? I'm afraid I'm not 
versed in dev stuff.


Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-06 Thread Daan Hoogland
Lucian,

Did you apply it? Or just approved?

On Mon, May 5, 2014 at 8:19 PM, Nux Ro  wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20921/#review42174
> ---
>
> Ship it!
>
>
> Ship It!
>
> - Nux Ro
>
>
> On April 30, 2014, 10:24 p.m., David Bierce wrote:
>>
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/20921/
>> ---
>>
>> (Updated April 30, 2014, 10:24 p.m.)
>>
>>
>> Review request for cloudstack.
>>
>>
>> Bugs: CLOUDSTACK-6552
>> https://issues.apache.org/jira/browse/CLOUDSTACK-6552
>>
>>
>> Repository: cloudstack-git
>>
>>
>> Description
>> ---
>>
>> The RPM build cloud.spec creates a directory that is never used by 
>> cloudstack management and is just left as an empty directory name 
>> cloudstack-management in /var/log.
>>
>>
>> Diffs
>> -
>>
>>   packaging/centos63/cloud.spec 83c598b
>>
>> Diff: https://reviews.apache.org/r/20921/diff/
>>
>>
>> Testing
>> ---
>>
>> Build RPMs on CentOS 6.4 and 6.5 using master
>>
>> mvn clean install -Dnonoss -Dnoredist
>> package.sh -p noredist
>>
>> Resulting packages installed on Centos 6.5 without the unused directory.
>>
>>
>> Thanks,
>>
>> David Bierce
>>
>>
>



-- 
Daan


Re: 4.4 release-notes

2014-05-06 Thread Daan Hoogland
I think Pierre-Luc's idea is good and infra will surely pick up the
ticket one of these days. The dashboard is Animesh's work. I think he
used private filters.

On Tue, May 6, 2014 at 10:19 AM, Sebastien Goasguen  wrote:
> Maybe we can ask animesh how he did it for 4.3 ...
>
> -sebastien
>
>
> On Mon, May 5, 2014 at 4:35 PM, Pierre-Luc Dion  wrote:
>
>> Request Created under ASF infrastructure team; INFRA project.
>>
>> Thanks David,
>>
>>
>> Pierre-Luc Dion
>> Architecte de Solution Cloud | Cloud Solutions Architect
>> 855-OK-CLOUD (855-652-5683) x1101
>> - - -
>>
>> *CloudOps*420 rue Guy
>> Montréal QC  H3J 1S6
>> www.cloudops.com
>> @CloudOps_
>>
>>
>> On Mon, May 5, 2014 at 4:17 PM, David Nalley  wrote:
>>
>> > You'll need to file a ticket with infra.
>> >
>> > On Mon, May 5, 2014 at 2:52 PM, Daan Hoogland 
>> > wrote:
>> > > Animesh,
>> > >
>> > > Can you (give me rights to) add these filters to the release dashboard?
>> > >
>> > > On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion 
>> > wrote:
>> > >> 4.4.0 Fixed Issues
>> > >> project = CLOUDSTACK AND type = Bug AND affectedVersion in (4.2.0,
>> > 4.2.1,
>> > >> 4.3.0) AND fixVersion = 4.4.0 AND resolution != "\"Unresolved\"" ORDER
>> > BY
>> > >> created DESC, priority DESC, key ASC
>> > >>
>> > >> 4.4.0 Known Issues
>> > >> project = CLOUDSTACK AND type = Bug AND (affectedVersion = 4.4.0 OR
>> > >> fixVersion = 4.4.0) AND resolution is EMPTY AND level = "Public" ORDER
>> > BY
>> > >> priority DESC, key ASC
>> > >>
>> > >
>> > >
>> > >
>> > > --
>> > > Daan
>> >
>>
>
>
>
> --
> ---
> Sebastien Goasguen
> Open Source Cloud Evangelist, Cloudstack
> http://sites.google.com/site/runseb/



-- 
Daan


Re: 4.4 release-notes

2014-05-06 Thread Sebastien Goasguen
Maybe we can ask animesh how he did it for 4.3 ...

-sebastien


On Mon, May 5, 2014 at 4:35 PM, Pierre-Luc Dion  wrote:

> Request Created under ASF infrastructure team; INFRA project.
>
> Thanks David,
>
>
> Pierre-Luc Dion
> Architecte de Solution Cloud | Cloud Solutions Architect
> 855-OK-CLOUD (855-652-5683) x1101
> - - -
>
> *CloudOps*420 rue Guy
> Montréal QC  H3J 1S6
> www.cloudops.com
> @CloudOps_
>
>
> On Mon, May 5, 2014 at 4:17 PM, David Nalley  wrote:
>
> > You'll need to file a ticket with infra.
> >
> > On Mon, May 5, 2014 at 2:52 PM, Daan Hoogland 
> > wrote:
> > > Animesh,
> > >
> > > Can you (give me rights to) add these filters to the release dashboard?
> > >
> > > On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion 
> > wrote:
> > >> 4.4.0 Fixed Issues
> > >> project = CLOUDSTACK AND type = Bug AND affectedVersion in (4.2.0,
> > 4.2.1,
> > >> 4.3.0) AND fixVersion = 4.4.0 AND resolution != "\"Unresolved\"" ORDER
> > BY
> > >> created DESC, priority DESC, key ASC
> > >>
> > >> 4.4.0 Known Issues
> > >> project = CLOUDSTACK AND type = Bug AND (affectedVersion = 4.4.0 OR
> > >> fixVersion = 4.4.0) AND resolution is EMPTY AND level = "Public" ORDER
> > BY
> > >> priority DESC, key ASC
> > >>
> > >
> > >
> > >
> > > --
> > > Daan
> >
>



-- 
---
Sebastien Goasguen
Open Source Cloud Evangelist, Cloudstack
http://sites.google.com/site/runseb/


[GitHub] cloudstack-docs-install pull request: add Managing Networks

2014-05-06 Thread darrentangdt
Github user darrentangdt commented on the pull request:


https://github.com/apache/cloudstack-docs-install/pull/10#issuecomment-42274686
  
I got it.
Maybe you are right.
I'm so sorry.

only want work for me . so, i can modify this  for myself.
because i'm translat  cs doc from en to zh_CN now.
I've thought it is a bug. but i'm clearly now.

Thanks for you reply.





2014-05-06 13:41 GMT+08:00 runseb :

> Thanks for the patch, it looks good.
>
> However I did not include the managing_networks section in the toc because
> I wanted to keep a clean installation document and not mix
> management/configuration instructions with installation steps.
>
> The managing_networks section is also in the admin guide but not in the
> toc yet.
>
> I'd rather see it in the admin guide than in the install guide. But even
> before adding it, the file needs to be cleaned up and re-organized.
>
> Does that make sense ?
>
> —
> Reply to this email directly or view it on 
GitHub
> .
>


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Review Request 19758: Added a Network Service Provider for JuniperContrailvSRX. This NSP provides sourceNAT functionality.

2014-05-06 Thread Rajesh Battala

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



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


As this is a new file content. please run rat build to verify the license



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


Are there any UnitTests available to test this element?

any marvin tests are available?



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


systemvm service offering can be used or seed required offering and use it.



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


how this template will be seeded?
user will upload this template?



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


what if there are multiple "Juniper vSRX" templates are available?

findBytemplateName method will not search on "uniquename" it will search on 
template name 
and template name can be same for other templates also.
this should be documented that uploaded template name should be with 
"Juniper vSRx" otherwise even user uploads it, code wont be able to find it.

which hypervisor template is supported?




plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java


missed formatting


- Rajesh Battala


On May 3, 2014, 7:30 a.m., Sachchidanand Vaidya wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19758/
> ---
> 
> (Updated May 3, 2014, 7:30 a.m.)
> 
> 
> Review request for cloudstack and Rajesh Battala.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
>  - Added new Network Service Provider "JuniperContrailvSRX".
>  - Impelemted an element (ContrailVSrxElement) to support this Network 
> Service Provider.
>vSRX element currently supports only sourceNAT. It will be enhanced later 
> to support
>more features.
>  - Changes to service-instance creation code to use left virtual-network's 
> SourceNAT addr
>as an interface address on vSRX instance's Public-network interface.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Network.java ef3bcdf 
>   
> plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml
>  99ab02e 
>   
> plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVSrxElementImpl.java
>  PRE-CREATION 
>   
> plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServiceManagerImpl.java
>  f34eacc 
>   
> plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/VMInterfaceModel.java
>  49060f1 
> 
> Diff: https://reviews.apache.org/r/19758/diff/
> 
> 
> Testing
> ---
> 
> Performed unit tests with vSRX network service provider. Also other unit 
> tests pass in local testbed.
> 
> 
> Thanks,
> 
> Sachchidanand Vaidya
> 
>



Re: How is this working?

2014-05-06 Thread Mike Tutkowski
Sorry...I see that you have already updated the ticket.

Thanks!


On Tue, May 6, 2014 at 1:25 AM, Mike Tutkowski  wrote:

> Could you please update this ticket with the dashboard part?
>
> https://issues.apache.org/jira/browse/CLOUDSTACK-6579
>
> Thanks!
>
>
> On Mon, May 5, 2014 at 9:55 PM, Koushik Das wrote:
>
>> The dashboard in CS UI is impacted by this.
>>
>> On 05-May-2014, at 11:25 PM, Mike Tutkowski 
>> wrote:
>>
>> > OK, I can log a JIRA ticket for this.
>> >
>> > Thanks
>> >
>> >
>> > On Mon, May 5, 2014 at 11:49 AM, Nitin Mehta 
>> wrote:
>> >
>> >> Ideally, we should deprecate the column since its not used and causes
>> >> confusion.
>> >> Setting to this value wouldn't help because this column is never
>> updated
>> >> regularly.
>> >>
>> >> On 05/05/14 10:28 AM, "Mike Tutkowski" 
>> >> wrote:
>> >>
>> >>> How's about I just check this code into master?
>> >>>
>> >>> poolVO.setUsedBytes(mspAnswer.getPoolInfo().getCapacityBytes() -
>> >>> mspAnswer.getPoolInfo().getAvailableBytes());
>> >>>
>> >>> It is patterned off of the PrimaryDataStoreHelper.attachHost logic,
>> which
>> >>> looks like this:
>> >>>
>> >>> pool.setUsedBytes(existingInfo.getCapacityBytes() -
>> >>> existingInfo.getAvailableBytes());
>> >>>
>> >>>
>> >>> On Mon, May 5, 2014 at 10:21 AM, Nitin Mehta 
>> >>> wrote:
>> >>>
>>  This column is not used for calculating capacity for pool.
>>  We have always used op host capacity table. Nevertheless please do
>> file
>>  a
>>  bug
>> 
>>  Thanks,
>>  -Nitin
>>  
>>  From: Mike Tutkowski [mike.tutkow...@solidfire.com]
>>  Sent: Monday, May 05, 2014 9:12 PM
>>  To: dev@cloudstack.apache.org
>>  Subject: Re: How is this working?
>> 
>>  My storage plug-in actually uses a custom host listener, so I have
>> not
>>  encountered this issue.
>> 
>>  I don't remember off hand if it was in 4.2 or 4.3, but at some point
>>  someone changed the storage_pool table's available_bytes column to be
>>  used_bytes.
>> 
>>  It looks like this code you reference was missed.
>> 
>> 
>>  On Mon, May 5, 2014 at 4:35 AM, Koushik Das 
>>  wrote:
>> 
>> > I came across this code snippet in hostConnect() method in
>> > DefaultHostListener.java. Look at the line where the used bytes is
>>  set on
>> > the poolVO. This looks like a serious bug. Looking at the history
>> this
>>  code
>> > has been there since a year. Has anyone encountered any issues with
>>  primary
>> > storage capacity?
>> >
>> > ModifyStoragePoolAnswer mspAnswer = (ModifyStoragePoolAnswer)answer;
>> > Š..
>> > StoragePoolVO poolVO = this.primaryStoreDao.findById(poolId);
>> > poolVO.setUsedBytes(mspAnswer.getPoolInfo().getAvailableBytes());
>> > poolVO.setCapacityBytes(mspAnswer.getPoolInfo().getCapacityBytes());
>> > primaryStoreDao.update(pool.getId(), poolVO);
>> >
>> >
>> > -Koushik
>> >
>> 
>> 
>> 
>>  --
>>  *Mike Tutkowski*
>>  *Senior CloudStack Developer, SolidFire Inc.*
>>  e: mike.tutkow...@solidfire.com
>>  o: 303.746.7302
>>  Advancing the way the world uses the
>>  cloud
>>  * *
>> 
>> >>>
>> >>>
>> >>>
>> >>> --
>> >>> *Mike Tutkowski*
>> >>> *Senior CloudStack Developer, SolidFire Inc.*
>> >>> e: mike.tutkow...@solidfire.com
>> >>> o: 303.746.7302
>> >>> Advancing the way the world uses the
>> >>> cloud
>> >>> * *
>> >>
>> >>
>> >
>> >
>> > --
>> > *Mike Tutkowski*
>> > *Senior CloudStack Developer, SolidFire Inc.*
>> > e: mike.tutkow...@solidfire.com
>> > o: 303.746.7302
>> > Advancing the way the world uses the
>> > cloud
>> > *™*
>>
>>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the 
> cloud
> *™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: How is this working?

2014-05-06 Thread Mike Tutkowski
Could you please update this ticket with the dashboard part?

https://issues.apache.org/jira/browse/CLOUDSTACK-6579

Thanks!


On Mon, May 5, 2014 at 9:55 PM, Koushik Das  wrote:

> The dashboard in CS UI is impacted by this.
>
> On 05-May-2014, at 11:25 PM, Mike Tutkowski 
> wrote:
>
> > OK, I can log a JIRA ticket for this.
> >
> > Thanks
> >
> >
> > On Mon, May 5, 2014 at 11:49 AM, Nitin Mehta 
> wrote:
> >
> >> Ideally, we should deprecate the column since its not used and causes
> >> confusion.
> >> Setting to this value wouldn't help because this column is never updated
> >> regularly.
> >>
> >> On 05/05/14 10:28 AM, "Mike Tutkowski" 
> >> wrote:
> >>
> >>> How's about I just check this code into master?
> >>>
> >>> poolVO.setUsedBytes(mspAnswer.getPoolInfo().getCapacityBytes() -
> >>> mspAnswer.getPoolInfo().getAvailableBytes());
> >>>
> >>> It is patterned off of the PrimaryDataStoreHelper.attachHost logic,
> which
> >>> looks like this:
> >>>
> >>> pool.setUsedBytes(existingInfo.getCapacityBytes() -
> >>> existingInfo.getAvailableBytes());
> >>>
> >>>
> >>> On Mon, May 5, 2014 at 10:21 AM, Nitin Mehta 
> >>> wrote:
> >>>
>  This column is not used for calculating capacity for pool.
>  We have always used op host capacity table. Nevertheless please do
> file
>  a
>  bug
> 
>  Thanks,
>  -Nitin
>  
>  From: Mike Tutkowski [mike.tutkow...@solidfire.com]
>  Sent: Monday, May 05, 2014 9:12 PM
>  To: dev@cloudstack.apache.org
>  Subject: Re: How is this working?
> 
>  My storage plug-in actually uses a custom host listener, so I have not
>  encountered this issue.
> 
>  I don't remember off hand if it was in 4.2 or 4.3, but at some point
>  someone changed the storage_pool table's available_bytes column to be
>  used_bytes.
> 
>  It looks like this code you reference was missed.
> 
> 
>  On Mon, May 5, 2014 at 4:35 AM, Koushik Das 
>  wrote:
> 
> > I came across this code snippet in hostConnect() method in
> > DefaultHostListener.java. Look at the line where the used bytes is
>  set on
> > the poolVO. This looks like a serious bug. Looking at the history
> this
>  code
> > has been there since a year. Has anyone encountered any issues with
>  primary
> > storage capacity?
> >
> > ModifyStoragePoolAnswer mspAnswer = (ModifyStoragePoolAnswer)answer;
> > Š..
> > StoragePoolVO poolVO = this.primaryStoreDao.findById(poolId);
> > poolVO.setUsedBytes(mspAnswer.getPoolInfo().getAvailableBytes());
> > poolVO.setCapacityBytes(mspAnswer.getPoolInfo().getCapacityBytes());
> > primaryStoreDao.update(pool.getId(), poolVO);
> >
> >
> > -Koushik
> >
> 
> 
> 
>  --
>  *Mike Tutkowski*
>  *Senior CloudStack Developer, SolidFire Inc.*
>  e: mike.tutkow...@solidfire.com
>  o: 303.746.7302
>  Advancing the way the world uses the
>  cloud
>  * *
> 
> >>>
> >>>
> >>>
> >>> --
> >>> *Mike Tutkowski*
> >>> *Senior CloudStack Developer, SolidFire Inc.*
> >>> e: mike.tutkow...@solidfire.com
> >>> o: 303.746.7302
> >>> Advancing the way the world uses the
> >>> cloud
> >>> * *
> >>
> >>
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the
> > cloud
> > *™*
>
>


-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*