[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157785310 Ping @DaanHoogland @remibergsma @miguelaferreira @bhaisaab @karuturi @borisroman First round of tests done and got the same results! I will run

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157782057 @DaanHoogland those warnings are in files that don't belong to my pull request. I could see in

Re: cloudstack UI

2015-11-18 Thread Miguel Ferreira
I think that would be a good move too. The only impediment I see are the hardcoved url endpoints. Specifically "client/" and "client/api". The UI code is already a separate module. But that gets bundled inside the management server WAR. We could separate that in two WARs if te UI would allow

[j.bac.o] Job configurations

2015-11-18 Thread Frank Maximus
Hi, I want to set up a jenkins server, and would to copy jobs from jenkins.buildacloud.org. I'm interested to get either the job config files, or a read-only account on jenkins, Thanks In Avdance, Frank *Frank Maximus * Senior Software Development Engineer *nuage*networks.net

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-157764335 Ping @miguelaferreira @remibergsma Tests are fine, but of course we can only be sure if they work with you as well. There was also a last

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157748781 @nvazquez can you please look at http://jenkins.buildacloud.org/job/build-master-slowbuild/2660/findbugsResult/new/ I think these are trivial, you just have

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

2015-11-18 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-157754858 Hey @wilderrodrigues @resmo, I was looking to fixing this as well. Is there any way I can contribute? What is the status of this? Thanks, pdube

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

2015-11-18 Thread jenkins
See Changes: [nick.livens] CLOUDSTACK-8832 : Update Nuage VSP plugin to work with Nuage VSP release [nicovazquez90] CLOUDSTACK-8956: Add VMware Api v5.5 and change pom.xml to use VMware [nicovazquez90] CLOUDSTACK-8956:

[WIKI] Create Design Document

2015-11-18 Thread Nicolás Vázquez
Hi all, I would like to propose a new feature for NiciraNVP Plugin. I would like to create a design document for 4.7 version in https://cwiki.apache.org/confluence/display/CLOUDSTACK/4.7+Design+Documents but I don't have the privileges. How can I proceed? Thanks, Nicolas

Re: [WIKI] Create Design Document

2015-11-18 Thread Nicolás Vázquez
Done, thanks 2015-11-18 12:50 GMT-03:00 Nicolás Vázquez : > Hi all, > > I would like to propose a new feature for NiciraNVP Plugin. I would like > to create a design document for 4.7 version in > https://cwiki.apache.org/confluence/display/CLOUDSTACK/4.7+Design+Documents

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157805785 sorry, @nlivens can you have a look? --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157810368 @wilderrodrigues maybe we can ask @anshul1886 to run the test wuite on hyperv? --- If your project is set up for it, you can reply to this email and have

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

2015-11-18 Thread jenkins
See -- [...truncated 27715 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157818838 If @anshul1886 can test it, just to make sure all is fine, would be very nice! @karuturi, do you have a HyperV test environment? :)

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread KrisSterckx
Github user KrisSterckx commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-157825620 Thanks @remibergsma and all reviewers - This is great. This is a great add to CloudStack and a great milestone for Nuage Networks. CloudStack + SDN : a

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

2015-11-18 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-157853386 @pdube you can create a pull request on github when you finish the coding and testing. This PR have lots of issues, so it will not be merged of course. I

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

2015-11-18 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-157854104 @ustcweizhou I have checked it out into another branch. I am working on it. And will create a PR when I am done. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157859746 More test results: * Environment - Management Server on CentOS 7.1 - One KVM Host on CentOS 7.1 - Agent + Common RPMs built from

[GitHub] cloudstack pull request: [4.6] Cannot list vlanipranges by keyword

2015-11-18 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1085#issuecomment-157652943 OK, please double check is always necessary. --- 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-9062: Improve S3 implementatio...

2015-11-18 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-157657472 Initially it looks good, but with S3 it's very hard to test this. At a first glance the code looks good. Much better then it was --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-9047 rename enums

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1049#issuecomment-157673706 @wilderrodrigues @wido can you review. The test keeps giving an exception but the destroyed suystem vms come back without problems and the test suite can even

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/935 --- 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-9067 - As I developer I want t...

2015-11-18 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1084#discussion_r45184075 --- Diff: utils/src/main/java/com/cloud/utils/StringUtils.java --- @@ -310,6 +310,11 @@ public static String mapToString(final Map

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-157680581 LGTM, based on a set of tests that I run on this branch (which I rebased myself first now that master moved on): ``` nosetests --with-marvin

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-157693735 @PaulAngus, every PR needs an issue on Apache Jira. In addition, how did you test your changes? Some input will help us to test it as well.

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread PaulAngus
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-157708006 Apologies. https://issues.apache.org/jira/browse/CLOUDSTACK-9073 created removal of /var/log/cloudstack-management and addition of catalina log

[GitHub] cloudstack pull request: CLOUDSTACK-9063 CitrixResourceBase refact...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1066#issuecomment-157676447 @franklouwers like doing a review on 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

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-157683273 Awesome! Thanks @jburwell for the thorough review, thanks @remibergsma for merging it in, thanks to all who helped getting this PR in! Much appreciated! :) --- If

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45187932 --- Diff: core/src/com/cloud/storage/template/S3TemplateDownloader.java --- @@ -19,303 +19,236 @@ package com.cloud.storage.template;

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-157683314 @borisroman checkstyle reports unused imports --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

Re: cloudstack UI

2015-11-18 Thread Abhinandan Prateek
On 17/11/15, 1:49 PM, "Sebastien Goasguen" wrote: > >> On Nov 17, 2015, at 7:24 AM, Abhinandan Prateek >> wrote: >> >> Hi, >> >> I am considering some improvements to cloudstack UI. Like delinking the >> quick view with the details page.

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157701917 According to Jenkins the build was fine: ``` Maven: -B -f /home/jenkins/jenkins-slave/workspace/cloudstack-pull-analysis/pom.xml

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-157681280 Congrats @nlivens @KrisSterckx this is now merged in master and will be included in 4.7.0. Thanks for all the work to get this in :-) --- If your project is

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread PaulAngus
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1087 Remove extraenous log dir and add catalina.out log rotation. remove unused /var/log/cloudstack-management directory add cloudstack-catalina to logrotate.d directory so that

Build failed in Jenkins: build-master-noredist #4746

2015-11-18 Thread jenkins
See Changes: [nicovazquez90] CLOUDSTACK-8956: Add VMware Api v5.5 and change pom.xml to use VMware [nicovazquez90] CLOUDSTACK-8956: Log NSX Api Version [nicovazquez90] CLOUDSTACK-8956: Deploy VM on NSX managed network

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157705996 good points, I do want to have a more generic way of creating complex log messages though. might be useful to construct womething like this for it anyway.

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-157709261 @PaulAngus the change looks reasonable. Have you checked upgrading the RPM to ensure that the any modified logrotate configuration files are left alone? --- If

[GitHub] cloudstack pull request: CLOUDSTACK-9050 Virtual router should onl...

2015-11-18 Thread dsclose
Github user dsclose closed the pull request at: https://github.com/apache/cloudstack/pull/1063 --- 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-9050 Virtual router should onl...

2015-11-18 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1063#issuecomment-157678223 CLosing this pull request as the patched script is already deprecated. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/801 --- 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-9067 - As I developer I want t...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157693480 @miguelaferreira discussed with me the over-engineering wrapper method just to add 1 test and we agreed on getting rid of it for the following reasons:

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

2015-11-18 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-157697285 @DaanHoogland You're right. Forgot to remove it in my last iteration. Thanks! --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157700801 @nvazquez Can you look at this build failure: http://jenkins.buildacloud.org/job/build-master-noredist/4746/console Does the slave need the 5.5 jar or

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-157655322 Although it’s based on an assumption, enforced by good pattern recognition, what happened to @miguelaferreira and @remibergsma was: 1. the code

[GitHub] cloudstack pull request: CLOUDSTACK-9058 - Respond with "saved_pas...

2015-11-18 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1079#issuecomment-157672288 @wilderrodrigues I'd be pleased to write a Marvin test though I've not worked with them before. I think the first step would be to get the tests running on my

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-18 Thread davidamorimfaria
Github user davidamorimfaria commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1075#discussion_r45172947 --- Diff: packaging/package.sh --- @@ -18,26 +18,37 @@ function usage() { echo "" -echo "usage: ./package.sh

[GitHub] cloudstack pull request: CLOUDSTACK-9063 CitrixResourceBase refact...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1066#issuecomment-157671857 Ping @DaanHoogland @remibergsma @davidamorimfaria Just finished some tests here. Based on the output + the code review I did, this PR LGTM :+1:

[GitHub] cloudstack pull request: [4.6] Cannot list vlanipranges by keyword

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1085#issuecomment-157647900 This time you tested and showed us how to do it. Good! The code LGTM, but I will also run the same test on this PR. Cheers, Wilder

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1084#discussion_r45176128 --- Diff: utils/src/main/java/com/cloud/utils/StringUtils.java --- @@ -310,6 +310,11 @@ public static String mapToString(final Map

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

2015-11-18 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1086 CLOUDSTACK-9051: update nic IP address of stopped vm This provides the feature to change ip address of NIC on a stopped vm by API and UI. You can merge this pull request into a Git

Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

2015-11-18 Thread Wido den Hollander
On 11/18/2015 08:23 AM, Qian Shaohua wrote: > Hi all, > > > > CS 4.5.2 KVM > > KVMStorageProcessor checks source volume of a snapshot for the ImageFormat > in createVolumeFromSnapshot(CopyCommand cmd). > > If the source volume is deleted, we get NPE. > > > > 2015-11-18 14:02:57,259

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157654321 less is more! reviewed java code only, didn't check the removed scripts. LGTM --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-18 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1075#issuecomment-157647291 As for installing the package, having distinct package version numbers allows one to update packages instead of having to remove and install them in

Read/Write splitting on database

2015-11-18 Thread Alireza Eskandari
Hi all, According to link below, does CloudStack split read/write request between master and slaves? https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=34838207 regards, Alireza

Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

2015-11-18 Thread Wei ZHOU
Qian, This is fixed in 4.6/master by commit bef92052ee9a7303b427782838a8a97623de231c diff --git a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImp~ index

[GitHub] cloudstack pull request: CLOUDSTACK-9058 - Respond with "saved_pas...

2015-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1079 --- 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-9047 rename enums

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1049#issuecomment-157671208 As for ```test_09_destroy_ssvm``` the exception is a false positive due to timeout. s-1-VM is destroyed and s-58-VM comes back but the timeout is reached

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1075#issuecomment-157638452 Quick question: were the generated packages installed successfully? Cheers, Wilder --- If your project is set up for it, you can reply to this

答复: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

2015-11-18 Thread Qian Shaohua
Oh, I see it, thanks! -邮件原件- 发件人: Wei ZHOU [mailto:ustcweiz...@gmail.com] 发送时间: 2015年11月18日 17:20 收件人: dev@cloudstack.apache.org 主题: Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted Qian, This is fixed in 4.6/master by commit

[GitHub] cloudstack pull request: CLOUDSTACK-9058 - Respond with "saved_pas...

2015-11-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1079#issuecomment-157670606 Gave LGTM on #1062, extending it here. --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-157647040 And how are we suppose to help reviewing/testing this PR, @ustcweizhou ? Any time to add some unit/integration tests and write few steps so we can

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-18 Thread NuxRo
Github user NuxRo commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1075#discussion_r45174353 --- Diff: packaging/package.sh --- @@ -18,26 +18,37 @@ function usage() { echo "" -echo "usage: ./package.sh [-p|--pack]

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-18 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1075#issuecomment-157643811 This is the package release version number, nothing to do with the thing that is packaged. Its purpose is to allow for this script to be used with

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1073#issuecomment-157648979 let's get this merged. I need no intergration tests for this improvement. a disect of ```updateRoutersRedundantState``` to make the bit that is fixed and

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

2015-11-18 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-157651712 @remibergsma and @miguelaferreira faced problems related to SSH timeout whilst testing this PR. I retested it and did not face the same issue: ```

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157736873 Ok, shouldn't it be removed from the install script and then? --- If your project is set up for it, you can reply to this email and have your reply appear on

Jenkins build is back to normal : build-master-noredist #4749

2015-11-18 Thread jenkins
See

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157727789 @nvazquez I tried to mv and copy the jar in the job but it doesn't work. now trying to see if your link yields a newer version then our job has --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157729881 @nvazquez is the older one still needed? --- 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-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157718592 @remibergsma It looks like it is not finding 5.5 jar. These are the steps I followed: * Download VMware-vSphere-SDK-5.5.0-1284541.zip file from

Build failed in Jenkins: build-master-noredist #4747

2015-11-18 Thread jenkins
See -- [...truncated 1710 lines...] [debug] execute contextualize [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] skip non existing resourceDirectory

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157727519 @DaanHoogland Hi! No, just renaming the old jar (version 5.1) will fail because it doesn't contain opaque network classes. Those ones are introduced in the 5.5

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157739765 @DaanHoogland Ok, sure --- 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-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157724852 @nvazquez Do you mean to say that we can just rename the old jar to contain the new version in the name and the sun shines again? --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-11-18 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-157731551 @DaanHoogland No, it's not. --- 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

Build failed in Jenkins: build-master-noredist #4748

2015-11-18 Thread jenkins
See -- [...truncated 1703 lines...] [debug] execute contextualize [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] skip non existing resourceDirectory

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

2015-11-18 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-157874321 @ustcweizhou Also if you have a fix for it, why not port it forward? --- 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-9053 security upgrade as per C...

2015-11-18 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1088 CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580 cloustack is not vulnerable but as the classes are in they might be used in the future so we upgrade to prevent accidental

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/1088 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1088#issuecomment-157887108 should have been against 4.6 --- 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

[STABILITY]} Large KVM Infrastructure with ACS

2015-11-18 Thread ilya
I'm curious if anyone runs ACS with atleast 250+ KVM hosts. We've been noticing weird issues with KVM where occasionally lots of KVM agents get Nio connection closed issue followed by barrage of alerts. In some instances the agent reconnects right away and in other - it attempts to reconnect but

Re: [STABILITY]} Large KVM Infrastructure with ACS

2015-11-18 Thread Rafael Weingärtner
When you say 250+, you mean 250+ host spread in lots of cluster, right? If I am not mistaken, ACS limits the number of KVM hosts in a cluster, something like 50? I do not remember now if that value can be configured, may it can be. I recall to have read something in a Red Hat doc about the KVM

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

2015-11-18 Thread jenkins
See -- [...truncated 27715 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>

Re: [STABILITY]} Large KVM Infrastructure with ACS

2015-11-18 Thread Daan Hoogland
sounds like a bad limit Ilya, i'll keep an eye out. On Wed, Nov 18, 2015 at 10:10 PM, ilya wrote: > I'm curious if anyone runs ACS with atleast 250+ KVM hosts. > > We've been noticing weird issues with KVM where occasionally lots of KVM > agents get Nio connection

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-8746: VM Snapshotting im...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/977#issuecomment-157883615 reran the tests. I noticed time out on destroy ssvm en cpvm but all others succeed. These timeouts, I have seen before in unrelated PRs. the systemvms come up

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

2015-11-18 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1089 CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580 cloustack is not vulnerable but as the classes are in they might be used in the future so we upgrade to prevent accidental

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

2015-11-18 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1089#issuecomment-157899791 Assuming Jenkin passes, 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

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-157871920 @nlivens please have a look at http://jenkins.buildacloud.org/job/build-master-slowbuild/2662/ two occasions of default encoding that you should make

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

2015-11-18 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157945578 @wilderrodrigues nope. No HyperV environment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

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

2015-11-18 Thread jenkins
See -- [...truncated 27715 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

2015-11-18 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-157971218 Guys can we get some +1s on this, so let's at least merge on master? cc @wilderrodrigues @DaanHoogland @NuxRo @wido @abhinandanprateek and others

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

2015-11-18 Thread jenkins
See -- [...truncated 27715 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>

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

2015-11-18 Thread jenkins
See -- [...truncated 27715 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>