[Doc] Please review: VM root disk reset on reboot doc

2013-08-09 Thread Jessica Tomechak
Hi,
New doc on resetting VM on reboot is ready for review. You can view the
updated text at the link below. This is checked in to both master and 4.2
branch.

Please provide your feedback.

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

Bug:
https://issues.apache.org/jira/browse/CLOUDSTACK-1682

Regards,
Jessica T.


Re: [Discuss] Using XenServer 6.2's Clone-on-Boot feature on CloudStack

2013-08-09 Thread Harikrishna Patnala
If we can control fall back of IntelliCache reads and writes back to the 
storage then it won't hinder with host maintenance mode and Xenmotion, and I 
think it helps handling volatile VMs without recreating the root disks.

Thanks,
Harikrishna

On 09-Aug-2013, at 11:39 AM, Prasanna Santhanam t...@apache.org wrote:

 On Fri, Aug 09, 2013 at 09:34:38AM +0700, Nguyen Anh Tu wrote:
 2013/8/9 Chiradeep Vittal chiradeep.vit...@citrix.com
 
 I think Intellicache is different. I found a discussion here:
 http://s.apache.org/NIX
 
 
 I see no different. Maybe I didn't explain obviously. Let see it:
 http://speakvirtual.com/2012/02/06/intellicache-what-how-and-when/
 http://blogs.citrix.com/2011/06/22/xendesktop-and-local-storage-intellicache/
 
 
 It appears Intellicache hinders with putting hosts into maintenance
 and the recently introduced XenMotion:
 https://cwiki.apache.org/confluence/x/TiTVAQ feature in cloudstack
 
 -- 
 Prasanna.,
 
 
 Powered by BigRock.com
 



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 imusa...@webmd.net 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 imusa...@webmd.net
 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
  chip.child...@sungard.comwrote:
 
   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
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*
 




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 dcah...@midokura.com 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 daan.hoogl...@gmail.com
 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 alex.hu...@citrix.com 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
  t...@apache.orgmailto: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
   dcah...@midokura.commailto:dcah...@midokura.com 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

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?


/interface
interface type='bridge'
source bridge='cloudbr0'/
mac address='06:45:c4:00:00:0c'/
model type='virtio'/
bandwidth
inbound average='25600' peak='25600'/
outbound average='25600' peak='25600'/
/bandwidth
/interface


 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 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



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 edison...@citrix.com

 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:

 /interface
 interface type='bridge'
 source bridge='cloudbr0'/
 mac address='06:45:c4:00:00:0c'/
 model type='virtio'/
 bandwidth
 inbound average='25600' peak='25600'/
 outbound average='25600' peak='25600'/
 /bandwidth
 /interface


 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



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: OVS

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


On 8/9/13 6:13 AM, Thomas Schneider thomas.schnei...@euskill.com 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: 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
https://reviews.apache.org/r/12849/#comment49071

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:uuid 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
  https://reviews.apache.org/r/12849/diff/9/?file=338752#file338752line172
 
  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:uuid 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: [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 imusa...@webmd.net
 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 imusa...@webmd.net
  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
   chip.child...@sungard.comwrote:
  
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
   cloudhttp://solidfire.com/solution/overview/?video=play
   *(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.
 
  /interface
  interface type='bridge'
  source bridge='cloudbr0'/
  mac address='06:45:c4:00:00:0c'/
  model type='virtio'/
  bandwidth
  inbound average='25600' peak='25600'/ outbound average='25600'
  peak='25600'/ /bandwidth /interface
 
 
   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
bandwidth
inbound average='25600' peak='25600'/
outbound average='25600' peak='25600'/
  /bandwidth

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.
 
   /interface
   interface type='bridge'
   source bridge='cloudbr0'/
   mac address='06:45:c4:00:00:0c'/
   model type='virtio'/
   bandwidth
   inbound average='25600' peak='25600'/ outbound average='25600'
   peak='25600'/ /bandwidth /interface
  
  
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 bandwidth
 inbound average='25600' peak='25600'/
 outbound average='25600' peak='25600'/
   /bandwidth
 
 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 donal.laffe...@citrix.comwrote:



  -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
  donal.laffe...@citrix.comwrote:
 
   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 thomas.schnei...@euskill.com
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 thomas.schnei...@euskill.com
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.


Re: HyperV support in 4.2

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

On 8/9/13 2:02 PM, Alena Prokharchyk alena.prokharc...@citrix.com
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 
animesh.chaturv...@citrix.commailto:animesh.chaturv...@citrix.com
Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Date: Friday, August 9, 2013 2:16 PM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org, Chip Childers 
chip.child...@sungard.commailto:chip.child...@sungard.com
Cc: Donal Lafferty donal.laffe...@citrix.commailto: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.orgmailto: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
 donal.laffe...@citrix.comwrote:
 
 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=loginusername= + 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=jsonid=' + 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=jsonid=' + 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=subprocess.PIPE)
(output, err) = 

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%2Fts=1331225219use_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 rohityada...@gmail.com 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 bhais...@apache.org 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 t...@apache.org 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 daan.hoogl...@gmail.comwrote:
 
  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 alex.hu...@citrix.com 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 t...@apache.org
  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 dcah...@midokura.com
  mailto:dcah...@midokura.com 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/service/pom.xml with no type specified, which AFAIK
  defaults to
 jar.
 So my understanding is that it's trying to download version
  4.3.0-SNAPSHOT
 of the cloud-engine-compute jar from repository.apache.org
  http://repository.apache.org. How does the
 jar get
 uploaded there in the first place? 

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