[GitHub] cloudstack pull request: CLOUDSTACK-8686:Verify data disk attachme...

GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/631 CLOUDSTACK-8686:Verify data disk attachment for clusters with only zone wide primary -Attaching the uploaded/allocated volume to a VM on zwps You can merge this pull request into a Git rep

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125850427 Hi @kishankavala I'm on it. Cheers, Wilder --- If your project is set up for it, you can reply to this email and have your reply appea

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

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125849581 Moved the test to a new file test_browse_templates2.py. This is the command to run it nosetests-2.7 --with-marvin --marvin-config=setup/dev/advanced.cfg

[GitHub] cloudstack pull request: Fixed Coverity issue "Dereference null re...

GitHub user manuiiit opened a pull request: https://github.com/apache/cloudstack/pull/630 Fixed Coverity issue "Dereference null return value" Fixed Coverity issue "Dereference null return value" You can merge this pull request into a Git repository by running: $ git pull http

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125846995 Hi @wilderrodrigues Please look into CLOUDSTACK-8683 when possible --- If your project is set up for it, you can reply to this email and have your reply ap

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

GitHub user kansal opened a pull request: https://github.com/apache/cloudstack/pull/629 Coverity Issue: Null pointer dereferencing fixed This is in continuation with the PR #618. You can merge this pull request into a Git repository by running: $ git pull https://github.com/ka

Re: Git Rebase Question

So, I don't know if it's because it's an older version of Git or because it's running on Ubuntu, but when I try to perform the same action on Ubuntu (with an older version of Git), it works fine. The rebase was performed there without issue. On Tue, Jul 28, 2015 at 1:18 PM, Mike Tutkowski < mike.t

Re: Git Rebase Question

Interesting...after running "git rebase --abort" and then "git rebase upstream/4.5" again, I have the same issue, but on a new file: Applying: Make a Spring Bean for ApiHelper Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... error: Your local chang

Re: Git Rebase Question

Here's an example, Daan. I performed a "git rebase upstream/4.5" today and received the following (I have no local changes to overwrite, yet it thinks that's going to happen): Applying: Get IQN from newly created volume, store in DB, and return from API call Using index info to reconstruct a base

[GitHub] cloudstack pull request:

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12402350 Okay, cool! I hope it works fine for you. I think there was a 4.5.2 release, but I can't remember now

[GitHub] cloudstack pull request:

Github user creategui commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12402319 sorrymy bad... from 4.3.1. Although looks like the change was introduced for 4.5.1. I made the change to my DB

[GitHub] cloudstack pull request:

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12402083 It's not, dude... From which version of **CloudStack** are you upgrading? You said you are going to 4

[GitHub] cloudstack pull request:

Github user creategui commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12401927 6.0.2... I thought that would have been obvious. ;) The resource column in my host table has "com.cloud.hypervisor

Re: Failed upgrade from 4.3 to 4.5.1

So the problem is that code was changed so that XenServer600Resource could handle stuff that XenServer602Resource used to handle. However the upgrade code was not changed to make this change to existing entries in the resource column of the host table. I'll file a JIRA for this although not sure

[GitHub] cloudstack pull request:

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12401772 From which version are you trying to upgrade? Then we can have a look at the update file and fix it. Cheers,

[GitHub] cloudstack pull request:

Github user creategui commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12401616 Ok... I see that 602 can be handled by 600. However this change was not reflected in the upgrade code. --- If your projec

[GitHub] cloudstack pull request:

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12401593 Hi @creategui , From which version are you migrating? It was removed almost 1 year ago and nobody noticed, wh

[GitHub] cloudstack pull request:

Github user creategui commented on the pull request: https://github.com/apache/cloudstack/commit/81c5e184ba998a6ccd75dfffad32cbe5dbc8c2ec#commitcomment-12401470 Any reason why XenServer602Resource.java was removed? This is causing upgrades to 4.5.1 to fail with: java.lang.ClassNotFou

Re: [DISCUSS] Release principles for Apache CloudStack

I’m back online and will continue working on this as soon as I worked through the many mails ;-) Happy to be your RM together with Rajani! Regards, Remi On 17 Jul 2015, at 17:04, sebgoa mailto:run...@gmail.com>> wrote: Finally read the thread, It seems to me that a way forward is to have Re

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125639101 Hi @kishankavala Are you looking at the other problem or can I have a look into it? I was able to reproduce here... just let me know if you

[GitHub] cloudstack pull request: Coverity Issue: Null Pointer Dereferencin...

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

Github user sateesh-chodapuneedi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35655350 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,490 @@ +// Licensed to the Apache Software Foundation

Re: Failed upgrade from 4.3 to 4.5.1

Ok... I think I found the problem but need advice how to fix. Looks like XenServer 6.0.2 got dropped somewhere along the way. 4.3 has the XenServer602Resource.java file in: https://github.com/apache/cloudstack/tree/4.3/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource but 4.5 does no

[GitHub] cloudstack pull request: Coverity Issue: Null Pointer Dereferencin...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/628#issuecomment-125625465 LGTM :+1: Thanks for the test! Cheers, Wilder --- If your project is set up for it, you can reply to this email and have your reply ap

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

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35649303 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,490 @@ +// Licensed to the Apache Software Foundation (ASF) un

[GitHub] cloudstack pull request: Coverity Issue: Null Pointer Dereferencin...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/628#issuecomment-125613820 please put the coverity - or jira issue number in the commit message otherwise LGTM --- If your project is set up for it, you can reply to this email a

Re: Failed upgrade from 4.3 to 4.5.1

Adding dev... Looks like the first error is a common one from what I see in the mail archives, so the second one is the one causing my hosts to be in a disconnected state. I have tried running with jdk 8 also but same error. any ideas? forgot to mention in previous email that I used shapeblue a

Jenkins build is back to normal : build-systemvm64-master #583

See

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

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/626 --- 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: fix systemvm64template build, fix .deb pa...

Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/626#issuecomment-125581299 I will go ahead and apply this PR to fix builds, @rsafonseca can you create a PR with your line that does not look at line 22 ? @wilderrodrigues, I know! it

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

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/626#issuecomment-125581160 I'm merging it as the line#22 will be fixed in another PR: https://github.com/apache/cloudstack/commit/e6f81dbbbc8b597755007d9e6d247d2d6ec6c74c#commi

[GitHub] cloudstack pull request:

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/e6f81dbbbc8b597755007d9e6d247d2d6ec6c74c#commitcomment-12394433 In debian/rules: In debian/rules on line 13: Yes, it does. I was going to reply on the PR that I will merge it

[GitHub] cloudstack pull request: Coverity Issue: Null Pointer Dereferencin...

GitHub user kansal opened a pull request: https://github.com/apache/cloudstack/pull/628 Coverity Issue: Null Pointer Dereferencing fixed and Test cases added Discussed this issue in a separate PR. Creating a new PR for this. You can merge this pull request into a Git repository by r

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125580306 Yes @wilderrodrigues "VR started successfully ". But port 3922 is not open. Once I add iptables rule to allow 3922, MS is able to program rules. I'm looking

[GitHub] cloudstack pull request:

Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/commit/e6f81dbbbc8b597755007d9e6d247d2d6ec6c74c#commitcomment-12394387 In debian/rules: In debian/rules on line 13: @rsafonseca, this line look much better then the line#22 ;-) can you cr

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

Github user rsafonseca commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/626#discussion_r35639533 --- Diff: debian/rules --- @@ -1,6 +1,6 @@ #!/usr/bin/make -f # -*- makefile -*- -VERSION := $(shell mvn org.apache.maven.plugins:maven-h

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

See Changes: [wrodrigues] VR type in shared network is dhcpsrvr. Ips are being removed due to this issue -- [...truncated 495 lines...] at org.junit.runners.ParentRunner$3.run(Par

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

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/626#issuecomment-125573670 @pdion891 Thanks for the PR. I'm still a bit puzzled because the latest build (from 27th July) also failed by did not mention the dependency.

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

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125573161 @DaanHoogland I will create a separate test file with new the test case for now. Once the issues in test_browse_templates.py are fixed, will merge it back later

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125572581 Hi @kishankavala But you said you tested "VR successfully started on shared network. All IPs are configured properly."... Based on that, I assumed i

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125571871 Thanks @wilderrodrigues for merging. Ran into another blocker after VR came up: https://issues.apache.org/jira/browse/CLOUDSTACK-8683 --- If your project i

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/627 --- 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-8668: VR type in shared networ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125566567 Thanks for the PR, @kishankavala ! It LGTM :+1: and I will also test it Cheers, Wilder --- If your project is set up for it, you can r

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/627#issuecomment-125565419 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: Dereference NULL return value

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

Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/618#issuecomment-125552998 Closing this pull request. Will open one for each issue. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

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

Created PR: https://github.com/apache/cloudstack/pull/627 -Original Message- From: Wilder Rodrigues [mailto:wrodrig...@schubergphilis.com] Sent: 27 July 2015 06:23 PM To: dev@cloudstack.apache.org Subject: Re: [BLOCKER] unable to start VR in basic zone on master Just replied to the previ

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

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125546704 I would like to see the test run pass but don't think it is fair that this should be blocked by an older error in test code. @koushik-das please comment on how

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

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

[GitHub] cloudstack pull request: CLOUDSTACK-8668: VR type in shared networ...

GitHub user kishankavala opened a pull request: https://github.com/apache/cloudstack/pull/627 CLOUDSTACK-8668: VR type in shared network is dhcpsrvr. Ips are being removed due to this issue - VR IP config is loaded from /var/cache/cloud/cmdline - For shared network VR type is "d

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

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/626#issuecomment-125522926 LGTM The thing with the line number could be done in a different PR. Cheers, Wilder Sent from my iPhone On 28 Jul 2015,

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

Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/626#discussion_r35628826 --- Diff: debian/rules --- @@ -1,6 +1,6 @@ #!/usr/bin/make -f # -*- makefile -*- -VERSION := $(shell mvn org.apache.maven.plugins:maven-

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

Github user sailajamada commented on the pull request: https://github.com/apache/cloudstack/pull/607#issuecomment-125512446 Hi , I have fixed the test data file to move the browser volume/template specific data from "configurableData” section. But automation

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

Github user rsafonseca commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/626#discussion_r35627314 --- Diff: debian/rules --- @@ -1,6 +1,6 @@ #!/usr/bin/make -f # -*- makefile -*- -VERSION := $(shell mvn org.apache.maven.plugins:maven-h

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

Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35627322 --- 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 ...

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/625#issuecomment-125505835 @DaanHoogland I will. Although I don't understand why these are happening --- If your project is set up for it, you can reply to this email and have your reply appear o

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

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/625#issuecomment-125505270 @wido you are introducing two findbugs findings, can you look at those? --- If your project is set up for it, you can reply to this email and have your reply ap

RE: Automation data on master...

Daan, Basiczone cases/setup are failing because VR is not coming up and as a result VM deployment is failing. The bug is common for KVM/XS basic zones - defect is https://issues.apache.org/jira/browse/CLOUDSTACK-8668. Looks like Wilder is looking at this. VMWare - creation of volumes seems t

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

Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/626#discussion_r35620265 --- Diff: debian/rules --- @@ -1,6 +1,6 @@ #!/usr/bin/make -f # -*- makefile -*- -VERSION := $(shell mvn org.apache.maven.plugins:maven-

Re: Automation data on master...

Raja, this sounds very bad: 61 failures not considering some reported failures are in setup methods, not in tests. Can you shed your light on the results and causes? On Tue, Jul 28, 2015 at 9:10 AM, Raja Pullela wrote: > Hi, > > Here an automation summary on Master - ran it on a local env and hap

Automation data on master...

Hi, Here an automation summary on Master - ran it on a local env and happy to share logs if needed. For the bugs filed, the details are there in the tickets. Raja Config Passrate Bugs filed Failed Cases Simulator Basic 100 NA NA Simulator Adv 100 NA NA KVM Adv 94.3 TBF Test Resu