[GitHub] cloudstack pull request: CLOUDSTACK-8651: [Browser Based Upload Te...

2015-07-27 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125183196 never mind that last bot-comment, i found a working config its running. will get results in a minute. if impatient and a hero, go with your own result ;) ---

Re: Edit rights to Wiki

2015-07-27 Thread Daan Hoogland
On Mon, Jul 27, 2015 at 2:26 PM, Sarath Kasi sarath.k...@citrix.com wrote: sarathkouk is in -- Daan

Re: [DISCUSS] Automatic usage reporting / call-home for CloudStack

2015-07-27 Thread Wido den Hollander
On 27-07-15 11:52, Daan Hoogland wrote: gogogo The PR is out there: https://github.com/apache/cloudstack/pull/625 On Mon, Jul 27, 2015 at 10:28 AM, Wido den Hollander w...@widodh.nl wrote: Hi, It has been quite some time and this feature hasn't merged yet. I created a issue for this:

RE: Edit rights to Wiki

2015-07-27 Thread Sarath Kasi
HI, My uid is sarathkouk Thanks, Sarath -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Monday, July 27, 2015 5:53 PM To: dev dev@cloudstack.apache.org Subject: Re: Edit rights to Wiki please give your cwiki uid? On Mon, Jul 27, 2015 at 2:13 PM, Sarath

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/625 CLOUDSTACK-8677: Call-home functionality for CloudStack With this commit the Management Server will be default generate a anonymous Usage report every 7 (seven) days and submit this

Re: [BLOCKER] unable to start VR in basic zone on master

2015-07-27 Thread Wilder Rodrigues
Hi there Rajani, My apologies, I missed that one. I haven’t tested basic zone in a while, to be honest. I will have a look to see if I can be of any help. Cheers, Wilder On 27 Jul 2015, at 11:11, Rajani Karuturi raj...@apache.org wrote: I faced the same issue with basic zone using

[GitHub] cloudstack pull request: Unit tests for HA manager investigate met...

2015-07-27 Thread koushik-das
GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/624 Unit tests for HA manager investigate method. Refer to CLOUDSTACK-866… …6 for the code chenges You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread kansal
Github user kansal commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/618#discussion_r35524999 --- Diff: plugins/user-authenticators/saml2/src/org/apache/cloudstack/saml/SAML2AuthManagerImpl.java --- @@ -358,7 +363,8 @@ private boolean setup() {

[GitHub] cloudstack pull request: boring resourse leaks again

2015-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/602 --- 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:

2015-07-27 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12371406 Hi @wilderrodrigues, I saw the job success on 22th, I'm wondering if dependencies for keepalived package would have

[GitHub] cloudstack pull request:

2015-07-27 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12372033 So we should just change the install of libnl to use the backports ? --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request:

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12372819 I don't think that would be the solution because I still don't understand what happened. The build from 22nd, which

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8651: [Browser Based Upload Te...

2015-07-27 Thread Daan Hoogland
can't find a config that runs. I can do with some help. last state was: \\sbpltc2kag3qr\dhoogland:~/cloudstack/cloudstack/test/integration (CLOUDSTACK-8651) nosetests --with-marvin --marvin-config=../../tools/devcloud/devcloud.cfg -s -a tags=advanced,required_hardware=false

Re: [BLOCKER] unable to start VR in basic zone on master

2015-07-27 Thread Wilder Rodrigues
Just replied to the previous email to the list. Sorry, I missed that one. I haven’t tested basic zone in a while… will give it a try. Cheers, Wilder On 27 Jul 2015, at 14:51, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi there Rajani, My apologies, I missed that one. I

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35532842 --- Diff: setup/db/db/schema-452to460.sql --- @@ -398,3 +398,5 @@ CREATE TABLE `cloud`.`external_bigswitch_bcf_devices` ( CONSTRAINT

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35533953 --- Diff: setup/db/db/schema-452to460.sql --- @@ -398,3 +398,5 @@ CREATE TABLE `cloud`.`external_bigswitch_bcf_devices` ( CONSTRAINT

aws in pom

2015-07-27 Thread Daan Hoogland
H, aws is still metioned in our pom.xml. It can be removed, afaics. Now that aws compatibility is taken care of in a proxy, we should not be depending on the sdk anymore or is there some other stuff that still needs it? -- Daan

Re: aws in pom

2015-07-27 Thread Daan Hoogland
yes :( found it in my latest experiment :) On Mon, Jul 27, 2015 at 3:43 PM, Wido den Hollander w...@widodh.nl wrote: On 27-07-15 15:36, Daan Hoogland wrote: H, aws is still metioned in our pom.xml. It can be removed, afaics. Now that aws compatibility is taken care of in a proxy, we

Edit rights to Wiki

2015-07-27 Thread Sarath Kasi
Hi, I'd like to update Testing section of LDAP: Trust AD and Auto Importhttps://cwiki.apache.org/confluence/display/CLOUDSTACK/LDAP%3A+Trust+AD+and+Auto+Import on wiki . Could someone please grant me the permissions needed? Thanks, Sarath

[GitHub] cloudstack pull request:

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12371179 Hi again @pdion891 That's my SSVM built upon the template 17th July * SSVM Linux distro: *

[GitHub] cloudstack pull request:

2015-07-27 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/4cdfbf337d7e8a221e6e227836a72ee26a49641f#commitcomment-12371467 In debian/rules: In debian/rules on line 3: look like this line is not working on when building .deb the ${VERSION}

[GitHub] cloudstack pull request:

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12371572 In the build that failed it mentioned the libnl-genl-3-200, which is already installed in my SSVM, based on the

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35534192 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,473 @@ +// Licensed to the Apache Software Foundation (ASF) under

Re: Loading a custom HostAllocator

2015-07-27 Thread Wido den Hollander
On 27-07-15 12:28, Amy Niklasson wrote: I have tried this and I still can't find it. I have tried to find it both in the regular installation and the source installation of CloudStack. However I have managed to find this link

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/625#issuecomment-125199454 any way to verify this? --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35535873 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,473 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: Edit rights to Wiki

2015-07-27 Thread Daan Hoogland
please give your cwiki uid? On Mon, Jul 27, 2015 at 2:13 PM, Sarath Kasi sarath.k...@citrix.com wrote: Hi, I'd like to update Testing section of LDAP: Trust AD and Auto Importhttps://cwiki.apache.org/confluence/display/CLOUDSTACK/LDAP%3A+Trust+AD+and+Auto+Import on wiki . Could someone

[GitHub] cloudstack pull request:

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12370885 @pdion891 ping... I'm checking the dependencies on Debian packages website:

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35534267 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,473 @@ +// Licensed to the Apache Software Foundation (ASF) under

Re: aws in pom

2015-07-27 Thread Wido den Hollander
On 27-07-15 15:36, Daan Hoogland wrote: H, aws is still metioned in our pom.xml. It can be removed, afaics. Now that aws compatibility is taken care of in a proxy, we should not be depending on the sdk anymore or is there some other stuff that still needs it? Yes, the S3 Secondary

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread kansal
Github user kansal commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/618#discussion_r35524889 --- Diff: plugins/user-authenticators/saml2/src/org/apache/cloudstack/saml/SAML2AuthManagerImpl.java --- @@ -119,12 +123,12 @@ @Override

RE: [PROPOSAL] Deploy user instance from VM snapshot

2015-07-27 Thread Somesh Naidu
+1 The FS LGTM. Regards, Somesh -Original Message- From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] Sent: Monday, July 27, 2015 2:11 AM To: dev@cloudstack.apache.org Subject: [PROPOSAL] Deploy user instance from VM snapshot Hi all, I am working on an enhancement

[GitHub] cloudstack pull request:

2015-07-27 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12377173 The jenkins job is triggered on new commits or daily. I've triggered it manually over the weekend because the last automated

[GitHub] cloudstack pull request: Unit tests for HA manager investigate met...

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/624#issuecomment-125273928 LGTM :+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 have

Re: [PROPOSAL] Deploy user instance from VM snapshot

2015-07-27 Thread Wido den Hollander
On 07/27/2015 07:32 PM, Somesh Naidu wrote: +1 The FS LGTM. Indeed, +1. Even in production this is very useful. Wido Regards, Somesh -Original Message- From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] Sent: Monday, July 27, 2015 2:11 AM To:

[GitHub] cloudstack pull request: Unit tests for HA manager investigate met...

2015-07-27 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/624#issuecomment-125350061 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

Using cloudstack RabbitMQ events

2015-07-27 Thread Yiping Zhang
Hi, list: First, please pardon me for some rant. rant I have been using cloudstack rabbitmq events to integrate with external apps since 4.3.x. Recently we upgraded to CS 4.5.1 and noticed that CS 4.5 events are quite different from 4.3 events, at least for ones we are working with (for

Re: Using cloudstack RabbitMQ events

2015-07-27 Thread Daan Hoogland
Yiping, good rant; You are absolutely right. Did you create a ticket (or more tickets) for this? On Mon, Jul 27, 2015 at 11:47 PM, Yiping Zhang yzh...@marketo.com wrote: Hi, list: First, please pardon me for some rant. rant I have been using cloudstack rabbitmq events to integrate with

Build failed in Jenkins: build-systemvm64-master #582

2015-07-27 Thread jenkins
See http://jenkins.buildacloud.org/job/build-systemvm64-master/582/changes Changes: [sanjeev] CLOUDSTACK-8664: Verify if queryAsyncJobResult api return jobinstanceid [sanjeev] CLOUDSTACK-8659: Verify presentation of volume id in description of events table for 'SNAPSHOT.CREATE' type [Daan

Re: Using cloudstack RabbitMQ events

2015-07-27 Thread Yiping Zhang
Just files a Doc bug, CLOUDSTACK-8679 and a code bug CLOUDSTACK-8680. Yiping On 7/27/15, 3:29 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: Yiping, good rant; You are absolutely right. Did you create a ticket (or more tickets) for this? On Mon, Jul 27, 2015 at 11:47 PM, Yiping Zhang

[GitHub] cloudstack pull request: CLOUDSTACK-8652: remove old upgrade code

2015-07-27 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/608#issuecomment-125378707 +1 for the concept, we need to get rid of old crappy stuff, none of those upgrade path are tested during RC and dev a release. Some of them have been removed from

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605790 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605766 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605842 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35606107 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,487 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/618#discussion_r35521326 --- Diff: plugins/user-authenticators/saml2/src/org/apache/cloudstack/saml/SAML2AuthManagerImpl.java --- @@ -119,12 +123,12 @@ @Override

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/618#discussion_r35521408 --- Diff: plugins/user-authenticators/saml2/src/org/apache/cloudstack/saml/SAML2AuthManagerImpl.java --- @@ -358,7 +363,8 @@ private boolean setup() {

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/618#issuecomment-125154356 @kansal Please see my comments. While it's okay, please squash the commits into two separate commits; one targeting the saml related stuff and second one targeting

[GitHub] cloudstack pull request:

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/08348593a2ec03767e92ff247f6279df76434194#commitcomment-12368701 Hi @pdion891 Quite weird since it was tested with a SVM having the old version of keepalived (1.2.2) and

[GitHub] cloudstack pull request: CLOUDSTACK-8610. Unable to attach 7th Dis...

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/554#issuecomment-125155019 +1 for @DaanHoogland's comments: please add unit tests to cover the changes. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8666: Put host in Alert state ...

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/621#issuecomment-12513 Hi @koushik-das Perhaps refactoring the code in a way that its design is improved could make easy to add tests for it. --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-8666: Put host in Alert state ...

2015-07-27 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/621#issuecomment-125156829 @wilderrodrigues Agree that refactoring the code might help with the tests. But that will be a bigger effort. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-8651: [Browser Based Upload Te...

2015-07-27 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125157246 @DaanHoogland Are you running some private simulator test? If so is it done? --- If your project is set up for it, you can reply to this email and have your

Re: Loading a custom HostAllocator

2015-07-27 Thread Amy Niklasson
I have tried this and I still can't find it. I have tried to find it both in the regular installation and the source installation of CloudStack. However I have managed to find this link http://cloudstack-release-notes.readthedocs.org/en/latest/upgrade/upgrade-4.0.html which implies that the file

[4.6] RC1 soon ?

2015-07-27 Thread Pierre-Luc Dion
Hi, I've create this jira filter[1] for the Release Manager, based on it, there would be only 4 maybe just 3 blockers on 4.6. Based on this, should we consider placing a target date for RC1 somewhere like end of August ? What's missing and to do in 4.6 as far as I know: 1. Basic documentation

[GitHub] cloudstack pull request: fix systemvm64template build, fix .deb pa...

2015-07-27 Thread pdion891
GitHub user pdion891 opened a pull request: https://github.com/apache/cloudstack/pull/626 fix systemvm64template build, fix .deb package creation. - build .deb package job must be change for ``dpkg-buildpackage -uc -us`` ubuntu 14.04 enforce signature of package. CLOUDSTACK-8638

[CLEANUP] j.bac.o: duplicate jobs and outdated

2015-07-27 Thread Pierre-Luc Dion
Hi, I've spend some times understanding Jenkins job on j.bac.o and it look like there is few job that should be removed and pause? Maybe persons aware of them can answer? 1. There is 2 build-master-slowbuild http://jenkins.buildacloud.org/job/build-master-slowbuild/ running every 3hours , this

[PROPOSAL] Deploy user instance from VM snapshot

2015-07-27 Thread Sateesh Chodapuneedi
Hi all, I am working on an enhancement to ACS - Deploy user instance from VM snapshot. Current work flow of deploying instance from volume snapshot is involves copying the snapshot image from primary storage to secondary storage and then to the primary storage where the instance is being

[GitHub] cloudstack pull request: CLOUDSTACK-8664: Verify if queryAsyncJobR...

2015-07-27 Thread pritisarap12
Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/619#discussion_r35508814 --- Diff: test/integration/testpaths/testpath_queryAsyncJobResult.py --- @@ -0,0 +1,133 @@ +# Licensed to the Apache Software Foundation (ASF)

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8664: Verify if queryAsyncJobR...

2015-07-27 Thread Sanjeev N
LGTM!! On Mon, Jul 27, 2015 at 12:11 PM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/619#discussion_r35508814 --- Diff: test/integration/testpaths/testpath_queryAsyncJobResult.py

[GitHub] cloudstack pull request: CLOUDSTACK-8664: Verify if queryAsyncJobR...

2015-07-27 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/619#discussion_r35507858 --- Diff: test/integration/testpaths/testpath_queryAsyncJobResult.py --- @@ -0,0 +1,133 @@ +# Licensed to the Apache Software Foundation (ASF) under

RE: [Proposal] Replacing Openswan ipsec with Strongswan ipsec

2015-07-27 Thread Suresh Sadhu
+1 Kindly share any practical limitation or any know issues exists in strong swan[if anybody using this protocol in real time ,please share your experience ]... that will really helpful us(both dev and qa) to design ,code and test this feature in a better way. Regards Sadhu

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread kansal
Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/618#issuecomment-125106632 @DaanHoogland Added the test file. Checking only the false cases. @wilderrodrigues Got away with the IF statements and replaced with the api @karuturi suggested.

[GitHub] cloudstack pull request: CLOUDSTACK-8664: Verify if queryAsyncJobR...

2015-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/619 --- 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

Re: [DISCUSS] Automatic usage reporting / call-home for CloudStack

2015-07-27 Thread Wido den Hollander
Hi, It has been quite some time and this feature hasn't merged yet. I created a issue for this: https://issues.apache.org/jira/browse/CLOUDSTACK-8677 The code is ready to merge and is in the reporter [0] branch on the Git repo. Even if the VM isn't ready, I would like to merge this feature

[GitHub] cloudstack pull request: CLOUDSTACK-8651: [Browser Based Upload Te...

2015-07-27 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125133501 I think this one can be merged after simulator is done, right? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

Re: [BLOCKER] unable to start VR in basic zone on master

2015-07-27 Thread Rajani Karuturi
I faced the same issue with basic zone using xenserver on master. Wilder, Can you please comment on this? ~Rajani On Fri, Jul 24, 2015 at 4:00 PM, Kishan Kavala kishan.kav...@citrix.com wrote: @Wilder, can you please look into this issue. Looks like VR config file is incomplete and it is

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/613 --- 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: Dereference NULL return value

2015-07-27 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/618#issuecomment-125150991 LGTM :+1: Thanks for the improvements! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [DISCUSS] Automatic usage reporting / call-home for CloudStack

2015-07-27 Thread Daan Hoogland
gogogo On Mon, Jul 27, 2015 at 10:28 AM, Wido den Hollander w...@widodh.nl wrote: Hi, It has been quite some time and this feature hasn't merged yet. I created a issue for this: https://issues.apache.org/jira/browse/CLOUDSTACK-8677 The code is ready to merge and is in the reporter [0]

[GitHub] cloudstack pull request: CLOUDSTACK-8666: Put host in Alert state ...

2015-07-27 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/621#issuecomment-125152445 @DaanHoogland Writing an unit test for changes in AgentManagerImpl.java is hard due to the way the method gets invoked. Let me know if you have any suggestion.

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-27 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/613#issuecomment-125133196 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 enabled

[GitHub] cloudstack pull request: initial dockerization commit.

2015-07-27 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/623#issuecomment-125132961 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 enabled

[GitHub] cloudstack pull request: Dereference NULL return value

2015-07-27 Thread kishankavala
Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/618#issuecomment-125136388 LGTM. @bhaisaab can you also take a look at the changes? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub