Loading a custom HostAllocator

2015-07-08 Thread Amy Niklasson
Hi, How do you load a custom HostAllocator in CloudStack 4.5? The only documentation I've managed to find is http://docs.cloudstack.apache.org/en/master/alloc.html#implementing-a-custom-hostallocator However,I can't find the files mentioned, components.xml and components-premium.xml, in /client/ to

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/558#issuecomment-119490399 @DaanHoogland @remibergsma @karuturi It LGTM. There is still 1 issue, that was also mentioned by Ian, but I'm fixing that. The deta

Re: [ACS4.5] status

2015-07-08 Thread Daan Hoogland
On Tue, Jul 7, 2015 at 10:55 AM, Rohit Yadav wrote: >> On 07-Jul-2015, at 11:47 am, Daan Hoogland wrote: >> >> I haven't heard of this in a few days. Does anyone keep track of 4.5 >> releasebility at the moment? > > Yes. I’ve shared this earlier that I plan to start this in 1-2 weeks time, > til

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/558#discussion_r34125971 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -407,10 +410,10 @@ def fw_vpcrouter(self):

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/558#discussion_r34126048 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -407,10 +410,10 @@ def fw_vpcrouter(self):

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/558#discussion_r34126397 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -407,10 +410,10 @@ def fw_vpcrouter(self):

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/558#discussion_r34126781 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -407,10 +410,10 @@ def fw_vpcrouter(self):

Re: Loading a custom HostAllocator

2015-07-08 Thread Wido den Hollander
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08-07-15 10:08, Amy Niklasson wrote: > Hi, How do you load a custom HostAllocator in CloudStack 4.5? The > only documentation I've managed to find is > http://docs.cloudstack.apache.org/en/master/alloc.html#implementing-a- custom-hostallocator >

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread pavanb018
GitHub user pavanb018 opened a pull request: https://github.com/apache/cloudstack/pull/566 Privtmpl Comments provided in PR 241 are addressed and the commits for those are in this PR. You can merge this pull request into a Git repository by running: $ git pull https://github.c

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-07-08 Thread pavanb018
Github user pavanb018 closed the pull request at: https://github.com/apache/cloudstack/pull/241 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack pull request: Automation for issue, Private template no...

2015-07-08 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/241#issuecomment-119507984 All the comments in this PR are addressed and those commits are in PR 566 --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/558#discussion_r34128899 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -407,10 +410,10 @@ def fw_vpcrouter(self):

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/558#issuecomment-119509684 L reasonably GTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have t

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/566#issuecomment-119511153 @pavanb018 You will have to add license headers to the new files --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/558#issuecomment-119515211 Thanks, @DaanHoogland Will proceed from here. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/558 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...

2015-07-08 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/567 CLOUDSTACK-8583 : fixing issue related to script test/integration/com… issue --- Few testcases are failing because of unsupported storage type .Deploy vm with data disk is

Build failed in Jenkins: simulator-singlerun #1345

2015-07-08 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2162 originally caused by: Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely o

Build failed in Jenkins: simulator-singlerun #1346

2015-07-08 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2163 originally caused by: Started by upstream project "build-master" build number 2451 originally caused by: Started

resource leaks

2015-07-08 Thread Daan Hoogland
I have started working on this in [1] and found that a lot of this is in the upgrade code. I fixed a few of those and all of them in other code. I will triage the upgrade issues separately and like to have a review on [1] now, please. [1] https://github.com/apache/cloudstack/pull/564 On Tue, Jul

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/566#issuecomment-119533762 test_project_private_template (integration.component.test_project_template.TestProjectPrivateTemplate) ... === TestName: test_project_private_template | Status : S

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/566#issuecomment-119533859 On Advanced Zone: Test Deploy Virtual Machine ... === TestName: test_deploy_vm_with_tags | Status : SUCCESS === ok ---

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/566#issuecomment-119533805 On Basic Zone: Test Deploy Virtual Machine ... === TestName: test_deploy_vm_with_tags | Status : SUCCESS === ok --

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134668 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under on

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134700 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under on

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134758 --- Diff: test/integration/component/test_user_taglisting.py --- @@ -0,0 +1,129 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...

2015-07-08 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/567#issuecomment-119536872 LGTM!! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134916 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under on

[GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...

2015-07-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/567 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Build failed in Jenkins: simulator-singlerun #1347

2015-07-08 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2164 originally caused by: Started by an SCM change Started by upstream project "build-master" build number 2452 origin

[GitHub] cloudstack pull request: CLOUDSTACK-8615 :skip all tests in test_p...

2015-07-08 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/565#issuecomment-119545250 please wait for 2 manual reviews before pushing to master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: coverity resource leaks

2015-07-08 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/564#issuecomment-119546107 changes looks good. :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread Daan Hoogland
guys and dolss, this was closed with only 1 LGTM! we agreed not to submit with less then two ok reviews on master. So next steps? revert? some extra justification or else revert? . On Wed, Jul 8, 2015 at 12:39 PM, asfgit wrote: > Github user asfgit closed the pull request at: > > https:/

Re: revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread Sanjeev N
I thought there is a limitation on LGTMs for integration tests? On Wed, Jul 8, 2015 at 5:11 PM, Daan Hoogland wrote: > guys and dolss, > > this was closed with only 1 LGTM! we agreed not to submit with less > then two ok reviews on master. So next steps? revert? some extra > justification or els

Re: revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread Daan Hoogland
That was never formalized, hence I put it as a shakespearian question. I did sugest it otherwise I would have just reverted. I don't like the exception but if the number of people working on integration is to low to have a good flow we could. On Wed, Jul 8, 2015 at 1:46 PM, Sanjeev N wrote: > I t

Deleting sparse snapshots from XenServer SR

2015-07-08 Thread Francois Gaudreault
Hi Folks, So we are in a situation where we have (a lot of) "sparse" snapshots, so snapshots hanging there without the actual VM still existing, on one SR, and they don't clean up. Is it safe to flush them manually from XenServer or there is another way of doing it? Thanks -- Francois Gaudr

Introducing Boris Schrijver

2015-07-08 Thread Wido den Hollander
Hi all, As you might know, we at PCextreme are using CloudStack for our public cloud offering and have been looking for a CloudStack dev for a long time. We found one; so let me introduce Boris Schrijver. Boris will be our fulltime CloudStack developer with no other purpose then making CloudStac

Re: Introducing Boris Schrijver

2015-07-08 Thread sebgoa
On Jul 8, 2015, at 5:08 PM, Wido den Hollander wrote: > Hi all, > > As you might know, we at PCextreme are using CloudStack for our public > cloud offering and have been looking for a CloudStack dev for a long time. > > We found one; so let me introduce Boris Schrijver. > > Boris will be our

Re: Introducing Boris Schrijver

2015-07-08 Thread Daan Hoogland
good luck Boris, Do not hesitate to make PRs for any strange oddities you see. On Wed, Jul 8, 2015 at 5:24 PM, sebgoa wrote: > > On Jul 8, 2015, at 5:08 PM, Wido den Hollander wrote: > >> Hi all, >> >> As you might know, we at PCextreme are using CloudStack for our public >> cloud offering and h

Re: Introducing Boris Schrijver

2015-07-08 Thread Mike Tutkowski
Welcome aboard, Boris! Please don't be shy about asking questions. :) On Wed, Jul 8, 2015 at 9:08 AM, Wido den Hollander wrote: > Hi all, > > As you might know, we at PCextreme are using CloudStack for our public > cloud offering and have been looking for a CloudStack dev for a long time. > > W

Re: Introducing Boris Schrijver

2015-07-08 Thread Rohit Yadav
Welcome Boris, looking forward to your PRs. Cheers. On 08-Jul-2015, at 8:38 pm, Wido den Hollander mailto:w...@widodh.nl>> wrote: Hi all, As you might know, we at PCextreme are using CloudStack for our public cloud offering and have been looking for a CloudStack dev for a long time. We found

Re: Introducing Boris Schrijver

2015-07-08 Thread Nux!
Welcome, Boris :-) Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Wido den Hollander" > To: dev@cloudstack.apache.org > Cc: bo...@pcextreme.nl > Sent: Wednesday, 8 July, 2015 16:08:09 > Subject: Introducing Boris Schrijver >

Re: revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread David Nalley
More tests are good, on the other hand exceptions tend to beget exceptions at tremendous speed. On Wed, Jul 8, 2015 at 7:48 AM, Daan Hoogland wrote: > That was never formalized, hence I put it as a shakespearian question. > I did sugest it otherwise I would have just reverted. I don't like the >

Re: revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread Rajani Karuturi
PR #565 is also closed with one review. I dont agree to adding an exception. Reviewing would be one way to get others engaged and become aware. Otherwise, they lay there untouched and unused. I am sure we have enough committers who can review. Lately, lot more folks are adding marvin tests as wel

[GitHub] cloudstack pull request: dhcp/dns offload feature config dirve cha...

2015-07-08 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/568 dhcp/dns offload feature config dirve changes for vm migration Added vm migration changes for config drive. Added integration test for this. You can merge this pull request into a Git repos

[GitHub] cloudstack pull request: Config drive changes for vm migration

2015-07-08 Thread kishankavala
Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/552#issuecomment-119812602 Looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[4.5] Issue logging in under User account

2015-07-08 Thread Mike Tutkowski
Hi, I recently updated my local 4.5 branch to what's in the CloudStack repo and I noticed a problem trying to log in as a user of a User account (I get back a 432 and the CloudStack GUI doesn't display an error message or anything). (If I type in an incorrect password, I get back a proper error me

[GitHub] cloudstack pull request: dhcp/dns offload feature config dirve cha...

2015-07-08 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/568#discussion_r34226433 --- Diff: test/integration/component/test_dhcp_dns_offload.py --- @@ -0,0 +1,2126 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...

2015-07-08 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/567#issuecomment-119841442 The commit looks good to me --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does no