Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-13 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/ --- (Updated March 13, 2014, 7:02 a.m.) Review request for cloudstack, John Burwell

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread punith s
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19021/ --- (Updated March 13, 2014, 12:52 p.m.) Review request for cloudstack, edison su a

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-13 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/ --- (Updated March 13, 2014, 7:31 a.m.) Review request for cloudstack, John Burwell

Build failed in Jenkins: build-master-slowbuild #418

2014-03-13 Thread jenkins
See -- [...truncated 18098 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] [INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<< [INFO] [INFO] -

Re: How writing a new api of Pause VirtualMachine

2014-03-13 Thread Wei ZHOU
entityType=UserVmResponse.class

Re: [VOTE] Apache CloudStack 4.3.0 (seventh round)

2014-03-13 Thread Daan Hoogland
If we have n features, there may be conflict in n! ways. Some of thos are relatively simple conflicts as these involve only two new features. Others are potentiatlly very complex as these concern more then just two features interacting with each other. The worrying thing about this is that it is a

ISO image

2014-03-13 Thread María Noelia Gil
Hello, I added an ISO image to Templates -> ISO, but this image does not appear when I want to add one instance. What is the problem? Thanks.

Re: [VOTE] Apache CloudStack 4.3.0 (seventh round)

2014-03-13 Thread Daan Hoogland
good point Paul, auw On Thu, Mar 13, 2014 at 1:14 AM, Paul Angus wrote: > I agree with Alex and would go further and say that the fact that for 4.3 RC7 > the noredist build was broken was an epic fail in the automated testing, > which needs addressing. > > Regards, > > Paul Angus > Cloud Archit

Re: Just checked in a minor change to the DB

2014-03-13 Thread Daan Hoogland
Amoch thanks, but please send one with the tag and a proper subject, next time I mean. On Thu, Mar 13, 2014 at 3:50 AM, Rajesh Battala wrote: > Thanks a lot Amogh for the info. > > -Original Message- > From: Amogh Vasekar [mailto:amogh.vase...@citrix.com] > Sent: Thursday, March 13, 2014

Re: ISO image

2014-03-13 Thread Daan Hoogland
That is not a lot of info, Maria. Can you send some more? logs, zones configured? did the template finish uploading? and possibly more...? regards, Daan On Thu, Mar 13, 2014 at 9:58 AM, María Noelia Gil wrote: > Hello, I added an ISO image to Templates -> ISO, but this image does not > appear w

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread punith s
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19021/ --- (Updated March 13, 2014, 2:46 p.m.) Review request for cloudstack, edison su an

Re: ISO image

2014-03-13 Thread Wei ZHOU
Is the ISO ready (Successfully Installed) ? Is it public or featured ? 2014-03-13 9:58 GMT+01:00 María Noelia Gil : > Hello, I added an ISO image to Templates -> ISO, but this image does not > appear when I want to add one instance. What is the problem? > > Thanks.

RE: ISO image

2014-03-13 Thread Suresh Sadhu
Check your uploaded ISO ready state if its yes then it will show in your addInsatnce wizard. If its yes but still not shown in UI then please provide logs Regards Sadhu -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: 13 March 2014 14:40 To: us...@clouds

RE: FindBugs

2014-03-13 Thread Sanjay Tripathi
Hi Hugo, I have fixed the bug reported by findbugs. --Sanjay -Original Message- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Tuesday, March 11, 2014 9:13 PM To: Subject: FindBugs Hey, Just a reminder, findbugs is now checking the code periodically via Jen

RE: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

2014-03-13 Thread Ram Ganesh
> > Thanks for your reply, Sanjay. I understand that it's only K1 and K2 cards > > at the moment, but when they add more, do we really want to have > > to edit the code to match? What is the notification process by which we > > know there are new types? Does the user have to upgrade CloudStack >

Re: ISO image

2014-03-13 Thread María Noelia Gil
I think the problem is that I tried adding the ISO from an internal IP address. I have seen that this is disabled by default. How to activate? El 13/03/2014, a las 10:25, Suresh Sadhu escribió: > Check your uploaded ISO ready state if its yes then it will show in your > addInsatnce wizard. >

RE: [VOTE] Apache CloudStack 4.3.0 (seventh round)

2014-03-13 Thread Alex Hitchins
Daan, I see where you are coming from in your use case scenario. I just think that winning minds over to Open Source platforms could be more difficult if when asked "How often is there a new version?" we need to break the news that a new one is out every week. It won't sound like a solid, mature

RE: Regarding Cloudstack-4322

2014-03-13 Thread Sanjay Tripathi
Namita, Exception “Failed to delete domain” comes if there is an error while cleaning up the accounts. For accounts, there is account.cleanup thread which cleanup up the removed/inactive accounts but for domain, CS doesn’t has any specific thread to delete inactive domains and thows the excepti

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Wilder Rodrigues
Hi guys, I'm testing the CS 4.3 RC (8th round), and will keep doing so, but for now I already did the following and it looks good (sorry for the long email): * Environment - Management Server: Debian 7 VM under VirtualBox - DevCloud: XenServer 6.2 - MySQL: running on the DevCloud - Syste

Re: [PROPOSAL] Bridge functionality

2014-03-13 Thread Hugo Trippaers
On 11 mrt. 2014, at 21:51, Chiradeep Vittal wrote: > Is there a more detailed writeup? I am interested in the api and Œflow’ No, no more details at the moment. Still thinking a lot on how i want it to work. I’m trying to get this as generic as possible so other SDN solution can benefit from t

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Guo Star
+1 2014-03-13 18:27 GMT+08:00 Wilder Rodrigues : > Hi guys, > > I'm testing the CS 4.3 RC (8th round), and will keep doing so, but for now > I already did the following and it looks good (sorry for the long email): > > * Environment > - Management Server: Debian 7 VM under VirtualBox > - Dev

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Madan Ganesh Velayudham
+1 On 13-Mar-2014, at 4:18 pm, Guo Star wrote: > +1 > > > 2014-03-13 18:27 GMT+08:00 Wilder Rodrigues : > >> Hi guys, >> >> I'm testing the CS 4.3 RC (8th round), and will keep doing so, but for now >> I already did the following and it looks good (sorry for the long email): >> >> * Enviro

Build failed in Jenkins: build-master-slowbuild #419

2014-03-13 Thread jenkins
See Changes: [sanjay.tripathi] findbugs: impossible cast in CreateServiceOfferingCmd file. -- [...truncated 18096 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] [

Unable to download CentOS template

2014-03-13 Thread Tejas Gadaria
Hi, I have new cs setup on xenserver 6.0.2. I am not able to download CentOS 5.6 template. its says not "No route to host" I tried to stop SSVM firewall but no luck. Also Consol proxy's pblic ip is not pinging. need help on this. Regards, Tejas

Re: Unable to download CentOS template

2014-03-13 Thread Ian Duffy
Hi, Please see the steps outlined at the below link on debugging the SSVM. This will help identify your issue. https://cwiki.apache.org/confluence/display/CLOUDSTACK/SSVM,+templates,+Secondary+storage+troubleshooting Thanks, Ian On 13 March 2014 11:09, Tejas Gadaria wrote: > Hi, > > I have new

RE: ISO image

2014-03-13 Thread Suresh Sadhu
Set this global configuration secstorage.allowed.internal.sites to your internal network cidr and restart the management server. Regards Sadhu -Original Message- From: María Noelia Gil [mailto:marianoelia@um.es] Sent: 13 March 2014 15:47 To: us...@cloudstack.apache.org; dev@cl

Re: Unable to download CentOS template

2014-03-13 Thread Tejas Gadaria
Hi Ian, Thanks for immediate response, I followed your link, 1) SSVM heath check.. ok Also in host table NFS Secondary storage status | 2 | nfs://10.129.151.51/vol/secondary | c876744c-b0bc-48ac-8779-b73bb124687c | Alert | SecondaryStorage | nfs I confirm all possibilities from doc but s

RE: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

2014-03-13 Thread Stephen Turner
I believe XenServer exposes everything you need in the VGPU_type class: # xe vgpu-type-list params=all uuid ( RO) : 6e175f2c-e8f1-2dce-aadf-8a8962b6f0e8 vendor-name ( RO): NVIDIA Corporation model-name ( RO): GRID K260Q framebuffer-

Re: Review Request 18970: CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

2014-03-13 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18970/#review37042 --- Gentle reminder - Gaurav Aradhye On March 10, 2014, 9:24 p.m., Ga

Re: Review Request 19119: CLOUDSTACK-5887: Resolved issue in test_base_image_updation.py

2014-03-13 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19119/#review37043 --- Gentle reminder - Gaurav Aradhye On March 12, 2014, 7:08 p.m., Ga

Re: Review Request 18971: CLOUDSTACK-6216: Skip the test if run on already existing setup

2014-03-13 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18971/#review37044 --- Gentle reminder - Gaurav Aradhye On March 10, 2014, 9:58 p.m., Ga

Re: Review Request 19022: List VM enhancement to support querying with multiple VM IDs

2014-03-13 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19022/#review37045 --- Ship it! Ship It! - Koushik Das On March 11, 2014, 6:29 a.m., Ko

Re: Review Request 19022: List VM enhancement to support querying with multiple VM IDs

2014-03-13 Thread Koushik Das
> On March 13, 2014, 12:19 p.m., Koushik Das wrote: > > Ship It! Commit 5779292e93fe52a7fb5ec0abffe1e3f6b9c10121 in cloudstack's branch refs/heads/master from [~koushikd] [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5779292 ] CLOUDSTACK-6052: List VM enhancement to support quer

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread punith s
> On March 12, 2014, 10:19 p.m., Mike Tutkowski wrote: > > plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/datastore/util/ElastistorUtil.java, > > line 495 > > > > > > I like how you removed the sta

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Daan Hoogland
Are you guys +1'ing Wilders testing or the release? And if the release, then why, what did you test that made you decide this? Please note that the PMC has to decide based on what they read here as arguments. We need more input then a number of +1, please. Thanks, Daan -Original Message---

Build failed in Jenkins: build-master-slowbuild #420

2014-03-13 Thread jenkins
See Changes: [koushik] CLOUDSTACK-6052: List VM enhancement to support querying with multiple VM IDs -- [...truncated 18096 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

Re: [CS4.1] Error during ip range creation

2014-03-13 Thread nicolas.lamirault
nobody has any ideas for this error ? Le 12/03/2014 14:02, nicolas.lamira...@orange.com a écrit : A lot of networks : mysql> select * from networks where broadcast_uri ='vlan://2013' ; +-+---+---+--+---+---+---

Re: [CS4.1] Error during ip range creation

2014-03-13 Thread Daan Hoogland
Nicolas, Which version are you running on? What is your setup (Basic/Advanced) is this network the only one? is it isolated/shared? etc. In short I have no idea, let's keep looking. On Thu, Mar 13, 2014 at 3:08 PM, wrote: > nobody has any ideas for this error ? > > Le 12/03/2014 14:02, nicolas

Re: [PROPOSAL] Bridge functionality

2014-03-13 Thread Ian Rae
Hugo, This is a common request from our SP customers but most have balked at the cost of developing a custom solution. It is an important feature for hybrid architectures, let us know how we can help. Certainly we will give feedback on the functional spec. Ian Ian Rae CEO, CloudOps Inc. 514-944-

Re: [CS4.1] Error during ip range creation

2014-03-13 Thread nicolas.lamirault
we're using CS 4.1 we are in Advanced (Cf zone in the Cloudstack UI) All networks are 'Share', associated to 'Account'. Le 13/03/2014 15:16, Daan Hoogland a écrit : Nicolas, Which version are you running on? What is your setup (Basic/Advanced) is this network the only one? is it isolated/shar

Re: Review Request 18956: CLOUDSTACK-6147: Adding next set of tests for Dynamic Compute Offering

2014-03-13 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18956/ --- (Updated March 13, 2014, 3:35 p.m.) Review request for cloudstack, Santhosh Edu

RE: [20/31] git commit: updated refs/heads/distributedrouter to a8d43ba

2014-03-13 Thread Konstantina Chremmou
Hi, This commit adds a new parameter to ListVMsCmd, but no 'since' has been specified in the parameter annotation. Please rectify. Thanks, Tina --- Guidelines for writing/modifying CS APIs: https://cwiki.apache

Re: [CS4.1] Error during ip range creation

2014-03-13 Thread Wei ZHOU
Could you give more information? the vlan range you added, the vlans already exists in vlan table. 2014-03-13 15:43 GMT+01:00 : > we're using CS 4.1 > we are in Advanced (Cf zone in the Cloudstack UI) > All networks are 'Share', associated to 'Account'. > > > > Le 13/03/2014 15:16, Daan Hooglan

RE: [20/31] git commit: updated refs/heads/distributedrouter to a8d43ba

2014-03-13 Thread Konstantina Chremmou
Hi, This commit adds a new parameter to ListVMsCmd, but no 'since' has been specified in the parameter annotation. Please rectify. Thanks, Tina --- Guidelines for writing/modifying CS APIs: https://cwiki.apache

Re: [CS4.1] Error during ip range creation

2014-03-13 Thread nicolas.lamirault
Yes : mysql> select * from vlan where vlan_gateway = '10.200.244.129' and vlan_netmask = '255.255.255.192' ; +-+-++-+---++++--+

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Wilder Rodrigues
Hi all, Outcome of more tests... So far, so good: * Environment - Management Server: Debian 7 VM under VirtualBox - DevCloud: XenServer 6.2 - MySQL: running on the DevCloud - System VM: Latest from http://jenkins.buildacloud.org/view/4.3/job/cloudstack-4.3-systemvm/ * Create VPC - 1

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Daan Hoogland
Created bridged network with ip-range reservation. Validated that https://issues.apache.org/jira/browse/CLOUDSTACK-6232 is no longer an issue. Vm can be created in this network. The same bridged offering is created with Connectivity through NVP. VM gets created and seems fine. Created vpcs with

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
Chiradeep, Any comments on them? Thanks Alex Ough On Wed, Mar 12, 2014 at 10:58 AM, Alex Ough wrote: > And I also uploaded the patch B that includes new implementation to > support multi regions. > > Thanks > Alex Ough > > > On Wed, Mar 12, 2014 at 10:17 AM, Alex Ough wrote: > >> I uploaded

Release cadence

2014-03-13 Thread David Nalley
The RC7 vote thread contained a lot of discussion around release cadence, and I figured I'd move that to a thread that has a better subject so there is better visibility to list participants who don't read every thread. When I look at things schedule wise, I see our aims and our reality. We have a

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-13 Thread John Burwell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/#review37061 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Re: Release cadence

2014-03-13 Thread Mike Tutkowski
I wanted to add a little comment/question in general about our release process: Right now we typically have a one-month overlap between releases. That being the case, if you are focusing on the current release until it is out the door, you effectively lose a month of development for the future rel

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-13 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/#review37064 --- Also why don't start routers in parallel? First, get all the router

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-13 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17736/#review37066 --- Also why don't start routers in parallel? First, get all the router

Re: Release cadence

2014-03-13 Thread Marcus
The overlap is simply a byproduct of cutting the branch, I'm not sure there's a way around it. It's a good point though, that essentially the window is 1 month shorter than I think was intended. Better testing will help that, however, with the point being that we shouldn't be doing a ton of work to

Build failed in Jenkins: build-master-slowbuild #421

2014-03-13 Thread jenkins
See -- [...truncated 18096 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] [INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<< [INFO] [INFO] -

Re: Release cadence

2014-03-13 Thread Mike Tutkowski
Yeah, if you "abandon" the "old" release as soon as a release branch is cut for it, then you essentially have three months on the new release before its release branch is cut and you move on to the newer release. I'm not sure that was the intent when such a schedule was created. It means we're rele

Re: [Merge] CloudStack IAM branch to master

2014-03-13 Thread Min Chen
Since we haven't heard of any objections to this merge for 3 days, I am going to merge it to master today. Thanks -min On 3/11/14 12:23 PM, "Hugo Trippaers" wrote: > >On 11 mrt. 2014, at 19:52, Min Chen wrote: > >> Also, have already run FingBugs on our branch and addressed all new >> findings

Re: Release cadence

2014-03-13 Thread Prasanna Santhanam
On Thu, Mar 13, 2014 at 12:42:26PM -0400, David Nalley wrote: > Radical proposition: > > Because we have two problems, of different nature, we are in a > difficult situation. This is a possible solution, and I'd appreciate > you reading and considering it. Feedback is welcome. I propose that > af

Re: Release cadence

2014-03-13 Thread David Nalley
This was (IIRC) part of the explicit decision in how to do things. The thought being that if you are restricting what people can do with a release branch, people still need to be able to have a place to base their ongoing work; and master should be that place. Some features will take more than a cy

[DB-CHANGE]

2014-03-13 Thread Alena Prokharchyk
Please update your DB after getting commit 1b83698dac712e3497d4140b15c9fcba3870c705 in master ALTER TABLE `cloud`.`vm_reservation` ADD COLUMN `deployment_planner` varchar(40) DEFAULT NULL COMMENT 'Preferred deployment planner for the vm’; -Alena.

Re: Release cadence

2014-03-13 Thread Mike Tutkowski
OK, so it sounds like a 3-month dev cycle for a four-month release was on purpose. Just curious...thanks :) On Thu, Mar 13, 2014 at 11:31 AM, David Nalley wrote: > This was (IIRC) part of the explicit decision in how to do things. The > thought being that if you are restricting what people can

[QUESTION] Creating an Ubuntu VM on XenServer from ISO

2014-03-13 Thread Mike Tutkowski
Hi, I'm having trouble creating a 64-bit Ubuntu VM on XenServer from an ISO, but if I manually go to XenServer and select the ISO, it works just fine. Well, it works just fine after I run this command on the VM: xe vm-param-set uuid=9e7ffaad-0c16-4d71-d17c-b1d58aa65bc2 HVM-boot-policy=BIOS\ order

Re: [20/31] git commit: updated refs/heads/distributedrouter to a8d43ba

2014-03-13 Thread Alena Prokharchyk
Konstantina, feel free to raise the Jira ticket for it. Thanks, Alena. On 3/13/14, 8:46 AM, "Konstantina Chremmou" wrote: >Hi, > >This commit adds a new parameter to ListVMsCmd, but no 'since' has been >specified in the parameter annotation. Please rectify. > >Thanks, >Tina > >-

Re: [20/31] git commit: updated refs/heads/distributedrouter to a8d43ba

2014-03-13 Thread David Nalley
Better yet, feel free to submit a patch that fixes it in addition to the Jira :) --David On Thu, Mar 13, 2014 at 2:00 PM, Alena Prokharchyk wrote: > Konstantina, feel free to raise the Jira ticket for it. > > Thanks, > Alena. > > On 3/13/14, 8:46 AM, "Konstantina Chremmou" > wrote: > >>Hi, >> >

Re: Review Request 19195: Automation of CCP Objects Verification after external changes made to the orignial setup

2014-03-13 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19195/ --- (Updated March 13, 2014, 6:50 p.m.) Review request for cloudstack, Girish Shila

Review Request 19195: Automation of CCP Objects Verification after external changes made to the orignial setup

2014-03-13 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19195/ --- Review request for cloudstack, Girish Shilamkar, sanjeev n, Santhosh Edukulla, a

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Nux!
On 13.03.2014 00:26, Animesh Chaturvedi wrote: Hi All, I've created a 4.3.0 release, with the following artifacts up for a vote: -1 In Adv SG zone assigning secondary IPs to a NIC doesn't update the ipset accordingly on the agent/hv; it requires stopping/starting the VM. It's not a traged

Re: Review Request 19195: Automation of CCP Objects Verification after external changes made to the orignial setup

2014-03-13 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19195/ --- (Updated March 13, 2014, 7 p.m.) Review request for cloudstack, Girish Shilamka

Re: Release cadence

2014-03-13 Thread Daan Hoogland
Just a thought, Why isn't the freshly cut branch the first RC from the get go? It is quite sure not to pass but it should cantain what we ant to ship feature wise. On Thu, Mar 13, 2014 at 6:35 PM, Mike Tutkowski wrote: > OK, so it sounds like a 3-month dev cycle for a four-month release was on >

Re: Review Request 19195: Automation of CCP Objects Verification after external changes made to the orignial setup

2014-03-13 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19195/ --- (Updated March 13, 2014, 7:06 p.m.) Review request for cloudstack, Girish Shila

Re: Review Request 19195: Automation of CCP Objects Verification after external changes made to the original setup

2014-03-13 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19195/ --- (Updated March 13, 2014, 7:08 p.m.) Review request for cloudstack, Girish Shila

Re: How writing a new api of Pause VirtualMachine

2014-03-13 Thread Chiradeep Vittal
But why would you want to pause a VM? Is this an end-user API or an admin API? From: Yitao Jiang mailto:willier...@gmail.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Wednesday, March 12, 2014 at 5:52 PM To: "dev@cloudstack.

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Nux!
On 13.03.2014 18:58, Nux! wrote: On 13.03.2014 00:26, Animesh Chaturvedi wrote: Hi All, I've created a 4.3.0 release, with the following artifacts up for a vote: -1 In Adv SG zone assigning secondary IPs to a NIC doesn't update the ipset accordingly on the agent/hv; it requires stopping/sta

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review37097 --- Patch B. * You shouldn’t make your service a part of cloud-mom-rabb

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alena Prokharchyk
Alex, I have some comments. Patch B. * You shouldn’t make your service a part of cloud-mom-rabbitmq plugin. Your subscribers/implementation are specific to your feature, and you need to introduce a special plugin just for your service. * AccountInterface and BaseInterface are still regular classe

Re: Release cadence

2014-03-13 Thread Marcus
Its a good point. I had thought about. Essentially we are saying that we know the features we just merged need another few months of work. On Mar 13, 2014 1:01 PM, "Daan Hoogland" wrote: > Just a thought, > > Why isn't the freshly cut branch the first RC from the get go? It is > quite sure not to

apache jenkins

2014-03-13 Thread Daan Hoogland
H, Anybody have any idea what's eating the rat test? -- Daan

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
Hi Alena, Patch B, I'm not quite familiar with java, so I have a little difficulty in following your recommendation. Can you send me an example using 'BaseInterface' and/or 'AccountInterface'? It will be very helpful and appreciated. Patch A, To reduce the number of requests to the remote regions

Build failed in Jenkins: build-master-slowbuild #422

2014-03-13 Thread jenkins
See Changes: [Alena Prokharchyk] deployVm/startVm APIs: ability to define deploymentPlanner for VmToStart in the api call (available to ROOT admin only) [Alena Prokharchyk] More generic parameters that are not unknown [Ale

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alena Prokharchyk
Alex, see inline. -Alena. From: Alex Ough mailto:alex.o...@sungard.com>> Date: Thursday, March 13, 2014 at 1:00 PM To: Alena Prokharchyk mailto:alena.prokharc...@citrix.com>> Cc: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>>, Chiradeep Vittal

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Animesh Chaturvedi
> -Original Message- > From: Nux! [mailto:n...@li.nux.ro] > Sent: Thursday, March 13, 2014 12:41 PM > To: dev@cloudstack.apache.org > Subject: Re: [VOTE] Apache CloudStack 4.3.0 (eighth round) > > On 13.03.2014 18:58, Nux! wrote: > > On 13.03.2014 00:26, Animesh Chaturvedi wrote: > >> Hi

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
Patch B, 1. The reason why I use abstract classes instead of interfaces is because there are some basic methods that are used among the inherited classes, so I'm not sure why it has to be an interface. 2. These are the abstract base classes along with their inherited classes and they are grouped

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19021/#review37109 --- The patch looks OK to me. When Mike is OK with your patch, then we c

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alena Prokharchyk
You extract stuff into interfaces when the methods are meant to be called from different classes/Managers. Do you implement to add APIs for your plugins? Can your plugin be used by any other CS manager - RegionManager for example? If the answer is yes, then you would need an interface. If not, abst

Re: apache jenkins

2014-03-13 Thread Laszlo Hornyak
A jenkins bug maybe? But fortunately it generates the rat report before it fails. Caused by: java.lang.ClassNotFoundException: org.apache.http.client.CredentialsProvider at org.apache.tools.ant.AntClassLoader.findClassInComponents(AntClassLoader.java:1365) at org.apache.tools.ant

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Animesh Chaturvedi
> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Thursday, March 13, 2014 1:25 PM > To: dev@cloudstack.apache.org > Subject: RE: [VOTE] Apache CloudStack 4.3.0 (eighth round) > > > > > -Original Message- > > From: Nux! [mailto:n...@

review request hook for git

2014-03-13 Thread Daan Hoogland
H, I concocted a git prepare-commit-msg hook. Please rant about my evening hobby after testing and reading DB=`git status | grep -e "modified.*\.sql$" -e "new file.*\.sql$"` OLD_MSG=`cat $1` if [ -z "$DB" ] then MSG="[DB-CHANGE] $OLD_MSG" else MSG="$OLD_MSG" fi echo "$MSG" >$1 regards, --

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
They are not called outside and only called from 'subscriber' classes and FullScanner class. Do you think these name changes are ok? - BaseInterface - UserInterface, AccountInterface, DomainInterface => APICaller - APIUserCaller, APIAccountCaller, APIDomainCaller - BaseService - U

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-13 Thread Nux!
On 13.03.2014 21:24, Animesh Chaturvedi wrote: [Animesh] Did you see this with prior RC too? [Animesh] Nux, security group support for advanced zone is limited and that too was developed in 4.2. I don’t think any changes have been made to that support since then. Can you call out what specific i

Re: Release cadence

2014-03-13 Thread David Nalley
Thats a very good point - we are effectively saying we know the features we merged in have potentially months worth of bugs. Though really, our hiccups don't seem to generally be in new features, it's old features. On Thu, Mar 13, 2014 at 3:44 PM, Marcus wrote: > Its a good point. I had thought a

Re: Release cadence

2014-03-13 Thread Mike Tutkowski
I think many people (myself included) are used to performing rigorous, but focused feature-specific testing before feature freeze, but are under the impression that once feature freeze arrives that we are in integration-testing mode (where our feature is tested in combination with other features...

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19021/#review37123 --- plugins/storage/volume/cloudbyte/src/org/apache/cloudstack/storage/

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread Mike Tutkowski
Hi Punith, Once you close out the open issues, we can work to get your code in. Feature freeze is Friday, March 14th, so we should do it sometime tomorrow. Thanks! On Thu, Mar 13, 2014 at 4:24 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: >This is an automatically generated e-

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alena Prokharchyk
Alex, There are so many classes, and it makes it hard to see/review the feature. Can you come up with some sort of visual diagram, so its easier to see which component is responsible for what task, and how they interact with each other. My suggestions: 1) I think it would make sense to merge al

Re: Release cadence

2014-03-13 Thread Daan Hoogland
That's how i like to see it and why I asked. Is there a reason people merge and then commit their features instead of rebasing and running a standard set of integration tests to validate before merging. I am not better then average on this myself but I think here is where we have room to improve if

RE: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread Edison Su
Hi Mike, Seems some of your comments are related to the scope of their implementation, such as do you support this, do you support that, can you do this, can you do that etc. From my point of view, it's not important, as the capability of driver is totally controlled by the vendors, vendors c

Build failed in Jenkins: build-master-slowbuild #423

2014-03-13 Thread jenkins
See Changes: [Alena Prokharchyk] Removed getUUID() method from Domain interface as this interface already implements @Identity where getUUID is defined [Alena Prokharchyk] Fixed some problems reported by FindBugs -

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread Mike Tutkowski
OK, if that's all we're checking, then I'm good with checking it in. Punith - Can you just close out those last issues in the review? Thanks! On Thu, Mar 13, 2014 at 5:02 PM, Edison Su wrote: > Hi Mike, > >Seems some of your comments are related to the scope of their > implementation, su

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-13 Thread Mike Tutkowski
Also, Punith has already refactored his code as I asked to put it in CloudByte-specific packages, so I think we're good to go. On Thu, Mar 13, 2014 at 5:04 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > OK, if that's all we're checking, then I'm good with checking it in. > > Punith

Master is broken?

2014-03-13 Thread Kelven Yang
Anyone encounter this problem on master? WARN [c.c.a.d.ParamGenericValidationWorker] (163545706@qtp-538020828-6:ctx-afe77737 ctx-06c20804) Received unknown parameters for command listRegions. Unknown parameters : listall WARN [c.c.a.d.ParamGenericValidationWorker] (1770041501@qtp-538020828-3:

  1   2   >