Re: Review Request 13223: (CLOUDSTACK-2729) use file lock to prevent concurrent refreshPool/deleteVolume on KVM shared storage pool

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 6d86af85baa57bb20ec9642cbd1cd82c002dcfef in branch refs/heads/master 
from Wei Zhou
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6d86af8 ]

CLOUDSTACK-2729: increase waiting timeout for global file lock to 300 seconds


- ASF Subversion and Git Services


On Aug. 5, 2013, 6:07 p.m., Wei Zhou wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13223/
> ---
> 
> (Updated Aug. 5, 2013, 6:07 p.m.)
> 
> 
> Review request for cloudstack, edison su and Wido den Hollander.
> 
> 
> Bugs: CLOUDSTACK-2729
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The storage pool issue (CLOUDSTACK-2729) is because of a bug in libvirt 
> (https://bugzilla.redhat.com/show_bug.cgi?id=977706)
> We need to prevent deleting a volume when refreshing the pool.
> This patch use a simple file lock to implement it.
> 
> PS: I have tested another file lock similar to Read/Write file lock, but it 
> was very unstable.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
>  b8a9f0f 
> 
> Diff: https://reviews.apache.org/r/13223/diff/
> 
> 
> Testing
> ---
> 
> Applied on 4.0.2 and 4.0.1
> 
> Testing On 4.0.1 From 20,June
> 3 nodes, create a VM on each node every 15 minutes. Destroy the VMs 5 minutes 
> later. expunge.inteval = 600 (10 minutes), expunge.worker = 2
>  
> 
> Testing On 4.0.2 From 01,July
> 2 nodes, create two VMs on each node every 5 minutes. Destroy the VMs 4 
> minutes later. expunge.inteval = 600 (10 minutes), expunge.worker = 2 
> 
> 
> Thanks,
> 
> Wei Zhou
> 
>



Re: Review Request 13223: (CLOUDSTACK-2729) use file lock to prevent concurrent refreshPool/deleteVolume on KVM shared storage pool

2013-08-09 Thread ASF Subversion and Git Services

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


Commit dacc4e1a4f7dc3763f16acc2cddd936fdc2fbd42 in branch refs/heads/4.2 from 
Wei Zhou
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=dacc4e1 ]

CLOUDSTACK-2729: increase waiting timeout for global file lock to 300 seconds
(cherry picked from commit 6d86af85baa57bb20ec9642cbd1cd82c002dcfef)


- ASF Subversion and Git Services


On Aug. 5, 2013, 6:07 p.m., Wei Zhou wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13223/
> ---
> 
> (Updated Aug. 5, 2013, 6:07 p.m.)
> 
> 
> Review request for cloudstack, edison su and Wido den Hollander.
> 
> 
> Bugs: CLOUDSTACK-2729
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The storage pool issue (CLOUDSTACK-2729) is because of a bug in libvirt 
> (https://bugzilla.redhat.com/show_bug.cgi?id=977706)
> We need to prevent deleting a volume when refreshing the pool.
> This patch use a simple file lock to implement it.
> 
> PS: I have tested another file lock similar to Read/Write file lock, but it 
> was very unstable.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
>  b8a9f0f 
> 
> Diff: https://reviews.apache.org/r/13223/diff/
> 
> 
> Testing
> ---
> 
> Applied on 4.0.2 and 4.0.1
> 
> Testing On 4.0.1 From 20,June
> 3 nodes, create a VM on each node every 15 minutes. Destroy the VMs 5 minutes 
> later. expunge.inteval = 600 (10 minutes), expunge.worker = 2
>  
> 
> Testing On 4.0.2 From 01,July
> 2 nodes, create two VMs on each node every 5 minutes. Destroy the VMs 4 
> minutes later. expunge.inteval = 600 (10 minutes), expunge.worker = 2 
> 
> 
> Thanks,
> 
> Wei Zhou
> 
>



Re: Review Request 13223: (CLOUDSTACK-2729) use file lock to prevent concurrent refreshPool/deleteVolume on KVM shared storage pool

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 64bb7007053d004fe7a8ac3b610e20df69f16bab in branch refs/heads/4.1 from 
Wei Zhou
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=64bb700 ]

CLOUDSTACK-2729: increase waiting timeout for global file lock to 300 seconds
(cherry picked from commit 6d86af85baa57bb20ec9642cbd1cd82c002dcfef)


- ASF Subversion and Git Services


On Aug. 5, 2013, 6:07 p.m., Wei Zhou wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13223/
> ---
> 
> (Updated Aug. 5, 2013, 6:07 p.m.)
> 
> 
> Review request for cloudstack, edison su and Wido den Hollander.
> 
> 
> Bugs: CLOUDSTACK-2729
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The storage pool issue (CLOUDSTACK-2729) is because of a bug in libvirt 
> (https://bugzilla.redhat.com/show_bug.cgi?id=977706)
> We need to prevent deleting a volume when refreshing the pool.
> This patch use a simple file lock to implement it.
> 
> PS: I have tested another file lock similar to Read/Write file lock, but it 
> was very unstable.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
>  b8a9f0f 
> 
> Diff: https://reviews.apache.org/r/13223/diff/
> 
> 
> Testing
> ---
> 
> Applied on 4.0.2 and 4.0.1
> 
> Testing On 4.0.1 From 20,June
> 3 nodes, create a VM on each node every 15 minutes. Destroy the VMs 5 minutes 
> later. expunge.inteval = 600 (10 minutes), expunge.worker = 2
>  
> 
> Testing On 4.0.2 From 01,July
> 2 nodes, create two VMs on each node every 5 minutes. Destroy the VMs 4 
> minutes later. expunge.inteval = 600 (10 minutes), expunge.worker = 2 
> 
> 
> Thanks,
> 
> Wei Zhou
> 
>



Re: [DISCUSS} Root/Domain Admin user should see ALL vms - based on their permisions

2013-08-09 Thread Daan Hoogland
My point is that all might be a lot and admin might want to filter
some classification he made using a naming convention, by typing it in
a filter box

On Thu, Aug 8, 2013 at 11:30 PM, Musayev, Ilya  wrote:
> Daan
>
> The point is to see everything ROOT Admin and Domain Admin are entitled to 
> see.
>
> They can see it anyway by scrolling through the projects, but its not getting 
> us - true - *all vms* view.
>
> I don't know if only *.mine* projects would achieve the objective.
>
> Regards
> ilya
>
>> -Original Message-
>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>> Sent: Thursday, August 08, 2013 4:38 PM
>> To: dev
>> Subject: Re: [DISCUSS} Root/Domain Admin user should see ALL vms - based
>> on their permisions
>>
>> maybe a column selectpr and/or a filter function; only projects matching
>> '.*mine.*'
>>
>> On Thu, Aug 8, 2013 at 10:03 PM, Musayev, Ilya 
>> wrote:
>> > One other challenge, if you have large number of VMs, the view is going to
>> be very long and some point hard to read. Perhaps we can add a filter
>> feature, such that user can choose how many VMs per page are presented,
>> i.e. 50, 100, 250, 500?
>> >
>> > This would apply to all UI Instance view pages.
>> >
>> >> -Original Message-
>> >> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>> >> Sent: Thursday, August 08, 2013 3:24 PM
>> >> To: dev@cloudstack.apache.org
>> >> Subject: Re: [DISCUSS} Root/Domain Admin user should see ALL vms -
>> >> based on their permisions
>> >>
>> >> +1
>> >>
>> >>
>> >> On Thu, Aug 8, 2013 at 12:59 PM, Chip Childers
>> >> wrote:
>> >>
>> >> > On Thu, Aug 08, 2013 at 06:54:02PM +, Musayev, Ilya wrote:
>> >> > > This question was asked several times on dev and user mailing list.
>> >> > >
>> >> > > I'd like to propose following enhancement and would like to get
>> >> > > the
>> >> > community feedback:
>> >> > >
>> >> > > Issue:
>> >> > >
>> >> > > Root Admin cannot see all VMs under Instance tab when "all"
>> >> > > instances
>> >> > are selected. Root Admin user need to switch between different
>> >> > projects to see more vms. This approach presents a challenge as
>> >> > there is no single view of all VMs in ALL projects. Same would
>> >> > apply to a domain admin, he should be able to see all vms in his
>> >> > domain at a glance without looping through each project.
>> >> > >
>> >> > > Suggested resolution:
>> >> > >
>> >> > > Change the behavior of show "all" instances by looping through
>> >> > > all
>> >> > projects and showing the vm names. We would also need to alter the
>> >> > Instance view and append project name column and append
>> >> domain/project.
>> >> > >
>> >> > > What do you think?
>> >> >
>> >> > +1, and I notice that Kirk J opened CLOUDSTACK-4196
>> >> >
>> >>
>> >>
>> >>
>> >> --
>> >> *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 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-09 Thread Gaurav Aradhye

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

(Updated Aug. 9, 2013, 7:51 a.m.)


Review request for cloudstack, sailaja mada, Sanjay Tripathi, and Prasanna 
Santhanam.


Changes
---

Automation: Limit Resources: Memory Limits - changes as per review


Repository: cloudstack-git


Description
---

Adding resource limit tests related to Memory. Changes suggested in CPU 
resource limit tests which are applicable here are incorporated too.

Updated test plan is available here: 
https://cwiki.apache.org/confluence/download/attachments/30757590/LimitResourcesTestPlanUpdate5.xlsx?version=1&modificationDate=1366952352000


Diffs (updated)
-

  test/integration/component/memory_limits/test_domain_limits.py PRE-CREATION 
  test/integration/component/memory_limits/test_maximum_limits.py PRE-CREATION 
  test/integration/component/memory_limits/test_memory_limits.py PRE-CREATION 
  test/integration/component/memory_limits/test_project_limits.py PRE-CREATION 
  tools/marvin/marvin/integration/lib/base.py 695edf7 
  tools/marvin/marvin/integration/lib/common.py 4f5acef 

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


Testing
---


Thanks,

Gaurav Aradhye



Re: Master build failure

2013-08-09 Thread Daan Hoogland
I have seen people ignoring a failure, but they had (good?) reason.
The build has been failing for other reason then their code.

another point is that the present implementation doesn't retry a
build, afaik. and it should retry after each new upload, not regarding
whether it failed or succeeded before.

Then we will encounter crossing patches at some point. I know I will
with this vpc thingy I want; two patches might build both on a given
HEAD , but not in combination :} I don't think we will fix that
scenario some time soon.

So I think it's what we all want, but still on the horizon.

On Fri, Aug 9, 2013 at 3:35 AM, Dave Cahill  wrote:
> Hi,
>
> Alex - sounds good, let's raise the Gerrit topic in a new thread after 4.2.
>
> Daan - you're right that if Jenkins builds, that should prove the code
> compiles cleanly.
> Have we actually been gating commits on the catofong report though? I looked
> at the
> Jenkins build behind it [1], and it hasn't succeeded in 4 days, so I'm
> guessing we're
> not gating based on it.
>
> Aside from the Jenkins integration, one of the reasons I wanted Gerrit is to
> put a step
> between writing code and it hitting master. Ideally, it would be nice to
> enforce that a
> positive code review from someone other than the author is present before
> pushing.
>
> Thanks,
> Dave.
>
> [1]
> http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/
>
>
> On Fri, Aug 9, 2013 at 4:16 AM, Daan Hoogland 
> wrote:
>>
>> Dave, wouldn't jenkins do a build like that? Meanng the positive cato
>> report is a must to allow commits
>>
>> On Thu, Aug 8, 2013 at 4:54 PM, Alex Huang  wrote:
>> > Yup.  That was the reason why it worked on my machine but doesn't work
>> > on jenkins.  It had a copy of the jar in the local maven cache so it
>> > continued to build even though it's missing.  Thanks for reverting.  Will
>> > fix.
>> >
>> > I still don't understand why we have problems with Gerrit.  It would
>> > have prevented exactly this type of problems.  I know a lot of people are
>> > busy on 4.2.  Will bring it up post 4.2 again.
>> >
>> > --Alex
>> >
>> >
>> >
>> > From: Dave Cahill [mailto:dcah...@midokura.com]
>> > Sent: Thursday, August 8, 2013 3:45 AM
>> > To: dev@cloudstack.apache.org; Dave Cahill; Alex Huang
>> > Subject: Re: Master build failure
>> >
>> > Thanks Prasanna.
>> >
>> > One suggestion I have is that if anyone makes pom file changes (or any
>> > large changes at
>> > all, to be honest), that they nuke their local Maven cache (rm -rf
>> > ~/.m2/repository) and do
>> > a "clean install" before submitting a patch. This way, you don't end up
>> > thinking your work
>> > builds when it just happens to build due to leftover, cached jars.
>> >
>> > Alex, do you have any ideas on build stability? There have been 2 build
>> > breakages in the
>> > past few days (ref: "compile error on 4.2" thread), and looking at git
>> > log I don't see much
>> > sign of review on the changes. We discussed Gerrit a while back, but I
>> > think the conclusion
>> > was that it would slow us down?
>> >
>> > Thanks,
>> > Dave.
>> >
>> > On Thu, Aug 8, 2013 at 6:13 PM, Prasanna Santhanam
>> > mailto:t...@apache.org>> wrote:
>> > Reverted the range 30c33415..f6a2c817bc. Master builds now. Others
>> > who've checked in after these will still retain their commits but
>> > please build before push. It's clear we are only worried about 4.2 now
>> > but we are breaking master for those who are beginning to work ahead
>> > for 4.3
>> >
>> > On Thu, Aug 08, 2013 at 02:15:58PM +0530, Prasanna Santhanam wrote:
>> >> Dave's been attempting to fix this and it looks to be intertwined in a
>> >> few commits. Large swaths of code has moved around. I'll revert to
>> >> last known state from Jenkins if we can't figure out a simple fix so
>> >> Dave can proceed.
>> >>
>> >> On Thu, Aug 08, 2013 at 04:11:36PM +0900, Dave Cahill wrote:
>> >> > Hi,
>> >> >
>> >> > As Prasanna noted in IRC, the cloudstack-master-maven Jenkins build
>> >> > is
>> >> > failing with a similar error:
>> >> >
>> >> > https://builds.apache.org/job/cloudstack-master-maven/2645/console
>> >> >
>> >> > Failed to execute goal on project cloud-engine-components-api: Could
>> >> > not resolve dependencies for project
>> >> > org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
>> >> > Could not find artifact
>> >> > org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
>> >> > Nexus (http://repository.apache.org/snapshots)
>> >> > cause : Could not resolve dependencies for project
>> >> > org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
>> >> > Could not find artifact
>> >> > org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
>> >> > Nexus (http://repository.apache.org/snapshots)
>> >> >
>> >> >
>> >> > Thanks,
>> >> >
>> >> > Dave.
>> >> >
>> >> >
>> >> > On Thu, Aug 8, 2013 at 3:53 PM, Dave Cahill
>> >> > mailto:dcah...@midokura.com>> wrote:
>> >> >
>> >> > > Hi,
>> 

Re: Review Request 13428: CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is inconsistent on the Upgraded Setup when compared with the fresh 4.2 Installation.

2013-08-09 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On Aug. 8, 2013, 12:25 p.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13428/
> ---
> 
> (Updated Aug. 8, 2013, 12:25 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4187
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is 
> inconsistent on the Upgraded Setup when compared with the fresh 4.2 
> Installation.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-4187
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  bb7f340 
>   setup/db/db/schema-410to420.sql 8e2feda 
>   setup/db/templates.sql e030852 
> 
> Diff: https://reviews.apache.org/r/13428/diff/
> 
> 
> Testing
> ---
> 
> Verified that the deployment of "Windows server 2003 enterprise edition" and 
> "Windows server 2000 sp4" in Xenserver 5.6FP1 is in PV mode.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: [DISCUSS] KVM io throttling

2013-08-09 Thread Wido den Hollander



On 08/09/2013 02:05 AM, Edison Su wrote:

Hi All,
  For basic network, the io limit for guest network(nic 2) is 200MB/s, we 
set the io limit on kvm host as:



I think you mean 200Mbit/s?














 From libvirt doc(http://libvirt.org/formatnetwork.html), it says above unit is 
KB/s, but per my testing, it sounds like it's Kb/s. Is there anybody have the 
same issue?



No, my testing tells me it's is KB/sec (KiloByte).


My libvirt version is 0.9.10 on Centos 6.3.



My libvirt version is 1.0.6 under Ubuntu 12.04 using the Cloud Archive 
for OpenStack Havana from Canonical: 
https://wiki.ubuntu.com/ServerTeam/CloudArchive


Wido


Re: Review Request 13428: CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is inconsistent on the Upgraded Setup when compared with the fresh 4.2 Installation.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit c18492509692d590aa72133a3974512c7dff119e in branch refs/heads/4.2 from 
Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c184925 ]

CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is 
inconsistent on the Upgraded Setup when compared with the fresh 4.2 
Installation.


- ASF Subversion and Git Services


On Aug. 8, 2013, 12:25 p.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13428/
> ---
> 
> (Updated Aug. 8, 2013, 12:25 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4187
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is 
> inconsistent on the Upgraded Setup when compared with the fresh 4.2 
> Installation.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-4187
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  bb7f340 
>   setup/db/db/schema-410to420.sql 8e2feda 
>   setup/db/templates.sql e030852 
> 
> Diff: https://reviews.apache.org/r/13428/diff/
> 
> 
> Testing
> ---
> 
> Verified that the deployment of "Windows server 2003 enterprise edition" and 
> "Windows server 2000 sp4" in Xenserver 5.6FP1 is in PV mode.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13431: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 8f3be62cbe11fb71e817d2e538de8c7d4e9bcbd8 in branch refs/heads/4.2 from 
Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=8f3be62 ]

CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.


- ASF Subversion and Git Services


On Aug. 9, 2013, 6:27 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13431/
> ---
> 
> (Updated Aug. 9, 2013, 6:27 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-3842
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3842
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja.properties 0c61c13 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 496c2e9 
>   ui/css/cloudstack3.css afebbd1 
> 
> Diff: https://reviews.apache.org/r/13431/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13431: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread Devdeep Singh

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


Can you submit an updated patch to apply it on master. Patch applied 
successfully on 4.2 but is failing on master.

- Devdeep Singh


On Aug. 9, 2013, 6:27 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13431/
> ---
> 
> (Updated Aug. 9, 2013, 6:27 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-3842
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3842
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja.properties 0c61c13 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 496c2e9 
>   ui/css/cloudstack3.css afebbd1 
> 
> Diff: https://reviews.apache.org/r/13431/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13428: CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is inconsistent on the Upgraded Setup when compared with the fresh 4.2 Installation.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit d6ebcd1924ad6fb0638a22386c921399fc68ae2d in branch refs/heads/master 
from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d6ebcd1 ]

CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is 
inconsistent on the Upgraded Setup when compared with the fresh 4.2 
Installation.


- ASF Subversion and Git Services


On Aug. 8, 2013, 12:25 p.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13428/
> ---
> 
> (Updated Aug. 8, 2013, 12:25 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-4187
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is 
> inconsistent on the Upgraded Setup when compared with the fresh 4.2 
> Installation.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-4187
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  bb7f340 
>   setup/db/db/schema-410to420.sql 8e2feda 
>   setup/db/templates.sql e030852 
> 
> Diff: https://reviews.apache.org/r/13428/diff/
> 
> 
> Testing
> ---
> 
> Verified that the deployment of "Windows server 2003 enterprise edition" and 
> "Windows server 2000 sp4" in Xenserver 5.6FP1 is in PV mode.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Review Request 13438: CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

2013-08-09 Thread Sanjay Tripathi

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

Review request for cloudstack, Abhinandan Prateek, Devdeep Singh, and Likitha 
Shetty.


Bugs: CLOUDSTACK-2448


Repository: cloudstack-git


Description
---

CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

Updated the guestOS support mapping of all the XenServer versions and also 
updated the mapping for XCP 1.0 and XCP 1.6


Diffs
-

  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java 
eec957b 

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


Testing
---

Verified the fix locally by deploying the VM for newly added guestOS.


Thanks,

Sanjay Tripathi



Review Request 13439: CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and ".

2013-08-09 Thread Sanjay Tripathi

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

Review request for cloudstack, Devdeep Singh and Likitha Shetty.


Bugs: CLOUDSTACK-2909


Repository: cloudstack-git


Description
---

CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "


Diffs
-

  services/console-proxy/server/js/ajaxkeys.js 4e4d775 

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


Testing
---

Verified the fix locally on cloudstack setup.


Thanks,

Sanjay Tripathi



Review Request 13440: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread Sanjay Tripathi

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

Review request for cloudstack, Devdeep Singh and Likitha Shetty.


Bugs: CLOUDSTACK-3842


Repository: cloudstack-git


Description
---

CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.


Diffs
-

  client/WEB-INF/classes/resources/messages_ja.properties 6970058 
  client/WEB-INF/classes/resources/messages_zh_CN.properties 2ea262d 
  ui/css/cloudstack3.css afebbd1 

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


Testing
---

Verified the fix locally on cloudstack setup.


Thanks,

Sanjay Tripathi



Re: Review Request 13431: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread Sanjay Tripathi


> On Aug. 9, 2013, 9:56 a.m., Devdeep Singh wrote:
> > Can you submit an updated patch to apply it on master. Patch applied 
> > successfully on 4.2 but is failing on master.

I have sent a seperate request for master branch, closing this one.


- Sanjay


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


On Aug. 9, 2013, 6:27 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13431/
> ---
> 
> (Updated Aug. 9, 2013, 6:27 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-3842
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3842
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja.properties 0c61c13 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 496c2e9 
>   ui/css/cloudstack3.css afebbd1 
> 
> Diff: https://reviews.apache.org/r/13431/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13438: CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

2013-08-09 Thread Devdeep Singh

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

Ship it!


Ship It!

- Devdeep Singh


On Aug. 9, 2013, 11:07 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13438/
> ---
> 
> (Updated Aug. 9, 2013, 11:07 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Devdeep Singh, and Likitha 
> Shetty.
> 
> 
> Bugs: CLOUDSTACK-2448
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.
> 
> Updated the guestOS support mapping of all the XenServer versions and also 
> updated the mapping for XCP 1.0 and XCP 1.6
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  eec957b 
> 
> Diff: https://reviews.apache.org/r/13438/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally by deploying the VM for newly added guestOS.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13424: CLOUDSTACK-4123: 3.0.6 to ASF 4.2 Upgrade: On Adding VMWare Cluster to the Setup, 3.0.6 System VM Template for VMWare is downloaded instead of 4.2

2013-08-09 Thread Harikrishna Patnala

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

(Updated Aug. 9, 2013, 11:50 a.m.)


Review request for cloudstack and Kishan Kavala.


Bugs: CLOUDSTACK-4123 and CLOUDSTACK-4205


Repository: cloudstack-git


Description (updated)
---

CLOUDSTACK-4123: 3.0.6 to ASF 4.2 Upgrade: On Adding VMWare Cluster to the 
Setup, 3.0.6 System VM Template for VMWare is downloaded instead of 4.2

Updating the new system template URLs for the existing templates during 
upgrade to 4.2.
If new 4.2 system template is registered before upgrade then marking the 
old templates as removed during upgrade.

CLOUDSTACK-4205: System VMs are not booting up with new templates after Upgrade 
from 3.0.6 to 4.2. 


Diffs (updated)
-

  engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 9243225 
  engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 0941408 
  server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java a583954 
  server/src/com/cloud/resource/ResourceManagerImpl.java e5441af 
  server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 
17cbf5c 

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


Testing
---


Thanks,

Harikrishna Patnala



Re: [DISCUSS] KVM io throttling

2013-08-09 Thread Wei ZHOU
My testing result is same to what Wido mentioned.



2013/8/9 Edison Su 

> Hi All,
>  For basic network, the io limit for guest network(nic 2) is 200MB/s,
> we set the io limit on kvm host as:
>
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
>
>
> From libvirt doc(http://libvirt.org/formatnetwork.html), it says above
> unit is KB/s, but per my testing, it sounds like it's Kb/s. Is there
> anybody have the same issue?
>
> My libvirt version is 0.9.10 on Centos 6.3.
>


Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-09 Thread Harikrishna Patnala

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

Review request for cloudstack and Koushik Das.


Bugs: CLOUDSTACK-3850


Repository: cloudstack-git


Description
---

CLOUDSTACK-3850:  CPU cap should be per VM not per VCPU 


Diffs
-

  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 e100211 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java
 d230be1 

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


Testing
---


Thanks,

Harikrishna Patnala



OVS

2013-08-09 Thread Thomas Schneider
Hello

I try to setup Cloudstack with OpenVSwitch, but SSVM and the proxy vm
can't reach internet.
I think it's a problem with cloud0 bridge in OVS.

My config is:

/etc/cloudstack/agent/agent.properties
network.bridge.type=openvswitch
libvirt.vif.driver=com.cloud.hypervisor.kvm.resource.OvsVifDriver

OVS
ovs-vsctl add-br cloudbr0
ovs-vsctl add-port cloudbr0 p49p1
ovs-vsctl add-br cloudbr1
ovs-vsctl add-port cloudbr1 p50p1

ovs-vsctl show
0771f39e-6e58-4563-b35e-c891197c858f
Bridge "cloud0"
Port "vnet0"
Interface "vnet0"
Port "cloud0"
Interface "cloud0"
type: internal
Port "vnet4"
Interface "vnet4"
Bridge "cloudbr1"
Port "cloudbr1"
Interface "cloudbr1"
type: internal
Port "p50p1"
Interface "p50p1"
Bridge "cloudbr0"
Port "vnet2"
Interface "vnet2"
Port "p49p1"
Interface "p49p1"
Port "vnet1"
Interface "vnet1"
Port "cloudbr0"
Interface "cloudbr0"
type: internal
Port "vnet3"
Interface "vnet3"
Port "vnet5"
Interface "vnet5"
Port "vnet6"
Interface "vnet6"
ovs_version: "1.9.0"

/etc/network/interfaces

# The primary network interface
auto p49p1
iface p49p1 inet manual

# The secondary network interface
auto p50p1
iface p50p1 inet manual

# Public network
auto cloudbr0
iface cloudbr0 inet static
bridge_ports p49p1
bridge_fd 5
bridge_stp off
bridge_maxwait 1
address 10.10.10.30
netmask 255.255.255.0
network 10.10.10.0
broadcast 10.10.10.255
gateway 10.10.10.254

auto cloudbr1
iface cloudbr1 inet manual
bridge_ports p50p1
bridge_fd 5
bridge_stp off
bridge_maxwait 1

THX
-- 
*Thomas Schneider*



Re: Review Request 13439: CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and ".

2013-08-09 Thread Likitha Shetty

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

Ship it!


Ship It!

- Likitha Shetty


On Aug. 9, 2013, 11:16 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13439/
> ---
> 
> (Updated Aug. 9, 2013, 11:16 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Likitha Shetty.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js 4e4d775 
> 
> Diff: https://reviews.apache.org/r/13439/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13438: CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit f168df3472b2b8415dcc397f2fdb5c9ef05f48fd in branch refs/heads/4.2 from 
Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=f168df3 ]

CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:07 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13438/
> ---
> 
> (Updated Aug. 9, 2013, 11:07 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Devdeep Singh, and Likitha 
> Shetty.
> 
> 
> Bugs: CLOUDSTACK-2448
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.
> 
> Updated the guestOS support mapping of all the XenServer versions and also 
> updated the mapping for XCP 1.0 and XCP 1.6
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  eec957b 
> 
> Diff: https://reviews.apache.org/r/13438/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally by deploying the VM for newly added guestOS.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13439: CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and ".

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 822991a0e94b8128082e0cf884f00049be0794ae in branch refs/heads/4.2 from 
Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=822991a ]

CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:16 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13439/
> ---
> 
> (Updated Aug. 9, 2013, 11:16 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Likitha Shetty.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js 4e4d775 
> 
> Diff: https://reviews.apache.org/r/13439/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13440: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread Likitha Shetty

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

Ship it!


Ship It!

- Likitha Shetty


On Aug. 9, 2013, 11:32 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13440/
> ---
> 
> (Updated Aug. 9, 2013, 11:32 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Likitha Shetty.
> 
> 
> Bugs: CLOUDSTACK-3842
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja.properties 6970058 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 2ea262d 
>   ui/css/cloudstack3.css afebbd1 
> 
> Diff: https://reviews.apache.org/r/13440/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13438: CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit d95e3dacd6ab6dba3d6e3bff0b0b9b0dc462648a in branch refs/heads/master 
from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d95e3da ]

CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:07 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13438/
> ---
> 
> (Updated Aug. 9, 2013, 11:07 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Devdeep Singh, and Likitha 
> Shetty.
> 
> 
> Bugs: CLOUDSTACK-2448
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.
> 
> Updated the guestOS support mapping of all the XenServer versions and also 
> updated the mapping for XCP 1.0 and XCP 1.6
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixHelper.java
>  eec957b 
> 
> Diff: https://reviews.apache.org/r/13438/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally by deploying the VM for newly added guestOS.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13439: CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and ".

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 7c708e00d24d1b7d79675e9044ffcea96a58a54e in branch refs/heads/master 
from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=7c708e0 ]

CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:16 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13439/
> ---
> 
> (Updated Aug. 9, 2013, 11:16 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Likitha Shetty.
> 
> 
> Bugs: CLOUDSTACK-2909
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2909: Fix for EN-US keys. Keys are ' and "
> 
> 
> Diffs
> -
> 
>   services/console-proxy/server/js/ajaxkeys.js 4e4d775 
> 
> Diff: https://reviews.apache.org/r/13439/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



Re: Review Request 13440: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 5c36766ad4f96fe2e8096f9519568deb9dc8c975 in branch refs/heads/master 
from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5c36766 ]

CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:32 a.m., Sanjay Tripathi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13440/
> ---
> 
> (Updated Aug. 9, 2013, 11:32 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and Likitha Shetty.
> 
> 
> Bugs: CLOUDSTACK-3842
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja.properties 6970058 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 2ea262d 
>   ui/css/cloudstack3.css afebbd1 
> 
> Diff: https://reviews.apache.org/r/13440/diff/
> 
> 
> Testing
> ---
> 
> Verified the fix locally on cloudstack setup.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>



RE: [DISCUSS] Binary diff and hot patch

2013-08-09 Thread Musayev, Ilya
I guess no way to do this :(

> -Original Message-
> From: Musayev, Ilya [mailto:imusa...@webmd.net]
> Sent: Wednesday, August 07, 2013 5:51 PM
> To: dev@cloudstack.apache.org
> Subject: [DISCUSS] Binary diff and hot patch
> 
> I was wondering if we can use binary diffs to hotpatch ACS.
> 
> Currently, the way for me to identify what patch could have caused issue X,
> would be  - try to find the suspects - it could be several. Revert 1 commit,
> rebuild and retry, if failed, restore previously reverted commit, revert 2nd
> commit, rebuild and retry - etc..
> 
> As you can see, this is very time consuming and painful, I'm wondering if
> there is a better way to do this. I came across a project called javaxdelta on
> sourceforge - curious what your thought on this may be, and if there is a
> better way to try out patches instead of what I'm doing now.
> 
> Thanks
> ilya



resourcecount.check.interval

2013-08-09 Thread Jeronimo Garcia
Hi List.

Today a number of my hosts stop reporting disk space free , and therefore
VMS will not launch on the given host due to space allocation exceptions.

I knew i needed to force resources recalculation somehow , so I've changed
this:
resourcecount.check.interval = 60

and that seemed to do the trick ,  can somebody explain this a little bit
deeper than the brief on global settings?

Thanks


Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 6b905528559f1f1b6ae472edc550c31d59e8461d in branch refs/heads/4.1 from 
Rajesh Battala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6b90552 ]

CLOUDSTACK-3215 Logs in the Software router are not being rotated


- ASF Subversion and Git Services


On June 28, 2013, 7:46 p.m., Donal Lafferty wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12131/
> ---
> 
> (Updated June 28, 2013, 7:46 p.m.)
> 
> 
> Review request for cloudstack, edison su and Min Chen.
> 
> 
> Bugs: CLOUDSTACK-3215
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix https://issues.apache.org/jira/browse/CLOUDSTACK-3215 by changing code to 
> not use a cache for image transfer if one can't be found.  Previously, the 
> management server entered a failure state.
> Also, added addition debug logging.
> 
> 
> Diffs
> -
> 
>   
> engine/storage/cache/src/org/apache/cloudstack/storage/cache/manager/StorageCacheManagerImpl.java
>  4b4e52106ffbf70bcf2f6a656a8b8e4cacd6f91e 
>   
> engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java
>  631de6a47a3eff510c84aa275fd87f8fa2f7780b 
> 
> Diff: https://reviews.apache.org/r/12131/diff/
> 
> 
> Testing
> ---
> 
> Code executed on deployement using S3 and no NFS cache.  Did not have 
> facilities to test on S3 with a cache. 
> 
> 
> Thanks,
> 
> Donal Lafferty
> 
>



Re: resourcecount.check.interval

2013-08-09 Thread Jeronimo Garcia
the updateresourcecount api call seem interesting when u combine it with
these resource ids:
 56 @Parameter(name=ApiConstants.RESOURCE_TYPE,
type=CommandType.INTEGER, description=  "Type of resource to update. If
specifies valid values are 0, 1, 2, 3, 4, 5, 6, 7, 8 and 9. If not
specified will update all resource counts" +
 57
"0 - Instance. Number of instances a user can create. " +
 58
"1 - IP. Number of public IP addresses a user can own. " +
 59
"2 - Volume. Number of disk volumes a user can create." +
 60
"3 - Snapshot. Number of snapshots a user can create." +
 61
"4 - Template. Number of templates that a user can
register/create." +
 62
"5 - Project. Number of projects that a user can create." +
 63
"6 - Network. Number of guest network a user can create." +
 64
"7 - VPC. Number of VPC a user can create." +
 65
"8 - CPU. Total number of CPU cores a user can use." +
 66
"9 - Memory. Total Memory (in MB) a user can use." )


On Fri, Aug 9, 2013 at 11:14 AM, Jeronimo Garcia wrote:

> Hi List.
>
> Today a number of my hosts stop reporting disk space free , and therefore
> VMS will not launch on the given host due to space allocation exceptions.
>
> I knew i needed to force resources recalculation somehow , so I've changed
> this:
> resourcecount.check.interval = 60
>
> and that seemed to do the trick ,  can somebody explain this a little bit
> deeper than the brief on global settings?
>
> Thanks
>


Re: OVS

2013-08-09 Thread Chiradeep Vittal
Is this 4.1/4.2/master?


On 8/9/13 6:13 AM, "Thomas Schneider"  wrote:

>Hello
>
>I try to setup Cloudstack with OpenVSwitch, but SSVM and the proxy vm
>can't reach internet.
>I think it's a problem with cloud0 bridge in OVS.
>
>My config is:
>
>/etc/cloudstack/agent/agent.properties
>network.bridge.type=openvswitch
>libvirt.vif.driver=com.cloud.hypervisor.kvm.resource.OvsVifDriver
>
>OVS
>ovs-vsctl add-br cloudbr0
>ovs-vsctl add-port cloudbr0 p49p1
>ovs-vsctl add-br cloudbr1
>ovs-vsctl add-port cloudbr1 p50p1
>
>ovs-vsctl show
>0771f39e-6e58-4563-b35e-c891197c858f
>Bridge "cloud0"
>Port "vnet0"
>Interface "vnet0"
>Port "cloud0"
>Interface "cloud0"
>type: internal
>Port "vnet4"
>Interface "vnet4"
>Bridge "cloudbr1"
>Port "cloudbr1"
>Interface "cloudbr1"
>type: internal
>Port "p50p1"
>Interface "p50p1"
>Bridge "cloudbr0"
>Port "vnet2"
>Interface "vnet2"
>Port "p49p1"
>Interface "p49p1"
>Port "vnet1"
>Interface "vnet1"
>Port "cloudbr0"
>Interface "cloudbr0"
>type: internal
>Port "vnet3"
>Interface "vnet3"
>Port "vnet5"
>Interface "vnet5"
>Port "vnet6"
>Interface "vnet6"
>ovs_version: "1.9.0"
>
>/etc/network/interfaces
>
># The primary network interface
>auto p49p1
>iface p49p1 inet manual
>
># The secondary network interface
>auto p50p1
>iface p50p1 inet manual
>
># Public network
>auto cloudbr0
>iface cloudbr0 inet static
>bridge_ports p49p1
>bridge_fd 5
>bridge_stp off
>bridge_maxwait 1
>address 10.10.10.30
>netmask 255.255.255.0
>network 10.10.10.0
>broadcast 10.10.10.255
>gateway 10.10.10.254
>
>auto cloudbr1
>iface cloudbr1 inet manual
>bridge_ports p50p1
>bridge_fd 5
>bridge_stp off
>bridge_maxwait 1
>
>THX
>-- 
>*Thomas Schneider*
>



Re: OVS

2013-08-09 Thread Thomas Schneider
this is 4.1

Le 09/08/2013 18:47, Chiradeep Vittal a écrit :
> Is this 4.1/4.2/master?
>
>
> On 8/9/13 6:13 AM, "Thomas Schneider"  wrote:
>
>> Hello
>>
>> I try to setup Cloudstack with OpenVSwitch, but SSVM and the proxy vm
>> can't reach internet.
>> I think it's a problem with cloud0 bridge in OVS.
>>
>> My config is:
>>
>> /etc/cloudstack/agent/agent.properties
>> network.bridge.type=openvswitch
>> libvirt.vif.driver=com.cloud.hypervisor.kvm.resource.OvsVifDriver
>>
>> OVS
>> ovs-vsctl add-br cloudbr0
>> ovs-vsctl add-port cloudbr0 p49p1
>> ovs-vsctl add-br cloudbr1
>> ovs-vsctl add-port cloudbr1 p50p1
>>
>> ovs-vsctl show
>> 0771f39e-6e58-4563-b35e-c891197c858f
>>Bridge "cloud0"
>>Port "vnet0"
>>Interface "vnet0"
>>Port "cloud0"
>>Interface "cloud0"
>>type: internal
>>Port "vnet4"
>>Interface "vnet4"
>>Bridge "cloudbr1"
>>Port "cloudbr1"
>>Interface "cloudbr1"
>>type: internal
>>Port "p50p1"
>>Interface "p50p1"
>>Bridge "cloudbr0"
>>Port "vnet2"
>>Interface "vnet2"
>>Port "p49p1"
>>Interface "p49p1"
>>Port "vnet1"
>>Interface "vnet1"
>>Port "cloudbr0"
>>Interface "cloudbr0"
>>type: internal
>>Port "vnet3"
>>Interface "vnet3"
>>Port "vnet5"
>>Interface "vnet5"
>>Port "vnet6"
>>Interface "vnet6"
>>ovs_version: "1.9.0"
>>
>> /etc/network/interfaces
>>
>> # The primary network interface
>> auto p49p1
>> iface p49p1 inet manual
>>
>> # The secondary network interface
>> auto p50p1
>> iface p50p1 inet manual
>>
>> # Public network
>> auto cloudbr0
>> iface cloudbr0 inet static
>>bridge_ports p49p1
>>bridge_fd 5
>>bridge_stp off
>>bridge_maxwait 1
>>address 10.10.10.30
>>netmask 255.255.255.0
>>network 10.10.10.0
>>broadcast 10.10.10.255
>>gateway 10.10.10.254
>>
>> auto cloudbr1
>> iface cloudbr1 inet manual
>>bridge_ports p50p1
>>bridge_fd 5
>>bridge_stp off
>>bridge_maxwait 1
>>
>> THX
>> -- 
>> *Thomas Schneider*
>>


-- 
*Thomas Schneider*
Directeur des Opérations
Euskill SARL
Web: www.euskill.com
Mobile: +33 (0)6 19 26 47 76
Mail: thomas.schnei...@euskill.com
5 rue de Phalsbourg
F-67000 Strasbourg


Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-09 Thread Toshiaki Hatano

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



api/src/com/cloud/network/Networks.java


Why don't we merge getValueFrom() and getValue()?

public String getValue(URI uri) {
if(uri.isOpaque()) {
return uri.getSchemeSpecificPart();
} else {
return uri.getHost();
}
}


- Toshiaki Hatano


On Aug. 8, 2013, 11:51 a.m., daan Hoogland wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12849/
> ---
> 
> (Updated Aug. 8, 2013, 11:51 a.m.)
> 
> 
> Review request for cloudstack, Chiradeep Vittal, Dave Cahill, Koushik Das, 
> and Sheng Yang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Both BroadcastDomainType and IsolationType needed some extra code for 
> backwards compatibility.
> 
> All over the code calls are done to URI.getHost() to retrieve ids of 
> broadcastdomains. These id obviously are not hosts so this call is confusing 
> and requires maintenance all over the code base. Also for different types the 
> value returned by getHost has different meaning. vlan://1 is id 1 of course 
> but others might be ranges of vlans or colon separated values. To make things 
> worse a NiciraNvp has an uri of the form lswitch: without the forward 
> slashes.
> 
> To make the system more maintainable in this perspect the changes in this 
> patch were made. It is my intention to replace the calls to getHost by the 
> member call getValueFrom or the static method getValue in time. In this way 
> maintenance is centralized and an overview of differnces and quirks is easily 
> found
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Networks.java c76c3d4 
>   api/test/com/cloud/network/NetworksTest.java 31114e8 
> 
> Diff: https://reviews.apache.org/r/12849/diff/
> 
> 
> Testing
> ---
> 
> unit tests for different kind of BroadcastDomainType.values
> 
> 
> Thanks,
> 
> daan Hoogland
> 
>



RE: Quick Question about System VM problem

2013-08-09 Thread Soheil Eizadi
I seem to remember that the command line options were different and there are 
calls had explicit hard coded path to /usr/bin/vhd-util that were failing if I 
placed the new vhd-util in that location. When we bring a new XenServer online 
and download to /opt/xensource, I don't think we remove the old vhd-util, so I 
thought this was by design meant to work this way.
-Soheil

From: Sanjay Tripathi [sanjay.tripa...@citrix.com]
Sent: Thursday, August 08, 2013 11:36 PM
To: dev@cloudstack.apache.org
Subject: RE: Quick Question about System VM problem

I don't think you need two versions of vhd-util. If you are adding new 
XenServer host, just
you need to copy the existing vhd-util from /usr/sbin to its correct locations 
and it works fine.

Can you share what are the issues you are getting if you use the existing 
vhd-util in XenServer host.

--Sanjay

> -Original Message-
> From: Soheil Eizadi [mailto:seiz...@infoblox.com]
> Sent: Friday, August 09, 2013 11:37 AM
> To: dev@cloudstack.apache.org
> Subject: RE: Quick Question about System VM problem
>
> My experience was that you had to keep two versions of it on the
> XenServer, the original one (/usr/sbin/vhd-util) and download a new one
> that ends up in /opt/xensource/bin/vhd-util -Soheil
> 
> From: Prasanna Santhanam [t...@apache.org]
> Sent: Thursday, August 08, 2013 9:22 PM
> To: dev@cloudstack.apache.org
> Subject: Re: Quick Question about System VM problem
>
> On Thu, Aug 08, 2013 at 10:25:28PM +, Donal Lafferty wrote:
> > There's a tag that is applied to you XenServer.  AFAIK, the tag tells
> > the management server not to download systemvm.iso
> >
> > Also, be careful not to overwrite the vhd-utils on a XenServer version
> > 6.x and beyond.  The vhd-util mentioned on the build from master page
> > does not work with new versions of XenServer.  It should be updated
> > soon.
>
> What is the issue with the vhd-util on download.cloud.com? Can you please
> file a bug? I've been using the vhd-util on download.cloud.com and haven't
> faced any issues with my Xen 6.0.2 hosts.
>
>
> --
> Prasanna.,
>
> 
> Powered by BigRock.com



Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-09 Thread Toshiaki Hatano


> On Aug. 9, 2013, 5:23 p.m., Toshiaki Hatano wrote:
> > api/src/com/cloud/network/Networks.java, line 172
> > 
> >
> > Why don't we merge getValueFrom() and getValue()?
> > 
> > public String getValue(URI uri) {
> > if(uri.isOpaque()) {
> > return uri.getSchemeSpecificPart();
> > } else {
> > return uri.getHost();
> > }
> > }

Or, maybe make getValueFrom private.

I think it will make confuse if we provide 2 public method to get value of URI.


- Toshiaki


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


On Aug. 8, 2013, 11:51 a.m., daan Hoogland wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12849/
> ---
> 
> (Updated Aug. 8, 2013, 11:51 a.m.)
> 
> 
> Review request for cloudstack, Chiradeep Vittal, Dave Cahill, Koushik Das, 
> and Sheng Yang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Both BroadcastDomainType and IsolationType needed some extra code for 
> backwards compatibility.
> 
> All over the code calls are done to URI.getHost() to retrieve ids of 
> broadcastdomains. These id obviously are not hosts so this call is confusing 
> and requires maintenance all over the code base. Also for different types the 
> value returned by getHost has different meaning. vlan://1 is id 1 of course 
> but others might be ranges of vlans or colon separated values. To make things 
> worse a NiciraNvp has an uri of the form lswitch: without the forward 
> slashes.
> 
> To make the system more maintainable in this perspect the changes in this 
> patch were made. It is my intention to replace the calls to getHost by the 
> member call getValueFrom or the static method getValue in time. In this way 
> maintenance is centralized and an overview of differnces and quirks is easily 
> found
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Networks.java c76c3d4 
>   api/test/com/cloud/network/NetworksTest.java 31114e8 
> 
> Diff: https://reviews.apache.org/r/12849/diff/
> 
> 
> Testing
> ---
> 
> unit tests for different kind of BroadcastDomainType.values
> 
> 
> Thanks,
> 
> daan Hoogland
> 
>



Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-09 Thread Toshiaki Hatano


> On Aug. 9, 2013, 5:23 p.m., Toshiaki Hatano wrote:
> > api/src/com/cloud/network/Networks.java, line 172
> > 
> >
> > Why don't we merge getValueFrom() and getValue()?
> > 
> > public String getValue(URI uri) {
> > if(uri.isOpaque()) {
> > return uri.getSchemeSpecificPart();
> > } else {
> > return uri.getHost();
> > }
> > }
> 
> Toshiaki Hatano wrote:
> Or, maybe make getValueFrom private.
> 
> I think it will make confuse if we provide 2 public method to get value 
> of URI.

Sorry I missed the point, getValueFrom is static method.

Please disregard and close this issue.


- Toshiaki


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


On Aug. 8, 2013, 11:51 a.m., daan Hoogland wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12849/
> ---
> 
> (Updated Aug. 8, 2013, 11:51 a.m.)
> 
> 
> Review request for cloudstack, Chiradeep Vittal, Dave Cahill, Koushik Das, 
> and Sheng Yang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Both BroadcastDomainType and IsolationType needed some extra code for 
> backwards compatibility.
> 
> All over the code calls are done to URI.getHost() to retrieve ids of 
> broadcastdomains. These id obviously are not hosts so this call is confusing 
> and requires maintenance all over the code base. Also for different types the 
> value returned by getHost has different meaning. vlan://1 is id 1 of course 
> but others might be ranges of vlans or colon separated values. To make things 
> worse a NiciraNvp has an uri of the form lswitch: without the forward 
> slashes.
> 
> To make the system more maintainable in this perspect the changes in this 
> patch were made. It is my intention to replace the calls to getHost by the 
> member call getValueFrom or the static method getValue in time. In this way 
> maintenance is centralized and an overview of differnces and quirks is easily 
> found
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/Networks.java c76c3d4 
>   api/test/com/cloud/network/NetworksTest.java 31114e8 
> 
> Diff: https://reviews.apache.org/r/12849/diff/
> 
> 
> Testing
> ---
> 
> unit tests for different kind of BroadcastDomainType.values
> 
> 
> Thanks,
> 
> daan Hoogland
> 
>



RE: Quick Question about System VM problem

2013-08-09 Thread Alex Huang
Yes.  Do not replace the original vhd-util.  

--Alex

> -Original Message-
> From: Soheil Eizadi [mailto:seiz...@infoblox.com]
> Sent: Friday, August 9, 2013 10:30 AM
> To: dev@cloudstack.apache.org
> Subject: RE: Quick Question about System VM problem
> 
> I seem to remember that the command line options were different and
> there are calls had explicit hard coded path to /usr/bin/vhd-util that were
> failing if I placed the new vhd-util in that location. When we bring a new
> XenServer online and download to /opt/xensource, I don't think we remove
> the old vhd-util, so I thought this was by design meant to work this way.
> -Soheil
> 
> From: Sanjay Tripathi [sanjay.tripa...@citrix.com]
> Sent: Thursday, August 08, 2013 11:36 PM
> To: dev@cloudstack.apache.org
> Subject: RE: Quick Question about System VM problem
> 
> I don't think you need two versions of vhd-util. If you are adding new
> XenServer host, just you need to copy the existing vhd-util from /usr/sbin to
> its correct locations and it works fine.
> 
> Can you share what are the issues you are getting if you use the existing vhd-
> util in XenServer host.
> 
> --Sanjay
> 
> > -Original Message-
> > From: Soheil Eizadi [mailto:seiz...@infoblox.com]
> > Sent: Friday, August 09, 2013 11:37 AM
> > To: dev@cloudstack.apache.org
> > Subject: RE: Quick Question about System VM problem
> >
> > My experience was that you had to keep two versions of it on the
> > XenServer, the original one (/usr/sbin/vhd-util) and download a new
> > one that ends up in /opt/xensource/bin/vhd-util -Soheil
> > 
> > From: Prasanna Santhanam [t...@apache.org]
> > Sent: Thursday, August 08, 2013 9:22 PM
> > To: dev@cloudstack.apache.org
> > Subject: Re: Quick Question about System VM problem
> >
> > On Thu, Aug 08, 2013 at 10:25:28PM +, Donal Lafferty wrote:
> > > There's a tag that is applied to you XenServer.  AFAIK, the tag
> > > tells the management server not to download systemvm.iso
> > >
> > > Also, be careful not to overwrite the vhd-utils on a XenServer
> > > version 6.x and beyond.  The vhd-util mentioned on the build from
> > > master page does not work with new versions of XenServer.  It should
> > > be updated soon.
> >
> > What is the issue with the vhd-util on download.cloud.com? Can you
> > please file a bug? I've been using the vhd-util on download.cloud.com
> > and haven't faced any issues with my Xen 6.0.2 hosts.
> >
> >
> > --
> > Prasanna.,
> >
> > 
> > Powered by BigRock.com



RE: [DISCUSS} Root/Domain Admin user should see ALL vms - based on their permisions

2013-08-09 Thread Musayev, Ilya
I see your point and I agree if we can implement flexible filtering!

> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Friday, August 09, 2013 3:07 AM
> To: dev
> Subject: Re: [DISCUSS} Root/Domain Admin user should see ALL vms - based
> on their permisions
> 
> My point is that all might be a lot and admin might want to filter some
> classification he made using a naming convention, by typing it in a filter box
> 
> On Thu, Aug 8, 2013 at 11:30 PM, Musayev, Ilya 
> wrote:
> > Daan
> >
> > The point is to see everything ROOT Admin and Domain Admin are entitled
> to see.
> >
> > They can see it anyway by scrolling through the projects, but its not 
> > getting
> us - true - *all vms* view.
> >
> > I don't know if only *.mine* projects would achieve the objective.
> >
> > Regards
> > ilya
> >
> >> -Original Message-
> >> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> >> Sent: Thursday, August 08, 2013 4:38 PM
> >> To: dev
> >> Subject: Re: [DISCUSS} Root/Domain Admin user should see ALL vms -
> >> based on their permisions
> >>
> >> maybe a column selectpr and/or a filter function; only projects
> >> matching '.*mine.*'
> >>
> >> On Thu, Aug 8, 2013 at 10:03 PM, Musayev, Ilya 
> >> wrote:
> >> > One other challenge, if you have large number of VMs, the view is
> >> > going to
> >> be very long and some point hard to read. Perhaps we can add a filter
> >> feature, such that user can choose how many VMs per page are
> >> presented, i.e. 50, 100, 250, 500?
> >> >
> >> > This would apply to all UI Instance view pages.
> >> >
> >> >> -Original Message-
> >> >> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
> >> >> Sent: Thursday, August 08, 2013 3:24 PM
> >> >> To: dev@cloudstack.apache.org
> >> >> Subject: Re: [DISCUSS} Root/Domain Admin user should see ALL vms -
> >> >> based on their permisions
> >> >>
> >> >> +1
> >> >>
> >> >>
> >> >> On Thu, Aug 8, 2013 at 12:59 PM, Chip Childers
> >> >> wrote:
> >> >>
> >> >> > On Thu, Aug 08, 2013 at 06:54:02PM +, Musayev, Ilya wrote:
> >> >> > > This question was asked several times on dev and user mailing list.
> >> >> > >
> >> >> > > I'd like to propose following enhancement and would like to
> >> >> > > get the
> >> >> > community feedback:
> >> >> > >
> >> >> > > Issue:
> >> >> > >
> >> >> > > Root Admin cannot see all VMs under Instance tab when "all"
> >> >> > > instances
> >> >> > are selected. Root Admin user need to switch between different
> >> >> > projects to see more vms. This approach presents a challenge as
> >> >> > there is no single view of all VMs in ALL projects. Same would
> >> >> > apply to a domain admin, he should be able to see all vms in his
> >> >> > domain at a glance without looping through each project.
> >> >> > >
> >> >> > > Suggested resolution:
> >> >> > >
> >> >> > > Change the behavior of show "all" instances by looping through
> >> >> > > all
> >> >> > projects and showing the vm names. We would also need to alter
> >> >> > the Instance view and append project name column and append
> >> >> domain/project.
> >> >> > >
> >> >> > > What do you think?
> >> >> >
> >> >> > +1, and I notice that Kirk J opened CLOUDSTACK-4196
> >> >> >
> >> >>
> >> >>
> >> >>
> >> >> --
> >> >> *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: [DISCUSS] KVM io throttling

2013-08-09 Thread Edison Su


> -Original Message-
> From: Wido den Hollander [mailto:w...@widodh.nl]
> Sent: Friday, August 09, 2013 2:41 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [DISCUSS] KVM io throttling
> 
> 
> 
> On 08/09/2013 02:05 AM, Edison Su wrote:
> > Hi All,
> >   For basic network, the io limit for guest network(nic 2) is 200MB/s, 
> > we
> set the io limit on kvm host as:
> >
> 
> I think you mean 200Mbit/s?
Oh, I see, it's 200Mb/s.
> 
> > 
> > 
> > 
> > 
> > 
> > 
> >   > peak='25600'/>  
> >
> >
> >  From libvirt doc(http://libvirt.org/formatnetwork.html), it says above unit
> is KB/s, but per my testing, it sounds like it's Kb/s. Is there anybody have 
> the
> same issue?
> >
> 
> No, my testing tells me it's is KB/sec (KiloByte).
On centos 6.3 + libvirt 0.9.10, the number doesn't match:
With the following simple test:

dd if=/dev/zero of=some-nfs-mount-point bs=1M count=10
I can get 104 MB/s if nic2 speed is unlimited, while only get 1MB/s, if nic2 is 
configured with



  

Maybe it's an issue on Centos 6.3?

> 
> > My libvirt version is 0.9.10 on Centos 6.3.
> >
> 
> My libvirt version is 1.0.6 under Ubuntu 12.04 using the Cloud Archive for
> OpenStack Havana from Canonical:
> https://wiki.ubuntu.com/ServerTeam/CloudArchive
> 
> Wido


RE: [DISCUSS] KVM io throttling

2013-08-09 Thread Edison Su
There is another issue, there is no way to change io speed from UI for system 
vm's guest network(nic2) in basic network model, which will impact all the 
template downloading in a negatively way. 
We'd better don't throttle system vm's io speed by default, or at least give 
admin the way to change the io speed.

> -Original Message-
> From: Edison Su [mailto:edison...@citrix.com]
> Sent: Friday, August 09, 2013 11:27 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [DISCUSS] KVM io throttling
> 
> 
> 
> > -Original Message-
> > From: Wido den Hollander [mailto:w...@widodh.nl]
> > Sent: Friday, August 09, 2013 2:41 AM
> > To: dev@cloudstack.apache.org
> > Subject: Re: [DISCUSS] KVM io throttling
> >
> >
> >
> > On 08/09/2013 02:05 AM, Edison Su wrote:
> > > Hi All,
> > >   For basic network, the io limit for guest network(nic 2) is
> > > 200MB/s, we
> > set the io limit on kvm host as:
> > >
> >
> > I think you mean 200Mbit/s?
> Oh, I see, it's 200Mb/s.
> >
> > > 
> > > 
> > > 
> > > 
> > > 
> > > 
> > >   > > peak='25600'/>  
> > >
> > >
> > >  From libvirt doc(http://libvirt.org/formatnetwork.html), it says
> > > above unit
> > is KB/s, but per my testing, it sounds like it's Kb/s. Is there
> > anybody have the same issue?
> > >
> >
> > No, my testing tells me it's is KB/sec (KiloByte).
> On centos 6.3 + libvirt 0.9.10, the number doesn't match:
> With the following simple test:
> 
> dd if=/dev/zero of=some-nfs-mount-point bs=1M count=10 I can get 104
> MB/s if nic2 speed is unlimited, while only get 1MB/s, if nic2 is configured
> with 
> 
> 
>   
> 
> Maybe it's an issue on Centos 6.3?
> 
> >
> > > My libvirt version is 0.9.10 on Centos 6.3.
> > >
> >
> > My libvirt version is 1.0.6 under Ubuntu 12.04 using the Cloud Archive
> > for OpenStack Havana from Canonical:
> > https://wiki.ubuntu.com/ServerTeam/CloudArchive
> >
> > Wido


Offline next week

2013-08-09 Thread Sheng Yang
For network part, please refer to Anthony, Alena, and Chiradeep.

Please refer to Animesh for emergency.

Thanks!

--Sheng


Re: [Discuss] Making CloudMonkey simpler to use for admin tasks

2013-08-09 Thread Rohit Yadav
On Mon, Aug 5, 2013 at 6:19 PM, Donal Lafferty wrote:

>
>
> > -Original Message-
> > From: rohityada...@gmail.com [mailto:rohityada...@gmail.com] On Behalf
> > Of Rohit Yadav
> > Sent: 02 August 2013 22:43
> > To: dev@cloudstack.apache.org
> > Subject: Re: [Discuss] Making CloudMonkey simpler to use for admin tasks
> >
> > On Thu, Aug 1, 2013 at 10:43 PM, Donal Lafferty
> > wrote:
> >
> > > I needed a different configuration than DevCloud provided, so I turned
> > > to CloudMonkey to automate setup of my test environment.
> > >
> > > This led to a blog on automating with CloudMonkey at
> > > http://dlafferty.blogspot.co.uk/2013/07/using-cloudmonkey-to-automate.
> > > html
> > >
> > >
> > Nice.
> >
> >
> > > What I forgot to mention is that automation would be a lot simpler if
> > > we were to do the following:
> > >
> > > 1.   Update Apache CloudStack logging to provide API calls in a
> tidy
> > > format that can be fed directly.  E.g. POST parameters are not logged,
> > > GET parameters are URL encoded.
> > >
> > > 2.   Update CloudMonkey to allow username / password authentication
> > >
> >
> > +1, send patch :)
> >
> How would you store the sessionkey?
>

In Python, you may use cookiejar. Another way is to bootstrap keys, i.e.
you use username:password to login and create api and secret keys. And then
use the keys to store them. In case keys exist, we load the api and secret
keys in memory in the config{} dictionary.


>
> E.g. see written in C#:
> https://github.com/lafferty/cloudstack_dotnetsdk/blob/master/CloudStack.SDK/client.cs
>
> Specifically, see public void Login(string userName, string password,
> string domainName, bool hashPassword)
>
> >
> > >
> > > 3.   Update CloudMonkey to not be picky about the case of command
> > > parameters when 'api' command used.
> > >
> >
> > +1, send patch :)
> >
>
> This is a one liner change, no?
>

+1


>
> >
> > >
> > > 4.   Update CloudMonkey to allow filter option to be used with
> 'api'
> > > command.
> > >
> >
> > How do you propose using that, example?
> >
>
> Example:
>
> apiresult=`cloudmonkey api createPhysicalNetwork zoneid=$zone
> name='"Physical Network 1"' isolationmethods='"VLAN"' `
> physnetid=`echo $apiresult | sed -e 's/^.*"id": //; s/,.*//'`
>
> becomes
>
> physnetid =`cloudmonkey filter=id api createPhysicalNetwork zoneid=$zone
> name='"Physical Network 1"' isolationmethods='"VLAN"' `
>

Cool.

Sorry for my late reply, I think moving forward I may not be even able to
work on CloudStack during all weekends, so I would love if someone could
lead this component though I'll be here.

Regards.


>
>
> > Regards.
>


Re: OVS

2013-08-09 Thread Chiradeep Vittal
I haven't tried it, but have you looked at
https://cwiki.apache.org/confluence/x/15XlAQ

On 8/9/13 10:23 AM, "Thomas Schneider" 
wrote:

>this is 4.1
>
>Le 09/08/2013 18:47, Chiradeep Vittal a écrit :
>> Is this 4.1/4.2/master?
>>
>>
>> On 8/9/13 6:13 AM, "Thomas Schneider" 
>>wrote:
>>
>>> Hello
>>>
>>> I try to setup Cloudstack with OpenVSwitch, but SSVM and the proxy vm
>>> can't reach internet.
>>> I think it's a problem with cloud0 bridge in OVS.
>>>
>>> My config is:
>>>
>>> /etc/cloudstack/agent/agent.properties
>>> network.bridge.type=openvswitch
>>> libvirt.vif.driver=com.cloud.hypervisor.kvm.resource.OvsVifDriver
>>>
>>> OVS
>>> ovs-vsctl add-br cloudbr0
>>> ovs-vsctl add-port cloudbr0 p49p1
>>> ovs-vsctl add-br cloudbr1
>>> ovs-vsctl add-port cloudbr1 p50p1
>>>
>>> ovs-vsctl show
>>> 0771f39e-6e58-4563-b35e-c891197c858f
>>>Bridge "cloud0"
>>>Port "vnet0"
>>>Interface "vnet0"
>>>Port "cloud0"
>>>Interface "cloud0"
>>>type: internal
>>>Port "vnet4"
>>>Interface "vnet4"
>>>Bridge "cloudbr1"
>>>Port "cloudbr1"
>>>Interface "cloudbr1"
>>>type: internal
>>>Port "p50p1"
>>>Interface "p50p1"
>>>Bridge "cloudbr0"
>>>Port "vnet2"
>>>Interface "vnet2"
>>>Port "p49p1"
>>>Interface "p49p1"
>>>Port "vnet1"
>>>Interface "vnet1"
>>>Port "cloudbr0"
>>>Interface "cloudbr0"
>>>type: internal
>>>Port "vnet3"
>>>Interface "vnet3"
>>>Port "vnet5"
>>>Interface "vnet5"
>>>Port "vnet6"
>>>Interface "vnet6"
>>>ovs_version: "1.9.0"
>>>
>>> /etc/network/interfaces
>>>
>>> # The primary network interface
>>> auto p49p1
>>> iface p49p1 inet manual
>>>
>>> # The secondary network interface
>>> auto p50p1
>>> iface p50p1 inet manual
>>>
>>> # Public network
>>> auto cloudbr0
>>> iface cloudbr0 inet static
>>>bridge_ports p49p1
>>>bridge_fd 5
>>>bridge_stp off
>>>bridge_maxwait 1
>>>address 10.10.10.30
>>>netmask 255.255.255.0
>>>network 10.10.10.0
>>>broadcast 10.10.10.255
>>>gateway 10.10.10.254
>>>
>>> auto cloudbr1
>>> iface cloudbr1 inet manual
>>>bridge_ports p50p1
>>>bridge_fd 5
>>>bridge_stp off
>>>bridge_maxwait 1
>>>
>>> THX
>>> -- 
>>> *Thomas Schneider*
>>>
>
>
>-- 
>*Thomas Schneider*
>Directeur des Opérations
>Euskill SARL
>Web: www.euskill.com
>Mobile: +33 (0)6 19 26 47 76
>Mail: thomas.schnei...@euskill.com
>5 rue de Phalsbourg
>F-67000 Strasbourg



HyperV support in 4.2

2013-08-09 Thread Alena Prokharchyk
Hi Chip,

Are we planning to officially support HyperV in 4.2, or is it pushed to the 
next release? I'm seeing HyperV listed in the values for "hypervisor.list" 
Global config (the list of hypervisors CS supports) - meaning that the code is 
in - but the bug below claims that there is no HyperV in 4.2:

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

Thank you,
Alena.


CloudMonkey 5.0 (was Re: CloudMonkey's new home)

2013-08-09 Thread Rohit Yadav
Hi folks,

Looks like the previous thread failed to capture attention on the dev ML.
I'm going forward with some decisions so as to move fast and as per our
philosophy to ask for forgiveness later than just waste time on too much
process polling now.

Here are some proposals;

- The version model would be to move fast, break things, release early and
release often
- Start with 5.0 version: Since cloudmonkey 4.x is already out there, I'm
proposing we start new cloudmonkey releases from 5.0; This is just to make
sure we don't end up releasing a 1.0 when we already have a 4.x
- Using semver, we don't deviate from major version "5" until we have
backward incompatible changes of configuration, paths, DSL etc.
- We'll use git tags to track (unvoted) releasable or testable candidates.
This is so we can release fast, release often. We can append a -rc for such
releases on git and pypi.
- A tested and voted release could take time and some process; but pypi
channel may not be necessarily used for only official releases, but all and
every release, even the test ones.

Suggestions, flames?

Moving forward, as it seems already, I may not be able to contribute on
weekends.

I may be only able to help with the first release, that too the non-process
oriented parts, perhaps people who already have some idea about the
internals of cloudmonkey like Prasanna or Sebastien can help lead the
component?

Regards.


On Sun, Jul 28, 2013 at 11:04 PM, Rohit Yadav  wrote:

> Based on our previous discussion thread[1], we've moved CloudMonkey out of
> ACS's repository to its new home [2]. Now,
> with 6f84e74a68d78705a06fe58f7927f42f61453a16 on master, we no longer have
> cloudmonkey in tools/cli. CloudMonkey will be within CloudStack project but
> now as an independent sub-project with its own repository and will have a
> faster need-basis release cycle.
>
> For doing that, please suggest on the release process or how it should
> work? If the present RM or someone wants to lead the release process?
> I just want to keep it simple with fast releases whenever we have a
> releasable candidate and semver[3] versioning. So, we ship things fast and
> don't worry if it breaks since we'll be shipping fast. We can after a fast
> lazy consensus/voting and publish via pypi and put the tarballs/zipballs
> under dists/ on ASF/CloudStack.
>
> Regards.
>
> [1] http://markmail.org/message/tjlr753xfhpw4uk4
> [2] https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git
> [3] http://semver.org/
>


Re: HyperV support in 4.2

2013-08-09 Thread Chiradeep Vittal
No, not supported.

On 8/9/13 2:02 PM, "Alena Prokharchyk" 
wrote:

>Hi Chip,
>
>Are we planning to officially support HyperV in 4.2, or is it pushed to
>the next release? I'm seeing HyperV listed in the values for
>"hypervisor.list" Global config (the list of hypervisors CS supports) -
>meaning that the code is in - but the bug below claims that there is no
>HyperV in 4.2:
>
>https://issues.apache.org/jira/browse/CLOUDSTACK-3893
>
>Thank you,
>Alena.



RE: HyperV support in 4.2

2013-08-09 Thread Animesh Chaturvedi


I see Donal's patch is pending in review board 
https://reviews.apache.org/r/12519/ but the branch mentioned is master. Donal 
can you confirm if this is for 4.2 as well.

Alena here is another thread that has more context 
http://markmail.org/thread/asg62idkvrnn6ba7


> -Original Message-
> From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
> Sent: Friday, August 09, 2013 2:02 PM
> To: Chip Childers; dev@cloudstack.apache.org
> Subject: HyperV support in 4.2
> 
> Hi Chip,
> 
> Are we planning to officially support HyperV in 4.2, or is it pushed to
> the next release? I'm seeing HyperV listed in the values for
> "hypervisor.list" Global config (the list of hypervisors CS supports) -
> meaning that the code is in - but the bug below claims that there is no
> HyperV in 4.2:
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3893
> 
> Thank you,
> Alena.


RE: HyperV support in 4.2

2013-08-09 Thread Animesh Chaturvedi
Donal if it is not for 4.2 then can you move this ticket 
https://issues.apache.org/jira/browse/CLOUDSTACK-999 out of 4.2. Copying Sudha 
as well

> -Original Message-
> From: Animesh Chaturvedi
> Sent: Friday, August 09, 2013 2:16 PM
> To: dev@cloudstack.apache.org; Chip Childers
> Cc: Donal Lafferty
> Subject: RE: HyperV support in 4.2
> 
> 
> 
> I see Donal's patch is pending in review board
> https://reviews.apache.org/r/12519/ but the branch mentioned is master.
> Donal can you confirm if this is for 4.2 as well.
> 
> Alena here is another thread that has more context
> http://markmail.org/thread/asg62idkvrnn6ba7
> 
> 
> > -Original Message-
> > From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
> > Sent: Friday, August 09, 2013 2:02 PM
> > To: Chip Childers; dev@cloudstack.apache.org
> > Subject: HyperV support in 4.2
> >
> > Hi Chip,
> >
> > Are we planning to officially support HyperV in 4.2, or is it pushed
> > to the next release? I'm seeing HyperV listed in the values for
> > "hypervisor.list" Global config (the list of hypervisors CS supports)
> > - meaning that the code is in - but the bug below claims that there
> is
> > no HyperV in 4.2:
> >
> > https://issues.apache.org/jira/browse/CLOUDSTACK-3893
> >
> > Thank you,
> > Alena.


Re: HyperV support in 4.2

2013-08-09 Thread Alena Prokharchyk
Animesh and Chiradeep, thanks for confirming.

Donal, I see that there are some references to HypervisorType.Hyperv exist in 
4.2 branch. We have to remove them all + all the type itself + global configs 
that I've mentioned, from 4.2

-Alena.

From: Animesh Chaturvedi 
mailto:animesh.chaturv...@citrix.com>>
Reply-To: "dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>
Date: Friday, August 9, 2013 2:16 PM
To: "dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>, Chip Childers 
mailto:chip.child...@sungard.com>>
Cc: Donal Lafferty mailto:donal.laffe...@citrix.com>>
Subject: RE: HyperV support in 4.2



I see Donal's patch is pending in review board 
https://reviews.apache.org/r/12519/ but the branch mentioned is master. Donal 
can you confirm if this is for 4.2 as well.

Alena here is another thread that has more context 
http://markmail.org/thread/asg62idkvrnn6ba7


-Original Message-
From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
Sent: Friday, August 09, 2013 2:02 PM
To: Chip Childers; dev@cloudstack.apache.org
Subject: HyperV support in 4.2
Hi Chip,
Are we planning to officially support HyperV in 4.2, or is it pushed to
the next release? I'm seeing HyperV listed in the values for
"hypervisor.list" Global config (the list of hypervisors CS supports) -
meaning that the code is in - but the bug below claims that there is no
HyperV in 4.2:
https://issues.apache.org/jira/browse/CLOUDSTACK-3893
Thank you,
Alena.



[DISCUSS] Releasing cloudmonkey 5.0.0

2013-08-09 Thread Rohit Yadav
Hi folks,

I've uploaded a new cloudmonkey tarball on pypi channel[1] using the latest
git HEAD 68d5fef065b882accae86a2e4cd327f7779fddb7 from the new repository
[2]. I've already proposed in another thread why we should start with 5.x
but any suggestions welcome.

I request if someone could help release it, do the release management,
voting thing etc. We don't have any tests to verify any recent changes, so
manual QA may be needed. There aren't many changes, so hopefully it will
work for everyone.

To test this candidate, just do: pip install cloudmonkey or pip install
--upgrade cloudmonkey

Suggestions, flames?

[1] https://pypi.python.org/pypi/cloudmonkey/5.0.0
[2] https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git

Regards.


Review Request 13453: Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create sever.xml, we need to check both file or symbolic link exist or not in saved folder

2013-08-09 Thread Rayees Namathponnan

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

Review request for cloudstack and Frank Zhang.


Bugs: CLOUDSTACK-3858


Repository: cloudstack-git


Description
---

Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
sever.xml under /etc/cloudstack/management, we need to check both file or 
symbolic link exist or not in save folder (/etc/cloud.saved/management) then 
create symbolic link 
 


Diffs
-

  packaging/centos63/cloud.spec 0a6327f 

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


Testing
---

Tested


Thanks,

Rayees Namathponnan



Re: [Discuss] Making CloudMonkey simpler to use for admin tasks

2013-08-09 Thread Pedro Roque Marques

On Aug 2, 2013, at 2:42 PM, Rohit Yadav wrote:

> On Thu, Aug 1, 2013 at 10:43 PM, Donal Lafferty
> wrote:
> 
>> I needed a different configuration than DevCloud provided, so I turned to
>> CloudMonkey to automate setup of my test environment.
>> 
>> This led to a blog on automating with CloudMonkey at
>> http://dlafferty.blogspot.co.uk/2013/07/using-cloudmonkey-to-automate.html
>> 
>> 
> Nice.
> 
> 
>> What I forgot to mention is that automation would be a lot simpler if we
>> were to do the following:
>> 
>> 1.   Update Apache CloudStack logging to provide API calls in a tidy
>> format that can be fed directly.  E.g. POST parameters are not logged,  GET
>> parameters are URL encoded.
>> 
>> 2.   Update CloudMonkey to allow username / password authentication
>> 
> 
> +1, send patch :)

I use following:
cloudauth.py retrieves a key pair...
cloudclient.py is a simple wrapper around cloudmonkey so that we can use it 
directly from python scripts and get json replies.

usage example:
def createZone(client):
try:
response = client.request('listZones')
if response['listzonesresponse']:
return response['listzonesresponse']['zone'][0]['id']

response = client.request('createZone',
  {'name': 'default',
   'networktype': 'Advanced',
   'domain': 'ROOT',
   'dns1': PARAMS['external_dns'],
   'internaldns1': PARAMS['internal_dns']})
return response['createzoneresponse']['zone']['id']

except Exception as e:
print 'Error while creating a zone:'
print e
raise



import cloudauth
from cloudmonkey.requester import monkeyrequest
import logging
import sys

class CloudClient(object):
class Error(Exception):
""" Base error."""
pass

def __init__(self, hostname, username, password):
self.host = hostname
self.username = username
self.password = password
self.port = 8080
self.path = '/client/api'
self.protocol = 'http'
self.timeout = 3600
keypair = cloudauth.getApiKey(self.host, self.username, self.password)
if not keypair:
sys.exit(1)
(self.apiKey, self.secretKey) = keypair

def request(self, command, args={}):
response, error = monkeyrequest(command, args, True,
"true", logging.getLogger(),
self.host, self.port,
self.apiKey, self.secretKey,
self.timeout, self.protocol, self.path)
if error is not None:
raise self.Error("Request %s:\n%s\nfailed with error:\n%s" %
 (command, str(args), error))
return response
import json
import logging
import os
import subprocess
import tempfile
import urllib

def encodeURIComponent(str):
return urllib.quote(str, safe='~()*!.\'')

def cloudLogin(hostname, username, password, tmp_name):
login = ("command=login&username=" + username + "&password=" + password +
 "&response=json")
cmd = ['curl',
   '-H', 'Content-Type: application/x-www-form-urlencoded',
   '-H', 'Accept: application/json',
   '-X', 'POST',
   '-d', login,
   '-c', tmp_name,
   'http://' + hostname + ':8080/client/api']
proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
(output, err) = proc.communicate()
response = json.loads(output)
if response.get('errorresponse'):
print response['errorresponse']['errortext']
return None
return response['loginresponse']

def getKeys(hostname, loginresp, tmp_name):
urlParam = '&response=json&id=' + loginresp['userid'] + '&sessionkey=' + encodeURIComponent(loginresp['sessionkey'])
cmd = ['curl',
   '-v',
   '-H', 'Content-Type: application/json',
   '-b', tmp_name,
   '-X', 'POST',
   'http://' + hostname + ':8080/client/api/?command=listUsers' + urlParam]
proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
(output, err) = proc.communicate()
response = json.loads(output)
logging.debug(response)
user = response['listusersresponse']['user'][0]
if not 'apikey' in user:
return None
return user['apikey'], user['secretkey']


def registerKeys(hostname, loginresp, tmp_name):
urlParam = '&response=json&id=' + loginresp['userid'] + '&sessionkey=' + encodeURIComponent(loginresp['sessionkey'])
cmd = ['curl',
   '-v',
   '-H', 'Content-Type: application/json',
   '-b', tmp_name,
   '-X', 'POST',
   'http://' + hostname + ':8080/client/api/?command=registerUserKeys' + urlParam]
proc = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subproc

Re: Review Request 12811: CLOUDSTACK-3612: 3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 52e10d6b202e99e2e4a24e859f2a3e16339875d7 in branch refs/heads/master 
from frank
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=52e10d6 ]

CLOUDSTACK-38333.0.6 to ASF 4.2 Upgrade: "cloud" Database Schema 
Inconsistencies on the Upgraded Setup - Tables "baremetal_dhcp_devices" and 
"baremetal_pxe_devices"

CloudStack CLOUDSTACK-3612
3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup


- ASF Subversion and Git Services


On July 24, 2013, 10:53 a.m., Saksham Srivastava wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12811/
> ---
> 
> (Updated July 24, 2013, 10:53 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk and Kishan Kavala.
> 
> 
> Bugs: 3612
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This fix is a part of missing db_upgrade bug 3612.
> Adding column virtual_size that will be a part of fix for CS-2508
> 
> 
> Diffs
> -
> 
>   setup/db/db/schema-410to420.sql f9417a0 
> 
> Diff: https://reviews.apache.org/r/12811/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Saksham Srivastava
> 
>



Re: Review Request 12811: CLOUDSTACK-3612: 3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup

2013-08-09 Thread ASF Subversion and Git Services

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


Commit ad508b7232e28ca02683bd8b60268cb87c71e36a in branch refs/heads/4.2 from 
frank
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=ad508b7 ]

CLOUDSTACK-38333.0.6 to ASF 4.2 Upgrade: "cloud" Database Schema 
Inconsistencies on the Upgraded Setup - Tables "baremetal_dhcp_devices" and 
"baremetal_pxe_devices"

CloudStack CLOUDSTACK-3612
3.0.6 to ASF 4.2 Upgrade: Database Schema Inconsistencies on the Upgraded Setup


- ASF Subversion and Git Services


On July 24, 2013, 10:53 a.m., Saksham Srivastava wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12811/
> ---
> 
> (Updated July 24, 2013, 10:53 a.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk and Kishan Kavala.
> 
> 
> Bugs: 3612
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> This fix is a part of missing db_upgrade bug 3612.
> Adding column virtual_size that will be a part of fix for CS-2508
> 
> 
> Diffs
> -
> 
>   setup/db/db/schema-410to420.sql f9417a0 
> 
> Diff: https://reviews.apache.org/r/12811/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Saksham Srivastava
> 
>



Re: Review Request 13453: Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create sever.xml, we need to check both file or symbolic link exist or not in saved folder

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 90457edb416198dcb7a67e7009c42302c0272f18 in branch refs/heads/4.2 from 
rayeesn
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=90457ed ]

Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
sever.xml, we need to check both file or symbolic link exist or not in save 
folder


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:09 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13453/
> ---
> 
> (Updated Aug. 9, 2013, 11:09 p.m.)
> 
> 
> Review request for cloudstack and Frank Zhang.
> 
> 
> Bugs: CLOUDSTACK-3858
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
> sever.xml under /etc/cloudstack/management, we need to check both file or 
> symbolic link exist or not in save folder (/etc/cloud.saved/management) then 
> create symbolic link 
>  
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 0a6327f 
> 
> Diff: https://reviews.apache.org/r/13453/diff/
> 
> 
> Testing
> ---
> 
> Tested
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



Re: Review Request 13453: Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create sever.xml, we need to check both file or symbolic link exist or not in saved folder

2013-08-09 Thread ASF Subversion and Git Services

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


Commit e80fdcda9452fc08cf227f50634e1c144af856cf in branch refs/heads/master 
from rayeesn
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e80fdcd ]

Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
sever.xml, we need to check both file or symbolic link exist or not in save 
folder


- ASF Subversion and Git Services


On Aug. 9, 2013, 11:09 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13453/
> ---
> 
> (Updated Aug. 9, 2013, 11:09 p.m.)
> 
> 
> Review request for cloudstack and Frank Zhang.
> 
> 
> Bugs: CLOUDSTACK-3858
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
> sever.xml under /etc/cloudstack/management, we need to check both file or 
> symbolic link exist or not in save folder (/etc/cloud.saved/management) then 
> create symbolic link 
>  
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 0a6327f 
> 
> Diff: https://reviews.apache.org/r/13453/diff/
> 
> 
> Testing
> ---
> 
> Tested
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



Re: Review Request 13453: Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create sever.xml, we need to check both file or symbolic link exist or not in saved folder

2013-08-09 Thread Frank Zhang

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

Ship it!


Ship It!

- Frank Zhang


On Aug. 9, 2013, 11:09 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13453/
> ---
> 
> (Updated Aug. 9, 2013, 11:09 p.m.)
> 
> 
> Review request for cloudstack and Frank Zhang.
> 
> 
> Bugs: CLOUDSTACK-3858
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create 
> sever.xml under /etc/cloudstack/management, we need to check both file or 
> symbolic link exist or not in save folder (/etc/cloud.saved/management) then 
> create symbolic link 
>  
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 0a6327f 
> 
> Diff: https://reviews.apache.org/r/13453/diff/
> 
> 
> Testing
> ---
> 
> Tested
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



CloudStack source which is on SourceForge but which should be with Apache?

2013-08-09 Thread Demetrius Tsitrelis


The admin and install guides recommend pulling files from SourceForge.  
Specifically:
 
SSH Key Gen script:

http://downloads.sourceforge.net/project/cloudstack/SSH%20Key%20Gen%20Script/cloud-set-guest-sshkey.in?r=http%3A%2F%2Fsourceforge.net%2Fprojects%2Fcloudstack%2Ffiles%2FSSH%2520Key%2520Gen%2520Script%2F&ts=1331225219&use_mirror=iweb

 

Password management scripts:

Linux:  http://cloudstack.org/dl/cloud-set-guest-password

Windows:  
http://sourceforge.net/projects/cloudstack/files/Password%20Management%20Scripts
 The CloudInstanceManage.msi in the last link is in binary form only; the C 
sources should be donated to Apache.  Also, some certificate will be needed 
when it is built so the build instructions should note that.

  

Re: Review Request 12500: fix CLOUDSTACK-3495 xenserver 6.1 and 6.2 can not open vnc console

2013-08-09 Thread ASF Subversion and Git Services

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


Commit f39937223310cffde9308b16f26157f2e883d018 in branch refs/heads/4.2 from 
anthony
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=f399372 ]

CLOUDSTACK-3495
CS used to access vnc server in xenserver dom0 to get VM console, now 
CS moves to use XenServer console API. getvncport plugin is not needed any more.
remove the code related to getvncport in XenServer


- ASF Subversion and Git Services


On July 12, 2013, 9:58 a.m., Hongtu Zang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12500/
> ---
> 
> (Updated July 12, 2013, 9:58 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-3495
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> change vmops getvncport function
> xenserver 6.1 and 6.2 will find the vnc-port in the same path with 6.0
> 
> 
> Diffs
> -
> 
>   scripts/vm/hypervisor/xenserver/vmops 650e955 
> 
> Diff: https://reviews.apache.org/r/12500/diff/
> 
> 
> Testing
> ---
> 
> can open vnc console in web UI
> 
> 
> Thanks,
> 
> Hongtu Zang
> 
>



Re: Review Request 12500: fix CLOUDSTACK-3495 xenserver 6.1 and 6.2 can not open vnc console

2013-08-09 Thread ASF Subversion and Git Services

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


Commit 4333209af36ca001ca88935ddf2b4d981cc0150e in branch refs/heads/master 
from anthony
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4333209 ]

CLOUDSTACK-3495
CS used to access vnc server in xenserver dom0 to get VM console, now 
CS moves to use XenServer console API. getvncport plugin is not needed any more.
remove the code related to getvncport in XenServer


- ASF Subversion and Git Services


On July 12, 2013, 9:58 a.m., Hongtu Zang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12500/
> ---
> 
> (Updated July 12, 2013, 9:58 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
> 
> 
> Bugs: CLOUDSTACK-3495
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> change vmops getvncport function
> xenserver 6.1 and 6.2 will find the vnc-port in the same path with 6.0
> 
> 
> Diffs
> -
> 
>   scripts/vm/hypervisor/xenserver/vmops 650e955 
> 
> Diff: https://reviews.apache.org/r/12500/diff/
> 
> 
> Testing
> ---
> 
> can open vnc console in web UI
> 
> 
> Thanks,
> 
> Hongtu Zang
> 
>



Review Request 13459: cloudstack-3759: deployVM with state set to be stopped, attach volume to it. The volume should not be created.

2013-08-09 Thread Fang Wang

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

Review request for cloudstack and Frank Zhang.


Bugs: cloudstack-3759


Repository: cloudstack-git


Description
---

When deploying a new VM, with its state set to be stopped. At that time, when 
attaching volume to it, we need to delay the volume creation. Because in DB, 
the host id and pod id, fields for the VM are empty. 


Diffs
-

  server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 

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


Testing
---


Thanks,

Fang Wang



Review Request 13460: VPC test case TestVMLifeCycleSharedNwVPC need to disabl, since its failing

2013-08-09 Thread Rayees Namathponnan

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

Review request for cloudstack, edison su, Frank Zhang, and Sheng Yang.


Repository: cloudstack-git


Description
---

TestVMLifeCycleSharedNwVPC need to disable
Netscaler config test case interface need to updated 


Diffs
-

  test/integration/component/test_netscaler_configs.py 3cde40e 
  test/integration/component/test_vpc_vm_life_cycle.py 0c5dd2f 

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


Testing
---

tested


Thanks,

Rayees Namathponnan



[DISCUSS][4.2] Using full-clone by default for VMware deployment

2013-08-09 Thread Kelven Yang
To better integrate with VMware features like storage live migration,  fully 
utilize third-party integrations from vSphere eco-system, I'd like to change 
our default volume deploy mode from linked-clone to full-clone in VMware.

If there is no objections, I'll make the change and starting from 4.2 release, 
we will use full-clone mode by default for VMware deployments.

Kelven



Re: Review Request 13460: VPC test case TestVMLifeCycleSharedNwVPC need to disabl, since its failing

2013-08-09 Thread edison su

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

Ship it!


Ship It!

- edison su


On Aug. 10, 2013, 12:53 a.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13460/
> ---
> 
> (Updated Aug. 10, 2013, 12:53 a.m.)
> 
> 
> Review request for cloudstack, edison su, Frank Zhang, and Sheng Yang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> TestVMLifeCycleSharedNwVPC need to disable
> Netscaler config test case interface need to updated 
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_netscaler_configs.py 3cde40e 
>   test/integration/component/test_vpc_vm_life_cycle.py 0c5dd2f 
> 
> Diff: https://reviews.apache.org/r/13460/diff/
> 
> 
> Testing
> ---
> 
> tested
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>



Re: CloudMonkey 5.0 (was Re: CloudMonkey's new home)

2013-08-09 Thread David Nalley
+1 move forward.

On Fri, Aug 9, 2013 at 5:02 PM, Rohit Yadav  wrote:
> Hi folks,
>
> Looks like the previous thread failed to capture attention on the dev ML.
> I'm going forward with some decisions so as to move fast and as per our
> philosophy to ask for forgiveness later than just waste time on too much
> process polling now.
>
> Here are some proposals;
>
> - The version model would be to move fast, break things, release early and
> release often
> - Start with 5.0 version: Since cloudmonkey 4.x is already out there, I'm
> proposing we start new cloudmonkey releases from 5.0; This is just to make
> sure we don't end up releasing a 1.0 when we already have a 4.x
> - Using semver, we don't deviate from major version "5" until we have
> backward incompatible changes of configuration, paths, DSL etc.
> - We'll use git tags to track (unvoted) releasable or testable candidates.
> This is so we can release fast, release often. We can append a -rc for such
> releases on git and pypi.
> - A tested and voted release could take time and some process; but pypi
> channel may not be necessarily used for only official releases, but all and
> every release, even the test ones.
>
> Suggestions, flames?
>
> Moving forward, as it seems already, I may not be able to contribute on
> weekends.
>
> I may be only able to help with the first release, that too the non-process
> oriented parts, perhaps people who already have some idea about the
> internals of cloudmonkey like Prasanna or Sebastien can help lead the
> component?
>
> Regards.
>
>
> On Sun, Jul 28, 2013 at 11:04 PM, Rohit Yadav  wrote:
>
>> Based on our previous discussion thread[1], we've moved CloudMonkey out of
>> ACS's repository to its new home [2]. Now,
>> with 6f84e74a68d78705a06fe58f7927f42f61453a16 on master, we no longer have
>> cloudmonkey in tools/cli. CloudMonkey will be within CloudStack project but
>> now as an independent sub-project with its own repository and will have a
>> faster need-basis release cycle.
>>
>> For doing that, please suggest on the release process or how it should
>> work? If the present RM or someone wants to lead the release process?
>> I just want to keep it simple with fast releases whenever we have a
>> releasable candidate and semver[3] versioning. So, we ship things fast and
>> don't worry if it breaks since we'll be shipping fast. We can after a fast
>> lazy consensus/voting and publish via pypi and put the tarballs/zipballs
>> under dists/ on ASF/CloudStack.
>>
>> Regards.
>>
>> [1] http://markmail.org/message/tjlr753xfhpw4uk4
>> [2] https://git-wip-us.apache.org/repos/asf?p=cloudstack-cloudmonkey.git
>> [3] http://semver.org/
>>


[ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-09 Thread Prasanna Santhanam
The Project Management Committee (PMC) for Apache CloudStack has asked Vijay
Bhamidipati to become a committer and we are pleased to announce that they have
accepted.

Being a committer allows many contributors to contribute more autonomously. For
developers, it makes it easier to submit changes and eliminates the need to
have contributions reviewed via the patch submission process. Whether
contributions are development-related or otherwise, it is a recognition of a
contributor's participation in the project and commitment to the project and
the Apache Way.

Please join me in congratulating Vijay!

-- 
Prasanna.,
on behalf of the Apache CloudStack PMC


Powered by BigRock.com



Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-09 Thread Min Chen
Congratulations, Vijay.

-min

On 8/9/13 10:00 PM, "Prasanna Santhanam"  wrote:

>The Project Management Committee (PMC) for Apache CloudStack has asked
>Vijay
>Bhamidipati to become a committer and we are pleased to announce that
>they have
>accepted.
>
>Being a committer allows many contributors to contribute more
>autonomously. For
>developers, it makes it easier to submit changes and eliminates the need
>to
>have contributions reviewed via the patch submission process. Whether
>contributions are development-related or otherwise, it is a recognition
>of a
>contributor's participation in the project and commitment to the project
>and
>the Apache Way.
>
>Please join me in congratulating Vijay!
>
>-- 
>Prasanna.,
>on behalf of the Apache CloudStack PMC
>
>
>Powered by BigRock.com
>



Re: Master build failure

2013-08-09 Thread Prasanna Santhanam
The jenkins integration is broken atm. If someone has time to look
into it please send a pull request with the fix here:
https://github.com/CloudStack-extras/reviewboard-tools/issues/2

On Fri, Aug 09, 2013 at 10:35:01AM +0900, Dave Cahill wrote:
> Hi,
> 
> Alex - sounds good, let's raise the Gerrit topic in a new thread after 4.2.
> 
> Daan - you're right that if Jenkins builds, that should prove the code
> compiles cleanly.
> Have we actually been gating commits on the catofong report though? I
> looked at the
> Jenkins build behind it [1], and it hasn't succeeded in 4 days, so I'm
> guessing we're
> not gating based on it.
> 
> Aside from the Jenkins integration, one of the reasons I wanted Gerrit is
> to put a step
> between writing code and it hitting master. Ideally, it would be nice to
> enforce that a
> positive code review from someone other than the author is present before
> pushing.
> 
> Thanks,
> Dave.
> 
> [1]
> http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/
> 
> On Fri, Aug 9, 2013 at 4:16 AM, Daan Hoogland wrote:
> 
> > Dave, wouldn't jenkins do a build like that? Meanng the positive cato
> > report is a must to allow commits
> >
> > On Thu, Aug 8, 2013 at 4:54 PM, Alex Huang  wrote:
> > > Yup.  That was the reason why it worked on my machine but doesn't work
> > on jenkins.  It had a copy of the jar in the local maven cache so it
> > continued to build even though it's missing.  Thanks for reverting.  Will
> > fix.
> > >
> > > I still don't understand why we have problems with Gerrit.  It would
> > have prevented exactly this type of problems.  I know a lot of people are
> > busy on 4.2.  Will bring it up post 4.2 again.
> > >
> > > --Alex
> > >
> > >
> > >
> > > From: Dave Cahill [mailto:dcah...@midokura.com]
> > > Sent: Thursday, August 8, 2013 3:45 AM
> > > To: dev@cloudstack.apache.org; Dave Cahill; Alex Huang
> > > Subject: Re: Master build failure
> > >
> > > Thanks Prasanna.
> > >
> > > One suggestion I have is that if anyone makes pom file changes (or any
> > large changes at
> > > all, to be honest), that they nuke their local Maven cache (rm -rf
> > ~/.m2/repository) and do
> > > a "clean install" before submitting a patch. This way, you don't end up
> > thinking your work
> > > builds when it just happens to build due to leftover, cached jars.
> > >
> > > Alex, do you have any ideas on build stability? There have been 2 build
> > breakages in the
> > > past few days (ref: "compile error on 4.2" thread), and looking at git
> > log I don't see much
> > > sign of review on the changes. We discussed Gerrit a while back, but I
> > think the conclusion
> > > was that it would slow us down?
> > >
> > > Thanks,
> > > Dave.
> > >
> > > On Thu, Aug 8, 2013 at 6:13 PM, Prasanna Santhanam  > > wrote:
> > > Reverted the range 30c33415..f6a2c817bc. Master builds now. Others
> > > who've checked in after these will still retain their commits but
> > > please build before push. It's clear we are only worried about 4.2 now
> > > but we are breaking master for those who are beginning to work ahead
> > > for 4.3
> > >
> > > On Thu, Aug 08, 2013 at 02:15:58PM +0530, Prasanna Santhanam wrote:
> > >> Dave's been attempting to fix this and it looks to be intertwined in a
> > >> few commits. Large swaths of code has moved around. I'll revert to
> > >> last known state from Jenkins if we can't figure out a simple fix so
> > >> Dave can proceed.
> > >>
> > >> On Thu, Aug 08, 2013 at 04:11:36PM +0900, Dave Cahill wrote:
> > >> > Hi,
> > >> >
> > >> > As Prasanna noted in IRC, the cloudstack-master-maven Jenkins build is
> > >> > failing with a similar error:
> > >> >
> > >> > https://builds.apache.org/job/cloudstack-master-maven/2645/console
> > >> >
> > >> > Failed to execute goal on project cloud-engine-components-api: Could
> > >> > not resolve dependencies for project
> > >> > org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
> > >> > Could not find artifact
> > >> > org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
> > >> > Nexus (http://repository.apache.org/snapshots)
> > >> > cause : Could not resolve dependencies for project
> > >> > org.apache.cloudstack:cloud-engine-components-api:jar:4.3.0-SNAPSHOT:
> > >> > Could not find artifact
> > >> > org.apache.cloudstack:cloud-framework-config:jar:4.3.0-SNAPSHOT in
> > >> > Nexus (http://repository.apache.org/snapshots)
> > >> >
> > >> >
> > >> > Thanks,
> > >> >
> > >> > Dave.
> > >> >
> > >> >
> > >> > On Thu, Aug 8, 2013 at 3:53 PM, Dave Cahill  > > wrote:
> > >> >
> > >> > > Hi,
> > >> > >
> > >> > > I'm seeing a build failure with master - I've checked the usual
> > gotchas,
> > >> > > but
> > >> > > this seems to be something else. Possibly related to the 4.2.0 to
> > 4.3.0
> > >> > > update?
> > >> > >
> > >> > > The missing dependency is cloud-engine-compute. It's listed as a
> > >> > > dependency in
> > >> > > engine/se

Re: Quick Question about System VM problem

2013-08-09 Thread Prasanna Santhanam
On Fri, Aug 09, 2013 at 06:53:55AM +, Donal Lafferty wrote:
> https://issues.apache.org/jira/browse/CLOUDSTACK-4082
> 
> Not sure who to assign it to ...  so I chose you :)

I wish vhd-util had versioning in it. But I can't see anywhere within
CloudStack where we invoke the -S (capital) option. Is it possible you
have some local changes to these scripts copy_vhd*?


> 
> > -Original Message-
> > From: Prasanna Santhanam [mailto:t...@apache.org]
> > Sent: 09 August 2013 05:22
> > To: dev@cloudstack.apache.org
> > Subject: Re: Quick Question about System VM problem
> > 
> > On Thu, Aug 08, 2013 at 10:25:28PM +, Donal Lafferty wrote:
> > > There's a tag that is applied to you XenServer.  AFAIK, the tag tells
> > > the management server not to download systemvm.iso
> > >
> > > Also, be careful not to overwrite the vhd-utils on a XenServer version
> > > 6.x and beyond.  The vhd-util mentioned on the build from master page
> > > does not work with new versions of XenServer.  It should be updated
> > > soon.
> > 
> > What is the issue with the vhd-util on download.cloud.com? Can you please
> > file a bug? I've been using the vhd-util on download.cloud.com and haven't
> > faced any issues with my Xen 6.0.2 hosts.
> > 
> > 
> > --
> > Prasanna.,
> > 
> > 
> > Powered by BigRock.com

-- 
Prasanna.,


Powered by BigRock.com



RE: [DISCUSS][4.2] Using full-clone by default for VMware deployment

2013-08-09 Thread Sateesh Chodapuneedi
+1 for this proposal to make full-clone as default.

-Sateesh

> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 10 August 2013 06:37
> To: dev@cloudstack.apache.org
> Subject: [DISCUSS][4.2] Using full-clone by default for VMware deployment
> 
> To better integrate with VMware features like storage live migration,  fully 
> utilize third-party integrations from vSphere eco-system, I'd like
> to change our default volume deploy mode from linked-clone to full-clone in 
> VMware.
> 
> If there is no objections, I'll make the change and starting from 4.2 
> release, we will use full-clone mode by default for VMware
> deployments.
> 
> Kelven



RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-09 Thread Sateesh Chodapuneedi
Hearty Congratulations, Vijay! 
Well deserved.

Regards,
Sateesh

> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: 10 August 2013 10:31
> To: CloudStack Dev
> Subject: [ANNOUNCE] New Committer: Vijay Bhamidipati
> 
> The Project Management Committee (PMC) for Apache CloudStack has asked Vijay 
> Bhamidipati to become a committer and we are pleased
> to announce that they have accepted.
> 
> Being a committer allows many contributors to contribute more autonomously. 
> For developers, it makes it easier to submit changes and
> eliminates the need to have contributions reviewed via the patch submission 
> process. Whether contributions are development-related or
> otherwise, it is a recognition of a contributor's participation in the 
> project and commitment to the project and the Apache Way.
> 
> Please join me in congratulating Vijay!
> 
> --
> Prasanna.,
> on behalf of the Apache CloudStack PMC
> 
> 
> Powered by BigRock.com



RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-09 Thread Sailaja Mada
Congratulations Vijay.

Regards,
Sailaja.M

-Original Message-
From: Prasanna Santhanam [mailto:t...@apache.org] 
Sent: Saturday, August 10, 2013 10:32 AM
To: CloudStack Dev
Subject: [ANNOUNCE] New Committer: Vijay Bhamidipati

The Project Management Committee (PMC) for Apache CloudStack has asked Vijay 
Bhamidipati to become a committer and we are pleased to announce that they have 
accepted.

Being a committer allows many contributors to contribute more autonomously. For 
developers, it makes it easier to submit changes and eliminates the need to 
have contributions reviewed via the patch submission process. Whether 
contributions are development-related or otherwise, it is a recognition of a 
contributor's participation in the project and commitment to the project and 
the Apache Way.

Please join me in congratulating Vijay!

--
Prasanna.,
on behalf of the Apache CloudStack PMC


Powered by BigRock.com