[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1274#issuecomment-166824866 Code changes 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

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-166823449 @nitin-maharana HypervisorUtilsTest test have failed, please do a force push again. Code changes LGTM --- If your project is set up for it, you can reply t

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-166820236 Github is not showing the diff for the below file as there are too many changes. Is there any way to display it? engine/orchestration/src/org/apache/cl

[GitHub] cloudstack pull request: CLOUDSTACK-9185: [VMware DRS] VM sync fai...

2015-12-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1256#issuecomment-166816696 LGTM Thanks for the explanation @sureshanaparti --- 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: CLOUDSTACK-9199: Fixed deployVirtualMachi...

2015-12-22 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1280 CLOUDSTACK-9199: Fixed deployVirtualMachine API does not throw an error when cpunumber is specified for static compute offering https://issues.apache.org/jira/browse/CLOUDSTACK-9199

[GitHub] cloudstack pull request: CLOUDSTACK-9185: [VMware DRS] VM sync fai...

2015-12-22 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/1256#issuecomment-166815006 @bhaisaab Got this exception on VM power sync when router is migrated back (due to a DRS rule set for VMs together) after manual migration from CCP. During t

[GitHub] cloudstack pull request: CLOUDSTACK-9185: [VMware DRS] VM sync fai...

2015-12-22 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1256#discussion_r48324398 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -3685,7 +3685,7 @@ public void setStoragePoolAllocators(final Lis

[GitHub] cloudstack pull request: update release to 4.7.0 so it use proper ...

2015-12-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1103#issuecomment-166814374 @pdion891 LGTM, but you'll need to fix the version again for master/4.8 and squash the commits --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack pull request: engine/schema: Use semantically correct u...

2015-12-22 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1279 engine/schema: Use semantically correct upgrade path 4.6.2->4.7.0 Adds an explicit upgrade path class from 4.6.2 to 4.7.0, that extends over the upgrade path from 4.6.1 to 4.7.0 cc

[GitHub] cloudstack pull request: [4.7.0/master] DB Paths fixes: 4.5.3->4.7...

2015-12-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1247#issuecomment-166814115 Closing as the 4.5.3 issue has been fixed on 4.7/master. Will open a new PR for the 4.6.2 to 4.7.0 semantic upgrade path/class fix. --- If your project is set up

[GitHub] cloudstack pull request: [4.7.0/master] DB Paths fixes: 4.5.3->4.7...

2015-12-22 Thread bhaisaab
Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack/pull/1247 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9198: Virtual router gets depl...

2015-12-22 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1278 CLOUDSTACK-9198: Virtual router gets deployed in disabled Pod While starting the router, send the user from the callingContext instead of defaulting to System user. https://issues.a

RE: Build failed in Jenkins: build-master-slowbuild #2783

2015-12-22 Thread Sateesh Chodapuneedi
Daan, Shall fix those listed in following. But unable to logon to Jenkins.buildacloud.org, can you please help with login? http://jenkins.buildacloud.org/login?from=/job/build-master-slowbuild/ws/vmware-base/target/findbugsXml.xml http://jenkins.buildacloud.org/job/build-master-slowbuild/ws/plugin

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2015-12-22 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1274#issuecomment-166799368 @DaanHoogland Updated the descrition --- 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 proj

Re: Build failed in Jenkins: build-master-slowbuild #2783

2015-12-22 Thread Abhinandan Prateek
At least one findbug is fixed in this pull request https://github.com/apache/cloudstack/pull/1254. Will look at others. [ShapeBlue] Abhinandan Prateek Software Architect , ShapeBlue d: +44 203 603 0540 | s: | m: +91 970 11 99011 e:

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

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

2015-12-22 Thread jenkins
See -- [...truncated 28732 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/1276#issuecomment-166714201 As for: > Is there a timing issue possible where both routers would be in the same state? If so, it might be good to > throw an exception before doi

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
Github user michaelandersen commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48290122 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -916,6 +916,9 @@ protected void updateSite2SiteVpnCo

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
Github user michaelandersen commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48288107 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -916,6 +916,9 @@ protected void updateSite2SiteVpnCo

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
Github user michaelandersen commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48286045 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -916,6 +916,9 @@ protected void updateSite2SiteVpnCo

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Removed unused methods from EventBus inte...

2015-12-22 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1260#issuecomment-166691207 Did you try rebuilding CS with each type of event queue? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48278591 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -916,6 +916,9 @@ protected void updateSite2SiteVpnConnectionSt

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1276#issuecomment-166675193 Good to see additional tests created. I would suggest adding a test that turns one of the routers off, and then seeing if the VPN connection works correctly. I quickl

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1276#issuecomment-18278 Did a code review on this PR and also talked to @michaelandersen on Slack about switching the MASTER router off. With the changes he applied on the Java fil

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1276#issuecomment-17779 Jenkins error is not related: ![image](https://cloud.githubusercontent.com/assets/5129209/11960321/6ed3c73e-a8d2-11e5-8b28-b90e81c79914.png) ---

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-13159 Ping @remibergsma @miguelaferreira @borisroman @michaelandersen PR created against 4.7. Please check all tests I ran and results in the previous PR.

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues closed the pull request at: https://github.com/apache/cloudstack/pull/1259 --- 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 f

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-12971 This PR is no longer needed as the PR #1277 contains the same fixes, but has been created against branch 4.7. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2015-12-22 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/1277 [4.7] Critical VPCVR issues fixed: CLOUDSTACK-9154; CLOUDSTACK-9187; and CLOUDSTACK-9188 This PR applies the same fixes as in the PR #1259, but against branch 4.7. Please refer

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread nitin-maharana
Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/1206 --- 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: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166619798 Ping @remibergsma @miguelaferreira @borisroman @michaelandersen Test are fine. It seems the the hiccups on network were caused by my environment. It

[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2015-12-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1274#issuecomment-166611273 LGTM but please describe (preferable in a unit test) what situation is prevented from happening here. --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166605170 wget google.nl works fine as well. ``` # wget google.com --2015-12-22 12:20:47-- http://google.com/ Resolving google.com... 2a00:1450:4

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
GitHub user michaelandersen opened a pull request: https://github.com/apache/cloudstack/pull/1276 FIX Site2SiteVPN on redundant VPC This PR: - fixes starting of Site2Site VPN on redundant VPC - fixes Site2Site VPN state checking on redundant VPC - improves the vpc_vp

[GitHub] cloudstack pull request: FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
Github user michaelandersen closed the pull request at: https://github.com/apache/cloudstack/pull/1275 --- 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 f

[GitHub] cloudstack pull request: FIX Site2SiteVPN on redundant VPC

2015-12-22 Thread michaelandersen
GitHub user michaelandersen opened a pull request: https://github.com/apache/cloudstack/pull/1275 FIX Site2SiteVPN on redundant VPC This PR: - fixes starting of Site2Site VPN on redundant VPC - fixes Site2Site VPN state checking on redundant VPC - improves the vpc_vp

[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2015-12-22 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1274 CLOUDSTACK-9196: Fixing null pointer exception when vm meta data is synced on upgraded setup https://issues.apache.org/jira/browse/CLOUDSTACK-9196 You can merge this pull request into a Git

Re: Results of a IPv6 brainstorm day

2015-12-22 Thread Daan Hoogland
Wido, consider me in. My main objective btw is to give users the possibility to run there VMs IPv6 only. The management layer is not such a biggy. I am still reading up on those rfcs and all the links in them. On Tue, Dec 22, 2015 at 11:17 AM, Wido den Hollander wrote: > > > On 12/22/2015 04:35

Re: Results of a IPv6 brainstorm day

2015-12-22 Thread Wido den Hollander
On 12/22/2015 04:35 AM, Ian Rae wrote: > Great to hear, next time I am happy to commit an engineer from CloudOps to > participate. We have done quite a bit of work around VPC and also need to > solve for IPv6 soon. > > Thanks for sharing, great initiative/goal and I will make sure the CloudOps >

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

2015-12-22 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166560296 Ping @remibergsma @miguelaferreira @borisroman @michaelandersen One test failed: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false. This tes

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1206#issuecomment-166556945 Made a new PR https://github.com/apache/cloudstack/pull/1273 with master. --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-166556707 Reference(4.6) : https://github.com/apache/cloudstack/pull/1206 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1273 CLOUDSTACK-9132: API createVolume takes empty string for name parameter Steps to Reproduce: Create a volume using createVolume API where parameter name is empty.

Re: template available on http://dl.openvm.eu/cloudstack/macchinina/x86_64/

2015-12-22 Thread Nux!
BTW, Cloudstack ships systemvm templates for HyperV also in ZIP, that's the reason I also pack it like that. AFAIK Windows can't deal with any other compression. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Nux!" > To: dev@cloudst

Re: template available on http://dl.openvm.eu/cloudstack/macchinina/x86_64/

2015-12-22 Thread Nux!
Hi, I do not know what you are running there, but: unzip macchinina-hyperv.vhd.zip Archive: macchinina-hyperv.vhd.zip inflating: macchinina-hyperv.vhd qemu-img info macchinina-hyperv.vhd image: macchinina-hyperv.vhd file format: vpc virtual size: 50M (52432896 bytes) disk size: 50M cluste

[GitHub] cloudstack pull request: CLOUDSTACK-9195: Cancelled/failed async j...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1272#issuecomment-166552880 Steps to test: - Prior to fix check for jobs in async_job table where job_status = 2 and job_complete_msid is NULL. - These entries won't get cleane

[GitHub] cloudstack pull request: CLOUDSTACK-9195: Cancelled/failed async j...

2015-12-22 Thread koushik-das
GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/1272 CLOUDSTACK-9195: Cancelled/failed async jobs not getting cleaned up f… …rom DB Setting 'job_complete_msid' field in async_job table when marking job as cancelled during MS rest

Re: DoNot host Copyrighted stuff on home.apache.org/

2015-12-22 Thread Sebastien Goasguen
> On Dec 22, 2015, at 1:34 AM, Srikanteswararao talluri > wrote: > > Hello All, > > Please do not host any copyrighted material like windows templates on your > apache home directory. There are concerns from Apache Infra team on These are not concerns, it is just plainly illegal. -sebastien