[GitHub] cloudstack pull request: messages: fix string in pt_BR translation...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/749#issuecomment-135657226 @milamberspace :+1: sorry I don't know how to use transifex or how to do translation the right way; I just found this issue in one of the production environment and

[GitHub] cloudstack pull request: Fix schema-421to430.sql issue after Maria...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/757#issuecomment-135658100 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: Coverity Issue: Null pointer dereferencin...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/629#issuecomment-135660397 LGTM. Looks like travis failures are unrelated --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: CLOUDSTACK-5863: revert volume snapshot f...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/732#issuecomment-135661166 @bhaisaab I updated this PR . The reponses is changed to SnapshotResponse for backward compatibility. @karuturi I will add some unit tests for snapshot (creat

[GitHub] cloudstack pull request: Fix schema-421to430.sql issue after Maria...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/757#issuecomment-135661617 Tested on Ubuntu 12.04 with mysql 5.5.41-0ubuntu0.12.04.1-log, it works . --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack pull request: CLOUDSTACK-8766: Fix infinite scrolling p...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/751#issuecomment-135661968 Did some API request size and time comparisons for various pagesizes against a large template collection (more than 4000 templates across 30+ zones); Page S

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135662629 I saw it a branch I am working on https://github.com/karuturi/cloudstack/blob/CLOUDSTACK-8647/utils/src/com/cloud/utils/log/CglibThrowableRenderer.java --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-8766: Fix infinite scrolling p...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/750#issuecomment-135663086 Please see the experimental analysis shared here: https://github.com/apache/cloudstack/pull/751#issuecomment-135661968 --- If your project is set up for it, you ca

Re: Discuss reserving memory on KVM hypervisors ref: CLOUDSTACK-8678

On 27-08-15 18:36, Josh Harshman wrote: > In a KVM cluster, CloudStack sees 100% of the compute node's RAM and treats > it as allocatable space which eventually leads to OOM killing guests. > > > There is an agent property named host.reserved.mem.mb which is able to be set > in the agent.prop

Re: [DISCUSS] Let's fix CloudStack Upgrades and DB migrations with CloudStack Chimp

On 28-08-15 08:51, Rohit Yadav wrote: > Hi all, > > Some of us have discussed in the past on fixing CloudStack’s upgrade and > DB migration, I’m trying to explore if we can really fix this. Please > review, advise changes, add suggestions along with your upgrade > experience so we can improve th

[GitHub] cloudstack pull request: Fix schema-421to430.sql issue after Maria...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/757 --- 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-8762: Check to confirm disk ac...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/753#issuecomment-135665672 I think we can close this one? Since #754 is the same? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135667196 Seems, good, but, all the logging is debug. Isn't there something which we have to print on info or error here? We want to make sure that we also print useful s

[GitHub] cloudstack pull request: CLOUDSTACK-5863: revert volume snapshot f...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/732#issuecomment-135667378 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: CLOUDSTACK-8762: Check to confirm disk ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135678520 @wido no need for info that, will fix to use error in case of errors --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/753#issuecomment-135679096 @wido this is for master branch, that is for 4.5 branch (not cherry-pickable) --- If your project is set up for it, you can reply to this email and have your reply

Re: How does the parameter startdate/enddate of api listEvents() use new time format like 'yyyy-MM-dd HH:mm:ss' ?

My scripts is just from cloudstack website https://cwiki.apache.org/confluence/display/CLOUDSTACK/Simple+class+for+making+API+calls%2C+Python and I just do some testing like this: 1. >>> from SignedAPICall import api 2. >>> api.listEvents(listall='true', startdate='2015-08-28') 3. >>> api.list

Re: How does the parameter startdate/enddate of api listEvents() use new time format like 'yyyy-MM-dd HH:mm:ss' ?

On Fri, Aug 28, 2015 at 10:40 AM, wrote: > 2015-08-28+16%3A20%3A00 ​very convincing , Coa Tong. Something is wrong in the calculation of the signature on this string. It seems to work for other people so can you provide environmental data: os version cloustack version language setting (both on

[GitHub] cloudstack-cloudmonkey pull request: Test01d

GitHub user cc494 opened a pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/9 Test01d You can merge this pull request into a Git repository by running: $ git pull https://github.com/Interoute/cloudmonkey-interoute master Alternatively you can review and app

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135694810 looks like it is removed from this commit 83fd8f60f3c503cf6fda7833b0f45c23a215f559 @miguelaferreira, tests from “cloud-plugin-network-nvp” project are faili

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135697615 @karuturi i noticed that while cleaning up my PR and I thought I had reverted that change. I will put that file back, and make a new PR --- If your project

[GitHub] cloudstack-cloudmonkey pull request: Test01d

Github user cc494 closed the pull request at: https://github.com/apache/cloudstack-cloudmonkey/pull/9 --- 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 fe

[GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

GitHub user miguelaferreira opened a pull request: https://github.com/apache/cloudstack/pull/758 Putting CglibThrowableRenderer.java back after it was removed in 83fd8f6 Also removing the entry 'log/' from .gitignore since that was the culprit for the removal of the file.

Re: How does the parameter startdate/enddate of api listEvents() use new time format like 'yyyy-MM-dd HH:mm:ss' ?

Hi Cao, Inside "_sort_request" function mentioned at the wiki, change the below line self.params.append(key + '=' + urllib.quote_plus(args[key])) to self.params.append( key + '=' + str.lower(urllib.quote_plus(args[key])). *replace(**"+", "%20") *) The time stamp parameter "2015-08-28 16:20:00

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135708224 @karuturi @pedro-martins file is restored in PR #758 --- 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-8762: Check to confirm disk ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135709691 @wido just checked again, most debug messages are in loop and changing them to info or errors would be unnecessary. In case of error, run time exceptions are thrown

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135712348 @bhaisaab It would be great if you could refactor the logic out, since it can be an object on its own, and that class is already huge as it is (> 5k LOC!). T

Apache Confluence Access

Hi all, I wanted to update some parts of Confluence. Pages regarding network infrastructure and database access. My username is borisroman, email: bo...@pcextreme.nl. Best regards, Boris Schrijver TEL: +31633784542 MAIL: bo...@pcextreme.nl

Re: Apache Confluence Access

done. ~Rajani On 28-Aug-2015, at 3:17 pm, Boris Schrijver wrote: > Hi all, > > I wanted to update some parts of Confluence. Pages regarding network > infrastructure and database access. > > My username is borisroman, email: bo...@pcextreme.nl. > > Best regards, > > Boris Schrijver > > TE

[GitHub] cloudstack pull request: CLOUDSTACK-8783: specify ciphersuite SSL_...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/759 CLOUDSTACK-8783: specify ciphersuite SSL_DH_anon_WITH_3DES_EDE_CBC_SHA in MockServerTest.java The ciphersuite could be different on os. Sometimes the MockServerTest fails due to the cip

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135733090 @miguelaferreira have moved the code to a new util class (so can be reusable perhaps by other hypervisors) ^^ --- If your project is set up for it, you can reply t

[GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/758#issuecomment-135739973 So that will fix the exceptions we are getting when running jetty, right? If so... LGTM. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135739986 @wilderrodrigues can you please test this PR, it LGTM but my approval is not binding! --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135740495 will test it now... Cheers, Wilder --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

Hi Sebastien, I am trying to configure and use ec2stack. This is what I have done $ pip install awscli $ cd cloudstack-ec2stack $ python setup.py install $ ec2stack-configure $ ec2stack & $ ec2stack-register http://localhost:5000 cs_api_key cs_secret_key $ aws ec2 describe-images --endpoint=http:

[GitHub] cloudstack pull request: CLOUDSTACK-8783: specify ciphersuite SSL_...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/759#issuecomment-135743960 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 enab

[GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/758#issuecomment-135744100 @wilderrodrigues that's the idea. --- 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 pro

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135745126 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: How does the parameter startdate/enddate of api listEvents() use new time format like 'yyyy-MM-dd HH:mm:ss' ?

Good catch Santhosh ! I had similar issue passing a space in ‘account name’. > On 28-Aug-2015, at 2:51 pm, Santhosh Edukulla > wrote: > > Hi Cao, > > Inside "_sort_request" function mentioned at the wiki, change the below > line > > self.params.append(key + '=' + urllib.quote_plus(args[key])) >

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135745806 I'm not able to build this branch with maven. ``` mvn clean install -P developer,systemvm ... [INFO] --

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-135747132 @DaanHoogland @kishankavala Added test case for checking the duplicate key-pair registrations. Please review. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/758#issuecomment-135749238 :+1: merging.. --- 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 f

[GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/758 --- 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-8727: API call listVirtualMach...

Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r38192659 --- Diff: setup/db/db/schema-452to460.sql --- @@ -354,6 +354,10 @@ CREATE VIEW `cloud`.`user_vm_view` AS left join `cloud`.`u

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135756504 @wido can you build it? --- 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-8762: Check to confirm disk ac...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135757828 @wido can you please try? --- 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 doe

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135757610 @miguelaferreira I didn't build it since all checks were green :) --- 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-8783: specify ciphersuite SSL_...

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/759#issuecomment-135759808 :+1: merging --- 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 feat

[GitHub] cloudstack pull request: CLOUDSTACK-8783: specify ciphersuite SSL_...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/759 --- 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: Changed variable s_logger to non-static a...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135761564 Thanks @miguelaferreira --- 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 doe

[GitHub] cloudstack pull request: use fakelocalhost in BasicRestClientTest

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/752#issuecomment-135762385 :+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: CLOUDSTACK-8785: Proper enforcement of re...

GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/760 CLOUDSTACK-8785: Proper enforcement of retry count (max.retries) for … …all work type handled by HighAvailability manager Retry count is properly enforced for all work types in H

Re: [DISCUSS] Let's fix CloudStack Upgrades and DB migrations with CloudStack Chimp

+1 and also +1 for Wido's idea. People are more forgiving with big changes during major version changes. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Wido den Hollander" > To: dev@cloudstack.apache.org > Sent: Friday, 28 August, 2

[GitHub] cloudstack pull request: use fakelocalhost in BasicRestClientTest

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/752#issuecomment-135767912 Ran all unit tests in that module and all passed: ``` mvn clean install -P developer,systemvm -pl :cloud-utils [INFO] Scanning for projects...

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135768876 @miguelaferreira your opinion feels like shackles to me. --- 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: Made following fixes in simulator

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135769042 @DaanHoogland I get the feeling that by this time tomorrow I might have a clue about what you meant --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135769001 Finished the tests... LGTM :+1: [root@cs2 cloudstack]# python -m marvin.deployDataCenter -i setup/dev/advanced.cfg Log Folder Path: /

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-135775262 @kansal it looks to me the new test tests exactly the same functionality as the previous one. Can you elaborate on the difference? And more specifically can we

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135776711 @miguelaferreira you know what ip my email lives on but to explain it to the world; your approval is very much appreciated and binding to me. --- If your proje

[GitHub] cloudstack pull request: Made following fixes in simulator

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/741#issuecomment-135777004 @DaanHoogland I figured it was something nice, but really wasn't sure about what you meant exactly. Thank you! --- If your project is set up for it, you can

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

See Changes: [miguelferreira] Putting CglibThrowableRenderer.java back after it was removed in 83fd8f6 [w.zhou] CLOUDSTACK-8783: specify ciphersuite SSL_DH_anon_WITH_3DES_EDE_CBC_SHA in MockServerTest.java --

[GitHub] cloudstack pull request: Coverity Issue: Null pointer dereferencin...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/629#issuecomment-135791640 2 LGTMs, merging --- 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 th

[GitHub] cloudstack pull request: Coverity Issue: Null pointer dereferencin...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/629 --- 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-8728: Testcase to Verify if VR...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-135792733 Something strange with this PR. The second commit merges master into the PR Was this an attempt to bring in new changes from master? If that was the

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

Rajani, Take a look at your ~/.aws/config and make sure the keys are correct. Also is you are using a current version of the AWS CLI make sure to add the signature_version block as mentioned in the ec2stack readme. >From the above looks like you are not using profiles therefore your config file s

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135814417 @miguelaferreira from the logs, looks like you were building 4.5 branch (the 4.5.2 artifact); on my system the build passes as well, are you using Java8? --- If y

Re: [GitHub] cloudstack pull request: Putting CglibThrowableRenderer.java back ...

This introduced an old findbugs issue as a new one because of which slow build failed. On Fri, Aug 28, 2015 at 17:27 PM, asfgit wrote: Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/758 --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8762: Check to confirm disk ac...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/754#issuecomment-135815149 @bhaisaab I've checked out this PR, which is your `shapeblue:4.5-CLOUDSTACK-8762` applied to `apache:4.5`. Yes, I do have Java8 installed, but the mvn config

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

Hi Carlos, I have the signature fix and config looks like below. I also checked that the keys are correct. On Fri, Aug 28, 2015 at 20:53 PM, Carlos Reategui wrote: Rajani, Take a look at your ~/.aws/config and make sure the keys are correct. Also is you are using a current version of the AWS C

[GitHub] cloudstack pull request: use fakelocalhost in BasicRestClientTest

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/752 --- 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] Let's fix CloudStack Upgrades and DB migrations with CloudStack Chimp

I REALLY like the idea. I wish we could gut the existing upgrade process and replace it wholesale. However, I'd urge us to use some existing tool as the foundation (while there links in the wiki page, it wasn't clear to me that this was assured), something open source that lots of other folks are a

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

See Changes: [kansal.k] Coverity Issue: NUll Pointer dereferncing fixed and commits squashed. -- [...truncated 28153 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-co

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

hmm. not sure. I am running: - ACS 4.5.1 - ec2stack 0.7.1 (on MS installed using pip) I have tested from: - MacOS 10.10.5 using aws cli 1.7.47 (installed using brew) - Ubuntu 14.04 using aws cli 1.3.8 and 1.8.0 (installed using pip) And it works, although I am using profiles but that shouldn't

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

See Changes: [w.zhou] use fakelocalhost in BasicRestClientTest -- [...truncated 28153 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud ---

[GitHub] cloudstack pull request: CLOUDSTACK-8728: Testcase to Verify if VR...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-135887682 @pritisarap12 Please squash the commits (get rid of the merge commit). We cannot merge it like this. Thanks! --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-8728: Testcase to Verify if VR...

Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-135894663 @remi: do you mean squash to get rid of the merge commit? Wouldn't the resulting commit have 2 parents? If so, that's not the solution. Or do you mea

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

See -- [...truncated 28153 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>> findbugs-maven-plugi

RE: Discuss reserving memory on KVM hypervisors ref: CLOUDSTACK-8678

Wido - Looks like the problem resides in the logic that does the detection of the host resources. It looks like most, if not all, the KVM host detection occurs in the LibvirtComputingResource class. Here I can modify it so it will take the value stored in dom0MinMem into account when calculati

Review wanted for PR 714 (CLOUDSTACK-8750)

Hi folks, This email is meant to ping the community about the PR (714) that solves the ticket CLOUDSTACK-8750 ( https://issues.apache.org/jira/browse/CLOUDSTACK-8750). The commit at first sight might look a bit too big to review, but at the end it is a pretty easy. It will remove a few lines of co

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

See -- [...truncated 28153 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>> findbugs-maven-plugi

[GitHub] cloudstack pull request: Changed variable s_logger to non-static a...

Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/714#issuecomment-135940383 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 featur

Re: Review wanted for PR 714 (CLOUDSTACK-8750)

It LGTM. I just updated the PR with that info. Thanks, Mike On Fri, Aug 28, 2015 at 7:50 PM, Rafael Weingärtner < rafaelweingart...@gmail.com> wrote: > Hi folks, > This email is meant to ping the community about the PR (714) that solves > the ticket CLOUDSTACK-8750 ( > https://issues.apache.org/

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

See -- [...truncated 28153 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>> findbugs-maven-plugi

Re: Discuss reserving memory on KVM hypervisors ref: CLOUDSTACK-8678

Make it configurable to enable/disable the check? Or even the size. For labs etc you might not have or want that much to spare. Erik Den lørdag 29. august 2015 skrev Josh Harshman følgende: > Wido - > Looks like the problem resides in the logic that does the detection of the > host resources.

[GitHub] cloudstack pull request: CLOUDSTACK-8728: Testcase to Verify if VR...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-135945903 @miguelaferreira If I recall correctly, `rebase -i` (where you'd usually squash stuff together and such) and doing nothing, will remove the merge commit already.

Re: Discuss reserving memory on KVM hypervisors ref: CLOUDSTACK-8678

Making the agent property work (host.reserved.mem.mb) would be awesome! On labs you can then set it to something low. Let me know if you need help with testing! Thanks, Remi Sent from my iPhone > On 29 Aug 2015, at 07:49, Erik Weber wrote: > > Make it configurable to enable/disable the che