[ACS4.5] move from xen 2 xenserver

2014-04-15 Thread Sebastien Goasguen
Folks,

I just applied a patch from Tim Mackey:

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=748b575af8a66c58b0d52e7457e4d4e1e875231f

commit: 748b575af8a66c58b0d52e7457e4d4e1e875231f 

This followed a [PROPOSAL] thread [1]

This was pushed into a separate branch: xen2server

This is a significant change that we should get consensus on and merge to 
master relatively quickly to avoid any conflicts later on.

Basically, we have been treating xen and xenserver the same so far, since the 
integration with Xen (i.e Xen Project) was/is done via xapi.

There has been discussion to integrate with Xen using straight up libvirt, by 
creating a new Xen agent probably based on the KVM agent and there was some 
discussion to that effect in the Denver Hackathon.

Making a clear split between Xen and Xenserver type of hypervisors will help 
support different integration methods.

I have asked Tim (in the review message) to create a wiki page, discussing all 
of this, but I wanted to give a heads up that this just hit the repo and that 
we should see a [MERGE] thread quickly.

thoughts, flames ?


[1] http://markmail.org/thread/yrl3ii7gqlaaexij

-Sebastien



Re: Review Request 20315: CLOUDSTACK-6401: [hyper-v] fixed correct host state in cloudstack if the hyper-v agent is down

2014-04-15 Thread ASF Subversion and Git Services

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


Commit b78aee8f117f9e89a533afba86b3ff91da19744e in cloudstack's branch 
refs/heads/4.4 from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=b78aee8 ]

CLOUDSTACK-6401: [hyper-v] fixed correct host state, if host is up and hyper-v 
agent is down then the host state will be shown as up(actual host state) in 
cloudstack


- ASF Subversion and Git Services


On April 15, 2014, 5:12 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20315/
> ---
> 
> (Updated April 15, 2014, 5:12 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6401
> https://issues.apache.org/jira/browse/CLOUDSTACK-6401
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> fixed correct host state in cloudstack if the hyper-v agent is down. Added 
> heartbeat file to monitor whether host is actually down or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
>  ec44568 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
> 5ddcbef 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/heartbeat.bat 
> PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20315/diff/
> 
> 
> Testing
> ---
> 
> verified various scenarios
> 
> 1. agent is down but host is up
> 2. both are down
> 3. both are up
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20313: CLOUDSTACK-6400:[hyper-v] [vmsync] Occasionally VM is not deleted from back-end when it is stopped from Hyper-V manager and then destroyed-expunged from CS

2014-04-15 Thread ASF Subversion and Git Services

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


Commit d45ee3ef21fd55c8a1cf1373225cab81389ad8c7 in cloudstack's branch 
refs/heads/4.4 from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d45ee3e ]

CLOUDSTACK-6400: [hyper-v] [vmsync] fixed occasionally VM is not deleted from 
back-end when it is stopped from Hyper-V manager and then destroyed-expunged 
from CS and returning only power VMs for vmsync


- ASF Subversion and Git Services


On April 14, 2014, 8:41 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20313/
> ---
> 
> (Updated April 14, 2014, 8:41 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6400
> https://issues.apache.org/jira/browse/CLOUDSTACK-6400
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6400: Occasionally VM is not deleted from back-end when it is 
> stopped from Hyper-V manager and then destroyed-expunged from CS. With this 
> patch hyper-v agent will delete the stopped VMs on hyper-v host so that 
> hyper-v host is in consistent state with cloudstack. This patch will also 
> make sure that VMs which are in process of creation they are returned as in 
> PowerOn state so that they are not accidentally deleted. Notes are added to 
> vm to identify whether it is created by cloudstack or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
>  8c682ad 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20313/diff/
> 
> 
> Testing
> ---
> 
> verified
> 1. if vm is stopped from hyper-v manager then it should be automatically 
> deleted
> 2. vms which is in process of creation is not getting accidentally deleted
> 3. proper notes are added to vm created by cloudstack
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20311: CLOUDSTACK-6398: [hyper-v] Change auto shutdown and startup actions of VMs to shutdown and nothing

2014-04-15 Thread ASF Subversion and Git Services

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


Commit 36103633c2764fe33374f5cbfb1ac0d5bf24ed1b in cloudstack's branch 
refs/heads/4.4 from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3610363 ]

CLOUDSTACK-6398:[hyper-v] changed auto shutdown and startup actions of VMs to 
shutdown and nothingauto shutdown and startup actions of VMs to shutdown and 
nothing so that they are consistent with cloudstack auto shudown and startup 
behavior


- ASF Subversion and Git Services


On April 14, 2014, 8:28 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20311/
> ---
> 
> (Updated April 14, 2014, 8:28 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6398
> https://issues.apache.org/jira/browse/CLOUDSTACK-6398
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Change auto shutdown and startup actions to shutdown and nothing so that they 
> are consistent with Cloudstack auto shutdown and startup behavior
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20311/diff/
> 
> 
> Testing
> ---
> 
> tested by verifying the state of VMs after hyper-v host restart 
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: CLVM volumes shown under the wrong hypervisor type (OVM) (with FIX!)

2014-04-15 Thread Nux!

On 19.01.2014 18:00, Nux! wrote:

Hi,

Opened https://issues.apache.org/jira/browse/CLOUDSTACK-5907 for this.

There doesn't seem to be any lack of functionality with these
volumes, but it seems wrong to display OVM there instead of KVM and
may cause other issues in the future.

Anyone aware of any workarounds?


Hi,

Can a dev have a look at Tanner's proposed fix and get it in master?

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

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


Re: Review Request 19917: Improvements on exception handlers (JIRA-6242)

2014-04-15 Thread daan Hoogland

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

Ship it!


Ship It!

- daan Hoogland


On April 5, 2014, 3:16 a.m., Ding Yuan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19917/
> ---
> 
> (Updated April 5, 2014, 3:16 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Alex Huang, Chiradeep 
> Vittal, daan Hoogland, edison su, David Nalley, Laszlo Hornyak, Mike 
> Tutkowski, Prachi Damle, and Venkata Siva Vijayendra Bhamidipati.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This is the patch for JIRA-6242. See 
> https://issues.apache.org/jira/browse/CLOUDSTACK-6242 for more details. 
> Thanks!
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java 
> 0d41bc1 
>   
> engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>  01508a4 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 3e088db 
>   
> engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/dao/EngineDataCenterDaoImpl.java
>  4b6818e 
>   engine/schema/src/com/cloud/dc/dao/DataCenterDaoImpl.java ea5039f 
>   engine/schema/src/com/cloud/host/dao/HostDaoImpl.java 426c90d 
>   engine/schema/src/com/cloud/storage/dao/StoragePoolHostDaoImpl.java e42eaf4 
>   engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 34fdca5 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade2214to30.java 58dd916 
>   engine/schema/src/com/cloud/vm/dao/ConsoleProxyDaoImpl.java 5e9c2f0 
>   engine/schema/src/com/cloud/vm/dao/SecondaryStorageVmDaoImpl.java 1f382d6 
>   
> engine/storage/src/org/apache/cloudstack/storage/datastore/DataObjectManagerImpl.java
>  6ed1274 
>   
> framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
>  83c8a42 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/discoverer/XcpServerDiscoverer.java
>  0ad6dc4 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerConnectionPool.java
>  b779085 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerStorageProcessor.java
>  e512046 
>   
> plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/lifecycle/SolidFirePrimaryDataStoreLifeCycle.java
>  af6a77a 
>   server/src/com/cloud/resource/ResourceManagerImpl.java f9a59ba 
>   server/src/com/cloud/server/ConfigurationServerImpl.java b8da4c8 
>   
> services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyThumbnailHandler.java
>  06f21d3 
>   utils/src/com/cloud/utils/net/NetUtils.java 6350986 
> 
> Diff: https://reviews.apache.org/r/19917/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Ding Yuan
> 
>



Re: Review Request 19917: Improvements on exception handlers (JIRA-6242)

2014-04-15 Thread daan Hoogland


> On April 15, 2014, 8:28 a.m., daan Hoogland wrote:
> > Ship It!

c031eb7d38200d680da85ef57367b21df3483c41


- daan


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


On April 5, 2014, 3:16 a.m., Ding Yuan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19917/
> ---
> 
> (Updated April 5, 2014, 3:16 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Alex Huang, Chiradeep 
> Vittal, daan Hoogland, edison su, David Nalley, Laszlo Hornyak, Mike 
> Tutkowski, Prachi Damle, and Venkata Siva Vijayendra Bhamidipati.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This is the patch for JIRA-6242. See 
> https://issues.apache.org/jira/browse/CLOUDSTACK-6242 for more details. 
> Thanks!
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java 
> 0d41bc1 
>   
> engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>  01508a4 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 3e088db 
>   
> engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/dao/EngineDataCenterDaoImpl.java
>  4b6818e 
>   engine/schema/src/com/cloud/dc/dao/DataCenterDaoImpl.java ea5039f 
>   engine/schema/src/com/cloud/host/dao/HostDaoImpl.java 426c90d 
>   engine/schema/src/com/cloud/storage/dao/StoragePoolHostDaoImpl.java e42eaf4 
>   engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 34fdca5 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade2214to30.java 58dd916 
>   engine/schema/src/com/cloud/vm/dao/ConsoleProxyDaoImpl.java 5e9c2f0 
>   engine/schema/src/com/cloud/vm/dao/SecondaryStorageVmDaoImpl.java 1f382d6 
>   
> engine/storage/src/org/apache/cloudstack/storage/datastore/DataObjectManagerImpl.java
>  6ed1274 
>   
> framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
>  83c8a42 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/discoverer/XcpServerDiscoverer.java
>  0ad6dc4 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerConnectionPool.java
>  b779085 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerStorageProcessor.java
>  e512046 
>   
> plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/lifecycle/SolidFirePrimaryDataStoreLifeCycle.java
>  af6a77a 
>   server/src/com/cloud/resource/ResourceManagerImpl.java f9a59ba 
>   server/src/com/cloud/server/ConfigurationServerImpl.java b8da4c8 
>   
> services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyThumbnailHandler.java
>  06f21d3 
>   utils/src/com/cloud/utils/net/NetUtils.java 6350986 
> 
> Diff: https://reviews.apache.org/r/19917/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Ding Yuan
> 
>



Re: Review Request 19917: Improvements on exception handlers (JIRA-6242)

2014-04-15 Thread Ding Yuan
HI Daan,
Thanks a lot for committing it! Sorry for the troubles, and I will do what you 
said next time if I upload the patch (thanks for the suggestions)!
Cheers!
Ding

On Apr 15, 2014, at 1:52 AM, Daan Hoogland  wrote:

> it applies,
> 
> I won't push it like this though. The comment says 'squash 6242 commits' so i 
> will take the liberty of changing it to something like 'CLOUDSTACK-6242: 
> exception handling improvements'.
> 
> Also I'd appriciate it if you upload to reviewboard next time. This make 
> reviewing easier for me. It gives me the chance to you diffs between your 
> diffs so after a big diff I can see if nothing deteriorated next time. For 
> now I'll just edit the comment and apply (after a unit test run).
> 
> regards,
> Daan
> 
> 
> 
> 
> On Mon, Apr 14, 2014 at 11:18 PM, Ding Yuan  wrote:
> Hi Daan,
> Sorry about that. Rebased my patch on the latest master. Attaching the patch. 
> Please let me know if it still doesn’t work...
> thanks!
> Ding
> 
> On Apr 14, 2014, at 3:04 PM, Daan Hoogland  wrote:
> 
>> thanks Ding,
>> 
>> it doesn't apply, I'm afraid. Can you rebase it to latest master? Make sure 
>> it is only one commit.
>> 
>> 
>> On Mon, Apr 14, 2014 at 8:48 PM, Ding Yuan  wrote:
>> Hi Daan,
>> Here you go! Please let me know if this is not what you want...
>> Thanks a lot for this!
>> 
>> Ding
>> 
>> 
>> On Apr 14, 2014, at 2:13 PM, daan Hoogland  wrote:
>> 
>>> 
>>> This is an automatically generated e-mail. To reply, visit: 
>>> https://reviews.apache.org/r/19917/
>>> 
>>> Ding,
>>> 
>>> sorry for the late reaction. I forgot to look at your patch last week.
>>> 
>>> Can you create the patch with 'git format-patch' ? I want to apply it with 
>>> 'git am -s '
>>> 
>>> regards,
>>> Daan
>>> 
>>> - daan Hoogland
>>> 
>>> 
>>> On April 5th, 2014, 3:16 a.m. UTC, Ding Yuan wrote:
>>> 
>>> Review request for cloudstack, Alena Prokharchyk, Alex Huang, Chiradeep 
>>> Vittal, daan Hoogland, edison su, David Nalley, Laszlo Hornyak, Mike 
>>> Tutkowski, Prachi Damle, and Venkata Siva Vijayendra Bhamidipati.
>>> By Ding Yuan.
>>> Updated April 5, 2014, 3:16 a.m.
>>> 
>>> Repository: cloudstack-git
>>> Description
>>> 
>>> This is the patch for JIRA-6242. See 
>>> https://issues.apache.org/jira/browse/CLOUDSTACK-6242 for more details. 
>>> Thanks!
>>> Diffs
>>> 
>>> engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java 
>>> (0d41bc1)
>>> engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>>>  (01508a4)
>>> engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
>>> (3e088db)
>>> engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/dao/EngineDataCenterDaoImpl.java
>>>  (4b6818e)
>>> engine/schema/src/com/cloud/dc/dao/DataCenterDaoImpl.java (ea5039f)
>>> engine/schema/src/com/cloud/host/dao/HostDaoImpl.java (426c90d)
>>> engine/schema/src/com/cloud/storage/dao/StoragePoolHostDaoImpl.java 
>>> (e42eaf4)
>>> engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java (34fdca5)
>>> engine/schema/src/com/cloud/upgrade/dao/Upgrade2214to30.java (58dd916)
>>> engine/schema/src/com/cloud/vm/dao/ConsoleProxyDaoImpl.java (5e9c2f0)
>>> engine/schema/src/com/cloud/vm/dao/SecondaryStorageVmDaoImpl.java (1f382d6)
>>> engine/storage/src/org/apache/cloudstack/storage/datastore/DataObjectManagerImpl.java
>>>  (6ed1274)
>>> framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
>>>  (83c8a42)
>>> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/discoverer/XcpServerDiscoverer.java
>>>  (0ad6dc4)
>>> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerConnectionPool.java
>>>  (b779085)
>>> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerStorageProcessor.java
>>>  (e512046)
>>> plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/lifecycle/SolidFirePrimaryDataStoreLifeCycle.java
>>>  (af6a77a)
>>> server/src/com/cloud/resource/ResourceManagerImpl.java (f9a59ba)
>>> server/src/com/cloud/server/ConfigurationServerImpl.java (b8da4c8)
>>> services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyThumbnailHandler.java
>>>  (06f21d3)
>>> utils/src/com/cloud/utils/net/NetUtils.java (6350986)
>>> View Diff
>>> 
>> 
>> 
>> 
>> 
>> 
>> -- 
>> Daan
> 
> 
> 
> 
> 
> -- 
> Daan



Review Request 20365: CLOUDSTACK-6411: some files are not loading in visual studio due to license header at top making them unparsable

2014-04-15 Thread Anshul Gangwar

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

Review request for cloudstack, Devdeep Singh and Rajesh Battala.


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


Repository: cloudstack-git


Description
---

 some files are not loading in visual studio due to license header at top 
making them unparsable. removed the license header and put them in exclude list 
of rat plugin


Diffs
-

  plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 5181cb8 
  
plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
 fb5b962 
  pom.xml 6837717 

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


Testing
---

verified with rat plugin that the license check is passed successfully


Thanks,

Anshul Gangwar



RE: [ACS4.5] move from xen 2 xenserver

2014-04-15 Thread Stephen Turner
As I said in the previous threaed, I'm +1 on the principle. It will avoid 
confusion between straight Xen and XenServer. It will also allow us to offer a 
non-XenServer Xen implementation.

Sebastien, as all the filenames have changed, it's not clear from the diff 
whether this is just a straight renaming with no other changes. Could you 
confirm that?

Also, are there any backwards compatibility implications for the API? Or did we 
already use "XenServer" instead of "Xen" there?

-- 
Stephen Turner


-Original Message-
From: Sebastien Goasguen [mailto:run...@gmail.com] 
Sent: 15 April 2014 08:36
To: dev@cloudstack.apache.org
Subject: [ACS4.5] move from xen 2 xenserver

Folks,

I just applied a patch from Tim Mackey:

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=748b575af8a66c58b0d52e7457e4d4e1e875231f

commit: 748b575af8a66c58b0d52e7457e4d4e1e875231f 

This followed a [PROPOSAL] thread [1]

This was pushed into a separate branch: xen2server

This is a significant change that we should get consensus on and merge to 
master relatively quickly to avoid any conflicts later on.

Basically, we have been treating xen and xenserver the same so far, since the 
integration with Xen (i.e Xen Project) was/is done via xapi.

There has been discussion to integrate with Xen using straight up libvirt, by 
creating a new Xen agent probably based on the KVM agent and there was some 
discussion to that effect in the Denver Hackathon.

Making a clear split between Xen and Xenserver type of hypervisors will help 
support different integration methods.

I have asked Tim (in the review message) to create a wiki page, discussing all 
of this, but I wanted to give a heads up that this just hit the repo and that 
we should see a [MERGE] thread quickly.

thoughts, flames ?


[1] http://markmail.org/thread/yrl3ii7gqlaaexij

-Sebastien



Re: [Responsiveness report] 2014w13

2014-04-15 Thread Daan Hoogland
I use google labels and thread display, the rest is mostly manual. I
check with markmail if the questions are really not answered. There is
a responsiveness report page on the wiki that explains my process and
that I hardly get any input on so please flame me.

On Tue, Apr 15, 2014 at 10:56 AM, Noah Slater  wrote:
> How are you compiling this?
>
> On 12 April 2014 14:32, Daan Hoogland  wrote:
>> Hello everybody,
>>
>> I want to bring under your attention that our responsiveness as a
>> community seems to deteriorate. I am busy with week 13 [1] and it is
>> the worst since I started doing this a year ago. I hope this is a
>> temporary failure on this part, maybe explained by the upcoming
>> cccna14 that has just come to pass. I do remember starting with this
>> because just before cccna13 it annoyed me that *6* messages had
>> remained unanswerred in the devlist. Week 13 of 2014 is a lot worse
>> and it looks like week 14 was as well.
>>
>> Let's leave no one behind!
>>
>> [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/2014W13
>>
>>
>> thanks
>> --
>> Daan
>
>
>
> --
> Noah Slater
> https://twitter.com/nslater



-- 
Daan


login API with MD5 is not working

2014-04-15 Thread Tejas Gadaria
Hi,

I am trying to login in to CS 4.3 though login API.

I am passing MD5 hash (1st) in password that works fine with CS 4.0.2 but
same doesn't works well with CS 4.3. Then I try to pass password in plain
text ( 2nd) & it worked, Is this a bug?

Both APIs are given below,


1)
http://10.129.151.55:8080/client/api?&command=login&username=admin&password=5f4dcc3b5aa765d61d8327deb882cf99


2)
http://10.129.151.55:8080/client/api?&command=login&username=admin&password=password

Regards,
Tejas


Re: Network state machine

2014-04-15 Thread Murali Reddy
On 15/04/14 3:08 AM, "Alena Prokharchyk" 
wrote:

>Murali,
>
>I have a question to you. In NetworkOrchestrator, implementNetwork code,
>when we update the state for the network, why don¹t we use state machine
>for Shared networks, but instead update it directly in the DB?
>
>
>if (isSharedNetworkWithServices(network)) {
>network.setState(Network.State.Implementing);
>} else {
>stateTransitTo(network, Event.ImplementNetwork);
>}
>
>
>That doesn¹t seem right to me as there should be no exception when it
>comes to state update - the state machine should always be used once its
>implemented for the object. If your code needs additional state
>transitions for the Setup state, why can¹t you add it?

Initially network just had states and states were set explicitly. Actually
I happened to add state changes to explicitly happen through the state
machine for resource state change events for EventBus. But the problem I
had with
shared networks was, it had different life cycle (before cloudstack
supported L4-L7 services in shared networks) and shared network with L4-L7
service has life cycle similar to that of isolated network. For the
network in 'Setup' state, ImplementNetwork event shall succeed only for
shared network with services but should fail for shared network without
L4-L7 services. Similarly an implemented network in 'Implemented' state on
'ShutdownNetwork' event can go to 'Allocated' or 'Setup' state depending
on the network type (isolate/shared). I needed some sort of conditional
state transition in the state machine which is not there.

Due to time constraint I added minimal disruptive changes. Doing it
cleaner way means revamping the network states and transitions to avoid
conflicting transitions. Please do open a bug to fix it.


>
>Thanks,
>Alena.
>
>




Re: Review Request 20311: CLOUDSTACK-6398: [hyper-v] Change auto shutdown and startup actions of VMs to shutdown and nothing

2014-04-15 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On April 14, 2014, 8:28 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20311/
> ---
> 
> (Updated April 14, 2014, 8:28 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6398
> https://issues.apache.org/jira/browse/CLOUDSTACK-6398
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Change auto shutdown and startup actions to shutdown and nothing so that they 
> are consistent with Cloudstack auto shutdown and startup behavior
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20311/diff/
> 
> 
> Testing
> ---
> 
> tested by verifying the state of VMs after hyper-v host restart 
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20311: CLOUDSTACK-6398: [hyper-v] Change auto shutdown and startup actions of VMs to shutdown and nothing

2014-04-15 Thread ASF Subversion and Git Services

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


Commit 0aa6cc60c73698a7be960d2503a5abceaf16d8c7 in cloudstack's branch 
refs/heads/master from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=0aa6cc6 ]

CLOUDSTACK-6398:[hyper-v] changed auto shutdown and startup actions of VMs to 
shutdown and nothingauto shutdown and startup actions of VMs to shutdown and 
nothing so that they are consistent with cloudstack auto shudown and startup 
behavior


- ASF Subversion and Git Services


On April 14, 2014, 8:28 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20311/
> ---
> 
> (Updated April 14, 2014, 8:28 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6398
> https://issues.apache.org/jira/browse/CLOUDSTACK-6398
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Change auto shutdown and startup actions to shutdown and nothing so that they 
> are consistent with Cloudstack auto shutdown and startup behavior
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20311/diff/
> 
> 
> Testing
> ---
> 
> tested by verifying the state of VMs after hyper-v host restart 
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20313: CLOUDSTACK-6400:[hyper-v] [vmsync] Occasionally VM is not deleted from back-end when it is stopped from Hyper-V manager and then destroyed-expunged from CS

2014-04-15 Thread ASF Subversion and Git Services

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


Commit 3ec7497e1ebcaa65a8cbd24c27326356355ac8cc in cloudstack's branch 
refs/heads/master from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3ec7497 ]

CLOUDSTACK-6400: [hyper-v] [vmsync] fixed occasionally VM is not deleted from 
back-end when it is stopped from Hyper-V manager and then destroyed-expunged 
from CS and returning only power VMs for vmsync


- ASF Subversion and Git Services


On April 14, 2014, 8:41 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20313/
> ---
> 
> (Updated April 14, 2014, 8:41 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6400
> https://issues.apache.org/jira/browse/CLOUDSTACK-6400
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6400: Occasionally VM is not deleted from back-end when it is 
> stopped from Hyper-V manager and then destroyed-expunged from CS. With this 
> patch hyper-v agent will delete the stopped VMs on hyper-v host so that 
> hyper-v host is in consistent state with cloudstack. This patch will also 
> make sure that VMs which are in process of creation they are returned as in 
> PowerOn state so that they are not accidentally deleted. Notes are added to 
> vm to identify whether it is created by cloudstack or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
>  8c682ad 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20313/diff/
> 
> 
> Testing
> ---
> 
> verified
> 1. if vm is stopped from hyper-v manager then it should be automatically 
> deleted
> 2. vms which is in process of creation is not getting accidentally deleted
> 3. proper notes are added to vm created by cloudstack
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20313: CLOUDSTACK-6400:[hyper-v] [vmsync] Occasionally VM is not deleted from back-end when it is stopped from Hyper-V manager and then destroyed-expunged from CS

2014-04-15 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On April 14, 2014, 8:41 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20313/
> ---
> 
> (Updated April 14, 2014, 8:41 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6400
> https://issues.apache.org/jira/browse/CLOUDSTACK-6400
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6400: Occasionally VM is not deleted from back-end when it is 
> stopped from Hyper-V manager and then destroyed-expunged from CS. With this 
> patch hyper-v agent will delete the stopped VMs on hyper-v host so that 
> hyper-v host is in consistent state with cloudstack. This patch will also 
> make sure that VMs which are in process of creation they are returned as in 
> PowerOn state so that they are not accidentally deleted. Notes are added to 
> vm to identify whether it is created by cloudstack or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
>  8c682ad 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
> 79fd0bc 
> 
> Diff: https://reviews.apache.org/r/20313/diff/
> 
> 
> Testing
> ---
> 
> verified
> 1. if vm is stopped from hyper-v manager then it should be automatically 
> deleted
> 2. vms which is in process of creation is not getting accidentally deleted
> 3. proper notes are added to vm created by cloudstack
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20315: CLOUDSTACK-6401: [hyper-v] fixed correct host state in cloudstack if the hyper-v agent is down

2014-04-15 Thread ASF Subversion and Git Services

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


Commit 55f53e13e82179c2cbde3f667869495ec944f496 in cloudstack's branch 
refs/heads/master from Anshul Gangwar
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=55f53e1 ]

CLOUDSTACK-6401: [hyper-v] fixed correct host state, if host is up and hyper-v 
agent is down then the host state will be shown as up(actual host state) in 
cloudstack


- ASF Subversion and Git Services


On April 15, 2014, 5:12 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20315/
> ---
> 
> (Updated April 15, 2014, 5:12 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6401
> https://issues.apache.org/jira/browse/CLOUDSTACK-6401
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> fixed correct host state in cloudstack if the hyper-v agent is down. Added 
> heartbeat file to monitor whether host is actually down or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
>  ec44568 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
> 5ddcbef 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/heartbeat.bat 
> PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20315/diff/
> 
> 
> Testing
> ---
> 
> verified various scenarios
> 
> 1. agent is down but host is up
> 2. both are down
> 3. both are up
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: Review Request 20315: CLOUDSTACK-6401: [hyper-v] fixed correct host state in cloudstack if the hyper-v agent is down

2014-04-15 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On April 15, 2014, 5:12 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20315/
> ---
> 
> (Updated April 15, 2014, 5:12 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6401
> https://issues.apache.org/jira/browse/CLOUDSTACK-6401
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> fixed correct host state in cloudstack if the hyper-v agent is down. Added 
> heartbeat file to monitor whether host is actually down or not
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
>  ec44568 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
>  4f4c14e 
>   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
> 5ddcbef 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/heartbeat.bat 
> PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20315/diff/
> 
> 
> Testing
> ---
> 
> verified various scenarios
> 
> 1. agent is down but host is up
> 2. both are down
> 3. both are up
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Review Request 20366: Integrating Windows wix tool to maven

2014-04-15 Thread Damodar Reddy Talakanti

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

Review request for cloudstack, Abhinandan Prateek and daan Hoogland.


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

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


Repository: cloudstack-git


Description
---

Integrated Wix tool set with maven build process so that no manual steps are 
required to create windows msi installer file.


Diffs
-

  Wix-cloudstack-maven-plugin/pom.xml PRE-CREATION 
  Wix-cloudstack-maven-plugin/src/org/apache/cloudstack/wix/HeatMojo.java 
PRE-CREATION 
  client/pom.xml d8dbde7 

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


Testing
---

Build process has been Tested on Windows server.
Regular profile Builds are tested 
Tested new profile "buildw" and verified msi file is getting created in target.


Thanks,

Damodar Reddy Talakanti



Review Request 20367: If already tmp directory exist inject Keys is failing while starting management server on windows

2014-04-15 Thread Damodar Reddy Talakanti

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

Review request for cloudstack and Abhinandan Prateek.


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

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


Repository: cloudstack-git


Description
---

For any reason if tmp directory present on windows users home directory the 
inject keys into system vm is failing on management server start up.


Diffs
-

  scripts/vm/systemvm/injectkeys.py bd840f9 

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


Testing
---

Tested on windows 2008 server on fresh setup


Thanks,

Damodar Reddy Talakanti



Re: Review Request 19616: Added check for null return.

2014-04-15 Thread Alex Hitchins

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

(Updated April 15, 2014, 10:18 a.m.)


Review request for cloudstack and daan Hoogland.


Changes
---

Updated


Repository: cloudstack-git


Description (updated)
---

Added check for returned null, if received then throw exception.

Amendments made as per dev mailing list comments.


Diffs (updated)
-

  server/src/com/cloud/storage/VolumeApiServiceImpl.java 680cd2e 

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


Testing
---

Compiled & ran.


Thanks,

Alex Hitchins



Re: [PROPOSAL] region level VPC and guest network spanning multiple zones

2014-04-15 Thread Murali Reddy

Sanjeev,

Please see answers inline. You may want to also check out
https://cwiki.apache.org/confluence/display/CLOUDSTACK/OVS+Tunnel+Manager+f
or+CloudStack for general limitation etc.

Some of the question are good points that needed to be in FS, I will add
them where ever appropriate.

On 09/04/14 4:54 AM, "Sanjeev Neelarapu" 
wrote:

>Hi ,
>
>Following are the review comments on the FS posted @
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/Region+level+VPC+an
>d
>+guest+network+spanning+multiple+zones:
>
>How many guest networks we can scale using SDN?

We could have theoretical maximum of 2^32 -1 GRE keys, so if CloudStack
can spin up networks in that order, there should not be any limitation
from OVS to create overlay networks.

>Do we support mixed networks in a zone (VLAN+Tunnel)?

Yes its possible to use VLAN and overlay networks in same zone. There is
no constraint preventing.

>Any limitation on the number of zones an SDN network can span?

No

>What are the services supported by SDN networks?
>What are the services supported by region level VPC?

There is no restriction on the services supported. Think OVS overlay
networking solution as a L2/L3 solution. You can still choose all the
services available for VLAN networks.

>With SDN networks do we support any external devices?

No not at the moment as external devices can not understand tunnelled
(GRE) traffic. 

>Do we support SGs with SDN?

No.

>Do we support monitoring for SDN controller and the GRE tunnels between
>the hosts across zones?

There is no real-time status gathering if tunnelling functioning as
expected. Ideally once you establish tunnel it should just work as
seamlessly as VLAN's.

>What are the alerts and events generated by the controller?

None.

>With SDNs do we support zones with mixed hypervisors?(e.g: zone1 with xen
>and zone2 with ESX)

Only XenServer, KVM are supported.

>If vpc is tied to a zone can we have the tiers spanned across multiple
>zones?

Yes

>If vpc is region level do we have control on the zone in which VR has to
>be deployed?

Yes, you still have start creating a VPC in a zone/

>Networks spanning across multiple zones is applicable only to vpc
>networks(tiers) or any guest network which is not part of vpc as well?

Both for the VPC tiers and regular isolated/shared networks.

>Do we support SDNs in an upgraded environment?

Yes

>Can we upgrade vlan based networks to SDN and vice-versa?

Yes, its possible

>Upgrading zone level vpc to region level vpc and vice-versa?

No

>What are the external dependencies for the SDN to work (ports need to be
>opened on the zone level firewalls)?

Yes, GRE traffic need to be permuted across the zones.




Build failed in Jenkins: build-systemvm64-4.2 #236

2014-04-15 Thread jenkins
See 

--
[...truncated 32989 lines...]
24/24 KB   
   
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-parent/8/maven-parent-8.pom
 (24 KB at 444.7 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-artifact/2.0.9/maven-artifact-2.0.9.pom
2/2 KB 
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-artifact/2.0.9/maven-artifact-2.0.9.pom
 (2 KB at 38.6 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/1.5.1/plexus-utils-1.5.1.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-utils/1.5.1/plexus-utils-1.5.1.pom
 (3 KB at 65.9 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-core/2.0.9/maven-core-2.0.9.pom
4/8 KB   
8/8 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-core/2.0.9/maven-core-2.0.9.pom
 (8 KB at 223.6 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-settings/2.0.9/maven-settings-2.0.9.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-settings/2.0.9/maven-settings-2.0.9.pom
 (3 KB at 60.9 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-model/2.0.9/maven-model-2.0.9.pom
4/4 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-model/2.0.9/maven-model-2.0.9.pom
 (4 KB at 76.7 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-parameter-documenter/2.0.9/maven-plugin-parameter-documenter-2.0.9.pom
2/2 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-parameter-documenter/2.0.9/maven-plugin-parameter-documenter-2.0.9.pom
 (2 KB at 53.3 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-profile/2.0.9/maven-profile-2.0.9.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-profile/2.0.9/maven-profile-2.0.9.pom
 (3 KB at 58.9 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-repository-metadata/2.0.9/maven-repository-metadata-2.0.9.pom
2/2 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-repository-metadata/2.0.9/maven-repository-metadata-2.0.9.pom
 (2 KB at 53.0 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-error-diagnostics/2.0.9/maven-error-diagnostics-2.0.9.pom
2/2 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-error-diagnostics/2.0.9/maven-error-diagnostics-2.0.9.pom
 (2 KB at 50.1 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-project/2.0.9/maven-project-2.0.9.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-project/2.0.9/maven-project-2.0.9.pom
 (3 KB at 73.5 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-artifact-manager/2.0.9/maven-artifact-manager-2.0.9.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-artifact-manager/2.0.9/maven-artifact-manager-2.0.9.pom
 (3 KB at 77.6 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-registry/2.0.9/maven-plugin-registry-2.0.9.pom
2/2 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-registry/2.0.9/maven-plugin-registry-2.0.9.pom
 (2 KB at 53.5 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-descriptor/2.0.9/maven-plugin-descriptor-2.0.9.pom
3/3 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-plugin-descriptor/2.0.9/maven-plugin-descriptor-2.0.9.pom
 (3 KB at 61.4 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-monitor/2.0.9/maven-monitor-2.0.9.pom
2/2 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-monitor/2.0.9/maven-monitor-2.0.9.pom
 (2 KB at 38.0 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-toolchain/1.0/maven-toolchain-1.0.pom
4/4 KB   
 
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/maven/maven-toolchain/1.0/maven-toolchain-1.0.pom
 (4 KB at 95.3 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-compiler-api/1.9.1/plexus-compiler-api-1.9.1.pom
867/867 B   

Downloaded: 
http://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-compiler-api/1.9.1/plexus-compiler-api-1.9.1.pom
 (867 B at 24.9 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-compiler/1.9.1/plexus-compiler-1.9.1.pom
4/4 KB  
 
Downloaded: 
http://repo.mav

Re: [ACS4.5] move from xen 2 xenserver

2014-04-15 Thread Sebastien Goasguen

On Apr 15, 2014, at 4:46 AM, Stephen Turner  wrote:

> As I said in the previous threaed, I'm +1 on the principle. It will avoid 
> confusion between straight Xen and XenServer. It will also allow us to offer 
> a non-XenServer Xen implementation.
> 
> Sebastien, as all the filenames have changed, it's not clear from the diff 
> whether this is just a straight renaming with no other changes. Could you 
> confirm that?
> 

That's what it looks like, basically it creates a 'xenserver' plugin
I know Tim has a prototype of Xen support as well, but it was not in this 
commit it seems.


> Also, are there any backwards compatibility implications for the API? Or did 
> we already use "XenServer" instead of "Xen" there?
> 

In the CloudStack API ?

We are probably using Xen as value in several api calls…so we will need to 
check this carefully before any merge, we definitely don't want to break 
backward compatibility, or if we do we will need to move to 5.0

> -- 
> Stephen Turner
> 
> 
> -Original Message-
> From: Sebastien Goasguen [mailto:run...@gmail.com] 
> Sent: 15 April 2014 08:36
> To: dev@cloudstack.apache.org
> Subject: [ACS4.5] move from xen 2 xenserver
> 
> Folks,
> 
> I just applied a patch from Tim Mackey:
> 
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=748b575af8a66c58b0d52e7457e4d4e1e875231f
> 
> commit: 748b575af8a66c58b0d52e7457e4d4e1e875231f 
> 
> This followed a [PROPOSAL] thread [1]
> 
> This was pushed into a separate branch: xen2server
> 
> This is a significant change that we should get consensus on and merge to 
> master relatively quickly to avoid any conflicts later on.
> 
> Basically, we have been treating xen and xenserver the same so far, since the 
> integration with Xen (i.e Xen Project) was/is done via xapi.
> 
> There has been discussion to integrate with Xen using straight up libvirt, by 
> creating a new Xen agent probably based on the KVM agent and there was some 
> discussion to that effect in the Denver Hackathon.
> 
> Making a clear split between Xen and Xenserver type of hypervisors will help 
> support different integration methods.
> 
> I have asked Tim (in the review message) to create a wiki page, discussing 
> all of this, but I wanted to give a heads up that this just hit the repo and 
> that we should see a [MERGE] thread quickly.
> 
> thoughts, flames ?
> 
> 
> [1] http://markmail.org/thread/yrl3ii7gqlaaexij
> 
> -Sebastien
> 



Re: [Responsiveness report] 2014w13

2014-04-15 Thread Daan Hoogland
On Tue, Apr 15, 2014 at 11:11 AM, Noah Slater  wrote:
> No, I think this is great.
>
> Two improvements I can think of:
>
> 1. Automate it, so it's parsing the mbox files on people.a.o.
Sound good. The problem I have is that an answer to a question on user
to user-cn. Another one is that questions might have a follow-up but
this might be a quetion as well. I want to include those in the
report. Any sugestions that help automate the proces further are
welcome.

>
> 2. Quantify it, and visualise the data.
sure, need to do this.

>
> On 15 April 2014 11:00, Daan Hoogland  wrote:
>> I use google labels and thread display, the rest is mostly manual. I
>> check with markmail if the questions are really not answered. There is
>> a responsiveness report page on the wiki that explains my process and
>> that I hardly get any input on so please flame me.
>>
>> On Tue, Apr 15, 2014 at 10:56 AM, Noah Slater  wrote:
>>> How are you compiling this?
>>>
>>> On 12 April 2014 14:32, Daan Hoogland  wrote:
 Hello everybody,

 I want to bring under your attention that our responsiveness as a
 community seems to deteriorate. I am busy with week 13 [1] and it is
 the worst since I started doing this a year ago. I hope this is a
 temporary failure on this part, maybe explained by the upcoming
 cccna14 that has just come to pass. I do remember starting with this
 because just before cccna13 it annoyed me that *6* messages had
 remained unanswerred in the devlist. Week 13 of 2014 is a lot worse
 and it looks like week 14 was as well.

 Let's leave no one behind!

 [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/2014W13


 thanks
 --
 Daan
>>>
>>>
>>>
>>> --
>>> Noah Slater
>>> https://twitter.com/nslater
>>
>>
>>
>> --
>> Daan
>
>
>
> --
> Noah Slater
> https://twitter.com/nslater



-- 
Daan


Re: Review Request 20250: Fixed errors with simulator VM snapshot delete and revert

2014-04-15 Thread Koushik Das

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

Ship it!


Ship It!

- Koushik Das


On April 11, 2014, 2:37 a.m., Chris Suich wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20250/
> ---
> 
> (Updated April 11, 2014, 2:37 a.m.)
> 
> 
> Review request for cloudstack and Alex Huang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> When doing some testing with the simulator, I found that VM snapshot delte 
> and revert were failing with an error saying that a vm with the specified 
> name didn't exist. It appears that a '!=' should have been an '=='. After 
> that, there was a NPE because the simulator manager wasn't sending a volume 
> TO list back.
> 
> This patch fixes those issues so that simulator VM snapshot delete and revert 
> now works as expected.
> 
> I just wanted to throw this up for review since the change that caused this 
> error was fairly recent. Alex - can you confirm whether this was just a typo 
> of '!=' instead of '=='?
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockVmManagerImpl.java
>  f487eb4 
> 
> Diff: https://reviews.apache.org/r/20250/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Chris Suich
> 
>



Re: [Responsiveness report] 2014w13

2014-04-15 Thread Sebastien Goasguen
In my crazy head I sometimes wish that we would have a big red button on our 
website.

It would be a twilio app that would forward the call 24/7 to members of our 
list willing to help out.

A volunteer based 24/7 hot-line for cloudstack ! 

-sebastien

On Apr 15, 2014, at 6:34 AM, Daan Hoogland  wrote:

> On Tue, Apr 15, 2014 at 11:11 AM, Noah Slater  wrote:
>> No, I think this is great.
>> 
>> Two improvements I can think of:
>> 
>> 1. Automate it, so it's parsing the mbox files on people.a.o.
> Sound good. The problem I have is that an answer to a question on user
> to user-cn. Another one is that questions might have a follow-up but
> this might be a quetion as well. I want to include those in the
> report. Any sugestions that help automate the proces further are
> welcome.
> 
>> 
>> 2. Quantify it, and visualise the data.
> sure, need to do this.
> 
>> 
>> On 15 April 2014 11:00, Daan Hoogland  wrote:
>>> I use google labels and thread display, the rest is mostly manual. I
>>> check with markmail if the questions are really not answered. There is
>>> a responsiveness report page on the wiki that explains my process and
>>> that I hardly get any input on so please flame me.
>>> 
>>> On Tue, Apr 15, 2014 at 10:56 AM, Noah Slater  wrote:
 How are you compiling this?
 
 On 12 April 2014 14:32, Daan Hoogland  wrote:
> Hello everybody,
> 
> I want to bring under your attention that our responsiveness as a
> community seems to deteriorate. I am busy with week 13 [1] and it is
> the worst since I started doing this a year ago. I hope this is a
> temporary failure on this part, maybe explained by the upcoming
> cccna14 that has just come to pass. I do remember starting with this
> because just before cccna13 it annoyed me that *6* messages had
> remained unanswerred in the devlist. Week 13 of 2014 is a lot worse
> and it looks like week 14 was as well.
> 
> Let's leave no one behind!
> 
> [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/2014W13
> 
> 
> thanks
> --
> Daan
 
 
 
 --
 Noah Slater
 https://twitter.com/nslater
>>> 
>>> 
>>> 
>>> --
>>> Daan
>> 
>> 
>> 
>> --
>> Noah Slater
>> https://twitter.com/nslater
> 
> 
> 
> -- 
> Daan



Re: [Responsiveness report] 2014w13

2014-04-15 Thread Daan Hoogland
hihi, I think you are stampeding. amok.

On Tue, Apr 15, 2014 at 12:38 PM, Sebastien Goasguen  wrote:
> In my crazy head I sometimes wish that we would have a big red button on our 
> website.
>
> It would be a twilio app that would forward the call 24/7 to members of our 
> list willing to help out.
>
> A volunteer based 24/7 hot-line for cloudstack !
>
> -sebastien
>
> On Apr 15, 2014, at 6:34 AM, Daan Hoogland  wrote:
>
>> On Tue, Apr 15, 2014 at 11:11 AM, Noah Slater  wrote:
>>> No, I think this is great.
>>>
>>> Two improvements I can think of:
>>>
>>> 1. Automate it, so it's parsing the mbox files on people.a.o.
>> Sound good. The problem I have is that an answer to a question on user
>> to user-cn. Another one is that questions might have a follow-up but
>> this might be a quetion as well. I want to include those in the
>> report. Any sugestions that help automate the proces further are
>> welcome.
>>
>>>
>>> 2. Quantify it, and visualise the data.
>> sure, need to do this.
>>
>>>
>>> On 15 April 2014 11:00, Daan Hoogland  wrote:
 I use google labels and thread display, the rest is mostly manual. I
 check with markmail if the questions are really not answered. There is
 a responsiveness report page on the wiki that explains my process and
 that I hardly get any input on so please flame me.

 On Tue, Apr 15, 2014 at 10:56 AM, Noah Slater  wrote:
> How are you compiling this?
>
> On 12 April 2014 14:32, Daan Hoogland  wrote:
>> Hello everybody,
>>
>> I want to bring under your attention that our responsiveness as a
>> community seems to deteriorate. I am busy with week 13 [1] and it is
>> the worst since I started doing this a year ago. I hope this is a
>> temporary failure on this part, maybe explained by the upcoming
>> cccna14 that has just come to pass. I do remember starting with this
>> because just before cccna13 it annoyed me that *6* messages had
>> remained unanswerred in the devlist. Week 13 of 2014 is a lot worse
>> and it looks like week 14 was as well.
>>
>> Let's leave no one behind!
>>
>> [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/2014W13
>>
>>
>> thanks
>> --
>> Daan
>
>
>
> --
> Noah Slater
> https://twitter.com/nslater



 --
 Daan
>>>
>>>
>>>
>>> --
>>> Noah Slater
>>> https://twitter.com/nslater
>>
>>
>>
>> --
>> Daan
>



-- 
Daan


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

2014-04-15 Thread jenkins
See 



Re: Review Request 20366: Integrating Windows wix tool to maven

2014-04-15 Thread daan Hoogland

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


please test on other platforms to make sure it doesn't break non-windows builds


Wix-cloudstack-maven-plugin/pom.xml


don't forget the license



Wix-cloudstack-maven-plugin/src/org/apache/cloudstack/wix/HeatMojo.java


don't forget the license


- daan Hoogland


On April 15, 2014, 9:29 a.m., Damodar Reddy Talakanti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20366/
> ---
> 
> (Updated April 15, 2014, 9:29 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and daan Hoogland.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6410
> 
> https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6410
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Integrated Wix tool set with maven build process so that no manual steps are 
> required to create windows msi installer file.
> 
> 
> Diffs
> -
> 
>   Wix-cloudstack-maven-plugin/pom.xml PRE-CREATION 
>   Wix-cloudstack-maven-plugin/src/org/apache/cloudstack/wix/HeatMojo.java 
> PRE-CREATION 
>   client/pom.xml d8dbde7 
> 
> Diff: https://reviews.apache.org/r/20366/diff/
> 
> 
> Testing
> ---
> 
> Build process has been Tested on Windows server.
> Regular profile Builds are tested 
> Tested new profile "buildw" and verified msi file is getting created in 
> target.
> 
> 
> Thanks,
> 
> Damodar Reddy Talakanti
> 
>



Build failed in Jenkins: build-systemvm64-4.2 #238

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

Fetching changes from the remote Git repository
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 config 
remote.origin.url https://git-wip-us.apache.org/repos/asf/cloudstack.git"; 
returned status code 255:
stdout: 
stderr: error: could not lock config file .git/config: Permission denied

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1086)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1063)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:900)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:910)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.setRemoteUrl(CliGitAPIImpl.java:644)
at hudson.plugins.git.GitAPI.setRemoteUrl(GitAPI.java:119)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.perform(RemoteInvocationHandler.java:299)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:280)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:239)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:326)
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 hudson.remoting.Engine$1$1.run(Engine.java:63)
at java.lang.Thread.run(Thread.java:679)


Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

2014-04-15 Thread Saurav Lahiri

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

(Updated April 15, 2014, 11:50 a.m.)


Review request for cloudstack, Jayapal Reddy, Rajani Karuturi, and Rajesh 
Battala.


Changes
---

Addressed review comments.


Repository: cloudstack-git


Description
---

With multiple java processes writing to the same logfile, each is not aware of 
the log4j's internal counter state, this needs to be prevented. So before 
starting new java process via the _run.sh , a check is made to ensure that 
there are no existing java processes running. This will prevent multiple java 
process writing to the same log file namely cloud.out. 


Diffs (updated)
-

  systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
  systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
  systemvm/scripts/run.sh 146d96f 
  systemvm/scripts/run.sh 146d96f 
  systemvm/scripts/utils.sh PRE-CREATION 
  systemvm/scripts/utils.sh PRE-CREATION 

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


Testing
---

Tested the changes with console proxy vm and secondary storage vm. They start 
and stop as expected.


Thanks,

Saurav Lahiri



Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

2014-04-15 Thread Saurav Lahiri


> On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote:
> > systemvm/scripts/utils.sh, line 9
> > 
> >
> > Where is the variable CLOUD_COM_HOME defined?

This is a typo. I have fixed this with the new patch file. 


> On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote:
> > systemvm/scripts/utils.sh, line 3
> > 
> >
> > Where is this variable used?

This is a typo. I have fixed this with the new patch file. 


> On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote:
> > systemvm/scripts/run.sh, line 30
> > 
> >
> > How do we achieve syncrhonization and avoid few race flows here? I mean 
> > by the time pid value is retrieved at 30, and went to line 32, there could 
> > be a pid available genuinely for process check we are doing? 
> > 
> > As well, if run.sh is called multiple times at same time, the value 
> > will not be synchronous, then there is no lock protection here?

Added locking code via the shells flock. Since run.sh is invoked only via 
/etc/init.d/cloud it perhaps is not a serious issue. But still to avoid 
parallel invocations of /etc/init.d/cloud start, locking code has been added.


> On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote:
> > systemvm/scripts/utils.sh, line 6
> > 
> >
> > This is not lock protected and can be run and called multiple times, 
> > may lead to unwarranted values.

The only two scripts which uses get_pids subroutines are the cloud script and 
the run.sh script and each of them have a local copy of the subroutine as they 
have sourced utils.sh. which contains the definition. Currently I think there 
might not be a need to lock protect this one. If you think otherwise, then a 
little bit more details would be very helpful.


- Saurav


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


On April 15, 2014, 11:50 a.m., Saurav Lahiri wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20123/
> ---
> 
> (Updated April 15, 2014, 11:50 a.m.)
> 
> 
> Review request for cloudstack, Jayapal Reddy, Rajani Karuturi, and Rajesh 
> Battala.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> With multiple java processes writing to the same logfile, each is not aware 
> of the log4j's internal counter state, this needs to be prevented. So before 
> starting new java process via the _run.sh , a check is made to ensure that 
> there are no existing java processes running. This will prevent multiple java 
> process writing to the same log file namely cloud.out. 
> 
> 
> Diffs
> -
> 
>   systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
>   systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
>   systemvm/scripts/run.sh 146d96f 
>   systemvm/scripts/run.sh 146d96f 
>   systemvm/scripts/utils.sh PRE-CREATION 
>   systemvm/scripts/utils.sh PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20123/diff/
> 
> 
> Testing
> ---
> 
> Tested the changes with console proxy vm and secondary storage vm. They start 
> and stop as expected.
> 
> 
> Thanks,
> 
> Saurav Lahiri
> 
>



Build failed in Jenkins: build-systemvm64-4.2 #239

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

Fetching changes from the remote Git repository
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 config 
remote.origin.url https://git-wip-us.apache.org/repos/asf/cloudstack.git"; 
returned status code 255:
stdout: 
stderr: error: could not lock config file .git/config: Permission denied

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1086)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1063)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:900)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:910)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.setRemoteUrl(CliGitAPIImpl.java:644)
at hudson.plugins.git.GitAPI.setRemoteUrl(GitAPI.java:119)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.perform(RemoteInvocationHandler.java:299)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:280)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:239)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:326)
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 hudson.remoting.Engine$1$1.run(Engine.java:63)
at java.lang.Thread.run(Thread.java:679)


Build failed in Jenkins: build-systemvm64-4.2 #240

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

Fetching changes from the remote Git repository
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 config 
remote.origin.url https://git-wip-us.apache.org/repos/asf/cloudstack.git"; 
returned status code 255:
stdout: 
stderr: error: could not lock config file .git/config: Permission denied

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1086)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1063)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:900)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:910)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.setRemoteUrl(CliGitAPIImpl.java:644)
at hudson.plugins.git.GitAPI.setRemoteUrl(GitAPI.java:119)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.perform(RemoteInvocationHandler.java:299)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:280)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:239)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:326)
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 hudson.remoting.Engine$1$1.run(Engine.java:63)
at java.lang.Thread.run(Thread.java:679)


Build failed in Jenkins: build-systemvm64-4.2 #241

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

Fetching changes from the remote Git repository
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 config 
remote.origin.url https://git-wip-us.apache.org/repos/asf/cloudstack.git"; 
returned status code 255:
stdout: 
stderr: error: could not lock config file .git/config: Permission denied

at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1086)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1063)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:900)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:910)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.setRemoteUrl(CliGitAPIImpl.java:644)
at hudson.plugins.git.GitAPI.setRemoteUrl(GitAPI.java:119)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.perform(RemoteInvocationHandler.java:299)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:280)
at 
hudson.remoting.RemoteInvocationHandler$RPCRequest.call(RemoteInvocationHandler.java:239)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:326)
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 hudson.remoting.Engine$1$1.run(Engine.java:63)
at java.lang.Thread.run(Thread.java:679)


Build failed in Jenkins: build-systemvm64-4.2 #242

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

Deleting old artifacts from #239
[copy-to-slave] Copying 
'debian-7.0.0-amd64-netinst.iso,VBoxGuestAdditions_4.2.6.iso', excluding 
nothing, from 'file:/var/lib/jenkins/userContent/' on the master to 
' on 
'devcloud-continuous-tests'.
FATAL: Pipe is already closed
java.io.IOException: Pipe is already closed
at hudson.remoting.PipeWindow.checkDeath(PipeWindow.java:108)
at hudson.remoting.PipeWindow$Real.get(PipeWindow.java:203)
at hudson.remoting.ProxyOutputStream._write(ProxyOutputStream.java:135)
at hudson.remoting.ProxyOutputStream.write(ProxyOutputStream.java:103)
at 
java.io.BufferedOutputStream.flushBuffer(BufferedOutputStream.java:82)
at java.io.BufferedOutputStream.write(BufferedOutputStream.java:126)
at 
com.jcraft.jzlib.DeflaterOutputStream.deflate(DeflaterOutputStream.java:144)
at 
com.jcraft.jzlib.DeflaterOutputStream.write(DeflaterOutputStream.java:102)
at java.io.BufferedOutputStream.write(BufferedOutputStream.java:122)
at org.apache.tools.tar.TarBuffer.writeBlock(TarBuffer.java:410)
at org.apache.tools.tar.TarBuffer.writeRecord(TarBuffer.java:351)
at 
hudson.org.apache.tools.tar.TarOutputStream.writeEOFRecord(TarOutputStream.java:356)
at 
hudson.org.apache.tools.tar.TarOutputStream.finish(TarOutputStream.java:137)
at 
hudson.org.apache.tools.tar.TarOutputStream.close(TarOutputStream.java:149)
at hudson.util.io.TarArchiver.close(TarArchiver.java:125)
at 
com.michelin.cio.hudson.plugins.copytoslave.MyFilePath.writeToTar(MyFilePath.java:230)
at 
com.michelin.cio.hudson.plugins.copytoslave.MyFilePath.copyRecursiveTo(MyFilePath.java:143)
at 
com.michelin.cio.hudson.plugins.copytoslave.CopyToSlaveBuildWrapper.setUp(CopyToSlaveBuildWrapper.java:133)
at hudson.model.Build$BuildExecution.doRun(Build.java:154)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:523)
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: java.io.IOException: Pipe is already closed
at 
hudson.remoting.FastPipedOutputStream.write(FastPipedOutputStream.java:147)
at 
hudson.remoting.FastPipedOutputStream.write(FastPipedOutputStream.java:131)
at 
hudson.remoting.ProxyOutputStream$Chunk$1.run(ProxyOutputStream.java:250)
at hudson.remoting.PipeWriter$1.run(PipeWriter.java:158)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
at java.util.concurrent.FutureTask.run(FutureTask.java:166)
at 
hudson.remoting.SingleLaneExecutorService$1.run(SingleLaneExecutorService.java:111)
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 hudson.remoting.Engine$1$1.run(Engine.java:63)
at java.lang.Thread.run(Thread.java:679)
Caused by: hudson.remoting.FastPipedInputStream$ClosedBy: The pipe was closed 
at...
at 
hudson.remoting.FastPipedInputStream.close(FastPipedInputStream.java:112)
at 
com.michelin.cio.hudson.plugins.copytoslave.MyFilePath$2.invoke(MyFilePath.java:138)
at 
com.michelin.cio.hudson.plugins.copytoslave.MyFilePath$2.invoke(MyFilePath.java:130)
at hudson.FilePath$FileCallableWrapper.call(FilePath.java:2462)
at hudson.remoting.UserRequest.perform(UserRequest.java:118)
at hudson.remoting.UserRequest.perform(UserRequest.java:48)
at hudson.remoting.Request$2.run(Request.java:326)
... 7 more


Re: Review Request 20123: Fix for cloudstack-6328 to Prevent console proxy support scripts from spawning multiple java processes

2014-04-15 Thread Saurav Lahiri

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

(Updated April 15, 2014, 12:13 p.m.)


Review request for cloudstack, Jayapal Reddy, Rajani Karuturi, and Rajesh 
Battala.


Changes
---

Looks like the earlier patch had a problem. Please use this one instead which 
addresses the review comments.


Repository: cloudstack-git


Description
---

With multiple java processes writing to the same logfile, each is not aware of 
the log4j's internal counter state, this needs to be prevented. So before 
starting new java process via the _run.sh , a check is made to ensure that 
there are no existing java processes running. This will prevent multiple java 
process writing to the same log file namely cloud.out. 


Diffs (updated)
-

  systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
  systemvm/scripts/run.sh 146d96f 
  systemvm/scripts/utils.sh PRE-CREATION 

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


Testing
---

Tested the changes with console proxy vm and secondary storage vm. They start 
and stop as expected.


Thanks,

Saurav Lahiri



Build failed in Jenkins: build-systemvm64-4.2 #243

2014-04-15 Thread jenkins
See 

--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace 

[build-systemvm64-4.2] $ /bin/sh -xe /tmp/hudson7016518448720689080.sh
+ whoami
prasanna
+ export 
PATH=/home/jenkins/.rvm/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
+ export rvm_path=/home/jenkins/.rvm
+ export HOME=/home/jenkins/
+ cd tools/appliance
+ [ -d iso ]
+ rm -fvr iso
rm: cannot remove `iso/VBoxGuestAdditions_4.2.6.iso': Permission denied
rm: cannot remove `iso/debian-7.0.0-i386-netinst.iso': Permission denied
rm: cannot remove `iso/debian-7.0.0-amd64-netinst.iso': Permission denied
Build step 'Execute shell' marked build as failure
Archiving artifacts
ERROR: Failed to archive artifacts: tools/appliance/dist/*
java.io.IOException: java.io.IOException: Failed to extract 
 of 4 files
at hudson.FilePath.readFromTar(FilePath.java:2089)
at hudson.FilePath.copyRecursiveTo(FilePath.java:2001)
at 
jenkins.model.StandardArtifactManager.archive(StandardArtifactManager.java:57)
at hudson.tasks.ArtifactArchiver.perform(ArtifactArchiver.java:140)
at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:742)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:714)
at hudson.model.Build$BuildExecution.post2(Build.java:183)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:663)
at hudson.model.Run.execute(Run.java:1713)
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: java.io.IOException: unexpected EOF with 2560 bytes unread
at 
hudson.org.apache.tools.tar.TarInputStream.read(TarInputStream.java:349)
at java.io.FilterInputStream.read(FilterInputStream.java:107)
at org.apache.commons.io.IOUtils.copyLarge(IOUtils.java:1792)
at org.apache.commons.io.IOUtils.copyLarge(IOUtils.java:1769)
at org.apache.commons.io.IOUtils.copy(IOUtils.java:1744)
at hudson.util.IOUtils.copy(IOUtils.java:40)
at hudson.FilePath.readFromTar(FilePath.java:2079)
... 12 more

at hudson.FilePath.copyRecursiveTo(FilePath.java:2008)
at 
jenkins.model.StandardArtifactManager.archive(StandardArtifactManager.java:57)
at hudson.tasks.ArtifactArchiver.perform(ArtifactArchiver.java:140)
at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:742)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:714)
at hudson.model.Build$BuildExecution.post2(Build.java:183)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:663)
at hudson.model.Run.execute(Run.java:1713)
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: java.util.concurrent.ExecutionException: 
java.io.FileNotFoundException: 

 (Permission denied)
at hudson.remoting.Channel$3.adapt(Channel.java:755)
at hudson.remoting.Channel$3.adapt(Channel.java:750)
at hudson.remoting.FutureAdapter.get(FutureAdapter.java:59)
at hudson.FilePath.copyRecursiveTo(FilePath.java:2004)
... 11 more
Caused by: java.io.FileNotFoundException: 

 (Permission denied)
at java.io.FileInputStream.open(Native Method)
at java.io.FileInputStream.(FileInputStream.java:137)
at hudson.util.io.TarArchiver.visit(TarArchiver.java:110)
at hudson.util.DirScanner.scanSingle(DirScanner.java:49)
at 
hudson.FilePath$ExplicitlySpecifiedDirScanner.scan(FilePath.java:2541)
at hudson.FilePath.writeToTar(FilePath.java:2053)
at hudson.FilePath.access$1000(FilePath.java:172)
at hudson.FilePath$41.invoke(FilePath.java:1994)
at hudson.FilePath$41.invoke(FilePath.java:1990)
at hudson.FilePath$FileCallableWrapper.call(FilePath.java:2462)
at hudson.remoting.UserRequest.perform(Use

Re: Review Request 18677: CLOUDSTACK-3272

2014-04-15 Thread Sonal Ojha

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

(Updated April 15, 2014, 12:37 p.m.)


Review request for cloudstack, Chiradeep Vittal and Murali Reddy.


Changes
---

Removed the earlier not working patch files and added a new patch file 
(0001-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch) which works 
on master with commit 55f53e13e. I could successfully apply the latest patch 
attached on master and test the alert/action/resource/usage events published on 
rabbitmq message bus. Earlier patch didnt work as it was built on the older 
version and had some changes which werent required. Kindly take a look and 
revert back.


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


Repository: cloudstack-git


Description
---

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


Diffs
-

  server/src/com/cloud/configuration/Config.java 2d55f46 
  server/src/com/cloud/event/ActionEventUtils.java 00122de 
  server/src/com/cloud/test/DatabaseConfig.java 9370218 

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


Testing
---

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


File Attachments (updated)


Working patch for master (latest git commit 55f53e13e)
  
https://reviews.apache.org/media/uploaded/files/2014/04/15/a6b65b2b-32ea-41d1-97e5-b9958d8b8841__0001-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch


Thanks,

Sonal Ojha



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

2014-04-15 Thread jenkins
See 



Re: [Responsiveness report] 2014w13

2014-04-15 Thread David Nalley
As long as yours is the first number in the rotation :)
On Apr 15, 2014 6:39 AM, "Sebastien Goasguen"  wrote:

> In my crazy head I sometimes wish that we would have a big red button on
> our website.
>
> It would be a twilio app that would forward the call 24/7 to members of
> our list willing to help out.
>
> A volunteer based 24/7 hot-line for cloudstack !
>
> -sebastien
>
> On Apr 15, 2014, at 6:34 AM, Daan Hoogland 
> wrote:
>
> > On Tue, Apr 15, 2014 at 11:11 AM, Noah Slater 
> wrote:
> >> No, I think this is great.
> >>
> >> Two improvements I can think of:
> >>
> >> 1. Automate it, so it's parsing the mbox files on people.a.o.
> > Sound good. The problem I have is that an answer to a question on user
> > to user-cn. Another one is that questions might have a follow-up but
> > this might be a quetion as well. I want to include those in the
> > report. Any sugestions that help automate the proces further are
> > welcome.
> >
> >>
> >> 2. Quantify it, and visualise the data.
> > sure, need to do this.
> >
> >>
> >> On 15 April 2014 11:00, Daan Hoogland  wrote:
> >>> I use google labels and thread display, the rest is mostly manual. I
> >>> check with markmail if the questions are really not answered. There is
> >>> a responsiveness report page on the wiki that explains my process and
> >>> that I hardly get any input on so please flame me.
> >>>
> >>> On Tue, Apr 15, 2014 at 10:56 AM, Noah Slater 
> wrote:
>  How are you compiling this?
> 
>  On 12 April 2014 14:32, Daan Hoogland 
> wrote:
> > Hello everybody,
> >
> > I want to bring under your attention that our responsiveness as a
> > community seems to deteriorate. I am busy with week 13 [1] and it is
> > the worst since I started doing this a year ago. I hope this is a
> > temporary failure on this part, maybe explained by the upcoming
> > cccna14 that has just come to pass. I do remember starting with this
> > because just before cccna13 it annoyed me that *6* messages had
> > remained unanswerred in the devlist. Week 13 of 2014 is a lot worse
> > and it looks like week 14 was as well.
> >
> > Let's leave no one behind!
> >
> > [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/2014W13
> >
> >
> > thanks
> > --
> > Daan
> 
> 
> 
>  --
>  Noah Slater
>  https://twitter.com/nslater
> >>>
> >>>
> >>>
> >>> --
> >>> Daan
> >>
> >>
> >>
> >> --
> >> Noah Slater
> >> https://twitter.com/nslater
> >
> >
> >
> > --
> > Daan
>
>


release note documentation

2014-04-15 Thread Pierre-Luc Dion
At the hackathon of CCCNA14 with Sebastien, we made few tests the RTD for
documentation versioning.

Look like we can use branch instead of tag for versioning and we can also
select which branches are published on RTD.org

So, in order to have doc version match product version, would it make sense
to create a branch call 4.3.0 for the current CS version and start updating
master for next version 4.4 ? once we will have 4.4 pretty much ready we
will be able to create new branch 4.4.

By using branches it allow us to update documentation without having to
update is version.

Make sense?  Have we missing a potential GIT behaviour  doing it this way?


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_


Re: Review Request 19912: [CLOUDSTACK-6322] Fixed Contrail: Params validation is missing while launching a service instance thru cloudmonkey

2014-04-15 Thread Daan Hoogland
Sachchidanand,

It helps to find a dev that has been to the code before and add them
to the reviewers list. I'll have a quick look to see if I can help.

Daan

On Mon, Apr 14, 2014 at 8:19 PM, Sachchidanand Vaidya
 wrote:
> Hi,
> Can someone pls review these changes.
>
> Thanks & Regards,
> Sachin
>
> On 4/2/14 2:14 AM, "Sachchidanand Vaidya"  wrote:
>
>>
>>---
>>This is an automatically generated e-mail. To reply, visit:
>>https://reviews.apache.org/r/19912/
>>---
>>
>>Review request for cloudstack.
>>
>>
>>Repository: cloudstack-git
>>
>>
>>Description
>>---
>>
>>Don't allow service instance creation with empty or null service-instance
>>"name"
>>
>>
>>Diffs
>>-
>>
>>
>>plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/networ
>>k/contrail/api/command/CreateServiceInstanceCmd.java d2cb4de
>>
>>Diff: https://reviews.apache.org/r/19912/diff/
>>
>>
>>Testing
>>---
>>
>>Unit tested for null and empty "name" parameter.
>>
>>
>>Thanks,
>>
>>Sachchidanand Vaidya
>>
>
>



-- 
Daan


Re: Review Request 19892: Add Juniper Contrail support to devCloud environment

2014-04-15 Thread daan Hoogland

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


please remove trailing spaces in the json definitions

- daan Hoogland


On April 1, 2014, 10:05 p.m., Sachchidanand Vaidya wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19892/
> ---
> 
> (Updated April 1, 2014, 10:05 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Add OpenContrail support to devCloud environment:
> - Created new devcloud advanced networking config file 
> (devcloud-advanced_juniper-contrail.cfg).
> - Modification to deployDataCenter script, to enable 
> "JuniperContrailRouter" &
>   "JuniperContrailVpcRouter" providers.
> 
> 
> Diffs
> -
> 
>   tools/devcloud/devcloud-advanced_juniper-contrail.cfg PRE-CREATION 
>   tools/marvin/marvin/deployDataCenter.py c4f6e1e 
> 
> Diff: https://reviews.apache.org/r/19892/diff/
> 
> 
> Testing
> ---
> 
> Unit tested with devCloud2 and ACS4.3 code base.
> 
> 
> Thanks,
> 
> Sachchidanand Vaidya
> 
>



Re: Review Request 20365: CLOUDSTACK-6411: some files are not loading in visual studio due to license header at top making them unparsable

2014-04-15 Thread daan Hoogland

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



plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config


Did you check if it worked with the license header below the xml header?


- daan Hoogland


On April 15, 2014, 8:44 a.m., Anshul Gangwar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20365/
> ---
> 
> (Updated April 15, 2014, 8:44 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Rajesh Battala.
> 
> 
> Bugs: CLOUDSTACK-6411
> https://issues.apache.org/jira/browse/CLOUDSTACK-6411
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
>  some files are not loading in visual studio due to license header at top 
> making them unparsable. removed the license header and put them in exclude 
> list of rat plugin
> 
> 
> Diffs
> -
> 
>   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 
> 5181cb8 
>   
> plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
>  fb5b962 
>   pom.xml 6837717 
> 
> Diff: https://reviews.apache.org/r/20365/diff/
> 
> 
> Testing
> ---
> 
> verified with rat plugin that the license check is passed successfully
> 
> 
> Thanks,
> 
> Anshul Gangwar
> 
>



Re: release note documentation

2014-04-15 Thread Daan Hoogland
makes sense! the only behavior that needs to be taken into account is
that of any publication scripts that we might write. So it seems to me
this is the best result we have from this version of the hackathon :(
& :)

On Tue, Apr 15, 2014 at 2:54 PM, Pierre-Luc Dion  wrote:
> At the hackathon of CCCNA14 with Sebastien, we made few tests the RTD for
> documentation versioning.
>
> Look like we can use branch instead of tag for versioning and we can also
> select which branches are published on RTD.org
>
> So, in order to have doc version match product version, would it make sense
> to create a branch call 4.3.0 for the current CS version and start updating
> master for next version 4.4 ? once we will have 4.4 pretty much ready we
> will be able to create new branch 4.4.
>
> By using branches it allow us to update documentation without having to
> update is version.
>
> Make sense?  Have we missing a potential GIT behaviour  doing it this way?
>
>
> 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_



-- 
Daan


Re: Review Request 19616: Added check for null return.

2014-04-15 Thread daan Hoogland

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



server/src/com/cloud/storage/VolumeApiServiceImpl.java


calling equals(null) on an object will never return true. It will throw a 
npe or return false. Do you mean to fix this?


- daan Hoogland


On April 15, 2014, 10:18 a.m., Alex Hitchins wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19616/
> ---
> 
> (Updated April 15, 2014, 10:18 a.m.)
> 
> 
> Review request for cloudstack and daan Hoogland.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Added check for returned null, if received then throw exception.
> 
> Amendments made as per dev mailing list comments.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeApiServiceImpl.java 680cd2e 
> 
> Diff: https://reviews.apache.org/r/19616/diff/
> 
> 
> Testing
> ---
> 
> Compiled & ran.
> 
> 
> Thanks,
> 
> Alex Hitchins
> 
>



Re: Review Request 19912: [CLOUDSTACK-6322] Fixed Contrail: Params validation is missing while launching a service instance thru cloudmonkey

2014-04-15 Thread daan Hoogland

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


It would seem that this is a behavioral change that would break backwards 
compatibility, is it? Right now empty service names are accepted. What is the 
consequence and why change the behavior, can you shed light on the subject?

otherwise the change seems trivial enough and is applicable.

- daan Hoogland


On April 2, 2014, 9:14 a.m., Sachchidanand Vaidya wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19912/
> ---
> 
> (Updated April 2, 2014, 9:14 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Don't allow service instance creation with empty or null service-instance 
> "name"
> 
> 
> Diffs
> -
> 
>   
> plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/api/command/CreateServiceInstanceCmd.java
>  d2cb4de 
> 
> Diff: https://reviews.apache.org/r/19912/diff/
> 
> 
> Testing
> ---
> 
> Unit tested for null and empty "name" parameter.
> 
> 
> Thanks,
> 
> Sachchidanand Vaidya
> 
>



Latest 4.2 template corrupted

2014-04-15 Thread Rayees Namathponnan
Hi All,

Latest 4.2 template built from jenkins.buildacloud.org , is corrupted.

Even though Jenkins job 
http://jenkins.buildacloud.org/view/4.2/job/build-systemvm-4.2 complete 
successfully,   template is not useful;  I cannot see any files under 
/opt/cloud/bin and SSVM and CPVM never getting programmed.

Look like there are disk space issue with template build machine, anyone having 
access on build machines please help on this ?

There is a defect for this already 
https://issues.apache.org/jira/browse/CLOUDSTACK-6416


Regards,
Rayees





Resource counts

2014-04-15 Thread Gaurav Aradhye
Hi all,

The resource counts for primary and secondary storage listed in GBs,  and I
haven't seen these in decimals. They are always in int.

What happens if suppose I register an ISO of smaller size (less than 1 GB)?
Shouldn't it show the resource count in decimals?

I am currently seeing the secondary storage resource count as 0 for an ISO
I registered. And for test case purposes I believe smaller ISOs are better
to use because they will have less overhead on execution and on management
server.

Regards,
Gaurav


Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread daan Hoogland

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



server/src/com/cloud/api/ApiDBUtils.java


I don't fully understand the logic of querying .next() and then .previous()

It seems like guessing or is there some hard restrained that makes this the 
way to determine this is KVM.

Is this really the place to decide on hypervisor type?


- daan Hoogland


On April 15, 2014, 3:43 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 15, 2014, 3:43 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: release note documentation

2014-04-15 Thread sebgoa
To add to what Pierre-Luc said:

Readthedocs has something they call "releases" but those are in fact builds 
that point to a branch. Not a specific tag.

So the release version of the doc we would see on the website will be the live 
state of the release branch, not a tag that we could vote on.

That said, we have not yet discussed whether or not we need to formally vote on 
doc releases :)

-sebastien


On Apr 15, 2014, at 3:56 PM, Daan Hoogland  wrote:

> makes sense! the only behavior that needs to be taken into account is
> that of any publication scripts that we might write. So it seems to me
> this is the best result we have from this version of the hackathon :(
> & :)
> 
> On Tue, Apr 15, 2014 at 2:54 PM, Pierre-Luc Dion  wrote:
>> At the hackathon of CCCNA14 with Sebastien, we made few tests the RTD for
>> documentation versioning.
>> 
>> Look like we can use branch instead of tag for versioning and we can also
>> select which branches are published on RTD.org
>> 
>> So, in order to have doc version match product version, would it make sense
>> to create a branch call 4.3.0 for the current CS version and start updating
>> master for next version 4.4 ? once we will have 4.4 pretty much ready we
>> will be able to create new branch 4.4.
>> 
>> By using branches it allow us to update documentation without having to
>> update is version.
>> 
>> Make sense?  Have we missing a potential GIT behaviour  doing it this way?
>> 
>> 
>> 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_
> 
> 
> 
> -- 
> Daan



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey


> On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
> > server/src/com/cloud/api/ApiDBUtils.java, line 1088
> > 
> >
> > I don't fully understand the logic of querying .next() and then 
> > .previous()
> > 
> > It seems like guessing or is there some hard restrained that makes this 
> > the way to determine this is KVM.
> > 
> > Is this really the place to decide on hypervisor type?

I had nearly finished a somewhat wordy reply when my tablet decided to reboot 
itself, so please forgive me if I seem brief in reply this time around.

1) From what I understand of Iterators, using .next() and .previous() in 
succession allow iterating over the same element twice. If this is not so, it's 
an easy enough fix.

2&3) This is a guess, yes. There is a similar guess above for deciding whether 
VHD should be associated with HyperV or Xenserver because of a similar 
situation (2 hypervisors, 1 format vs. 1 hypervisor, two formats) so this 
seemed like a logical place to put a similar fix. As illustrated in CS-6396, 
for the function getHypervisorTypeFromFormat() there is only one possible 
return for each format, which is not true for KVM as it can support RAW images 
(in the case of RBD or CLVM) as well as qcow2 images. The only thing I can 
think of that would remedy this is implementing a way to return multiple 
supported types of formats for hypervisors OR a separate format for RBD raw 
images & CLVM raw images, but that's a fair bit more legwork than I feel 
comfortable doing being as I am relatively new to the codebase.

Another way to patch this that I thought of while writing this reply would be 
to check the datacenter for KVM and OVM clusters, and for example the presence 
of OVM clusters but not KVM clusters would indicate that RAW images are for 
OVM, whereas in the presence of KVM clusters without OVM cluststers RAW images 
could be assigned to KVM for RBD / CLVM.

Let me know what you think, I hope I answered all your questions.


- Tanner


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


On April 15, 2014, 3:43 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 15, 2014, 3:43 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey


> On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
> > server/src/com/cloud/api/ApiDBUtils.java, line 1088
> > 
> >
> > I don't fully understand the logic of querying .next() and then 
> > .previous()
> > 
> > It seems like guessing or is there some hard restrained that makes this 
> > the way to determine this is KVM.
> > 
> > Is this really the place to decide on hypervisor type?
> 
> Tanner Danzey wrote:
> I had nearly finished a somewhat wordy reply when my tablet decided to 
> reboot itself, so please forgive me if I seem brief in reply this time around.
> 
> 1) From what I understand of Iterators, using .next() and .previous() in 
> succession allow iterating over the same element twice. If this is not so, 
> it's an easy enough fix.
> 
> 2&3) This is a guess, yes. There is a similar guess above for deciding 
> whether VHD should be associated with HyperV or Xenserver because of a 
> similar situation (2 hypervisors, 1 format vs. 1 hypervisor, two formats) so 
> this seemed like a logical place to put a similar fix. As illustrated in 
> CS-6396, for the function getHypervisorTypeFromFormat() there is only one 
> possible return for each format, which is not true for KVM as it can support 
> RAW images (in the case of RBD or CLVM) as well as qcow2 images. The only 
> thing I can think of that would remedy this is implementing a way to return 
> multiple supported types of formats for hypervisors OR a separate format for 
> RBD raw images & CLVM raw images, but that's a fair bit more legwork than I 
> feel comfortable doing being as I am relatively new to the codebase.
> 
> Another way to patch this that I thought of while writing this reply 
> would be to check the datacenter for KVM and OVM clusters, and for example 
> the presence of OVM clusters but not KVM clusters would indicate that RAW 
> images are for OVM, whereas in the presence of KVM clusters without OVM 
> cluststers RAW images could be assigned to KVM for RBD / CLVM.
> 
> Let me know what you think, I hope I answered all your questions.

I just did some additional reading and there is a better way to do that loop, I 
will have to wait until I'm at my home computer to submit a revision.


- Tanner


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


On April 15, 2014, 3:43 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 15, 2014, 3:43 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread sebgoa
I think it's better if we make one patch per bug..



On Apr 15, 2014, at 5:59 PM, Tanner Danzey  wrote:

> 
> 
>> On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
>>> server/src/com/cloud/api/ApiDBUtils.java, line 1088
>>> 
>>> 
>>>I don't fully understand the logic of querying .next() and then 
>>> .previous()
>>> 
>>>It seems like guessing or is there some hard restrained that makes this 
>>> the way to determine this is KVM.
>>> 
>>>Is this really the place to decide on hypervisor type?
>> 
>> Tanner Danzey wrote:
>>I had nearly finished a somewhat wordy reply when my tablet decided to 
>> reboot itself, so please forgive me if I seem brief in reply this time 
>> around.
>> 
>>1) From what I understand of Iterators, using .next() and .previous() in 
>> succession allow iterating over the same element twice. If this is not so, 
>> it's an easy enough fix.
>> 
>>2&3) This is a guess, yes. There is a similar guess above for deciding 
>> whether VHD should be associated with HyperV or Xenserver because of a 
>> similar situation (2 hypervisors, 1 format vs. 1 hypervisor, two formats) so 
>> this seemed like a logical place to put a similar fix. As illustrated in 
>> CS-6396, for the function getHypervisorTypeFromFormat() there is only one 
>> possible return for each format, which is not true for KVM as it can support 
>> RAW images (in the case of RBD or CLVM) as well as qcow2 images. The only 
>> thing I can think of that would remedy this is implementing a way to return 
>> multiple supported types of formats for hypervisors OR a separate format for 
>> RBD raw images & CLVM raw images, but that's a fair bit more legwork than I 
>> feel comfortable doing being as I am relatively new to the codebase.
>> 
>>Another way to patch this that I thought of while writing this reply 
>> would be to check the datacenter for KVM and OVM clusters, and for example 
>> the presence of OVM clusters but not KVM clusters would indicate that RAW 
>> images are for OVM, whereas in the presence of KVM clusters without OVM 
>> cluststers RAW images could be assigned to KVM for RBD / CLVM.
>> 
>>Let me know what you think, I hope I answered all your questions.
> 
> I just did some additional reading and there is a better way to do that loop, 
> I will have to wait until I'm at my home computer to submit a revision.
> 
> 
> - Tanner
> 
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/#review40394
> ---
> 
> 
> On April 15, 2014, 3:43 a.m., Tanner Danzey wrote:
>> 
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/20357/
>> ---
>> 
>> (Updated April 15, 2014, 3:43 a.m.)
>> 
>> 
>> Review request for cloudstack.
>> 
>> 
>> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
>>https://issues.apache.org/jira/browse/CLOUDSTACK-5907
>>https://issues.apache.org/jira/browse/CLOUDSTACK-6396
>> 
>> 
>> Repository: cloudstack-git
>> 
>> 
>> Description
>> ---
>> 
>> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
>> OVM volumes (not able to snapshot)
>> 
>> 
>> Diffs
>> -
>> 
>>  server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
>> 
>> Diff: https://reviews.apache.org/r/20357/diff/
>> 
>> 
>> Testing
>> ---
>> 
>> Applied on otherwise clean 4.4 branch and tested in a live testing 
>> environment with KVM hypervisors and RBD primary storage pool that would 
>> otherwise identify as OVM. No errors and no weird behavior. Just the 
>> expected result.
>> 
>> 
>> Thanks,
>> 
>> Tanner Danzey
>> 
>> 
> 



Re: NodeJS Client for CloudStack

2014-04-15 Thread Madan Ganesh V
Thank Chip,

Yes,  I was referring to those two modules in my email.

csclient appears to be broken while cloudstack module looks ok.

Cheers,
Madan


On 14 April 2014 at 10:49:08 pm, Chip Childers (chipchild...@apache.org) wrote:

https://nodejsmodules.org/tags/cloudstack  

I have not used them myself, so YMMV.  

On Mon, Apr 14, 2014 at 10:42:02PM +0530, Madan Ganesh V wrote:  
> Thanks Sebastien for the reply,  
>  
> We could get started with node-cloudstack. So far so good.  
>  
> We will roll out our cloud analytics beta next week. Hope the nodeJS works 
> like a charm with ACS.  
>  
> Cheers,  
> Madan  
> www.actonmagic.com  
>  
> On 14 April 2014 at 9:28:50 pm, Sebastien Goasguen (run...@gmail.com) wrote:  
>  
>  
> On Apr 11, 2014, at 6:46 AM, Madan Ganesh V  wrote:  
>  
> > Hello there,  
> >  
> > I’m writing nodeJS client application to interact with CloudStack.  
> >  
> > I notice csclient and node-cloudstack modules. csclient throws API error 
> > 432 for listVirtualMachines.  
> >  
> > Anybody has played with these modules in Node JS.  
>  
> Unfortunately I have not. Would love to hear about it though, when you get it 
> working :)  
>  
> >  
> > Appreciate your pointers/ recommendations.  
> >  
> > Cheers,  
> > Madan  
> >  
> >  
>  


Re: release note documentation

2014-04-15 Thread David Nalley
So I think it makes sense for most documents to point to feature
version (e.g. 4.3 branch, 4.4 branch.) E.g. docs for 4.3.1 should be
materially the same as 4.3.0 from a docs standpoint. Release notes are
the exception here, though perhaps they could be dealt with in an
additive way.

--David

On Tue, Apr 15, 2014 at 11:17 AM, sebgoa  wrote:
> To add to what Pierre-Luc said:
>
> Readthedocs has something they call "releases" but those are in fact builds 
> that point to a branch. Not a specific tag.
>
> So the release version of the doc we would see on the website will be the 
> live state of the release branch, not a tag that we could vote on.
>
> That said, we have not yet discussed whether or not we need to formally vote 
> on doc releases :)
>
> -sebastien
>
>
> On Apr 15, 2014, at 3:56 PM, Daan Hoogland  wrote:
>
>> makes sense! the only behavior that needs to be taken into account is
>> that of any publication scripts that we might write. So it seems to me
>> this is the best result we have from this version of the hackathon :(
>> & :)
>>
>> On Tue, Apr 15, 2014 at 2:54 PM, Pierre-Luc Dion  wrote:
>>> At the hackathon of CCCNA14 with Sebastien, we made few tests the RTD for
>>> documentation versioning.
>>>
>>> Look like we can use branch instead of tag for versioning and we can also
>>> select which branches are published on RTD.org
>>>
>>> So, in order to have doc version match product version, would it make sense
>>> to create a branch call 4.3.0 for the current CS version and start updating
>>> master for next version 4.4 ? once we will have 4.4 pretty much ready we
>>> will be able to create new branch 4.4.
>>>
>>> By using branches it allow us to update documentation without having to
>>> update is version.
>>>
>>> Make sense?  Have we missing a potential GIT behaviour  doing it this way?
>>>
>>>
>>> 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_
>>
>>
>>
>> --
>> Daan
>


Re: Network state machine

2014-04-15 Thread Alena Prokharchyk
Done, thank you, Murali

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


On 4/15/14, 2:13 AM, "Murali Reddy"  wrote:

>On 15/04/14 3:08 AM, "Alena Prokharchyk" 
>wrote:
>
>>Murali,
>>
>>I have a question to you. In NetworkOrchestrator, implementNetwork code,
>>when we update the state for the network, why don¹t we use state machine
>>for Shared networks, but instead update it directly in the DB?
>>
>>
>>if (isSharedNetworkWithServices(network)) {
>>network.setState(Network.State.Implementing);
>>} else {
>>stateTransitTo(network, Event.ImplementNetwork);
>>}
>>
>>
>>That doesn¹t seem right to me as there should be no exception when it
>>comes to state update - the state machine should always be used once its
>>implemented for the object. If your code needs additional state
>>transitions for the Setup state, why can¹t you add it?
>
>Initially network just had states and states were set explicitly. Actually
>I happened to add state changes to explicitly happen through the state
>machine for resource state change events for EventBus. But the problem I
>had with
>shared networks was, it had different life cycle (before cloudstack
>supported L4-L7 services in shared networks) and shared network with L4-L7
>service has life cycle similar to that of isolated network. For the
>network in 'Setup' state, ImplementNetwork event shall succeed only for
>shared network with services but should fail for shared network without
>L4-L7 services. Similarly an implemented network in 'Implemented' state on
>'ShutdownNetwork' event can go to 'Allocated' or 'Setup' state depending
>on the network type (isolate/shared). I needed some sort of conditional
>state transition in the state machine which is not there.
>
>Due to time constraint I added minimal disruptive changes. Doing it
>cleaner way means revamping the network states and transitions to avoid
>conflicting transitions. Please do open a bug to fix it.
>
>
>>
>>Thanks,
>>Alena.
>>
>>
>
>



Re: Review Request 19584: CLOUDSTACK-6258: Disable systemvm cloud startup script from logging messages to /var/log/cloud/cloud.out

2014-04-15 Thread Rajesh Battala

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

Ship it!


Ship It!

- Rajesh Battala


On April 14, 2014, 7:10 a.m., Saurav Lahiri wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19584/
> ---
> 
> (Updated April 14, 2014, 7:10 a.m.)
> 
> 
> Review request for cloudstack and Jayapal Reddy.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This will prevent the /etc/init.d/cloud script from logging messages to 
> /var/log/cloud/cloud.out if the CLOUD_DEBUG flag is not defined. If the flag 
> is defined only then will messages be logged. This is because the cloud.out 
> file gets rolled over once max size is reached via the log4j settings, since 
> the script is not aware of the log4j settings it causes the log file to 
> exceed its max size and fill up the file system. 
> 
> 
> Diffs
> -
> 
>   systemvm/patches/debian/config/etc/init.d/cloud 83853bc 
> 
> Diff: https://reviews.apache.org/r/19584/diff/
> 
> 
> Testing
> ---
> 
> The console proxy vm was started and console sessions tested. The secondary 
> storage vm was succesfully started.
> 
> 
> Thanks,
> 
> Saurav Lahiri
> 
>



Re: [4.4]copy_vhd_from_secondarystorage is timing out

2014-04-15 Thread Murali Reddy

I am still running into this issue. Alex reverted complete change
replacing xapi.jar, which was also resulting XmlRpc exceptions but system
VM's were launching fine after revert. I may be wrong but looking at the
stack trace and recent commits, commit
9f44909e63a19b2b8034baa13b92e24ea2eb0227 (use event instead of poll for
xapi async call in XS 6.2 and above to reduce the pressure on XAPI) may be
causing this issue. Anthony can you please take a look?


On 14/04/14 8:12 AM, "Mike Tutkowski"  wrote:

>I was seeing a similar issue when XAPI was changed when I tried to create
>an SR.
>
>I wonder if that XAPI issue hasn't been fully resolved.
>
>
>On Sun, Apr 13, 2014 at 4:16 PM, Srikanteswararao Talluri <
>srikanteswararao.tall...@citrix.com> wrote:
>
>> Did anybody see this on latest 4.4?
>>
>>
>> 2014-04-14 00:01:34,305 DEBUG [c.c.h.x.r.XenServerConnectionPool]
>> (DirectAgent-71:ctx-12d9be59) XmlRpcException for method: event.from
>>due to
>> org.apache.xmlrpc.XmlRpcException: Failed to create input stream: Read
>> timed out
>>
>> 2014-04-14 00:01:34,305 WARN  [c.c.h.x.r.CitrixResourceBase]
>> (DirectAgent-71:ctx-12d9be59) callHostPlugin failed for cmd:
>> copy_vhd_from_secondarystorage with args mountpoint:
>>10.147.28.7:/export/home/talluri/psec/template/tmpl/1/1/,
>> sruuid: b3dd21aa-8e82-a923-ed29-ee58220d158b, namelabel:
>> cloud-8498d88f-be6b-41b6-bece-a58bbc1fbc32,  due to Failed to create
>>input
>> stream: Read timed out
>>
>> org.apache.xmlrpc.XmlRpcException: Failed to create input stream: Read
>> timed out
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.getInputStream(XmlRpcSunH
>>ttpTransport.java:99)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcStreamTransport.sendRequest(XmlRpcStreamTr
>>ansport.java:152)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcHttpTransport.sendRequest(XmlRpcHttpTransp
>>ort.java:143)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.sendRequest(XmlRpcSunHttp
>>Transport.java:69)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcClientWorker.execute(XmlRpcClientWorker.ja
>>va:56)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:167)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:137)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:126)
>>
>> at com.xensource.xenapi.Connection.dispatch(Connection.java:285)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerConnectionPool$XenServerConnec
>>tion.dispatch(XenServerConnectionPool.java:456)
>>
>> at com.xensource.xenapi.Event.properFrom(Event.java:310)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer620Resource.waitForTask(XenSer
>>ver620Resource.java:125)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.CitrixResourceBase.callHostPluginAsync(
>>CitrixResourceBase.java:3558)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copy_vhd_from
>>_secondarystorage(XenServerStorageProcessor.java:826)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copyTemplateT
>>oPrimaryStorage(XenServerStorageProcessor.java:962)
>>
>> at
>> 
>>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(Sto
>>rageSubsystemCommandHandlerBase.java:77)
>>
>> at
>> 
>>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleStora
>>geCommands(StorageSubsystemCommandHandlerBase.java:52)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.CitrixResourceBase.executeRequest(Citri
>>xResourceBase.java:542)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer56Resource.executeRequest(XenS
>>erver56Resource.java:60)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer610Resource.executeRequest(Xen
>>Server610Resource.java:92)
>>
>> at
>> 
>>com.cloud.agent.manager.DirectAgentAttache$Task.runInContext(DirectAgentA
>>ttache.java:216)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(Manage
>>dContextRunnable.java:49)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(D
>>efaultManagedContext.java:56)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWith
>>Context(DefaultManagedContext.java:103)
>>
>>
>>   547,2-9
>>
>
>
>
>-- 
>*Mike Tutkowski*
>*Senior CloudStack Developer, SolidFire Inc.*
>e: mike.tutkow...@solidfire.com
>o: 303.746.7302
>Advancing the way the world uses the
>cloud
>*(tm)*
>




RE: unable to create a instance in 4.3 using new devcloud and solution

2014-04-15 Thread Rajesh Battala
Punit,  
There is a change in the template.
Download it from http://jenkins.buildacloud.org/job/build-systemvm64-master/
You can replace the base system vm template in devcloud and use it.

Thanks
Rajesh Battala

-Original Message-
From: Punith S [mailto:punit...@cloudbyte.com] 
Sent: Tuesday, April 15, 2014 11:59 AM
To: cloudstack
Subject: unable to create a instance in 4.3 using new devcloud and solution

hi,

i'm using new devcloud for 4.3, and configured the basic zone using devcloud 
python scripts.
when we launch a the downloaded tiny linux template, it throws the following 
exception :

Router requires upgrade. Unable to send command to router Unable to send 
command. Upgrade in progress. Please contact administrator.
and finally insufficient server capacity exception.

and then found that the virtual router needed an upgrade from current 4.2 to 
4.3 version, so initiated the upgrade for virtual router, *the UI shows the 
upgrade was completed successfully but the version still remains the 
same(4.2)*, hence it fails again to create a instance again. and i also checked 
the /opt/storage/secondary template in devcloud, it remains the new vdh file.

my workaround solution :
 just make *false* for router.check.version in global settings and restart the 
MS.

any workaround for the upgrade for router from 4.2 to 4.3 in devcloud , is 
there any new template we need to seed in secondary storage ??

--
regards,

punith s
cloudbyte.com


RE: Review Request 19616: Added check for null return.

2014-04-15 Thread Alex Hitchins
Sorry - I picked that up but must have been after I'd done the patch. Will 
amend _again_!



Alex Hitchins | 07788 423 969 | 01892 523 587
-

-Original Message-
From: daan Hoogland [mailto:nore...@reviews.apache.org] On Behalf Of daan 
Hoogland
Sent: 15 April 2014 15:07
To: daan Hoogland
Cc: Alex Hitchins; cloudstack
Subject: Re: Review Request 19616: Added check for null return.


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



server/src/com/cloud/storage/VolumeApiServiceImpl.java


calling equals(null) on an object will never return true. It will throw a 
npe or return false. Do you mean to fix this?


- daan Hoogland


On April 15, 2014, 10:18 a.m., Alex Hitchins wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19616/
> ---
> 
> (Updated April 15, 2014, 10:18 a.m.)
> 
> 
> Review request for cloudstack and daan Hoogland.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Added check for returned null, if received then throw exception.
> 
> Amendments made as per dev mailing list comments.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/storage/VolumeApiServiceImpl.java 680cd2e 
> 
> Diff: https://reviews.apache.org/r/19616/diff/
> 
> 
> Testing
> ---
> 
> Compiled & ran.
> 
> 
> Thanks,
> 
> Alex Hitchins
> 
>




RE: [4.4]copy_vhd_from_secondarystorage is timing out

2014-04-15 Thread Anthony Xu
Hi Murali,

It is caused by the commit, I tested the commit on XS 6.2 FOX, which doesn't 
use copy_vhd_from_secondarystorage, that's why I didn't see this issue.

Just disabled XAPI Event in 4.4 branch.


Thanks,
Anthony






-Original Message-
From: Murali Reddy 
Sent: Tuesday, April 15, 2014 10:54 AM
To: dev@cloudstack.apache.org; Anthony Xu
Subject: Re: [4.4]copy_vhd_from_secondarystorage is timing out


I am still running into this issue. Alex reverted complete change replacing 
xapi.jar, which was also resulting XmlRpc exceptions but system VM's were 
launching fine after revert. I may be wrong but looking at the stack trace and 
recent commits, commit
9f44909e63a19b2b8034baa13b92e24ea2eb0227 (use event instead of poll for xapi 
async call in XS 6.2 and above to reduce the pressure on XAPI) may be causing 
this issue. Anthony can you please take a look?


On 14/04/14 8:12 AM, "Mike Tutkowski"  wrote:

>I was seeing a similar issue when XAPI was changed when I tried to 
>create an SR.
>
>I wonder if that XAPI issue hasn't been fully resolved.
>
>
>On Sun, Apr 13, 2014 at 4:16 PM, Srikanteswararao Talluri < 
>srikanteswararao.tall...@citrix.com> wrote:
>
>> Did anybody see this on latest 4.4?
>>
>>
>> 2014-04-14 00:01:34,305 DEBUG [c.c.h.x.r.XenServerConnectionPool]
>> (DirectAgent-71:ctx-12d9be59) XmlRpcException for method: event.from 
>>due to
>> org.apache.xmlrpc.XmlRpcException: Failed to create input stream: 
>>Read  timed out
>>
>> 2014-04-14 00:01:34,305 WARN  [c.c.h.x.r.CitrixResourceBase]
>> (DirectAgent-71:ctx-12d9be59) callHostPlugin failed for cmd:
>> copy_vhd_from_secondarystorage with args mountpoint:
>>10.147.28.7:/export/home/talluri/psec/template/tmpl/1/1/,
>> sruuid: b3dd21aa-8e82-a923-ed29-ee58220d158b, namelabel:
>> cloud-8498d88f-be6b-41b6-bece-a58bbc1fbc32,  due to Failed to create 
>>input
>> stream: Read timed out
>>
>> org.apache.xmlrpc.XmlRpcException: Failed to create input stream: 
>> Read timed out
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.getInputStream(XmlRpcS
>>unH
>>ttpTransport.java:99)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcStreamTransport.sendRequest(XmlRpcStrea
>>mTr
>>ansport.java:152)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcHttpTransport.sendRequest(XmlRpcHttpTra
>>nsp
>>ort.java:143)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.sendRequest(XmlRpcSunH
>>ttp
>>Transport.java:69)
>>
>> at
>> 
>>org.apache.xmlrpc.client.XmlRpcClientWorker.execute(XmlRpcClientWorker
>>.ja
>>va:56)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:167)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:137)
>>
>> at
>> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:126)
>>
>> at 
>> com.xensource.xenapi.Connection.dispatch(Connection.java:285)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerConnectionPool$XenServerCon
>>nec
>>tion.dispatch(XenServerConnectionPool.java:456)
>>
>> at com.xensource.xenapi.Event.properFrom(Event.java:310)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer620Resource.waitForTask(Xen
>>Ser
>>ver620Resource.java:125)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.CitrixResourceBase.callHostPluginAsy
>>nc(
>>CitrixResourceBase.java:3558)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copy_vhd_f
>>rom
>>_secondarystorage(XenServerStorageProcessor.java:826)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copyTempla
>>teT
>>oPrimaryStorage(XenServerStorageProcessor.java:962)
>>
>> at
>> 
>>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(
>>Sto
>>rageSubsystemCommandHandlerBase.java:77)
>>
>> at
>> 
>>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleSt
>>ora
>>geCommands(StorageSubsystemCommandHandlerBase.java:52)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.CitrixResourceBase.executeRequest(Ci
>>tri
>>xResourceBase.java:542)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer56Resource.executeRequest(X
>>enS
>>erver56Resource.java:60)
>>
>> at
>> 
>>com.cloud.hypervisor.xen.resource.XenServer610Resource.executeRequest(
>>Xen
>>Server610Resource.java:92)
>>
>> at
>> 
>>com.cloud.agent.manager.DirectAgentAttache$Task.runInContext(DirectAge
>>ntA
>>ttache.java:216)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(Man
>>age
>>dContextRunnable.java:49)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.cal
>>l(D
>>efaultManagedContext.java:56)
>>
>> at
>> 
>>org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callW
>>ith
>>Context(DefaultManagedContext.java:103)
>>
>>
>>   547,2-9
>>
>
>
>
>--
>*Mike Tutkowski*
>*Seni

Devcloud2 errors on 4.4

2014-04-15 Thread David Grizzanti
All,

I am facing an issue when trying to get devcloud2 running with latest code from 
4.4 branch. There was a similar email to this a few weeks back on master, but I 
did not see any resolution.

I'm running everything in side of devcloud2 (mgmt server, mysql, xen).

Seeing the following errors when trying to deploy an initial zone.  Don't see 
any issues with creating the /opt/cloud/bin directory inside the VM.

WARN  [c.c.a.m.DirectAgentAttache] (DirectAgent-37:ctx-0d57a0bf) Seq 
3-8906994163031998465: Exception Caught while executing command
com.cloud.utils.exception.CloudRuntimeException: Cannot create directory 
/opt/cloud/bin on XenServer hosts
at 
com.cloud.hypervisor.xen.resource.CitrixResourceBase.setupServer(CitrixResourceBase.java:4842)
at 
com.cloud.hypervisor.xen.resource.CitrixResourceBase.execute(CitrixResourceBase.java:4668)
at 
com.cloud.hypervisor.xen.resource.CitrixResourceBase.executeRequest(CitrixResourceBase.java:489)
at 
com.cloud.hypervisor.xen.resource.XcpOssResource.executeRequest(XcpOssResource.java:176)
at 
com.cloud.agent.manager.DirectAgentAttache$Task.runInContext(DirectAgentAttache.java:216)
at 
org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(ManagedContextRunnable.java:49)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(DefaultManagedContext.java:56)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWithContext(DefaultManagedContext.java:103)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext.runWithContext(DefaultManagedContext.java:53)
at 
org.apache.cloudstack.managed.context.ManagedContextRunnable.run(ManagedContextRunnable.java:46)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
at java.util.concurrent.FutureTask.run(FutureTask.java:166)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:724)
WARN  [c.c.h.x.d.XcpServerDiscoverer] (AgentTaskPool-13:ctx-5015bd03) Unable to 
setup agent 3 due to com.cloud.utils.exception.CloudRuntimeException: Cannot 
create directory /opt/cloud/bin on XenServer hosts
INFO  [c.c.u.e.CSExceptionErrorCode] (AgentTaskPool-13:ctx-5015bd03) Could not 
find exception: com.cloud.exception.ConnectionException in error code list for 
exceptions
WARN  [c.c.a.m.AgentManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Monitor 
XcpServerDiscoverer says there is an error in the connect process for 3 due to 
Reinitialize agent after setup.
INFO  [c.c.a.m.AgentManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Host 3 is 
disconnecting with event AgentDisconnected
WARN  [c.c.r.ResourceManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Unable to 
connect due to
com.cloud.exception.ConnectionException: Reinitialize agent after setup.
at 
com.cloud.hypervisor.xen.discoverer.XcpServerDiscoverer.processConnect(XcpServerDiscoverer.java:658)
at 
com.cloud.agent.manager.AgentManagerImpl.notifyMonitorsOfConnection(AgentManagerImpl.java:514)
at 
com.cloud.agent.manager.AgentManagerImpl.handleDirectConnectAgent(AgentManagerImpl.java:1428)
at 
com.cloud.resource.ResourceManagerImpl.createHostAndAgent(ResourceManagerImpl.java:1767)
at 
com.cloud.resource.ResourceManagerImpl.createHostAndAgent(ResourceManagerImpl.java:1893)
at sun.reflect.GeneratedMethodAccessor226.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.sun.proxy.$Proxy147.createHostAndAgent(Unknown Source)
at 
com.cloud.agent.manager.AgentManagerImpl$SimulateStartTask.runInContext(AgentManagerImpl.java:1070)
at 
org.apache.cloudstack.manage

Re: [4.4]copy_vhd_from_secondarystorage is timing out

2014-04-15 Thread Mike Tutkowski
Hi Anthony,

Can you go into a bit more depth about copy_vhd_from_secondarystorage not
being used with XenServer 6.2?

Does that mean "public Answer copyTemplateToPrimaryStorage(CopyCommand
cmd)" in XenServerStorageProcessor is not called when XenServer 6.2 is in
use?

I'm testing out a feature of mine for 4.4, but have only been using
XenServer 6.1 so far.

Thanks!
Mike


On Tue, Apr 15, 2014 at 12:44 PM, Anthony Xu  wrote:

> Hi Murali,
>
> It is caused by the commit, I tested the commit on XS 6.2 FOX, which
> doesn't use copy_vhd_from_secondarystorage, that's why I didn't see this
> issue.
>
> Just disabled XAPI Event in 4.4 branch.
>
>
> Thanks,
> Anthony
>
>
>
>
>
>
> -Original Message-
> From: Murali Reddy
> Sent: Tuesday, April 15, 2014 10:54 AM
> To: dev@cloudstack.apache.org; Anthony Xu
> Subject: Re: [4.4]copy_vhd_from_secondarystorage is timing out
>
>
> I am still running into this issue. Alex reverted complete change
> replacing xapi.jar, which was also resulting XmlRpc exceptions but system
> VM's were launching fine after revert. I may be wrong but looking at the
> stack trace and recent commits, commit
> 9f44909e63a19b2b8034baa13b92e24ea2eb0227 (use event instead of poll for
> xapi async call in XS 6.2 and above to reduce the pressure on XAPI) may be
> causing this issue. Anthony can you please take a look?
>
>
> On 14/04/14 8:12 AM, "Mike Tutkowski" 
> wrote:
>
> >I was seeing a similar issue when XAPI was changed when I tried to
> >create an SR.
> >
> >I wonder if that XAPI issue hasn't been fully resolved.
> >
> >
> >On Sun, Apr 13, 2014 at 4:16 PM, Srikanteswararao Talluri <
> >srikanteswararao.tall...@citrix.com> wrote:
> >
> >> Did anybody see this on latest 4.4?
> >>
> >>
> >> 2014-04-14 00:01:34,305 DEBUG [c.c.h.x.r.XenServerConnectionPool]
> >> (DirectAgent-71:ctx-12d9be59) XmlRpcException for method: event.from
> >>due to
> >> org.apache.xmlrpc.XmlRpcException: Failed to create input stream:
> >>Read  timed out
> >>
> >> 2014-04-14 00:01:34,305 WARN  [c.c.h.x.r.CitrixResourceBase]
> >> (DirectAgent-71:ctx-12d9be59) callHostPlugin failed for cmd:
> >> copy_vhd_from_secondarystorage with args mountpoint:
> >>10.147.28.7:/export/home/talluri/psec/template/tmpl/1/1/,
> >> sruuid: b3dd21aa-8e82-a923-ed29-ee58220d158b, namelabel:
> >> cloud-8498d88f-be6b-41b6-bece-a58bbc1fbc32,  due to Failed to create
> >>input
> >> stream: Read timed out
> >>
> >> org.apache.xmlrpc.XmlRpcException: Failed to create input stream:
> >> Read timed out
> >>
> >> at
> >>
> >>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.getInputStream(XmlRpcS
> >>unH
> >>ttpTransport.java:99)
> >>
> >> at
> >>
> >>org.apache.xmlrpc.client.XmlRpcStreamTransport.sendRequest(XmlRpcStrea
> >>mTr
> >>ansport.java:152)
> >>
> >> at
> >>
> >>org.apache.xmlrpc.client.XmlRpcHttpTransport.sendRequest(XmlRpcHttpTra
> >>nsp
> >>ort.java:143)
> >>
> >> at
> >>
> >>org.apache.xmlrpc.client.XmlRpcSunHttpTransport.sendRequest(XmlRpcSunH
> >>ttp
> >>Transport.java:69)
> >>
> >> at
> >>
> >>org.apache.xmlrpc.client.XmlRpcClientWorker.execute(XmlRpcClientWorker
> >>.ja
> >>va:56)
> >>
> >> at
> >> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:167)
> >>
> >> at
> >> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:137)
> >>
> >> at
> >> org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:126)
> >>
> >> at
> >> com.xensource.xenapi.Connection.dispatch(Connection.java:285)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.XenServerConnectionPool$XenServerCon
> >>nec
> >>tion.dispatch(XenServerConnectionPool.java:456)
> >>
> >> at com.xensource.xenapi.Event.properFrom(Event.java:310)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.XenServer620Resource.waitForTask(Xen
> >>Ser
> >>ver620Resource.java:125)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.CitrixResourceBase.callHostPluginAsy
> >>nc(
> >>CitrixResourceBase.java:3558)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copy_vhd_f
> >>rom
> >>_secondarystorage(XenServerStorageProcessor.java:826)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.XenServerStorageProcessor.copyTempla
> >>teT
> >>oPrimaryStorage(XenServerStorageProcessor.java:962)
> >>
> >> at
> >>
> >>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(
> >>Sto
> >>rageSubsystemCommandHandlerBase.java:77)
> >>
> >> at
> >>
> >>com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleSt
> >>ora
> >>geCommands(StorageSubsystemCommandHandlerBase.java:52)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.CitrixResourceBase.executeRequest(Ci
> >>tri
> >>xResourceBase.java:542)
> >>
> >> at
> >>
> >>com.cloud.hypervisor.xen.resource.XenServer56Resource.executeRequest(X
> >>enS
> >>erver56Resource.java:60)
> >>
> >> at
> >>
> >

Re: Review Request 19892: Add Juniper Contrail support to devCloud environment

2014-04-15 Thread Sachchidanand Vaidya

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

(Updated April 15, 2014, 1:18 p.m.)


Review request for cloudstack.


Changes
---

Updated diff with whitespaces removed from config file.


Repository: cloudstack-git


Description
---

Add OpenContrail support to devCloud environment:
- Created new devcloud advanced networking config file 
(devcloud-advanced_juniper-contrail.cfg).
- Modification to deployDataCenter script, to enable 
"JuniperContrailRouter" &
  "JuniperContrailVpcRouter" providers.


Diffs (updated)
-

  tools/devcloud/devcloud-advanced_juniper-contrail.cfg PRE-CREATION 
  tools/marvin/marvin/deployDataCenter.py c4f6e1e 

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


Testing
---

Unit tested with devCloud2 and ACS4.3 code base.


Thanks,

Sachchidanand Vaidya



Re: Review Request 19912: [CLOUDSTACK-6322] Fixed Contrail: Params validation is missing while launching a service instance thru cloudmonkey

2014-04-15 Thread Sachchidanand Vaidya


> On April 15, 2014, 7:23 a.m., daan Hoogland wrote:
> > It would seem that this is a behavioral change that would break backwards 
> > compatibility, is it? Right now empty service names are accepted. What is 
> > the consequence and why change the behavior, can you shed light on the 
> > subject?
> > 
> > otherwise the change seems trivial enough and is applicable.

Hi Daan,

 If user passes "null" string during instance creation then it's an issue since 
we can't search service-instance with a null string as part of Fully Qualified 
Name lookup on Contrail Controller. Hence user can't delete a service-instance 
with "null" string. Passing empty string is functionally ok but we want to 
avoid both "null" and "empty" string since  it's not a good idea from 
usability/debugability point of view.

Thanks & Regards,
Sachin


- Sachchidanand


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


On April 2, 2014, 2:14 a.m., Sachchidanand Vaidya wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19912/
> ---
> 
> (Updated April 2, 2014, 2:14 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Don't allow service instance creation with empty or null service-instance 
> "name"
> 
> 
> Diffs
> -
> 
>   
> plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/api/command/CreateServiceInstanceCmd.java
>  d2cb4de 
> 
> Diff: https://reviews.apache.org/r/19912/diff/
> 
> 
> Testing
> ---
> 
> Unit tested for null and empty "name" parameter.
> 
> 
> Thanks,
> 
> Sachchidanand Vaidya
> 
>



Unable to start instance on 4.3 using Simulator

2014-04-15 Thread David Grizzanti
All,

Has anyone run across issues when standing up a new instances with the 
simulator on 4.3? Configuration goes fine, but whenever I try to add a new 
instance I get the following errors:

ERROR [c.c.v.VirtualMachineManagerImpl] (Job-Executor-1:ctx-8ed90afb 
ctx-7310e6f4) Failed to start instance 
VM[User|QA-10296ffe-d1e3-40c7-a5a8-df2c02447b10]
com.cloud.utils.exception.CloudRuntimeException: Unable to send command. 
Upgrade in progress. Please contact administrator.
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.sendCommandsToRouter(VirtualNetworkApplianceManagerImpl.java:3616)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl$7.execute(VirtualNetworkApplianceManagerImpl.java:3051)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.applyRules(VirtualNetworkApplianceManagerImpl.java:3903)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.applyDhcpEntry(VirtualNetworkApplianceManagerImpl.java:3043)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:622)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.sun.proxy.$Proxy221.applyDhcpEntry(Unknown Source)
at 
com.cloud.network.element.VirtualRouterElement.addDhcpEntry(VirtualRouterElement.java:921)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepareElement(NetworkOrchestrator.java:1187)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepareNic(NetworkOrchestrator.java:1309)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepare(NetworkOrchestrator.java:1245)
at 
com.cloud.vm.VirtualMachineManagerImpl.orchestrateStart(VirtualMachineManagerImpl.java:960)
at 
com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:761)
at 
com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:601)
at 
org.apache.cloudstack.engine.cloud.entity.api.VMEntityManagerImpl.deployVirtualMachine(VMEntityManagerImpl.java:228)
at 
org.apache.cloudstack.engine.cloud.entity.api.VirtualMachineEntityImpl.deploy(VirtualMachineEntityImpl.java:207)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3581)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3161)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3147)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:622)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
com.cloud.event.ActionEventInterceptor.invoke(ActionEventInterceptor.java:50)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:161)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.sun.proxy.$Proxy213.startVirtualMachine(Unknown Source)
at 
org.apache.cloudstack.api.command.user.vm.DeployVMCmd.execute(DeployVMCmd.java:443)
at com.cloud.api.ApiDispatcher.dispatch(ApiDispatcher.java:161)
at 
com.cloud.api.ApiAsyncJobDispatcher.runJobInContext(ApiAsyncJobDispatcher.java:109)
at 
com.cloud.api.ApiAsyncJobDispatcher$1.run(ApiAsyncJobDispatcher.java:66)

Re: git commit: updated refs/heads/4.4 to 46f6b26

2014-04-15 Thread Daan Hoogland
Anthony,

Can you remove instead of commenting out code? We do have git.
Op 15 apr. 2014 20:43 schreef :

> Repository: cloudstack
> Updated Branches:
>   refs/heads/4.4 b78aee8f1 -> 46f6b26d4
>
>
> disable XAPI event
>
>
> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
> Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/46f6b26d
> Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/46f6b26d
> Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/46f6b26d
>
> Branch: refs/heads/4.4
> Commit: 46f6b26d49c39d6ec1c98876799c04d8cd92b46e
> Parents: b78aee8
> Author: Anthony Xu 
> Authored: Tue Apr 15 11:26:17 2014 -0700
> Committer: Anthony Xu 
> Committed: Tue Apr 15 11:42:53 2014 -0700
>
> --
>  .../hypervisor/xen/resource/XenServer620Resource.java | 10 ++
>  1 file changed, 6 insertions(+), 4 deletions(-)
> --
>
>
>
> http://git-wip-us.apache.org/repos/asf/cloudstack/blob/46f6b26d/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
> --
> diff --git
> a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
> b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
> index 0c8bc71..4c9907b 100644
> ---
> a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
> +++
> b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java
> @@ -19,18 +19,20 @@ import com.xensource.xenapi.Connection;
>  import com.xensource.xenapi.Host;
>  import com.xensource.xenapi.HostPatch;
>  import com.xensource.xenapi.PoolPatch;
> +/*
> +import java.util.HashSet;
>  import com.xensource.xenapi.Event;
>  import com.xensource.xenapi.Task;
>  import com.xensource.xenapi.Types;
>  import com.xensource.xenapi.Types.XenAPIException;
>  import org.apache.xmlrpc.XmlRpcException;
> +import java.util.concurrent.TimeoutException;
> +*/
>  import org.apache.cloudstack.hypervisor.xenserver.XenserverConfigs;
>  import java.util.Map;
>  import java.util.Set;
> -import java.util.HashSet;
>  import javax.ejb.Local;
>  import org.apache.log4j.Logger;
> -import java.util.concurrent.TimeoutException;
>  import com.cloud.agent.api.StartupRoutingCommand;
>  import com.cloud.resource.ServerResource;
>
> @@ -110,7 +112,7 @@ public class XenServer620Resource extends
> XenServer610Resource {
>  cmd.setHostDetails(details);
>  }
>
> -
> +/*
>  protected void waitForTask(Connection c, Task task, long
> pollInterval, long timeout) throws XenAPIException, XmlRpcException,
> TimeoutException {
>  long beginTime = System.currentTimeMillis();
>  if (s_logger.isTraceEnabled()) {
> @@ -162,6 +164,6 @@ public class XenServer620Resource extends
> XenServer610Resource {
>  }
>  }
>  }
> -
> +*/
>
>  }
>
>


Re: login API with MD5 is not working

2014-04-15 Thread Nate Gordon
CloudStack changed password encoding in 4.2 (I believe, but I seem to
remember, but can't find documentation that it changed in 4.1) such that it
no longer assumes that passwords are MD5s coming in. It instead expects
that passwords are passed as plain text and are encoded by CS internally
according to how it is configured to store passwords. Documentation is at:

http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.0/html/Installation_Guide/management-server-install-flow.html#password-storage-engine

This threw me off as well and applies to both logins and user/account
create.

Thanks,

-Nate



On Tue, Apr 15, 2014 at 3:02 AM, Tejas Gadaria wrote:

> Hi,
>
> I am trying to login in to CS 4.3 though login API.
>
> I am passing MD5 hash (1st) in password that works fine with CS 4.0.2 but
> same doesn't works well with CS 4.3. Then I try to pass password in plain
> text ( 2nd) & it worked, Is this a bug?
>
> Both APIs are given below,
>
>
> 1)
>
> http://10.129.151.55:8080/client/api?&command=login&username=admin&password=5f4dcc3b5aa765d61d8327deb882cf99
>
>
> 2)
>
> http://10.129.151.55:8080/client/api?&command=login&username=admin&password=password
>
> Regards,
> Tejas
>



-- 


*Nate Gordon*Director of Technology | Appcore - the business of cloud
computing®

Office +1.800.735.7104  |  Direct +1.515.612.7787
nate.gor...@appcore.com  |  www.appcore.com

--

The information in this message is intended for the named recipients only.
It may contain information that is privileged, confidential or otherwise
protected from disclosure. If you are not the intended recipient, you are
hereby notified that any disclosure, copying, distribution, or the taking
of any action in reliance on the contents of this message is strictly
prohibited. If you have received this e-mail in error, do not print it or
disseminate it or its contents. In such event, please notify the sender by
return e-mail and delete the e-mail file immediately thereafter. Thank you.


Review Request 20387: CLOUDSTACK-6202: Add signatureversion and expires to server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java

2014-04-15 Thread Yichi Lu

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

Review request for cloudstack, Chiradeep Vittal and Rohit Yadav.


Repository: cloudstack-git


Description
---

add ApiConstants.SIGNATURE_VERSION and ApiConstants.EXPIRES to 
server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java to allow 
use of signature version and expires by cloudmonkey


Diffs
-

  server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java 7a73b8a 

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


Testing
---

API call: $ cloudmonkey list users


Thanks,

Yichi Lu



Review Request 20389: CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from appropriate branch.

2014-04-15 Thread Rayees Namathponnan

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

Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Nitin Mehta.


Repository: cloudstack-git


Description
---

CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from 
appropriate branch.


Diffs
-

  tools/appliance/definitions/systemvmtemplate/postinstall.sh 997d854 

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


Testing
---

Yes


Thanks,

Rayees Namathponnan



Review Request 20390: CLOUDSTACK-6202: Add signatureversion and expiring datetime to cloudmonkey, make it configurable

2014-04-15 Thread Yichi Lu

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

Review request for cloudstack, Chiradeep Vittal and Rohit Yadav.


Repository: cloudstack-cloudmonkey


Description
---

use signature version 3 for cloudmonkey api calls, with 600 seconds expiration 
time as default. The expiration time is configurable via .cloudmonkey/config


Diffs
-

  cloudmonkey/cloudmonkey.py b465bec 
  cloudmonkey/config.py 2f91608 
  cloudmonkey/requester.py b06e1fc 

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


Testing
---

for 600 seconds expiration time and CST:
now:  2014-04-15T16:36:46+ , expires:  2014-04-15T21:46:46+


Thanks,

Yichi Lu



Review Request 20391: CLOUDSTACK-6255: UI for supporting region level VPC, distributed routing enabled VPC and stretched L2 neworks (ui for SDN)

2014-04-15 Thread Gabor Apati-Nagy

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

Review request for cloudstack, Brian Federle and Jessica Wang.


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


Repository: cloudstack-git


Description
---

CLOUDSTACK-6255: UI for supporting region level VPC, distributed routing 
enabled VPC and stretched L2 neworks (ui for SDN)
(UI for SDN)


Diffs
-

  client/WEB-INF/classes/resources/messages.properties bfa9600 
  ui/dictionary.jsp f4d380c 
  ui/scripts/configuration.js ba6bdce 
  ui/scripts/docs.js df4bcf0 
  ui/scripts/network.js 6494704 
  ui/scripts/regions.js 66dae8c 
  ui/scripts/ui/dialog.js 50e12f9 
  ui/scripts/vpc.js 73d34df 

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


Testing
---


Thanks,

Gabor Apati-Nagy



Re: Review Request 20389: CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from appropriate branch.

2014-04-15 Thread Prachi Damle

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

Ship it!


Looks like the code change to get the snapshot from the correct branch - 4.2 in 
this case, looks ok. This changes is to be applied on 4.2 only.

- Prachi Damle


On April 15, 2014, 10:23 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20389/
> ---
> 
> (Updated April 15, 2014, 10:23 p.m.)
> 
> 
> Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Nitin 
> Mehta.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot 
> from appropriate branch.
> 
> 
> Diffs
> -
> 
>   tools/appliance/definitions/systemvmtemplate/postinstall.sh 997d854 
> 
> Diff: https://reviews.apache.org/r/20389/diff/
> 
> 
> Testing
> ---
> 
> Yes
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



Re: Review Request 20389: CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from appropriate branch.

2014-04-15 Thread Rayees Namathponnan

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

(Updated April 15, 2014, 11:02 p.m.)


Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Prachi 
Damle.


Repository: cloudstack-git


Description
---

CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from 
appropriate branch.


Diffs
-

  tools/appliance/definitions/systemvmtemplate/postinstall.sh 997d854 

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


Testing
---

Yes


Thanks,

Rayees Namathponnan



Re: Review Request 20389: CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from appropriate branch.

2014-04-15 Thread Rayees Namathponnan


> On April 15, 2014, 10:54 p.m., Prachi Damle wrote:
> > Looks like the code change to get the snapshot from the correct branch - 
> > 4.2 in this case, looks ok. This changes is to be applied on 4.2 only.

Yes, this required only in 4.2

64-bit template script already updated in below check-in 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6d5ca0ef7dc10d25d8c6c5312232caf06895d47


- Rayees


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


On April 15, 2014, 11:02 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20389/
> ---
> 
> (Updated April 15, 2014, 11:02 p.m.)
> 
> 
> Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Prachi 
> Damle.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot 
> from appropriate branch.
> 
> 
> Diffs
> -
> 
>   tools/appliance/definitions/systemvmtemplate/postinstall.sh 997d854 
> 
> Diff: https://reviews.apache.org/r/20389/diff/
> 
> 
> Testing
> ---
> 
> Yes
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



[ACS4.4, XenServer] Adding a XenServer host to a cluster

2014-04-15 Thread Mike Tutkowski
Hi,

Prior to 4.4, if I built a CS cluster with one XenServer host (that was not
in a XenServer resource pool) and then later wanted to add another
XenServer host to the CS cluster, CS would put both hosts in a XenServer
resource pool automatically and then show them both in the CS cluster.

In 4.4, this process fails.

What I need to do now is manually create the XenServer resource pool and
put my hosts in it, then ask CS to add the second host to the CS cluster.

Is this expected behavior for 4.4?

Thanks!

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


Re: Review Request 20389: CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot from appropriate branch.

2014-04-15 Thread ASF Subversion and Git Services

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


Commit 0610dbf803d1b1dc37b41b3a6319cfcd94cf6bbe in cloudstack's branch 
refs/heads/4.2 from rayeesn
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=0610dbf ]

Fixed provided for CLOUDSTACK-6416 need to be ported for 32 bit template also


- ASF Subversion and Git Services


On April 15, 2014, 11:02 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20389/
> ---
> 
> (Updated April 15, 2014, 11:02 p.m.)
> 
> 
> Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Prachi 
> Damle.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-6416: fixed postinstall.sh script to fetch cloudstack snapshot 
> from appropriate branch.
> 
> 
> Diffs
> -
> 
>   tools/appliance/definitions/systemvmtemplate/postinstall.sh 997d854 
> 
> Diff: https://reviews.apache.org/r/20389/diff/
> 
> 
> Testing
> ---
> 
> Yes
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



Re: [ACS4.5] move from xen 2 xenserver

2014-04-15 Thread Tim Mackey
This was a little more than a straight renaming, and I'll post all the
changes to the wiki for review in the morning.  At a high level this patch
did the following:

- Move the old plugins/hypervisors/xen to /plugins/hypervisors/xenserver
and change all namespaces to reflect the move (bulk of the work)
- Change any DDL/DML containing "xen"/"Xen" to become "XenServer" (took
care of both create and upgrade paths)
- Change any display areas containing "Xen" to become "XenServer"

iirc there was one API change in the network label.  It was
xennetworklabel, and I updated it to become xenservernetworklabel.  Since I
don't want to break backwards compatibility either, I'm open to how best to
handle the situation.  Also would like to understand a test case I can use
to validate.

-tim


On Tue, Apr 15, 2014 at 6:27 AM, Sebastien Goasguen wrote:

>
> On Apr 15, 2014, at 4:46 AM, Stephen Turner 
> wrote:
>
> > As I said in the previous threaed, I'm +1 on the principle. It will
> avoid confusion between straight Xen and XenServer. It will also allow us
> to offer a non-XenServer Xen implementation.
> >
> > Sebastien, as all the filenames have changed, it's not clear from the
> diff whether this is just a straight renaming with no other changes. Could
> you confirm that?
> >
>
> That's what it looks like, basically it creates a 'xenserver' plugin
> I know Tim has a prototype of Xen support as well, but it was not in this
> commit it seems.
>
>
> > Also, are there any backwards compatibility implications for the API? Or
> did we already use "XenServer" instead of "Xen" there?
> >
>
> In the CloudStack API ?
>
> We are probably using Xen as value in several api calls…so we will need to
> check this carefully before any merge, we definitely don't want to break
> backward compatibility, or if we do we will need to move to 5.0
>
> > --
> > Stephen Turner
> >
> >
> > -Original Message-
> > From: Sebastien Goasguen [mailto:run...@gmail.com]
> > Sent: 15 April 2014 08:36
> > To: dev@cloudstack.apache.org
> > Subject: [ACS4.5] move from xen 2 xenserver
> >
> > Folks,
> >
> > I just applied a patch from Tim Mackey:
> >
> >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=748b575af8a66c58b0d52e7457e4d4e1e875231f
> >
> > commit: 748b575af8a66c58b0d52e7457e4d4e1e875231f
> >
> > This followed a [PROPOSAL] thread [1]
> >
> > This was pushed into a separate branch: xen2server
> >
> > This is a significant change that we should get consensus on and merge
> to master relatively quickly to avoid any conflicts later on.
> >
> > Basically, we have been treating xen and xenserver the same so far,
> since the integration with Xen (i.e Xen Project) was/is done via xapi.
> >
> > There has been discussion to integrate with Xen using straight up
> libvirt, by creating a new Xen agent probably based on the KVM agent and
> there was some discussion to that effect in the Denver Hackathon.
> >
> > Making a clear split between Xen and Xenserver type of hypervisors will
> help support different integration methods.
> >
> > I have asked Tim (in the review message) to create a wiki page,
> discussing all of this, but I wanted to give a heads up that this just hit
> the repo and that we should see a [MERGE] thread quickly.
> >
> > thoughts, flames ?
> >
> >
> > [1] http://markmail.org/thread/yrl3ii7gqlaaexij
> >
> > -Sebastien
> >
>
>


[ACS4.4] Can't create an SR anymore (due to XmlRpcException)

2014-04-15 Thread Mike Tutkowski
Hi,

I have a problem that - at first - I thought was related to the XAPI
changes (although they were reverted and this problem seems to remain).

I am no longer able to create an SR in CitrixResourceBase (getIscsiSR
method) as XmlRpcException is being thrown with the following text:

Failed to create input stream: Read timed out
The "funny" part is that the SR does actually get created. However, due to
the exception being throw, an error is returned to CS by the XenServer
plug-in.

Does anyone know anything about this?

Thanks!

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


Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey

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

(Updated April 16, 2014, 1:31 a.m.)


Review request for cloudstack.


Changes
---

I changed the iteration method to be more proper, using alternating .next() and 
.prev() calls was hacky and prone to malfunction.


Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
https://issues.apache.org/jira/browse/CLOUDSTACK-5907
https://issues.apache.org/jira/browse/CLOUDSTACK-6396


Repository: cloudstack-git


Description
---

Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM 
volumes (not able to snapshot)


Diffs (updated)
-

  server/src/com/cloud/api/ApiDBUtils.java 67e47f7 

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


Testing
---

Applied on otherwise clean 4.4 branch and tested in a live testing environment 
with KVM hypervisors and RBD primary storage pool that would otherwise identify 
as OVM. No errors and no weird behavior. Just the expected result.


Thanks,

Tanner Danzey



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey

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

(Updated April 16, 2014, 1:34 a.m.)


Review request for cloudstack.


Changes
---

Removed trailing whitespace in comments.


Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
https://issues.apache.org/jira/browse/CLOUDSTACK-5907
https://issues.apache.org/jira/browse/CLOUDSTACK-6396


Repository: cloudstack-git


Description
---

Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM 
volumes (not able to snapshot)


Diffs (updated)
-

  server/src/com/cloud/api/ApiDBUtils.java 67e47f7 

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


Testing
---

Applied on otherwise clean 4.4 branch and tested in a live testing environment 
with KVM hypervisors and RBD primary storage pool that would otherwise identify 
as OVM. No errors and no weird behavior. Just the expected result.


Thanks,

Tanner Danzey



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey

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

(Updated April 16, 2014, 3:23 a.m.)


Review request for cloudstack.


Changes
---

Nasty typo errors from the past have been removed.


Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
https://issues.apache.org/jira/browse/CLOUDSTACK-5907
https://issues.apache.org/jira/browse/CLOUDSTACK-6396


Repository: cloudstack-git


Description
---

Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM 
volumes (not able to snapshot)


Diffs (updated)
-

  server/src/com/cloud/api/ApiDBUtils.java 67e47f7 

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


Testing
---

Applied on otherwise clean 4.4 branch and tested in a live testing environment 
with KVM hypervisors and RBD primary storage pool that would otherwise identify 
as OVM. No errors and no weird behavior. Just the expected result.


Thanks,

Tanner Danzey



Re: Security Group bug impeding system VMs functionality

2014-04-15 Thread Jayapal Reddy Uradi
Hi Nux,

The paste links are does not exist.
Can you please upload the logs again. Also upload rules/logs specific to system 
rules are not set.


Thanks,
Jayapal

On 11-Apr-2014, at 9:10 PM, Nux!  wrote:

> Hello,
> 
> I'm on 4.3 right now, CentOS6.5 + KVM and SG ADV zone.
> What happens is that after a reboot or after disabling a zone, when the 
> system VMs come back the iptables rules required for their proper functioning 
> do not get set.
> It seems to be happening randomly and it may not be affecting both VMs (S and 
> V) at the same time.
> 
> More info:
> http://paste.fedoraproject.org/93567/72307041/
> sg log: http://paste.fedoraproject.org/93564/23056713/
> 
> The problem always goes away if I stop/start the system VMs; the required 
> iptables rules get created, eg:
> -A s-105-VM -m physdev --physdev-in vnet3 --physdev-is-bridged -j RETURN
> -A s-105-VM -m physdev --physdev-in vnet2 --physdev-is-bridged -j RETURN
> -A s-105-VM -m physdev --physdev-in vnet1 --physdev-is-bridged -j RETURN
> -A s-105-VM -j ACCEPT
> -A v-106-VM -m physdev --physdev-in vnet6 --physdev-is-bridged -j RETURN
> -A v-106-VM -m physdev --physdev-in vnet5 --physdev-is-bridged -j RETURN
> -A v-106-VM -j ACCEPT
> 
> 
> If someone could have a look at this it'd be great. Let me know if more info 
> is needed.
> 
> Lucian
> 
> -- 
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Yoshikazu Nojima

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


It seems getHypervisorTypeFromFormat method is only called from 
VolumeJoinDaoImpl#newVolumeResponse. 
It should be fixed, but does it affect snapshot feature?

- Yoshikazu Nojima


On April 16, 2014, 3:23 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 16, 2014, 3:23 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



[REMINDER] Review Request 18677: CLOUDSTACK-3272

2014-04-15 Thread Sonal Ojha
Please take a look.

-- Forwarded message --
From: Sonal Ojha 
Date: Tue, Apr 15, 2014 at 6:07 PM
Subject: Re: Review Request 18677: CLOUDSTACK-3272
To: Murali Reddy , Chiradeep Vittal <
chirade...@gmail.com>
Cc: Sonal Ojha , cloudstack <
dev@cloudstack.apache.org>


   This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18677/
  Review request for cloudstack, Chiradeep Vittal and Murali Reddy.
By Sonal Ojha.

*Updated April 15, 2014, 12:37 p.m.*
Changes

Removed the earlier not working patch files and added a new patch file
(0001-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch)
which works on master with commit 55f53e13e. I could successfully
apply the latest patch attached on master and test the
alert/action/resource/usage events published on rabbitmq message bus.
Earlier patch didnt work as it was built on the older version and had
some changes which werent required. Kindly take a look and revert
back.

  *Bugs: * 
CLOUDSTACK-3272
 *Repository: * cloudstack-git
Description

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

  Testing

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

  Diffs

   - server/src/com/cloud/configuration/Config.java (2d55f46)
   - server/src/com/cloud/event/ActionEventUtils.java (00122de)
   - server/src/com/cloud/test/DatabaseConfig.java (9370218)

View Diff 
File Attachments (updated)

   - Working patch for master (latest git commit
55f53e13e)




-- 

*Thanks and Regards,*

*Sonal Ojha ▪ Sr.Engineer - Product Development **▪ **Sungard Availability
Services, India*

2nd Floor, Wing 4, Cluster D, MIDC Kharadi Knowledge Park, Pune - 411 014 ▪
Office: 267-234-9014 ▪ Mobile: +91 9922412645 ▪ sonal.o...@sungardas.com ▪
www.sungardas. *in*

*[image: Logo]* 
*[image: cid:image019.png@01CF48EC.6617C7F0]*
 *[image:
cid:image020.png@01CF48EC.6617C7F0]* 
 *[image: 
cid:image021.png@01CF48EC.6617C7F0]*
 *[image: 
cid:image022.png@01CF48EC.6617C7F0]*
 *[image: 
cid:image023.png@01CF48EC.6617C7F0]*
 *[image: cid:image024.png@01CF48EC.6617C7F0]*
*CONFIDENTIALITY:*  This e-mail (including any attachments) may contain
confidential, proprietary and privileged information, and unauthorized
disclosure or use is prohibited.  If you received this e-mail in error,
please notify the sender and delete this e-mail from your system.


[OFFLIST] I will be offlist from 4/17 - 5/6

2014-04-15 Thread Animesh Chaturvedi
Folks

FYI I will be out  on PTO for 2 weeks starting 4/17


Thanks
Animesh


Re: [ACS4.4] Can't create an SR anymore (due to XmlRpcException)

2014-04-15 Thread Mike Tutkowski
Just an FYI that this is not only a problem for managed storage (as I
expected it wouldn't be).

If you try to add primary storage to CloudStack for XenServer using an
iSCSI target, it fails. You have to first create the SR from XenCenter (or
xe) and then use the PreSetup "protocol" to introduce the storage to CS as
primary storage.

Is anyone else having trouble with XML RPC?

Thanks


On Tue, Apr 15, 2014 at 7:22 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Hi,
>
> I have a problem that - at first - I thought was related to the XAPI
> changes (although they were reverted and this problem seems to remain).
>
> I am no longer able to create an SR in CitrixResourceBase (getIscsiSR
> method) as XmlRpcException is being thrown with the following text:
>
> Failed to create input stream: Read timed out
> The "funny" part is that the SR does actually get created. However, due to
> the exception being throw, an error is returned to CS by the XenServer
> plug-in.
>
> Does anyone know anything about this?
>
> Thanks!
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the 
> cloud
> *(tm)*
>



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


RE: Unable to start instance on 4.3 using Simulator

2014-04-15 Thread Koushik Das
In 4.3 you need to set global config router.version.check to false to 
workaround this issue in simulator. This is fixed in 4.4/master by commit 
38b8677e01fe1ec89df43abe9fac26295597080f.

-Koushik

-Original Message-
From: David Grizzanti [mailto:david.grizza...@sungard.com] 
Sent: Wednesday, 16 April 2014 2:10 AM
To: dev@cloudstack.apache.org
Subject: Unable to start instance on 4.3 using Simulator

All,

Has anyone run across issues when standing up a new instances with the 
simulator on 4.3? Configuration goes fine, but whenever I try to add a new 
instance I get the following errors:

ERROR [c.c.v.VirtualMachineManagerImpl] (Job-Executor-1:ctx-8ed90afb 
ctx-7310e6f4) Failed to start instance 
VM[User|QA-10296ffe-d1e3-40c7-a5a8-df2c02447b10]
com.cloud.utils.exception.CloudRuntimeException: Unable to send command. 
Upgrade in progress. Please contact administrator.
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.sendCommandsToRouter(VirtualNetworkApplianceManagerImpl.java:3616)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl$7.execute(VirtualNetworkApplianceManagerImpl.java:3051)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.applyRules(VirtualNetworkApplianceManagerImpl.java:3903)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.applyDhcpEntry(VirtualNetworkApplianceManagerImpl.java:3043)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:622)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.sun.proxy.$Proxy221.applyDhcpEntry(Unknown Source)
at 
com.cloud.network.element.VirtualRouterElement.addDhcpEntry(VirtualRouterElement.java:921)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepareElement(NetworkOrchestrator.java:1187)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepareNic(NetworkOrchestrator.java:1309)
at 
org.apache.cloudstack.engine.orchestration.NetworkOrchestrator.prepare(NetworkOrchestrator.java:1245)
at 
com.cloud.vm.VirtualMachineManagerImpl.orchestrateStart(VirtualMachineManagerImpl.java:960)
at 
com.cloud.vm.VirtualMachineManagerImpl.advanceStart(VirtualMachineManagerImpl.java:761)
at 
com.cloud.vm.VirtualMachineManagerImpl.start(VirtualMachineManagerImpl.java:601)
at 
org.apache.cloudstack.engine.cloud.entity.api.VMEntityManagerImpl.deployVirtualMachine(VMEntityManagerImpl.java:228)
at 
org.apache.cloudstack.engine.cloud.entity.api.VirtualMachineEntityImpl.deploy(VirtualMachineEntityImpl.java:207)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3581)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3161)
at 
com.cloud.vm.UserVmManagerImpl.startVirtualMachine(UserVmManagerImpl.java:3147)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:622)
at 
org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
at 
com.cloud.event.ActionEventInterceptor.invoke(ActionEventInterceptor.java:50)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:161)
at 
org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
at 
org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:172)
at 
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at com.

Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey


> On April 16, 2014, 4:02 a.m., Yoshikazu Nojima wrote:
> > It seems getHypervisorTypeFromFormat method is only called from 
> > VolumeJoinDaoImpl#newVolumeResponse. 
> > It should be fixed, but does it affect snapshot feature?

Yes. Without this fix, if you go into the Storage UI panel the volumes show as 
OVM without any snapshot button.


- Tanner


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


On April 16, 2014, 3:23 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 16, 2014, 3:23 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Yoshikazu Nojima


> On April 16, 2014, 4:02 a.m., Yoshikazu Nojima wrote:
> > It seems getHypervisorTypeFromFormat method is only called from 
> > VolumeJoinDaoImpl#newVolumeResponse. 
> > It should be fixed, but does it affect snapshot feature?
> 
> Tanner Danzey wrote:
> Yes. Without this fix, if you go into the Storage UI panel the volumes 
> show as OVM without any snapshot button.

I understand. Thank you for explanation.


- Yoshikazu


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


On April 16, 2014, 3:23 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 16, 2014, 3:23 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-04-15 Thread Tanner Danzey


> On April 16, 2014, 4:02 a.m., Yoshikazu Nojima wrote:
> > It seems getHypervisorTypeFromFormat method is only called from 
> > VolumeJoinDaoImpl#newVolumeResponse. 
> > It should be fixed, but does it affect snapshot feature?
> 
> Tanner Danzey wrote:
> Yes. Without this fix, if you go into the Storage UI panel the volumes 
> show as OVM without any snapshot button.
> 
> Yoshikazu Nojima wrote:
> I understand. Thank you for explanation.

You're most welcome!


- Tanner


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


On April 16, 2014, 3:23 a.m., Tanner Danzey wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20357/
> ---
> 
> (Updated April 16, 2014, 3:23 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
> https://issues.apache.org/jira/browse/CLOUDSTACK-5907
> https://issues.apache.org/jira/browse/CLOUDSTACK-6396
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as 
> OVM volumes (not able to snapshot)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/api/ApiDBUtils.java 67e47f7 
> 
> Diff: https://reviews.apache.org/r/20357/diff/
> 
> 
> Testing
> ---
> 
> Applied on otherwise clean 4.4 branch and tested in a live testing 
> environment with KVM hypervisors and RBD primary storage pool that would 
> otherwise identify as OVM. No errors and no weird behavior. Just the expected 
> result.
> 
> 
> Thanks,
> 
> Tanner Danzey
> 
>



Re: Devcloud2 errors on 4.4

2014-04-15 Thread Punith S
hi,

check whether you are using the new updated devcloud2(1.3GB) for 4.3
onwards!
ref - https://cwiki.apache.org/confluence/display/CLOUDSTACK/DevCloud

and its recommended to use devcloud as host only mode.
thanks,



On Wed, Apr 16, 2014 at 12:18 AM, David Grizzanti <
david.grizza...@sungard.com> wrote:

> All,
>
> I am facing an issue when trying to get devcloud2 running with latest code
> from 4.4 branch. There was a similar email to this a few weeks back on
> master, but I did not see any resolution.
>
> I'm running everything in side of devcloud2 (mgmt server, mysql, xen).
>
> Seeing the following errors when trying to deploy an initial zone.  Don't
> see any issues with creating the /opt/cloud/bin directory inside the VM.
>
> WARN  [c.c.a.m.DirectAgentAttache] (DirectAgent-37:ctx-0d57a0bf) Seq
> 3-8906994163031998465: Exception Caught while executing command
> com.cloud.utils.exception.CloudRuntimeException: Cannot create directory
> /opt/cloud/bin on XenServer hosts
> at
> com.cloud.hypervisor.xen.resource.CitrixResourceBase.setupServer(CitrixResourceBase.java:4842)
> at
> com.cloud.hypervisor.xen.resource.CitrixResourceBase.execute(CitrixResourceBase.java:4668)
> at
> com.cloud.hypervisor.xen.resource.CitrixResourceBase.executeRequest(CitrixResourceBase.java:489)
> at
> com.cloud.hypervisor.xen.resource.XcpOssResource.executeRequest(XcpOssResource.java:176)
> at
> com.cloud.agent.manager.DirectAgentAttache$Task.runInContext(DirectAgentAttache.java:216)
> at
> org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(ManagedContextRunnable.java:49)
> at
> org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(DefaultManagedContext.java:56)
> at
> org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWithContext(DefaultManagedContext.java:103)
> at
> org.apache.cloudstack.managed.context.impl.DefaultManagedContext.runWithContext(DefaultManagedContext.java:53)
> at
> org.apache.cloudstack.managed.context.ManagedContextRunnable.run(ManagedContextRunnable.java:46)
> at
> java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
> at
> java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
> at java.util.concurrent.FutureTask.run(FutureTask.java:166)
> at
> java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
> at
> java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
> at
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:724)
> WARN  [c.c.h.x.d.XcpServerDiscoverer] (AgentTaskPool-13:ctx-5015bd03)
> Unable to setup agent 3 due to
> com.cloud.utils.exception.CloudRuntimeException: Cannot create directory
> /opt/cloud/bin on XenServer hosts
> INFO  [c.c.u.e.CSExceptionErrorCode] (AgentTaskPool-13:ctx-5015bd03) Could
> not find exception: com.cloud.exception.ConnectionException in error code
> list for exceptions
> WARN  [c.c.a.m.AgentManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Monitor
> XcpServerDiscoverer says there is an error in the connect process for 3 due
> to Reinitialize agent after setup.
> INFO  [c.c.a.m.AgentManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Host 3 is
> disconnecting with event AgentDisconnected
> WARN  [c.c.r.ResourceManagerImpl] (AgentTaskPool-13:ctx-5015bd03) Unable
> to connect due to
> com.cloud.exception.ConnectionException: Reinitialize agent after setup.
> at
> com.cloud.hypervisor.xen.discoverer.XcpServerDiscoverer.processConnect(XcpServerDiscoverer.java:658)
> at
> com.cloud.agent.manager.AgentManagerImpl.notifyMonitorsOfConnection(AgentManagerImpl.java:514)
> at
> com.cloud.agent.manager.AgentManagerImpl.handleDirectConnectAgent(AgentManagerImpl.java:1428)
> at
> com.cloud.resource.ResourceManagerImpl.createHostAndAgent(ResourceManagerImpl.java:1767)
> at
> com.cloud.resource.ResourceManagerImpl.createHostAndAgent(ResourceManagerImpl.java:1893)
> at sun.reflect.GeneratedMethodAccessor226.invoke(Unknown Source)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:606)
> at
> org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:317)
> at
> org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:183)
> at
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:150)
> at
> org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:91)
> 

Re: unable to create a instance in 4.3 using new devcloud and solution

2014-04-15 Thread Punith S
thanks for the link rajesh.


On Tue, Apr 15, 2014 at 11:19 PM, Rajesh Battala
wrote:

> Punit,
> There is a change in the template.
> Download it from
> http://jenkins.buildacloud.org/job/build-systemvm64-master/
> You can replace the base system vm template in devcloud and use it.
>
> Thanks
> Rajesh Battala
>
> -Original Message-
> From: Punith S [mailto:punit...@cloudbyte.com]
> Sent: Tuesday, April 15, 2014 11:59 AM
> To: cloudstack
> Subject: unable to create a instance in 4.3 using new devcloud and solution
>
> hi,
>
> i'm using new devcloud for 4.3, and configured the basic zone using
> devcloud python scripts.
> when we launch a the downloaded tiny linux template, it throws the
> following exception :
>
> Router requires upgrade. Unable to send command to router Unable to send
> command. Upgrade in progress. Please contact administrator.
> and finally insufficient server capacity exception.
>
> and then found that the virtual router needed an upgrade from current 4.2
> to 4.3 version, so initiated the upgrade for virtual router, *the UI shows
> the upgrade was completed successfully but the version still remains the
> same(4.2)*, hence it fails again to create a instance again. and i also
> checked the /opt/storage/secondary template in devcloud, it remains the new
> vdh file.
>
> my workaround solution :
>  just make *false* for router.check.version in global settings and restart
> the MS.
>
> any workaround for the upgrade for router from 4.2 to 4.3 in devcloud , is
> there any new template we need to seed in secondary storage ??
>
> --
> regards,
>
> punith s
> cloudbyte.com
>



-- 
regards,

punith s
cloudbyte.com


Re: Review Request 20366: Integrating Windows wix tool to maven

2014-04-15 Thread Abhinandan Prateek

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

Ship it!


Ship It!

- Abhinandan Prateek


On April 15, 2014, 9:29 a.m., Damodar Reddy Talakanti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20366/
> ---
> 
> (Updated April 15, 2014, 9:29 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and daan Hoogland.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6410
> 
> https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6410
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Integrated Wix tool set with maven build process so that no manual steps are 
> required to create windows msi installer file.
> 
> 
> Diffs
> -
> 
>   Wix-cloudstack-maven-plugin/pom.xml PRE-CREATION 
>   Wix-cloudstack-maven-plugin/src/org/apache/cloudstack/wix/HeatMojo.java 
> PRE-CREATION 
>   client/pom.xml d8dbde7 
> 
> Diff: https://reviews.apache.org/r/20366/diff/
> 
> 
> Testing
> ---
> 
> Build process has been Tested on Windows server.
> Regular profile Builds are tested 
> Tested new profile "buildw" and verified msi file is getting created in 
> target.
> 
> 
> Thanks,
> 
> Damodar Reddy Talakanti
> 
>



Re: Review Request 20367: If already tmp directory exist inject Keys is failing while starting management server on windows

2014-04-15 Thread Abhinandan Prateek

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

Ship it!


Ship It!

- Abhinandan Prateek


On April 15, 2014, 9:54 a.m., Damodar Reddy Talakanti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20367/
> ---
> 
> (Updated April 15, 2014, 9:54 a.m.)
> 
> 
> Review request for cloudstack and Abhinandan Prateek.
> 
> 
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6298
> 
> https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6298
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> For any reason if tmp directory present on windows users home directory the 
> inject keys into system vm is failing on management server start up.
> 
> 
> Diffs
> -
> 
>   scripts/vm/systemvm/injectkeys.py bd840f9 
> 
> Diff: https://reviews.apache.org/r/20367/diff/
> 
> 
> Testing
> ---
> 
> Tested on windows 2008 server on fresh setup
> 
> 
> Thanks,
> 
> Damodar Reddy Talakanti
> 
>



Re: [QUESTION] Debugging CS VMware logic in Eclipse

2014-04-15 Thread Mike Tutkowski
Hi Kelven,

Just wanted to follow up on this.

I was running my scenario with VMware again and I see in VMwareGuru that
CopyCommand is sent to the SSVM.

Do you know if this was changed at some point? When I did a bunch of my
testing in the not-too-distant past, I was seeing my CopyCommands being
sent to the server resource for the hypervisor in question that lives
within the management server. I was able to trace through the logic in
copyTemplateToPrimaryStorage(CopyCommand) and watch my code execute.

Now this command seems to be routed to the SSVM, though. I don't see any
recent changes to VMwareGuru, though. I thought you might have some insight
into this.

Thanks!
Mike


On Tue, Apr 8, 2014 at 12:09 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Thanks, Kelven!
>
> Just so I can get a better understanding of this, is there any real-world
> scenario where a customer using VMware would run with storage delegation
> going to the server resource in the management server?
>
> Also, when you say "storage-related commands," do you just mean the
> commands in VmwareStorageProcessor?
>
> Thanks for clarifying this for me, Kelven. This is a bit different from
> how it works with XenServer and KVM and I just want to make sure I have it
> straight in my head. :)
>



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


  1   2   >