Build failed in Jenkins: build-master-jdk18 #202

See Changes: [koushik] CLOUDSTACK-8623: CPVM fails to start after MS is restarted during its initial start-up process [koushik] CLOUDSTACK-8623: CPVM fails to start after MS is restarted during its initial start-up process --

[GitHub] cloudstack pull request: Newb

GitHub user pavanb018 opened a pull request: https://github.com/apache/cloudstack/pull/581 Newb Test Create/Delete a LB rule and verify correct usage is recorded ... === TestName: test_01_lb_usage | Status : SUCCESS === ok ---

[GitHub] cloudstack pull request: CLOUDSTACK-8556: Unable to delete attache...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/481#issuecomment-121140868 The description of the pull request isn't really specific. Looking at the files changed it is only a test case which changed, right? No CS code itself. --- If your pro

[GitHub] cloudstack pull request: Coverity resource leaks

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/578#issuecomment-121140173 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: Embedded Tomcat & Jetty

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/372#issuecomment-121140056 Any progress on this PR? Since I really do like the proposal, but my knowledge of Jetty/Tomcat is indeed to limited. I can test the packaging, but I have no ide

[GitHub] cloudstack pull request: Findbugs issues with instance acces to st...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/573#issuecomment-121139921 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: CLOUDSTACK-8628: kvm: Disable Fencing whe...

GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/580 CLOUDSTACK-8628: kvm: Disable Fencing when no NFS storage pools are p… …resent On NFS we write a heartbeat, but without those we can not safely fence off a host. If we fenc

[GitHub] cloudstack pull request: CLOUDSTACK-8623: CPVM fails to start afte...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-121138730 Manually closing PR. I did a rebase to do away with unnecessary merge commit. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: CLOUDSTACK-8623: CPVM fails to start afte...

Github user koushik-das closed the pull request at: https://github.com/apache/cloudstack/pull/571 --- 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

[GitHub] cloudstack pull request: Coverity resource leaks

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/578#issuecomment-121138178 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 e

[GitHub] cloudstack pull request: Findbugs issues with instance acces to st...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/573#issuecomment-121137019 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 e

Build failed in Jenkins: simulator-singlerun #1361

See -- Started by upstream project "build-master-simulator" build number 2178 originally caused by: Started by an SCM change Started by an SCM change Started by upstream project "build-master"

Build failed in Jenkins: simulator-4.5-singlerun #253

See -- Started by upstream project "build-4.5-simulator" build number 308 originally caused by: Started by upstream project "build-4.5" build number 498 originally caused by: Started by an

Build failed in Jenkins: simulator-singlerun #1360

See -- Started by upstream project "build-master-simulator" build number 2177 originally caused by: Started by an SCM change Started by an SCM change Started by an SCM change Started by upstr

[GitHub] cloudstack pull request: CLOUDSTACK-8623: CPVM fails to start afte...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-121133952 Although RAT build status is showing ABORTED, the report is clean. Also the build failed due to some random test failure. Since travis is green, I am going ahead

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack/pull/575 --- 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 i

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-121131795 Squashing and merging commits. --- 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

Re: capitalized method name

Hi Daan, Please go ahead and change the name. Thanks for notifying. -Jayapal On 13-Jul-2015, at 8:45 PM, Daan Hoogland wrote: > Jayapal, > > I didn't notice this before but in PR 568, in commit > 638885a9de0fd651351f0b16cbdd9fc30c4a187d, you added a method name > starting with capital 'A' to

Re: [2/3] git commit: updated refs/heads/master to 63688e7

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/13/2015 05:30 PM, Daan Hoogland wrote: > LGTM but what PR did I miss here, Wido? > Uh, oops. None. I completely forgot! Won't happen again :) Wido > On Mon, Jul 13, 2015 at 4:59 PM, wrote: >> kvm: Write Libvirt instead of LibVirt >> >>

Re: [2/3] git commit: updated refs/heads/master to 63688e7

LGTM but what PR did I miss here, Wido? On Mon, Jul 13, 2015 at 4:59 PM, wrote: > kvm: Write Libvirt instead of LibVirt > > > Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo > Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/24f8e66e > Tree: http://git-wip-us.apac

Build failed in Jenkins: simulator-singlerun #1359

See -- Started by upstream project "build-master-simulator" build number 2176 originally caused by: Started by upstream project "build-master" build number 2462 originally caused by: Started

capitalized method name

Jayapal, I didn't notice this before but in PR 568, in commit 638885a9de0fd651351f0b16cbdd9fc30c4a187d, you added a method name starting with capital 'A' to citrixrecource base. Is this on purpose and can we change it to adhere to convention? public boolean AttachConfigDriveToMigratedVm(Conne

Build failed in Jenkins: simulator-singlerun #1358

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

New Defects reported by Coverity Scan for cloudstack

Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 3 new defect(s) introduced to cloudstack found with Coverity Scan. 32 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect

[GitHub] cloudstack pull request: CLOUDSTACK-8623: CPVM fails to start afte...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-120906494 test logic makes sense; LGTM waiting for travis --- 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-8623: CPVM fails to start afte...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-120903118 @karuturi @DaanHoogland Added unit tests for CPVM allocation. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-120897166 Thanks both, I pushed another UI change. Please see it too. This change allows admins to say if they want usernames with a domain (extracted from the IDP server).

Re: [Discuss] separate ML for PR and build notification?

On 13-Jul-2015, at 1:41 pm, Rene Moser mailto:m...@renemoser.net>> wrote: Since we "rebuild" our communication stack (slack, irc, ML, ...) I would bring up some discussion about the "noise" in the dev ML. I like to be in dev ML but I am not that interested in notifications about builds on PR, P

Re: [Discuss] separate ML for PR and build notification?

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 13-07-15 10:11, Rene Moser wrote: > Hey > > Since we "rebuild" our communication stack (slack, irc, ML, ...) I > would bring up some discussion about the "noise" in the dev ML. > > I like to be in dev ML but I am not that interested in > notific

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-120883355 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: CLOUDSTACK-8582: Skipping unsuitable test...

Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/532#issuecomment-120846548 @runseb Sorry for delayed response, I was away for two weeks. As said by Sanjeev, the tests were wrongly tagged. required_hardware = "true" implies test can

LAST CHANCE - speaker submissions for CCCEU15

All We've had some great talks submitted for Dublin so should now be able to put together a really exciting agenda , but we still have room on the schedule for some more, so... Please consider submitting a talk: The CFP closes in 2 days www.cloudstackcollab.org

[Discuss] separate ML for PR and build notification?

Hey Since we "rebuild" our communication stack (slack, irc, ML, ...) I would bring up some discussion about the "noise" in the dev ML. I like to be in dev ML but I am not that interested in notifications about builds on PR, PR comments, and Jenkins builds. I am suggest to make a seprate ML for t