Re: Review Request 11670: CLOUDSTACK-2288: NPE while creating volume from snapshot when the primary storage is in maintenance state.

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11670/#review22579 --- Applying: CLOUDSTACK-2288: NPE while creating volume from snapshot w

RE: Master build problem

2013-06-29 Thread Paul Angus
Is this fix in the new 4.2 branch? I'm getting the following error when trying to log in to the management UI. (UI reports 'Invalid Username or Password') 2013-06-30 07:12:33,779 DEBUG [cloud.user.AccountManagerImpl] (catalina-exec-5:null) Attempting to log in user: admin in domain 1 2013-06-30

Re: Review Request 9818: Bugfix CLOUDSTACK-1594: Secondary storage host always remains Alert status

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9818/#review22578 --- Roxanne, thanks for working on the patch and resubmitting! Since the

Re: Review Request 11977: Re-enabling baremetal for 4.2/master

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11977/#review22577 --- This was submitted. Closing the review http://cloudstack.markmail.or

Re: Review Request 10846: Selenium Headless configuration using PhantomJS

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10846/#review22576 --- 48540dcf5621d0cd7a925e8da299a3ef2d7d0c31 - Prasanna Santhanam On

Re: Review Request 11858: CLOUDSTACK-2340 [AWS Style Health Checks] Response of the API listLoadBalancerRuleInstances should show the service state of a VM if health check is configured for it

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11858/ --- (Updated June 30, 2013, 6:08 a.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 7018: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7018/#review22575 --- Don't think this is applicable anymore with the storage subsystem cha

Re: [GSOC] cloudstack-agent service and "unable to add host"

2013-06-29 Thread Prasanna Santhanam
On Sat, Jun 29, 2013 at 01:51:32PM -0400, Han,Meng wrote: > I encountered a few problems while following 4.0 quick install guide. > > 1. cloudstack-management service can be started, but dies quickly. > It is the same problem listed > here:https://issues.apache.org/jira/browse/CLOUDSTACK-2758 > I

Re: Review Request 11953: Remove code that does nothing but allocate a bunch of objects

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11953/ --- (Updated June 30, 2013, 5:52 a.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 12197: Updated VPC test case's service offering

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12197/#review22574 --- Ship it! b59bb25..1116968 master-6-17-stable -> master-6-17-stable

Re: Review Request 12129: CLOUDSTACK-3217: Fixed indentation issues in test_routers.py

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12129/#review22573 --- was fixed as part of 8dc99d92b6f82e435610dfcc01d3458a99503119 - Pra

Re: Review Request 12130: Tag missing for test integratin.component test_ldap.py

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12130/#review22572 --- Ship it! 56a001d1c49221397621759d784bd6000b38f5e4 - Prasanna Santh

Review Request 12197: Updated VPC test case's service offering

2013-06-29 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12197/ --- Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam. Reposit

RE: [ACS42] Branch created

2013-06-29 Thread Sateesh Chodapuneedi
Hi Animesh, Can you please let me know if commits into this branch are controlled? Regards, Sateesh > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: 29 June 2013 22:01 > To: dev@cloudstack.apache.org > Subject: [ACS42] Branch created > > I ha

Review Request 12192: set the field 'state' rather than the unread parameter 'state', if the parameter is null

2013-06-29 Thread Dave Brosius
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12192/ --- Review request for cloudstack. Repository: cloudstack-git Description ---

Re: Review Request 12162: Add documents on plugin development

2013-06-29 Thread Noel King
Good work Ian, enjoy the weekend On 29 June 2013 20:54, Sebastien Goasguen wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12162/#review22568 > ---

Re: Review Request 12162: Add documents on plugin development

2013-06-29 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12162/#review22568 --- Ship it! applied to master with 15a6844784142141af4489a57d7d748

Re: Review Request 12189: Apach CloudStack 4.1 Quick Install Guide (en-US)

2013-06-29 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12189/#review22567 --- Ship it! committed to master with 676d8744ef6801cf3fcf6425d37ba27a1

RE: [ACS42] Branch created

2013-06-29 Thread Sudha Ponnaganti
Just want to confirm if upgrades are all enabled for 4.2?? -Original Message- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Saturday, June 29, 2013 10:31 AM To: dev@cloudstack.apache.org Subject: Re: [ACS42] Branch created Great! I've setup the basic build j

Re: [GSOC] cloudstack-agent service and "unable to add host"

2013-06-29 Thread Han,Meng
Hi Sebastien, Thanks for your reply. On Fri, 28 Jun 2013 14:54:46 -0400, Sebastien Goasguen wrote: Meng, I am not sure where you stand on this, but you should not use VLANs. Just setup a basic zone (no vlans needed) , use the packages from the repository and don't even try to compile from so

Re: [ACS42] Branch created

2013-06-29 Thread Hugo Trippaers
Great! I've setup the basic build jobs for 4.2 on jenkins.cloudstack.org. I would like to send all build failures from those jobs directly to the dev list. Can a moderator add jenk...@cloudstack.org to the whitelist of the dev mailing list? Cheers, Hugo On Jun 29, 2013, at 9:30 AM, Animesh C

Review Request 12191: removed EnumerationImpl

2013-06-29 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12191/ --- Review request for cloudstack. Repository: cloudstack-git Description ---

Review Request 12190: removed CleanupDelegate

2013-06-29 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12190/ --- Review request for cloudstack. Repository: cloudstack-git Description ---

Re: Suggestion needed for CLOUDSTACK-212

2013-06-29 Thread David Nalley
On Fri, Jun 28, 2013 at 4:05 AM, Sebastien Goasguen wrote: > > On Jun 27, 2013, at 1:23 PM, Dharmesh Kakadia wrote: > >> Hi, >> >> I misunderstood the feature freeze. This has been assigned to fix in >> Version 4.2 on jira. As per this discussion, should I change it to 4.3 ? > > Dharmesh, can you

Re: [DISCUSS] If BVT breaks, revert the commits...

2013-06-29 Thread David Nalley
On Sat, Jun 29, 2013 at 12:45 AM, Prasanna Santhanam wrote: > On Sat, Jun 29, 2013 at 12:18:17AM +, Alex Huang wrote: >> After Dave's complain in the vmsync [MERGE] thread about BVT in >> horrible shape on master, I went around to figure out what exactly >> happened. The best I can figure is

[ACS42] Branch created

2013-06-29 Thread Animesh Chaturvedi
I have created 4.2 branch Animesh

Re: [DISCUSS][VOTE] Moving cloudmonkey to its own repository, have its own release process

2013-06-29 Thread David Nalley
Rohit: Do you plan on generating a cloudmonkey release? (and acting as the RM?, should be comparatively simple, but we might need to document it.) Setup.py will need to be fixed, but I have no other concerns than that. I plan to make this writable momentarily. --David On Sat, Jun 29, 2013 at 9:2

Review Request 12189: Apach CloudStack 4.1 Quick Install Guide (en-US)

2013-06-29 Thread meng han
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12189/ --- Review request for cloudstack. Repository: cloudstack-git Description ---

Re: Review Request 11942: ProcessUtil cleanup

2013-06-29 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11942/ --- (Updated June 29, 2013, 3:51 p.m.) Review request for cloudstack. Changes ---

Review Request 12188: removed interface "On"

2013-06-29 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12188/ --- Review request for cloudstack. Repository: cloudstack-git Description ---

Re: Review Request 11953: Remove code that does nothing but allocate a bunch of objects

2013-06-29 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11953/#review22566 --- Looks like the whole point is to calculate the poolId and the DataC

Testcase Failure

2013-06-29 Thread Dharmesh Kakadia
Hi, I am trying to switch com.cloud.network package to org.apache.network. While doing so, I am encountering following failure in test case. I tried looking around for few hours, but no success. I don't understand spring-junit testing much. I looked at https://cwiki.apache.org/CLOUDSTACK/unit-test

Re: Jenkins and IRC

2013-06-29 Thread Rohit Yadav
On Fri, Jun 28, 2013 at 10:39 PM, Hugo Trippaers wrote: > Heya, > > I've added a parameterised job to jenkins that will run a complete build > on any branch. You can kick it off on jenkins.cloudstack.org and enter > any branch as the parameter. It will do the regular maven build with the > clean

Re: [DISCUSS][VOTE] Moving cloudmonkey to its own repository, have its own release process

2013-06-29 Thread Rohit Yadav
More than 72 hours now. Let's make it writable and publish latest cloudmonkey on pypi. Cheers. On Wed, Jun 26, 2013 at 3:02 AM, David Nalley wrote: > > > > Not sure about setup.py related failures. > > > > Me either - setup.py is pretty dramatically different between the two > repos. > > --Davi

Re: [ACS42] Feature Freeze reminder

2013-06-29 Thread Prasanna Santhanam
And the RAT check failed. Fixed now in SolidFireUtil.java On Sat, Jun 29, 2013 at 04:51:59PM +0530, Prasanna Santhanam wrote: > I've merged the plugin changes for simulator into master. Also ran the > checkin tests without issue. The only change was to move Mike's plugin > over to componentContext

Re: [ACS42] Feature Freeze reminder

2013-06-29 Thread Prasanna Santhanam
I've merged the plugin changes for simulator into master. Also ran the checkin tests without issue. The only change was to move Mike's plugin over to componentContext.xml instead of applicationContext.xml. Thanks, -- Prasanna., On Fri, Jun 28, 2013 at 11:57:16PM +, Edison Su wrote: > Nice!

RE: [ACS42] Feature Freeze reminder

2013-06-29 Thread Sudha Ponnaganti
Mike, I have targeted the Ticket to 4.2 [1] Added subtasks for QA/Doc and automation Wanted to check if there is someone with SolidFire background that can add test plan and do some validation for this plug-in [1] https://issues.apache.org/jira/browse/CLOUDSTACK-2778 Thanks /sudha -Ori

RE: [ACS42] Feature Freeze reminder

2013-06-29 Thread Sudha Ponnaganti
Hi All, Calling on community to see if someone is interested to pick up this plugin for validation Require the following to close validation aspect for this plug-in: - Functional test plan/ Automated tests to test this plug-in - see cwiki for test plan template[1] and test results templates[2]

Is there any patch file CS 4.1 to xen-6.2.0

2013-06-29 Thread Keerthiraja SJ
Hi All, Is there any patch file for cloudstack-4.1 to added xenserver-6.2.0 . I knew that plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/discoverer/XcpServerDiscoverer.java this file plays vital role to added xenserver-6.2.0 but it support only upto 6.1 do u guys have any patch Just but

Re: Unable to register templates and ISOs

2013-06-29 Thread Nguyen Anh Tu
I checked SSVM. Everything fine. DNS checking's ok and SSVM can connect to MS port 8250. But it still can't register template. Log on MS said: 2013-06-29 15:38:35,133 DEBUG [storage.download.DownloadListener] (Timer-5:null) Scheduling timeout at 3 ms, TEMPLATE: 201 at host 0 2013-06-29 15:38:3

Re: Unable to register templates and ISOs

2013-06-29 Thread Harikrishna Patnala
Can you try troubleshooting the SSVM by following this link https://cwiki.apache.org/CLOUDSTACK/ssvm-templates-secondary-storage-troubleshooting.html Probably agent is not running in the SSVM due to which no entry is created in the host table. please try building systemvm and recreate ssvm -Harik

Test Result for test-smoke-matrix #559

2013-06-29 Thread Prasanna Santhanam
Test Run: #559 Total:94 Fail :7 Skip :2 name passfailskip test_vm_life_cycle/ 9 1 0 test_global_settings/ 1 0 0 test_affinity_groups/

Re: Review Request 12180: Fix NPE when adding invalid primary datastore

2013-06-29 Thread Sateesh Chodapuneedi
> On June 29, 2013, 7:33 a.m., Prasanna Santhanam wrote: > > engine/storage/volume/src/org/apache/cloudstack/storage/datastore/manager/PrimaryDataStoreProviderManagerImpl.java, > > line 59 > > > > > > This block should

Re: Review Request 12180: Fix NPE when adding invalid primary datastore

2013-06-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12180/#review22563 --- engine/storage/volume/src/org/apache/cloudstack/storage/datastore/m

Re: Review Request 12180: Fix NPE when adding invalid primary datastore

2013-06-29 Thread Sateesh Chodapuneedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12180/#review22562 --- engine/storage/volume/src/org/apache/cloudstack/storage/datastore/m