Review Request 15014: allow physical devices as guest traffic label, vxlan bridge name fix

2013-10-28 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15014/ --- Review request for cloudstack, Toshiaki Hatano and Yoshikazu Nojima. Bugs: CLOU

Re: ifconfig and MacAddress

2013-10-28 Thread Laszlo Hornyak
Hi, I noticed other problem with the MacAddress, it appears that it does not handle the Windows ipconfig output. (see related patch) https://reviews.apache.org/r/14514/ I tested it with win8, 7, and XP outputs As far as I remember this may be a problem when running CS cluster on windows, since ea

Re: [Merge] hyperv branch to master

2013-10-28 Thread Chiradeep Vittal
+1 On 10/28/13 4:41 AM, "Devdeep Singh" wrote: >Hi, > >I would like to merge the support for Hyperv to the master branch. >Development for this has been done by Donal, Rajesh, Anshul and I on >branch [1]. The feature was proposed for merge earlier [3] but unit tests >for hyperv agent code were r

Re: Review Request 14867: api call to import ldap users to the same domains in cloudstack

2013-10-28 Thread Rajani Karuturi
> On Oct. 23, 2013, 6:22 a.m., Prasanna Santhanam wrote: > > How is the import behaviour when OUs contain other OUs? > > Rajani Karuturi wrote: > That is still not done. (Its the second part of TODO:1, nested hierarchy) > > Prasanna Santhanam wrote: > Ah, didn't notice the TODO. Is ther

Re: Review Request 15012: CLOUDSTACK-4984: decrement MAX_VXLAN_VNI to be aligned with Linux kernel

2013-10-28 Thread Toshiaki Hatano
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15012/#review27660 --- Ship it! Ship It! - Toshiaki Hatano On Oct. 29, 2013, 5 a.m., Yo

Review Request 15012: CLOUDSTACK-4984: decrement MAX_VXLAN_VNI to be aligned with Linux kernel

2013-10-28 Thread Yoshikazu Nojima
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15012/ --- Review request for cloudstack, Marcus Sorensen and Toshiaki Hatano. Repository:

Re: Review Request 15001: CLOUDSTACK-4967: Bugfix: Guest network isn't created correctly in rare case when vxlan is used for isolation method

2013-10-28 Thread Toshiaki Hatano
> On Oct. 29, 2013, 4:35 a.m., Toshiaki Hatano wrote: > > Actual issue is not "the physical interface name length must be shorter than 5" but "generated bridge name cannot exceed 15 character". > On Oct. 29, 2013, 4:35 a.m., Toshiaki Hatano wrote: > > plugins/hypervisors/kvm/src/com/cloud/hyp

Re: Review Request 15001: CLOUDSTACK-4967: Bugfix: Guest network isn't created correctly in rare case when vxlan is used for isolation method

2013-10-28 Thread Toshiaki Hatano
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15001/#review27657 --- plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/Bridg

RE: Review Request 14827: hyperv unit tests

2013-10-28 Thread Anshul Gangwar
Hi Donal, We don't have to give -DskipTests=false specifically. They will be triggered by default i.e. mvn clean install -P hyperv-agent will also trigger the C# unit tests. Thanks, Anshul -Original Message- From: Donal Lafferty Sent: Monday, October 28, 2013 4:58 PM To: dev@cloudst

Re: Review Request 14889: Expose as much type information to commands.xml as well as to the generated python clasesss for Marvin

2013-10-28 Thread Ryan Dietrich
> On Oct. 25, 2013, 6:10 a.m., Prasanna Santhanam wrote: > > I actually don't quite see how marvin uses the typeInfo. Will it only act > > as a carrier for this information? > > > > How would a test case author use this? It would've been nice if we could > > use the type info to deserialize t

Re: Review Request 14889: Expose as much type information to commands.xml as well as to the generated python clasesss for Marvin

2013-10-28 Thread Ryan Dietrich
> On Oct. 28, 2013, 6:06 a.m., Prasanna Santhanam wrote: > > One last thing - if you change the ApiXmlDocWriter to include type > > information, you might want to include that in the API discovery plugin as > > well. Many of the downstream tools (cloudmonkey/marvin) use the listApis > > call t

Re: Wiki access

2013-10-28 Thread Nitin Mehta
Please grant me edit access. Email - nitin.me...@citrix.com On 28/10/13 5:17 AM, "Gaurav Aradhye" wrote: >Hi, > >My username is gauravaradhye. >Can anyone grant me edit permission? > >I need to update test plan automation columns. > > >Regards, >Gaurav > > >On Fri, Oct 18, 2013 at 5:26 PM, Kisha

Re: Tiered Quality

2013-10-28 Thread Darren Shepherd
Starting with the honor system might be good. It's not so easy some times to relate lines of code to functionality. Also just because it hits a line of code doesn't mean it's really tested. Can't we just get people to just put a check mark on some table in the wiki? Darren > On Oct 28, 2013,

Re: LXC and SSVM/CPVM on the host

2013-10-28 Thread Kelven Yang
As long as the container VNC console is reachable from console proxy agent, I don't see a technical issue here. Need some log details to understand the situation. For example, what information does LXC hypervisor resource agent returns on GetVncPort command? Kelven On 10/28/13 10:59 AM, "Franco

Review Request 15001: CLOUDSTACK-4967: Bugfix: Guest network isn't created correctly in rare case when vxlan is used for isolation method

2013-10-28 Thread Yoshikazu Nojima
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15001/ --- Review request for cloudstack, Marcus Sorensen and Toshiaki Hatano. Bugs: CLOUD

RE: RN-KnownIssuesIn4.2

2013-10-28 Thread Animesh Chaturvedi
> -Original Message- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: Monday, October 28, 2013 1:31 AM > To: dev@cloudstack.apache.org; Animesh Chaturvedi > Subject: Re: RN-KnownIssuesIn4.2 > > Animesh owns that filter, > > On Oct 27, 2013, at 6:39 AM, Marty Sweet wrote: >

Re: ifconfig and MacAddress

2013-10-28 Thread Kelven Yang
In a setup with multiple interfaces or Multiple IPs on one interfaces, it sounds more simple to just get the decision resolved through configuration. Looking for interfaces that has IP configured as what "cluster.node.IP" says so, can get us a deterministic mac address. I'm not sure about a KVM ca

Re: Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Alena Prokharchyk
Got it. Anything that is specific to the LB rule, should go to loadbalacingrules table. -Alena. From: Syed Ahmed mailto:sah...@cloudops.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Monday, October 28, 2013 2:13 PM To: "de

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marty Sweet
To add to my last reply, I have just created the following compute examples, tests all run with the VM launch on the same Hypervisor: === Offering with 50Mbps === # tc class show dev

RE: [ASF4.2.1] default to 64-bit system VM template

2013-10-28 Thread Animesh Chaturvedi
> -Original Message- > From: Wido den Hollander [mailto:w...@widodh.nl] > Sent: Tuesday, October 22, 2013 11:24 PM > To: dev@cloudstack.apache.org > Subject: Re: [ASF4.2.1] default to 64-bit system VM template > > On 10/23/2013 07:32 AM, Abhinandan Prateek wrote: > > > >We are planni

Re: Little GUI Issue

2013-10-28 Thread Mike Tutkowski
This one, as well: https://issues.apache.org/jira/browse/CLOUDSTACK-4981 On Mon, Oct 28, 2013 at 3:26 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > I logged this minor issue: > > https://issues.apache.org/jira/browse/CLOUDSTACK-4980 > > > On Mon, Oct 28, 2013 at 2:44 PM, Mike Tutk

Re: Little GUI Issue

2013-10-28 Thread Mike Tutkowski
I logged this minor issue: https://issues.apache.org/jira/browse/CLOUDSTACK-4980 On Mon, Oct 28, 2013 at 2:44 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > One other issue I see is when I try to log out: I can't. :) > > What happens is I click on the drop down to log out and the l

Re: Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Syed Ahmed
The problem with using protocol from firewall_rules is that it uses that field to implement firewall policies ( iptables etc ). The protocol should be tcp,udp or icmp. I would need lb_protocol for adding ssl and http protocols for loabalancing specifically. Thanks, -Syed On Mon 28 Oct 2013 0

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marty Sweet
Yeah, from the libvirt website (http://libvirt.org/cgroups.html): Network tuning The net_cls is not currently used. Instead traffic filter policies are set directly against individual virtual network interfaces. However, when bandwidth limiting is applied, I can't see any obvious rules with any

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marcus Sorensen
Check the XML that was generated when the VM in question was started: # virsh dumpxml i-2-15-VM | egrep "inbound|outbound" See if the settings match what you put in your network offering or properties (whichever applies to your situation). On Oct 28, 2013 1:44 PM, "Marty Swee

Re: Little GUI Issue

2013-10-28 Thread Mike Tutkowski
One other issue I see is when I try to log out: I can't. :) What happens is I click on the drop down to log out and the logout button shows up under the Default View combo box. If I move the mouse to click on it, the logout button disappears. :) On Mon, Oct 28, 2013 at 2:35 PM, Mike Tutkowski <

Re: Little GUI Issue

2013-10-28 Thread Mike Tutkowski
I'm using Chrome on Mac OS X 10.8.3. In my case, I can click the Notifications button at a very slow pace and reproduce the issue. Thanks! On Mon, Oct 28, 2013 at 2:34 PM, Brian Federle wrote: > Hey Mike, for some reason I can’t reproduce w/ the latest master, but I > have noticed that in the

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marcus Sorensen
It just uses the libvirt XML, which uses cgroups, which uses tc rules. On Mon, Oct 28, 2013 at 2:25 PM, Marty Sweet wrote: > Hi Marcus, > > My earlier email mentioned those configurations, unfortunately not really > complying with what was set out in the compute offering. > After setting the comp

RE: Little GUI Issue

2013-10-28 Thread Brian Federle
Hey Mike, for some reason I can't reproduce w/ the latest master, but I have noticed that in the past, if you accidentally double click on an item. Is that happening here, where repeated clicking is causing the overlay to load a bunch of times? Also, what browser did you reproduce on? -Brian F

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marty Sweet
Hi Marcus, My earlier email mentioned those configurations, unfortunately not really complying with what was set out in the compute offering. After setting the compute offering network limit and stop/starting the VMs the lines do not appear, and outbound traffic has returned to normal speeds but i

Little GUI Issue

2013-10-28 Thread Mike Tutkowski
Hey Brian, I could log a JIRA ticket for this, if you'd like, but I believe I had logged one a while ago and it was fixed (and is now broken again). The problem is the semi-transparent background when a dialog is displayed can get darker and darker if certain actions are taken. Here's a simple e

Re: Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Alena Prokharchyk
Why not use the protocol field in firewall_rules table? And make NS provider to read it from there. -Alena. From: Syed Ahmed mailto:sah...@cloudops.com>> Date: Monday, October 28, 2013 12:20 PM To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>>

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marty Sweet
Thanks for the links, while I have set 0 for all the properties the following results still occur: Guest -> Other Server : >900Mbps (As expected) Other Server -> Guest (so inbound to the VM) : Varies depending on hypervisor host: 121, 405, 233, 234Mbps Each hypervisor has 2 NICS in an LACP bond,

Re: Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Syed Ahmed
Thanks Alena, So adding lb_protocol in load_balancing_rules should work fine as long as the LB providers don't use the protocol field in the firewall_rules table to create the correct type of LB on the device which I don't see ( atleast for Netscaler). Thanks, -Syed On 13-10-28 03:11 PM,

Re: Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Alena Prokharchyk
Syed, each network rule – PF/LB – currently has entries in in 2 tables. PublicIp/PublicPort/Protocol + State/Purpose and other generic fields that are the same for all the networking rules, are being stored in firewall_rules. Other information is being stored in child tables. As an example, for

RE: Tiered Quality

2013-10-28 Thread Santhosh Edukulla
1.It seems we already have a code coverage numbers using sonar as below. It currently shows only the numbers for unit tests. https://analysis.apache.org/dashboard/index/100206 2. The below link has an explanation for using it for both integration and unit tests. http://docs.codehaus.org/displa

Adding "protocol" parameter to CreateLoadBalancerRule API

2013-10-28 Thread Syed Ahmed
Hi, As a part of implementing the SSL termination functioanlity, I was trying to add a protocol parameter to the CreateLoadBalancerRule API so that the correct type of LB can be created to which the certificate can be bound. I see that the LoadBalancerVO is an extension of FirewallVO which ha

Re: haPlanners error in fresh 4.2 checkout

2013-10-28 Thread Nitin Mehta
Probably cleaning up you workspace might help. On 28/10/13 10:51 AM, "Prachi Damle" wrote: >I had encountered this sometime back ... you need to check if your >componentContext.xml or its equivalent has following defined. However I >do see it is present in the 4.2 branch > > > > >

RE: VPC virtual router fails to start up - no route to CS Management Server.

2013-10-28 Thread Enrique Ladrón de Guevara Hernández
Thanks to everyone. CS 4.2 fixed this issue in my case ...!! Regards. > From: imusa...@webmd.net > To: dev@cloudstack.apache.org; us...@cloudstack.apache.org > Subject: RE: VPC virtual router fails to start up - no route to CS Management > Server. > Date: Fri, 6 Sep 2013 18:36:43 + > > Both

Re: LXC and SSVM/CPVM on the host

2013-10-28 Thread Francois Gaudreault
Thanks! Is the console proxy issue will be fixed at some point in the future or this won't be possible? Francois On 10/28/2013, 1:57 PM, Phong Nguyen wrote: Yes, the console proxy VM is unable to connect to the LXC console. Use virsh to connect from the host. virsh -c lxc:/// console i-2-20-

Re: LXC and SSVM/CPVM on the host

2013-10-28 Thread Phong Nguyen
Yes, the console proxy VM is unable to connect to the LXC console. Use virsh to connect from the host. virsh -c lxc:/// console i-2-20-VM -Phong On Mon, Oct 28, 2013 at 1:22 PM, Francois Gaudreault < fgaudrea...@cloudops.com> wrote: > Ok, I followed Chiradeep's comment here, and it now works f

RE: haPlanners error in fresh 4.2 checkout

2013-10-28 Thread Prachi Damle
I had encountered this sometime back ... you need to check if your componentContext.xml or its equivalent has following defined. However I do see it is present in the 4.2 branch -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Monday, O

Re: Tiered Quality

2013-10-28 Thread Laszlo Hornyak
Sonar already tracks the unit test coverage. It is also able to track the integration test coverage, however this might be a bit more sophisticated in CS since not all hardware/software requirements are available in the jenkins environment. However, this could be a problem in any environment. On

Re: LXC and SSVM/CPVM on the host

2013-10-28 Thread Francois Gaudreault
Ok, I followed Chiradeep's comment here, and it now works fine. I have CentOS 6 containers running. My only other questions would be: - Is it normal that the console proxy VM cannot proxy the container's console? - How can I access the container console from the host? (I am on CentOS) I can ss

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marcus Sorensen
Yeah, the bandwidth limiting for KVM was dropped into 4.2. You just need to tweak your settings, whether it's on network offerings or global. On Mon, Oct 28, 2013 at 8:25 AM, Wei ZHOU wrote: > Please read this artcle http://support.citrix.com/article/CTX132019 > Hope this help you. > > > 2013/10/

Jira is extremely slow

2013-10-28 Thread Gavin Lee
I could not access the jira (very slow actually) for cloudstack, who know the reason? Thanks. -- Gavin

Re: Review Request 14990: added formatter setting xml file for eclipse

2013-10-28 Thread Rajani Karuturi
> On Oct. 28, 2013, 1:52 p.m., daan Hoogland wrote: > > If this work the patch should include removing the epf file. It makes no > > sense having both around, unless... > > Is this xml format compatible with all eclipse versions? Did you test it > > with any? > > daan Hoogland wrote: > i m

Re: Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Wei ZHOU
Please read this artcle http://support.citrix.com/article/CTX132019 Hope this help you. 2013/10/28 Marty Sweet > Hi Guys, > > Following my upgrade from 4.1.1 -> 4.2.0, I have noticed that VM traffic is > now limited to 2Mbits. > My compute offerings were already 1000 for network limit and I hav

Re: Review Request 14990: added formatter setting xml file for eclipse

2013-10-28 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14990/ --- (Updated Oct. 28, 2013, 2:25 p.m.) Review request for cloudstack, daan Hoogland

RE: [PROPOSAL] Support GPU/VGPU in VMWare/XenServer

2013-10-28 Thread Sanjay Tripathi
Hi all, This is the first draft of the functional spec: https://cwiki.apache.org/confluence/display/CLOUDSTACK/GPU+and+vGPU+support+for+CloudStack+Guest+VMs Will keep updating it based on the feedback. --Sanjay > -Original Message- > From: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.

Re: Review Request 14990: added formatter setting xml file for eclipse

2013-10-28 Thread daan Hoogland
> On Oct. 28, 2013, 1:52 p.m., daan Hoogland wrote: > > If this work the patch should include removing the epf file. It makes no > > sense having both around, unless... > > Is this xml format compatible with all eclipse versions? Did you test it > > with any? i mean which ones? - daan

Re: Review Request 14990: added formatter setting xml file for eclipse

2013-10-28 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14990/#review27610 --- If this work the patch should include removing the epf file. It make

haPlanners error in fresh 4.2 checkout

2013-10-28 Thread sebgoa
Hi, I just checked out the latest from 4.2 and I get this: INFO [factory.annotation.AutowiredAnnotationBeanPostProcessor] (main:) JSR-330 'javax.inject.Inject' annotation found and supported for autowiring INFO [factory.annotation.AutowiredAnnotationBeanPostProcessor] (main:) JSR-330 'javax.i

Re: Intellij Idea Code Style settings jar

2013-10-28 Thread Rajani Karuturi
We already have eclipse settings file at tools/eclipse/eclipse.epf (Thanks Daan for updating the wiki with this info) Using the Eclipse Code Formatter plugin (http://plugins.jetbrains.com/plugin/?idea&id=6546 ) for intellij, we can use the same file for intellij as well. However, the plugin requ

Review Request 14990: added formatter setting xml file for eclipse

2013-10-28 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14990/ --- Review request for cloudstack, daan Hoogland and Prasanna Santhanam. Repository

Re: Wiki access

2013-10-28 Thread Gaurav Aradhye
Hi, My username is gauravaradhye. Can anyone grant me edit permission? I need to update test plan automation columns. Regards, Gaurav On Fri, Oct 18, 2013 at 5:26 PM, Kishan Kavala wrote: > Thanks Hugo. > > > -Original Message- > > From: Trippie [mailto:trip...@gmail.com] On Behalf O

[Merge] hyperv branch to master

2013-10-28 Thread Devdeep Singh
Hi, I would like to merge the support for Hyperv to the master branch. Development for this has been done by Donal, Rajesh, Anshul and I on branch [1]. The feature was proposed for merge earlier [3] but unit tests for hyperv agent code were requested [4]. Checklist: The development for the fea

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Prasanna Santhanam
> On Oct. 28, 2013, 8:25 a.m., Prasanna Santhanam wrote: > > Are we okay with putting in VisualStudio project files? We don't do the > > same for Eclipse. > > + > > plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj > > > > I guess there is a HOWTO somewhere on runni

RE: Review Request 14827: hyperv unit tests

2013-10-28 Thread Donal Lafferty
The mvn build flags are: -P hyperv-plugin for the C# portion of plugin -DskipTests=false for unit tests (platform independent tests) -DskipFunctionalTests=false for functional tests (tests that need to be run on a Hyper-V server) Setup for Mono is on the blog: http://dlafferty.blogspot.co.uk/20

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Prasanna Santhanam
> On Oct. 28, 2013, 8:25 a.m., Prasanna Santhanam wrote: > > Are we okay with putting in VisualStudio project files? We don't do the > > same for Eclipse. > > + > > plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj > > > > I guess there is a HOWTO somewhere on runni

Bandwidth Shaping - Ubuntu 12.04.3 KVM

2013-10-28 Thread Marty Sweet
Hi Guys, Following my upgrade from 4.1.1 -> 4.2.0, I have noticed that VM traffic is now limited to 2Mbits. My compute offerings were already 1000 for network limit and I have created new offerings to ensure this wasn't the issue (this fixed it for someone in the mailing list). Is there anything

[EVENT] Announcing CloudStack Mini-Conference @Open Source India 2013

2013-10-28 Thread Shanker Balan
Hi All, On November 13th, The Apache CloudStack community along with Citrix is hosting a CloudStack Mini-Conference in Bangalore. The half day conference starts with a beginners guide to CloudStack By Iliyas Shirol followed up with a live demo. CloudStack developer and Apache Committer Rajesh Ba

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Anshul Gangwar
> On Oct. 28, 2013, 8:25 a.m., Prasanna Santhanam wrote: > > Are we okay with putting in VisualStudio project files? We don't do the > > same for Eclipse. > > + > > plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj > > > > I guess there is a HOWTO somewhere on runni

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Donal Lafferty
> On Oct. 28, 2013, 8:25 a.m., Prasanna Santhanam wrote: > > Are we okay with putting in VisualStudio project files? We don't do the > > same for Eclipse. > > + > > plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj > > > > I guess there is a HOWTO somewhere on runni

Re: RN-KnownIssuesIn4.2

2013-10-28 Thread Sebastien Goasguen
Animesh owns that filter, On Oct 27, 2013, at 6:39 AM, Marty Sweet wrote: > Hi Guys, > > Following the issues I have had upgrading to 4.2.0, I have noticed that the > known issues filter in Jira is set to used fixedVersion as opposed to > affectedVersion. This changes the results dramatically,

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Prasanna Santhanam
> On Oct. 28, 2013, 8:25 a.m., Prasanna Santhanam wrote: > > Are we okay with putting in VisualStudio project files? We don't do the > > same for Eclipse. > > + > > plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj > > > > I guess there is a HOWTO somewhere on runni

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14827/#review27604 --- Are we okay with putting in VisualStudio project files? We don't do

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-10-28 Thread Sebastien Goasguen
I think the error is in the java code of AssignVirtualMachine: api/src/org/apache/cloudstack/api/command/admin/vm/AssignVMCmd.java: @APICommand(name = "assignVirtualMachine", description="Change ownership of a VM from one account to another. This API is available for Basic zones with security gr

Re: cloudmonkey exit value

2013-10-28 Thread Sebastien Goasguen
On Oct 27, 2013, at 7:49 PM, Darren Shepherd wrote: > It would be really nice if the API failed if cloudmonkey exited with > !=0 exit code. I've been scripting some stuff with cloud monkey and > this makes it difficult. It would also be nice to somehow specific > which field you want printed.

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14827/#review27603 --- Ship it! LGTM - Rajesh Battala On Oct. 23, 2013, 11:05 a.m., An

Re: Review Request 14925: Added few misc changes to marvin

2013-10-28 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14925/#review27601 --- Ship it! 03830c5 -> master Sorry to be an ass about this, but git

Re: Review Request 14925: Added few misc changes to marvin

2013-10-28 Thread Santhosh Edukulla
> On Oct. 28, 2013, 5:55 a.m., Prasanna Santhanam wrote: > > tools/marvin/marvin/marvinPlugin.py, line 148 > > > > > > Any reason to remove the debug logger? This will cause the write of all > > logs without timestamp

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-28 Thread Sowmya Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14426/ --- (Updated Oct. 28, 2013, 7:03 a.m.) Review request for cloudstack, Rajesh Battal

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-28 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14459/#review27597 --- test/integration/component/test_base_image_updation.py

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-28 Thread Sowmya Krishnan
> On Oct. 16, 2013, 4:43 p.m., Santhosh Edukulla wrote: > > Sowmya, > > > > > > Is it ok if we do a code walkthrough? Let me know i can create a gtm. > > Also,please add the path to test plan for this feature along with feature > > owner information, i will invite him\her as well. > > > > sa