RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Sanjay Tripathi]

Congratulations Vijay!

--Sanjay

> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: Saturday, August 10, 2013 10:31 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

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Abhinav Roy]

Congrats Vijay.

Regards,
Abhinav

-Original Message-
From: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.com] 
Sent: Sunday, August 11, 2013 12:53 PM
To: dev@cloudstack.apache.org
Subject: RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

Congratulations Vijay!

--Sanjay

> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: Saturday, August 10, 2013 10:31 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

Review Request 13482: Fix test_snapshots script to choose the right device name while mounting and other minor script issues

[SrikanteswaraRao Talluri]

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

Review request for cloudstack and Prasanna Santhanam.


Bugs: CLOUDSTACK-4096


Repository: cloudstack-git


Description
---

Following are the fixes:
1. don't rely on prepopulated data for choosing the disk to be mounted
   a. get the list of volumes
   b. take the latest volume and add the proper device name for each hypervisor
  xenserver - /dev/xvd{a|b...}
  vmware - /dev/sd{a|b|..}
  kvm - /dev/vd{a|b|...}
2. Get the right volume for taking snapshot.
and other script issues are also fixed.


Diffs
-

  test/integration/component/test_snapshots.py b97a83a 

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


Testing
---

tested


Thanks,

SrikanteswaraRao Talluri

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

[Daan Hoogland]

ok, the down side will be that the semantics of the versioning are
confusing when cloudstack 5 comes and the version numbers don't match.
still a +1 from me though.

On Sat, Aug 10, 2013 at 8:54 PM, Rohit Yadav  wrote:
> On Sat, Aug 10, 2013 at 10:32 PM, Daan Hoogland 
> wrote:
>
>> +1 +question
>>
>> Is cloudmonkey 5 backwards compatible in the sense that it can talk to a
>> 4.x ms?
>>
>
> Sorry for the confusion. Yes, in fact it is. It's the same cloudmonkey and
> is aimed to work with Apache CloudStack 4.0.0-incubating and its
> derivatives. If it does not it's a bug.
>
> We're gaming the version number, since cloudmonkey-4.x was already out
> here, it would create confusion if we release cloudmonkey-1.x. Therefore,
> it made sense to just start with 5.0.0 and use semver from this version.
>
> Regards.
>
>
>>
>> On Sat, Aug 10, 2013 at 5:50 AM, David Nalley  wrote:
>> > +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/
>> >>>
>>

Re: Review Request 13468: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm; Zone host is ready, but secondary storage vm template: 3 is not ready on secondary storag

[Abhinandan Prateek]

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

Ship it!


Ship It!

- Abhinandan Prateek


On Aug. 10, 2013, 11:23 a.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13468/
> ---
> 
> (Updated Aug. 10, 2013, 11:23 a.m.)
> 
> 
> Review request for cloudstack and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-3228
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3228:  system vms are not comming up in zone with two cluster xen 
> and kvm;Zone host is ready, but secondary storage vm template: 3 is not ready 
> on secondary storage: 2 
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java c5faccb 
>   server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 
> d1c148b 
> 
> Diff: https://reviews.apache.org/r/13468/diff/
> 
> 
> Testing
> ---
> 
> Tested the scenario mentioned in the bug ticket
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

Re: Review Request 13468: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm; Zone host is ready, but secondary storage vm template: 3 is not ready on secondary storag

[ASF Subversion and Git Services]

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


Commit 2868d4b3369b7d31047b1baf699765232cb99403 in branch refs/heads/master 
from Harikrishna Patnala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=2868d4b ]

CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and 
kvm; Zone host is ready, but secondary storage vm template: 3 is not ready on 
secondary storage: 2

Signed-off-by: Abhinandan Prateek 


- ASF Subversion and Git Services


On Aug. 10, 2013, 11:23 a.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13468/
> ---
> 
> (Updated Aug. 10, 2013, 11:23 a.m.)
> 
> 
> Review request for cloudstack and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-3228
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3228:  system vms are not comming up in zone with two cluster xen 
> and kvm;Zone host is ready, but secondary storage vm template: 3 is not ready 
> on secondary storage: 2 
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java c5faccb 
>   server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 
> d1c148b 
> 
> Diff: https://reviews.apache.org/r/13468/diff/
> 
> 
> Testing
> ---
> 
> Tested the scenario mentioned in the bug ticket
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

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

[Koushik Das]

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


'mvn clean install' on 4.2 branch resulted in build failure. XS unit test is 
failing.


[INFO] 
[INFO] Building Apache CloudStack Plugin - Hypervisor Xen 4.2.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Deleting 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/target 
(includes = [**/*], excludes = [])
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-hypervisor-xen ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/src/main/resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 19 source files to 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/target/classes
[WARNING] Unable to autodetect 'javac' path, using 'javac' from the environment.
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-hypervisor-xen ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/test/resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Compiling 1 source file to 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/target/test-classes
[WARNING] Unable to autodetect 'javac' path, using 'javac' from the environment.
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-hypervisor-xen ---
[INFO] Surefire report directory: 
/Users/koushik/code/cloudstack-apache/cloudstack/plugins/hypervisors/xen/target/surefire-reports

---
 T E S T S
---
Running com.cloud.hypervisor.xen.resource.CitrixResourceBaseTest
log4j:WARN No appenders could be found for logger 
(com.cloud.hypervisor.xen.resource.XenServerConnectionPool).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.884 sec <<< 
FAILURE!

Results :

Failed tests:   
testScaleVMF3(com.cloud.hypervisor.xen.resource.CitrixResourceBaseTest): (..)

Tests run: 3, Failures: 1, Errors: 0, Skipped: 0


- Koushik Das


On Aug. 10, 2013, 7:08 p.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13441/
> ---
> 
> (Updated Aug. 10, 2013, 7:08 p.m.)
> 
> 
> 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 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

[Harikrishna Patnala]

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

(Updated Aug. 11, 2013, 5:28 p.m.)


Review request for cloudstack and Koushik Das.


Changes
---

added unit test changes


Bugs: CLOUDSTACK-3850


Repository: cloudstack-git


Description
---

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


Diffs (updated)
-

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

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


Testing
---


Thanks,

Harikrishna Patnala

Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Harikrishna Patnala]

Congratulations

-Harikrishna

On 11-Aug-2013, at 12:55 PM, Abhinav Roy  wrote:

> Congrats Vijay.
> 
> Regards,
> Abhinav
> 
> -Original Message-
> From: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.com] 
> Sent: Sunday, August 11, 2013 12:53 PM
> To: dev@cloudstack.apache.org
> Subject: RE: [ANNOUNCE] New Committer: Vijay Bhamidipati
> 
> Congratulations Vijay!
> 
> --Sanjay
> 
>> -Original Message-
>> From: Prasanna Santhanam [mailto:t...@apache.org]
>> Sent: Saturday, August 10, 2013 10:31 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
> 

RE: HyperV support in 4.2

[Donal Lafferty]

Done!

> -Original Message-
> From: Animesh Chaturvedi
> Sent: 09 August 2013 22:18
> To: dev@cloudstack.apache.org; Chip Childers
> Cc: Donal Lafferty; Sudha Ponnaganti
> Subject: RE: HyperV support in 4.2
> 
> 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

[Donal Lafferty]

Hi Alena,

I had a closer look at this.

As before, Hyper-V is not available on 4.2.  It should be in place for 4.3. 

However, I noticed that the patch applied in June 27th that introduces 
"HypervisorType.Hyperv" to the list of configured hypervisors. See 
https://reviews.apache.org/r/12106/, especially 
server/src/com/cloud/configuration/Config.java@205

Removing this item should solve the problem of Hyper-V showing up where the 
hypervisor list is used to populate a dropdown box.  

I don't have committer status, so I'm not able to take care of this directly.  
:(

As before, for any other the settings, you could talk to Murali Reddy.  
However, he added them quite some time ago.  A look through the config may be 
the best approach.

DL


> -Original Message-
> From: Animesh Chaturvedi
> Sent: 09 August 2013 22:18
> To: dev@cloudstack.apache.org; Chip Childers
> Cc: Donal Lafferty; Sudha Ponnaganti
> Subject: RE: HyperV support in 4.2
> 
> 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: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Isaac Chiang]

Congratz! :)


On Mon, Aug 12, 2013 at 1:35 AM, Harikrishna Patnala <
harikrishna.patn...@citrix.com> wrote:

> Congratulations
>
> -Harikrishna
>
> On 11-Aug-2013, at 12:55 PM, Abhinav Roy  wrote:
>
> > Congrats Vijay.
> >
> > Regards,
> > Abhinav
> >
> > -Original Message-
> > From: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.com]
> > Sent: Sunday, August 11, 2013 12:53 PM
> > To: dev@cloudstack.apache.org
> > Subject: RE: [ANNOUNCE] New Committer: Vijay Bhamidipati
> >
> > Congratulations Vijay!
> >
> > --Sanjay
> >
> >> -Original Message-
> >> From: Prasanna Santhanam [mailto:t...@apache.org]
> >> Sent: Saturday, August 10, 2013 10:31 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
> >
>
>

Re: Review Request 13468: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm; Zone host is ready, but secondary storage vm template: 3 is not ready on secondary storag

[ASF Subversion and Git Services]

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


Commit 97b8f123a0e75b7ccfa01f3790062bfc25d17d98 in branch refs/heads/4.2 from 
Harikrishna Patnala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=97b8f12 ]

CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and 
kvm;Zone host is ready, but secondary storage vm template: 3 is not ready on 
secondary storage: 2

Signed-off-by: Abhinandan Prateek 


- ASF Subversion and Git Services


On Aug. 10, 2013, 11:23 a.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13468/
> ---
> 
> (Updated Aug. 10, 2013, 11:23 a.m.)
> 
> 
> Review request for cloudstack and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-3228
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> CLOUDSTACK-3228:  system vms are not comming up in zone with two cluster xen 
> and kvm;Zone host is ready, but secondary storage vm template: 3 is not ready 
> on secondary storage: 2 
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java c5faccb 
>   server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 
> d1c148b 
> 
> Diff: https://reviews.apache.org/r/13468/diff/
> 
> 
> Testing
> ---
> 
> Tested the scenario mentioned in the bug ticket
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

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

[Koushik Das]

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

Ship it!


Ship It!

- Koushik Das


On Aug. 11, 2013, 5:28 p.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13441/
> ---
> 
> (Updated Aug. 11, 2013, 5:28 p.m.)
> 
> 
> 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 
>   
> plugins/hypervisors/xen/test/com/cloud/hypervisor/xen/resource/CitrixResourceBaseTest.java
>  cb16ae2 
> 
> Diff: https://reviews.apache.org/r/13441/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

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

[ASF Subversion and Git Services]

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


Commit a7dea8eecdee071e09cc90d6e7e47c8b93746bb1 in branch refs/heads/4.2 from 
Harikrishna Patnala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a7dea8e ]

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

Signed-off-by: Koushik Das 


- ASF Subversion and Git Services


On Aug. 11, 2013, 5:28 p.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13441/
> ---
> 
> (Updated Aug. 11, 2013, 5:28 p.m.)
> 
> 
> 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 
>   
> plugins/hypervisors/xen/test/com/cloud/hypervisor/xen/resource/CitrixResourceBaseTest.java
>  cb16ae2 
> 
> Diff: https://reviews.apache.org/r/13441/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

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

[ASF Subversion and Git Services]

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


Commit 726114a95df0c0525617225e07295b8dff10a8af in branch refs/heads/master 
from Harikrishna Patnala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=726114a ]

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

Signed-off-by: Koushik Das 


- ASF Subversion and Git Services


On Aug. 11, 2013, 5:28 p.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13441/
> ---
> 
> (Updated Aug. 11, 2013, 5:28 p.m.)
> 
> 
> 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 
>   
> plugins/hypervisors/xen/test/com/cloud/hypervisor/xen/resource/CitrixResourceBaseTest.java
>  cb16ae2 
> 
> Diff: https://reviews.apache.org/r/13441/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Radhika Puthiyetath]

Congrats Vijay...:-)

-Original Message-
From: Prasanna Santhanam [mailto:t...@apache.org] 
Sent: Saturday, August 10, 2013 10:31 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

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Animesh Chaturvedi]

Congratulations Vijay

Animesh

> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: Friday, August 09, 2013 10:01 PM
> 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

Unable to access ASF repo

[Rayees Namathponnan]

Hi All,

We are unable to access ASF repo "git-wip-us.apache.org" from our cooperate 
network (Citrix) ,   both ssh or ping not working;  same time  I am not facing 
this issue from outside office network.

Is anyone can please help on this ?

Regards,
Rayees

RE: Unable to access ASF repo

[Animesh Chaturvedi]

Rayees did you open up a ticket with Apache INFRA 
https://issues.apache.org/jira/browse/INFRA


> -Original Message-
> From: Rayees Namathponnan [mailto:rayees.namathpon...@citrix.com]
> Sent: Sunday, August 11, 2013 9:16 PM
> To: dev@cloudstack.apache.org
> Subject: Unable to access ASF repo
> 
> Hi All,
> 
> We are unable to access ASF repo "git-wip-us.apache.org" from our
> cooperate network (Citrix) ,   both ssh or ping not working;  same time
> I am not facing this issue from outside office network.
> 
> Is anyone can please help on this ?
> 
> Regards,
> Rayees

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Likitha Shetty]

Congratulations Vijay !

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

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Saksham Srivastava]

Congrats Vijay :)

From: Prasanna Santhanam [t...@apache.org]
Sent: Saturday, August 10, 2013 10:30 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

RE: Unable to access ASF repo

[Rayees Namathponnan]

Ticket https://issues.apache.org/jira/browse/INFRA-6657 created  


-Original Message-
From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] 
Sent: Sunday, August 11, 2013 9:21 PM
To: dev@cloudstack.apache.org
Subject: RE: Unable to access ASF repo 

Rayees did you open up a ticket with Apache INFRA 
https://issues.apache.org/jira/browse/INFRA


> -Original Message-
> From: Rayees Namathponnan [mailto:rayees.namathpon...@citrix.com]
> Sent: Sunday, August 11, 2013 9:16 PM
> To: dev@cloudstack.apache.org
> Subject: Unable to access ASF repo
> 
> Hi All,
> 
> We are unable to access ASF repo "git-wip-us.apache.org" from our
> cooperate network (Citrix) ,   both ssh or ping not working;  same time
> I am not facing this issue from outside office network.
> 
> Is anyone can please help on this ?
> 
> Regards,
> Rayees

Re: Review Request 13472: make apache server listen on all the ipaliases to provide access to meta-data.

[bharat kumar]

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

(Updated Aug. 12, 2013, 4:37 a.m.)


Review request for cloudstack, Chiradeep Vittal, Murali Reddy, and Sheng Yang.


Changes
---

Adding Murali to the review request.


Bugs: Cloudstack-4231


Repository: cloudstack-git


Description
---

https://issues.apache.org/jira/browse/CLOUDSTACK-4231
make apache server listen on all the ipaliases to provide access to meta-data.

Earlier we were using the name data-server to access the meta-data in case of 
multiple subnets. 
This requires updating the guest templates. 

Now we make apache server listen on all the ipaliases. So no need to update the 
template.
we create separate file for each ipAlias 


Diffs
-

  patches/systemvm/debian/config/etc/init.d/cloud-early-config a552d44 
  patches/systemvm/debian/config/root/createIpAlias.sh 5498195 
  patches/systemvm/debian/config/root/deleteIpAlias.sh fa228fb 

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


Testing
---

Tested on 4.2
deployed vms in different subnets.
used the script cloud-set-guest-sshkey.in to fetch the public keys form the 
router.
was able to get the keys without using the name data-server.

deleted the subnet and checked if the apache config was removed.

ran a failure case, all the config files generated during this operation were 
moved to failure_directory.


Thanks,

bharat kumar

Re: Unable to access ASF repo

[Dave Brosius]

I assume you company has a network proxy?

|git config --global http.proxy http://your.proxy.server:8080|



On 08/12/2013 12:32 AM, Rayees Namathponnan wrote:

Ticket https://issues.apache.org/jira/browse/INFRA-6657 created


-Original Message-
From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
Sent: Sunday, August 11, 2013 9:21 PM
To: dev@cloudstack.apache.org
Subject: RE: Unable to access ASF repo

Rayees did you open up a ticket with Apache INFRA 
https://issues.apache.org/jira/browse/INFRA



-Original Message-
From: Rayees Namathponnan [mailto:rayees.namathpon...@citrix.com]
Sent: Sunday, August 11, 2013 9:16 PM
To: dev@cloudstack.apache.org
Subject: Unable to access ASF repo

Hi All,

We are unable to access ASF repo "git-wip-us.apache.org" from our
cooperate network (Citrix) ,   both ssh or ping not working;  same time
I am not facing this issue from outside office network.

Is anyone can please help on this ?

Regards,
Rayees

Re: Review Request 12948: Affinity Groups: Adding new test cases

[Prasanna Santhanam]

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


Can you please close this review as discarded if these tests are already 
corrected?

- Prasanna Santhanam


On July 26, 2013, 9:43 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12948/
> ---
> 
> (Updated July 26, 2013, 9:43 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Adding new test cases for affinity groups
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_affinity_groups.py 3ecab6f 
> 
> Diff: https://reviews.apache.org/r/12948/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: Review Request 13482: Fix test_snapshots script to choose the right device name while mounting and other minor script issues

[Prasanna Santhanam]

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



test/integration/component/test_snapshots.py


Remove the "hello" tag



test/integration/component/test_snapshots.py


It's better to have one master list with all device names to avoid putting 
the if-else logic everywhere. I guess that makes correcting the test expensive 
when a new device name needs to be added. 

Also you'll miss partitioned disks of the form /dev/hdb1, /dev/hdb2 etc 
that some vmware guests have using this patch.


- Prasanna Santhanam


On Aug. 11, 2013, 7:28 a.m., SrikanteswaraRao Talluri wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13482/
> ---
> 
> (Updated Aug. 11, 2013, 7:28 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4096
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Following are the fixes:
> 1. don't rely on prepopulated data for choosing the disk to be mounted
>a. get the list of volumes
>b. take the latest volume and add the proper device name for each 
> hypervisor
>   xenserver - /dev/xvd{a|b...}
>   vmware - /dev/sd{a|b|..}
>   kvm - /dev/vd{a|b|...}
> 2. Get the right volume for taking snapshot.
> and other script issues are also fixed.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_snapshots.py b97a83a 
> 
> Diff: https://reviews.apache.org/r/13482/diff/
> 
> 
> Testing
> ---
> 
> tested
> 
> 
> Thanks,
> 
> SrikanteswaraRao Talluri
> 
>

Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Anshul Gangwar]

Congrats Vijay!

On Saturday 10 August 2013 10:30 AM, 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!
>

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

[Krishnan Ganapathy]

Congratulations Vijay. 

> From: anshul.gang...@citrix.com
> To: dev@cloudstack.apache.org
> Subject: Re: [ANNOUNCE] New Committer: Vijay Bhamidipati
> Date: Mon, 12 Aug 2013 05:55:27 +
> 
> Congrats Vijay!
> 
> On Saturday 10 August 2013 10:30 AM, 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!
> >
  

Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

[Rajesh Battala]

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

(Updated Aug. 12, 2013, 6:48 a.m.)


Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.


Changes
---

rebased with 4.2


Bugs: CLOUDSTACK-3946


Repository: cloudstack-git


Description
---

while migrating volume, if the storage and destination pool are same, we are 
still do overhead of copying the volume to same pool and deleting the other src 
copy.

fixed the issue by restricting volumemigration if the source and destination 
pool are same.


Diffs (updated)
-

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

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


Testing
---

1. Successfully migrated the volume from Storagepool A to pool B.
2. migration is not allowed if the source and destination are same.
3. tried the volume with storage pool scope (ZONE and Cluster)


Thanks,

Rajesh Battala

Review Request 13490: [DOC] add document - custom cpu model for kvm guest.

[Jijun]

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

Review request for cloudstack and Wei Zhou.


Repository: cloudstack-git


Description
---

add document for custom cpu model for kvm guest.


Diffs
-

  docs/en-US/hypervisor-host-install-agent.xml 41b6719 

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


Testing
---


Thanks,

Jijun