[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162803159 @jayapalu Hmmm the job is not starting? --- 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 p

[GitHub] cloudstack pull request: CLOUDSTACK-9086: ACS allows to create iso...

2015-12-07 Thread kansal
Github user kansal closed the pull request at: https://github.com/apache/cloudstack/pull/1125 --- 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: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1187#issuecomment-162797756 @bhaisaab, @remibergsma, this is the ported commit for 4.6 described in PR#1185 --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread nlivens
GitHub user nlivens opened a pull request: https://github.com/apache/cloudstack/pull/1187 Adapted HypervisorUtilsTest to no longer give false positives You can merge this pull request into a Git repository by running: $ git pull https://github.com/nlivens/cloudstack 4.6_test_i

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162797528 @sanju1010 it works for XenServer as well atleast, haven't tried VMware. But you'll have to change your script to adjust to the different output --- If your proj

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread Sanjeev N
@michaelandersen, if we want to use that template it only works for kvm. So either we have to skip the tests for other hypervisors or make the similar templates for all other hypervisors as well. On Tue, Dec 8, 2015 at 2:00 AM, michaelandersen wrote: > Github user michaelandersen commented on th

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162789887 @terbolous Remote access vpn i have tested It. I have triggered a job for systemvm template on my github branch for strongswan changes. http://jenkins.build

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162788704 Thanks @jayapalu @terbolous can you test remote access vpn please? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request: CLOUDSTACK-9069: Newly added project is n...

2015-12-07 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1082#issuecomment-162762394 Hi @remibergsma, here I attached the snapshots. See the cursor arrows. ![step-1_nitin](https://cloud.githubusercontent.com/assets/12583725/11648062/cb47e4

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162761653 @remibergsma @bhaisaab I am working on uploading the test case results for the site2site vpn. Meanwhile some one can pick up this branch and generate the syste

Re: [VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Mike Tutkowski
I just tried it on 4.5, though: works fine on that version. On Mon, Dec 7, 2015 at 3:26 PM, Mike Tutkowski wrote: > Normally I run my system VMs on XenServer and that seems to work fine at > the moment. > > I just had a need to run a VMware-only environment today, so came across > this issue wit

Re: [VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Mike Tutkowski
Normally I run my system VMs on XenServer and that seems to work fine at the moment. I just had a need to run a VMware-only environment today, so came across this issue with the virtual router (on two different setups of mine). On Mon, Dec 7, 2015 at 3:25 PM, Mike Tutkowski wrote: > Well, it do

Re: [VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Mike Tutkowski
Well, it does have an IP address assigned on Eth0 that falls within my user-VM range. The only other IP address it states is the loopback. I'm not sure what's "normal." On Mon, Dec 7, 2015 at 3:19 PM, Erik Weber wrote: > open the console and poke into it :-) > > > > On Mon, Dec 7, 2015 at 11:1

Re: [VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Erik Weber
open the console and poke into it :-) On Mon, Dec 7, 2015 at 11:16 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Note: The CS MS is saying it can't connect to the virtual router. I can > ping the virtual router manually from the CS MS, but I can't SSH into it > (not sure if I shou

Re: [VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Mike Tutkowski
Note: The CS MS is saying it can't connect to the virtual router. I can ping the virtual router manually from the CS MS, but I can't SSH into it (not sure if I should be able to). On Mon, Dec 7, 2015 at 3:14 PM, Mike Tutkowski wrote: > Hi, > > I am having a problem getting the virtual router to

[VMware] Problem starting virtual router on 4.6 and 4.7

2015-12-07 Thread Mike Tutkowski
Hi, I am having a problem getting the virtual router to leave the "Starting" state on 4.6 and 4.7. I am making use of the correct system VM template in each case, but the virtual router claims it requires an upgrade. This is in a Basic Zone and using local storage for the system VMs. Thoughts?

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162668836 @jayapalu Unfortunately I got no response and I cannot merge it without knowing if it is even finished. Please send an update asap and if it is finished

Master frozen: please do not merge to master and/or 4.6

2015-12-07 Thread Remi Bergsma
Hi all, We managed to get quite some PRs merged in over the last few days. Thanks for helping out! It will be a great new release with interesting new features and of course some bugs fixed. One PR that didn’t make it, is the VPN improvement and that’s a pity. It seems Jayapal was working on i

[GitHub] cloudstack pull request: Updating pom.xml version numbers for rele...

2015-12-07 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1186 Updating pom.xml version numbers for release 4.6.2-SNAPSHOT Set next version in 4.6 release branch to version 4.6.2-SNAPSHOT. Using ` ./tools/build/setnextversion.sh`. Ping

[GitHub] cloudstack pull request: CLOUDSTACK-8892:If VR enters out-of band ...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1158 --- 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: L10N : Updated locale files with latest t...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1097 --- 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: Fixed issues with test_vpc_vpn script

2015-12-07 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162651522 So removed my last comment about field 7 to 6. We need to choose 1 type of template os for both kvm and xen and base the test on that. @sanju1010: the ma

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1142 --- 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-9106 - As a Developer I want t...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1184 --- 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-9106 - As a Developer I want t...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162647475 @wilderrodrigues I think we're fine. That test also passes here: ``` Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH

[GitHub] cloudstack pull request: Add support for not (re)starting server a...

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1162 --- 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: Add support for not (re)starting server a...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1162#issuecomment-162644589 LGTM: Old default behaviour is to enable service: ``` [root@cs1 x86_64]# cloudstack-setup-management --tomcat7 Starting to configure CloudSt

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162631636 * Environment - Hardware required: TRUE - Management Server + MySQL on CentOS 7.1 - One KVM Host on CentOS 7.1 - Agent + Common R

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162624822 Ping @remibergsma @miguelaferreira @DaanHoogland Test passed as expected! ``` Test redundant router internals ... === TestName: test_0

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162624037 agree with @bhaisaab , @nlivens this will certainly decrease the failure rate ;) --- If your project is set up for it, you can reply to this email and have yo

[GitHub] cloudstack pull request: Prevent live-lock in NSX API client

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1178 --- 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: Prevent live-lock in NSX API client

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1178#issuecomment-162616790 @miguelaferreira gave me some explanations about the problem and how he was going to fix it. I went through the code and also checked the tests he wrote.

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162616151 LGTM based on these tests: ``` test_01_vpc_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_01_vpc_priv

[GitHub] cloudstack pull request: Implement Quota service

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162615010 Built it again, turned feature on, still looks good. Merging! --- 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-9106 - As a Developer I want t...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162614546 Went through the code. Looks Good To Me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack pull request: Implement Quota service

2015-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/768 --- 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-8858: listVolumes API fails fo...

2015-12-07 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162612642 @ustcweizhou , Yes. I agree. In case on instance id, vm_state is not null. --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-07 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1150#issuecomment-162612180 @bhaisaab @DaanHoogland I updated the change with current 4.6 branch and removed the commented codes. @remibergsma This only supports in VMware. Thanks :)

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162611887 Although it worked manually, I'm running the test again... in an brand new DC. ``` nosetests --with-marvin --marvin-config=/data/shared/marvin/

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1185#issuecomment-162603163 Good idea. @nlivens please open the same patch/PR again 4.6 branch, thanks. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack pull request: Implement Quota service

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162601662 @remibergsma LGTM, final issue fixed. Let's merge now! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cloudstack pull request: Implement Quota service

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162601030 @agneya2001 thanks, squashing the change and pushing now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162598613 Ping @remibergsma @DaanHoogland @miguelaferreira @borisroman Deployed another DC and followed the same step as the test, but did it manually:

[GitHub] cloudstack pull request: Implement Quota service

2015-12-07 Thread agneya2001
Github user agneya2001 commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162595100 @bhaisaab the issue is resolved. --- 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 d

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1185#issuecomment-162588761 @nlivens @bhaisaab If you guys want this in 4.6 as well (which seems like a good idea) please open a PR against 4.6. That will then be forward merged to master.

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1185#issuecomment-162578278 Great thanks, LGTM. cc @remibergsma - let's merge this, just so to have less Travis failures? (possibly also merge on 4.6?) --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162574201 Ping @remibergsma @miguelaferreira @DaanHoogland @borisroman Problem with my PR! I haven’t seen that one failing before. Will investigate.

[GitHub] cloudstack pull request: Quota

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162573927 @remibergsma sure, I'm discussing with Abhi on that minor fix. After that, will squash/amend the message in the git log. --- If your project is set up for it, you

[GitHub] cloudstack pull request: Quota

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162572535 @bhaisaab Minor thing: could you also rename the title to "Implement Quota service"? I will make sure it will be merged before we freeze. --- If your p

[GitHub] cloudstack pull request: Quota

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162572129 @agneya2001 OK, please ping me when done! --- 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: [4.7] CLOUDSTACK-8746: VM Snapshotting im...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/977#issuecomment-162565931 Run the tests again. LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1185#issuecomment-162551073 @bhaisaab, this is the PR for the test adaptation discussed in PR#1142 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: Adapted HypervisorUtilsTest to no longer ...

2015-12-07 Thread nlivens
GitHub user nlivens opened a pull request: https://github.com/apache/cloudstack/pull/1185 Adapted HypervisorUtilsTest to no longer give false positives You can merge this pull request into a Git repository by running: $ git pull https://github.com/nlivens/cloudstack test_impro

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162549079 @nlivens I like your patch, can you send that as a separate PR please? LGTM, cc @remibergsma please merge this (do you need to run your tests, if necessary?)

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162548576 Jenkins known issue: ![image](https://cloud.githubusercontent.com/assets/5129209/11629982/7a0ef4a8-9cfb-11e5-8c46-8083306958b8.png) --- If your p

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162546287 :-1: Tests are failing, we need to fix it before we consider a merge, sorry. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162545461 Hi Sanju, Isn't it the point of the test to verify this exact connectivity? But I any case I don't believe connectivity is the reason this t

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162544664 @sanju1010 the error lies in the script due to different outputs. the test seems to use macchinina as template, which has the following output for ping: # ping

Re: CloudStack Logging, the log4j Configuration file

2015-12-07 Thread Miguel Ferreira
+1 on this question. I too would like to know the reason for having all these files in different modules. On 07/12/15 12:52, "Chaz PC" wrote: >Hello, > > > > >I am studying the logging process used by CloudStack and came accross the >configuration file that is used by Log4j. What is confusin

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162539384 Hi Remi, As I mentioned in my PR the second test failure is related to connectivity issues between the vms part of s2s vpn. It is not because of my cha

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162535442 Doesn't work for me either: ``` Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS === ok Test VP

[GitHub] cloudstack pull request: Removal of class AgentBasedStandaloneCons...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/855#issuecomment-162533617 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \ component/test_v

Re: Feature freeze ACS 4.7 next Monday

2015-12-07 Thread cs user
Hi Folks, Would anyone else be able to give https://github.com/apache/cloudstack/pull/1037 a try at some point? Quite a serious issue when running in clustered mode with Xen, and trying to perform VM migrations between hosts. Thanks! On Mon, Dec 7, 2015 at 12:08 PM, sebgoa wrote: > The rn rep

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-07 Thread serg38
Github user serg38 commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-162532791 This index comes from schema-40to410.sql: CREATE TABLE `cloud`.`nicira_nvp_router_map` ( `id` bigint unsigned NOT NULL AUTO_INCREMENT COMMENT 'id',

[GitHub] cloudstack pull request: Quota

2015-12-07 Thread agneya2001
Github user agneya2001 commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162529656 @bhaisaab @remibergsma there is still one issue that I am looking at now. --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1184#issuecomment-162527902 Ping @DaanHoogland @remibergsma @borisroman I will continue the tests, but PR is ready to be reviewed. * Environment - Hardware requi

[GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...

2015-12-07 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/1184 CLOUDSTACK-9106 - As a Developer I want the Redundant VPC private gateway feature fixed This PR contains the same fixes from PR #1179, which was created against the master branch.

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162526429 @jayapalu Nice! Did you also test site2site VPN? --- 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-9095 : Hypervisor changes to s...

2015-12-07 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162523432 @DaanHoogland, below you can find a possible alternative for this test file. [HypervisorUtilsTestPatch.txt](https://github.com/apache/cloudstack/files/53965/Hype

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162522267 ping @sanju1010 --- 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 ha

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162520618 @sureshanaparti I do not get NPE when listVolumes. please notice the code: long instanceId = volume.getVmId(); if (instanceId

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162520027 Just tested this on 4.6.1-SNAPSHOT with the command: ``` nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-xen1.cfg -a tags=adv

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-07 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-162517321 Remote access vpn test results: nosetests-2.7 --with-marvin --marvin-config=/Users/jayapalreddy/advanced.cfg /Users/jayapalreddy/dev/github/cloudstack/test/

[GitHub] cloudstack pull request: Marvin test to verify that adding TCP por...

2015-12-07 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1183 Marvin test to verify that adding TCP ports 500,4500 and 1701 in vpn should not fail Please refer to JIRA ticket for more details https://issues.apache.org/jira/browse/CLOUDSTACK-9117

Re: Feature freeze ACS 4.7 next Monday

2015-12-07 Thread sebgoa
The rn repo contains a script that polls JIRA for a release filter and generates the tables of issues fixed: https://github.com/apache/cloudstack-docs-rn/blob/master/utils/jira.py but we are not tracking the ones that really are *features*…. small detail that would be nice to sort out. On Dec

[GitHub] cloudstack pull request: Quota

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-162506613 Abhi has fixed the issues, changes are now incorporated, commits squashed into one. cc @remibergsma @jburwell LGTM, let's merge this before EOD. ---

CloudStack Logging, the log4j Configuration file

2015-12-07 Thread Chaz PC
Hello, I am studying the logging process used by CloudStack and came accross the configuration file that is used by Log4j. What is confusing to me is that all the resources is stating that the "log4j-cloud.xml" is supposed to be the file used to confiugre the logging, but what I have noticed

ROOT volume resize fails if the VM is stopped

2015-12-07 Thread Nux!
Hello, Just hit this today.. Opened the following issue with it. Functionality is restored if I use a v4.4 qcow2/resizevolume.sh script. https://issues.apache.org/jira/browse/CLOUDSTACK-9115 -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162498348 @nlivens there are several tests like this bugging us. Do you know an alternative for this one? --- If your project is set up for it, you can reply to this em

CloudStack Logging, the log4j Configuration file

2015-12-07 Thread Chaz PC
I am studying the logging process used by CloudStack and came accross the configuration file that is used by Log4j. What is confusing to me is that all the resources is stating that the "log4j-cloud.xml" is supposed to be the file used to confiugre the logging, but what I have noticed is that no

[GitHub] cloudstack pull request: CLOUDSTACK-9086: ACS allows to create iso...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1125#issuecomment-162497602 it is the NetUtilsTest.test31BitPrefix...() family of tests that are failing --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162496553 @bhaisaab, we've looked into the code of that test, and the results are unreliable. The steps that are done in that test (along with the implementation of the c

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162494808 The volumes, in Ready state and are not attached to any VM have their vm_state has null and querying such volumes may result in NPE. These changes are only to

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162493847 @remibergsma Do you remember any changes that might be causing HypervisorUtilsTest.checkVolumeFileForActivityTest to fail? --- If your project is set up for it, y

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162493826 @ustcweizhou , Same with my result set as well. mysql> select * from vm_instance where state is null; Empty set (0.00 sec) mysql> SELECT id, na

[GitHub] cloudstack pull request: CLOUDSTACK-9086: ACS allows to create iso...

2015-12-07 Thread kansal
Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/1125#issuecomment-162490625 @DaanHoogland I see a test case in the NetUtilsTest.java class' testIsSameIpRange() function. assertFalse(NetUtils.isSameIpRange(cidrFirst, "10.3.6.5/50"));

[GitHub] cloudstack pull request: CLOUDSTACK-9113: skip vm with inconsisten...

2015-12-07 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1182 CLOUDSTACK-9113: skip vm with inconsistent state when getVmStats/getVmDiskStats on KVM, if there is a vm has inconsistent state between hypervisor and db, the getVmStat will terminate and

[GitHub] cloudstack pull request: CLOUDSTACK-9113: skip vm with inconsisten...

2015-12-07 Thread ustcweizhou
Github user ustcweizhou closed the pull request at: https://github.com/apache/cloudstack/pull/1181 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-9113: skip vm with inconsisten...

2015-12-07 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1181#issuecomment-162488604 wrong branch --- 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-9113: skip vm with inconsisten...

2015-12-07 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1181 CLOUDSTACK-9113: skip vm with inconsistent state when getVmStats/getVmDiskStats on KVM, if there is a vm has inconsistent state between hypervisor and db, the getVmStat will terminate and

[GitHub] cloudstack pull request: CLOUDSTACK-9086: ACS allows to create iso...

2015-12-07 Thread kansal
Github user kansal commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1125#discussion_r46809694 --- Diff: utils/src/main/java/com/cloud/utils/net/NetUtils.java --- @@ -1523,7 +1524,15 @@ public static boolean isIpWithtInCidrRange(final String ipAddre

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162486767 @sureshanaparti mysql> SELECT id, name, volume_type, state, vm_state FROM cloud.volume_view where vm_state is null and vm_id is not null; Empty set

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1150#issuecomment-162485224 Hmm.. this isn't supported on KVM, is it? That may be why I can't find it. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1150#issuecomment-162484701 @nitin-maharana @bhaisaab I looked around but just don't find what's changed. Please post screenshots of what is changed or instruct exactly how it should work.

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162484552 @sureshanaparti How do we force such a situation to occur? It seems to me that should be guarded on input and alerted on on output instead of silently filter

[GitHub] cloudstack pull request: CLOUDSTACK-8858: listVolumes API fails fo...

2015-12-07 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/830#issuecomment-162483571 @DaanHoogland , @ustcweizhou This issue is reproduced when the volume associated vm instance has null or invalid state (vm_state in volume_view). The code

Re: Feature freeze ACS 4.7 next Monday

2015-12-07 Thread Remi Bergsma
Hi Seb, Now that we merge everything and have a smaller scope, it should be easier to get the list of pull requests that got in since 4.6. Quick attempt: git log --pretty=oneline --abbrev-commit upstream/4.6..master | grep Merge | grep -v release | grep -v 4\.6 | awk {'print $5'} | sed s/\#//g

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-12-07 Thread fmaximus
Github user fmaximus commented on the pull request: https://github.com/apache/cloudstack/pull/1142#issuecomment-162483085 Are we still taking Travis into account? It has a unit test failure in HypervisorUtilTest, which also failed in other builds on Travis. This test seems to fail ran

[GitHub] cloudstack pull request: CLOUDSTACK-9069: Newly added project is n...

2015-12-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1082#issuecomment-162481574 @nitin-maharana How do I test this? I tried adding a project but the page keeps loading forever. Which dropdown should I look at? ![screen shot 2015-12-

[GitHub] cloudstack pull request: CLOUDSTACK-9086: ACS allows to create iso...

2015-12-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1125#issuecomment-162480371 Not sure if it is related but ``` Running com.cloud.utils.net.NetUtilsTest 2015-12-07 11:33:08,471 INFO [utils.net.NetUtils] (main:) Invalid va

  1   2   >