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

2015-07-27 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/626#issuecomment-125471653 :+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 this feature ena

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

2015-07-27 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/624#issuecomment-125470801 Already merged, looks like bot is not closing PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

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

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

Re: Addition to CloudStack Project on Jira

2015-07-27 Thread Rajani Karuturi
added ~Rajani On Tue, Jul 28, 2015 at 12:14 PM, Ardhendu Shekhar < ardhendu.shek...@citrix.com> wrote: > Hi, > > I am the Program Manager at Citrix for CloudStack. Please add my profile ( > ardhendu.shek...@citrix.com) to > CloudStack project on Jira. > > Rega

Addition to CloudStack Project on Jira

2015-07-27 Thread Ardhendu Shekhar
Hi, I am the Program Manager at Citrix for CloudStack. Please add my profile (ardhendu.shek...@citrix.com) to CloudStack project on Jira. Regards, Ardhendu

Re: Using cloudstack RabbitMQ events

2015-07-27 Thread Daan Hoogland
thanks, the backwards compatibility thing is serious but less likely to be completely resolved. the string encoding thing should be easy to find. On Tue, Jul 28, 2015 at 1:20 AM, Yiping Zhang wrote: > Just files a Doc bug, CLOUDSTACK-8679 and a code bug CLOUDSTACK-8680. > > Yiping > > On 7/27/15,

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

2015-07-27 Thread Erik Weber
for item 4 i think you are right. When the old fastbuildserver went offline i provided a new host, but never knew the job, but i have been meaning to ask if it should test master and not its own stale branch Erik Den tirsdag 28. juli 2015 skrev Pierre-Luc Dion følgende: > Hi, > > I've spend som

[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 running every 3hours , this

[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 of

[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

[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: 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) under

[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) under

[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) under

[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_r35605705 --- 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: 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 4

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" 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 wrote: >> Hi, list:

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

2015-07-27 Thread jenkins
See 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 H

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 wrote: > Hi, list: > > First, please pardon me for some rant. > > > I have been using cloudstack rabbitmq events to integrate with external apps > si

Using cloudstack RabbitMQ events

2015-07-27 Thread Yiping Zhang
Hi, list: First, please pardon me for some 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 VM.CREATE

[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

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 > T

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 to

[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 this

[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:

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 wa

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 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 should

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 Secon

[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

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

[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: 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

[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

[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 `fk_external_bigsw

[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 not

[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 `fk_extern

[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 system

[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 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 chang

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 > wrote: > > Hi there Rajani, > > My apologies, I missed that one. > > I haven’t tested basic z

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 wrote: > > I faced the same issue with basic zone using xenserver on master.

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 > http://cloudstack-release-notes.readthedocs.org/en/latest/upgra

[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: * roo

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 wrote: >> Hi, >> >> It has been quite some time and this feature hasn't merged yet. >> >> I created a issue for this: >>

[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: 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 information

Re: Edit rights to Wiki

2015-07-27 Thread Daan Hoogland
On Mon, Jul 27, 2015 at 2:26 PM, Sarath Kasi wrote: > sarathkouk is in -- Daan

[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: https://packages.debian.org/wheez

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 Subject: Re: Edit rights to Wiki please give your cwiki uid? On Mon, Jul 27, 2015 at 2:13 PM, Sarath Kasi wrote: > Hi, > I'd

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 wrote: > Hi, > I'd like to update Testing section of LDAP: Trust AD and Auto > Import >on wiki . Could someone please grant me the

Edit rights to Wiki

2015-07-27 Thread Sarath Kasi
Hi, I'd like to update Testing section of LDAP: Trust AD and Auto Import on wiki . Could someone please grant me the permissions needed? Thanks, Sarath

[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: [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 component/test_browse_

[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: 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

[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 https://g

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 ha

[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 repl

[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 repl

[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-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:

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 ins

[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: 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 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: 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: 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 we

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 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] branch o

[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

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 wrote: > @Wilder, can you please look into this issue. > Looks like VR config file is incomplete and it is over-writing the IP > info

[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

[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 as

[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

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 in

[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

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

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