[GitHub] cloudstack pull request: CLOUDSTACK-8759 - Destroying VPC router r...

2015-09-08 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/773#issuecomment-138493272 Looks like this has broken regular isolated network. VR is not going to 'up' state as SSH is failing on the link local ip. The order of the interfaces on VR has

[GitHub] cloudstack pull request: UI Change (Adding a message while configu...

2015-09-08 Thread nitin-maharana
Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/785 --- 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-8727: API call listVirtualMach...

2015-09-08 Thread kansal
Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-138492797 @DaanHoogland @kishankavala Have made some changes to test cases. Please review. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138463931 IMHO we need some more changes here, and it might be more clear and easier to handle if we would make a package per ubuntu release and debian release, beginning from

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138466873 It depends, openjdk-8 would not work with current init script in my opinion. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-08 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138476262 @vincentbernat Agreed, if we want a persistent file we should fix it in the packaging and not in the Python code. Code should never create files in a dynamic

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

2015-09-08 Thread jenkins
See Changes: [wido] debian: Updated dependencies for Packages -- [...truncated 396 lines...] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.223 sec - in

[GitHub] cloudstack pull request: UI Change (Adding a message while configu...

2015-09-08 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/785 UI Change (Adding a message while configuring firewall rules) Changes made in network.js file. You can merge this pull request into a Git repository by running: $ git pull

CCP-4.5.0: Adding additional interfaces are not accepting on VMs if the VM has vm snapshots

2015-09-08 Thread anil lakineni
Hi All, We are facing issues when adding additional interfaces on VMs if VM has vm snapshots. Issue is VM not accepting to add additional interface due to VM snapshots. Error is: *Add netowrk to VM Action failed - Service Exception : NIC cannot be added to VM with VM Snapshots* I believe that

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138465225 @resmo Fair enough. These changes are just some small fixes. A new PR with those fixes is fine with me, but I think we can merge this one? --- If your project is set

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-08 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138465073 We have to LGTMs. Are we merging 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 your project

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138483590 Hi @jayapalu @remibergsma @miguelaferreira @DaanHoogland I executed the tests mentioned about. Results below: VPN remote access user limit

[GitHub] cloudstack pull request: CLOUDSTACK-8799 fixed the defalut routes

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/784#issuecomment-138450694 Hi @bvbharatk couple of questions regarding your changes: 1. There was also an issue with VPC VRs, which was reported/fixed here ->

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-08 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138465486 I prefer a persistent config (you know where to search for it) and not a dynamic one with hard coded values. If these would be dynamic, then it is clear to use the

[GitHub] cloudstack pull request: CLOUDSTACK-8799 fixed the defalut routes

2015-09-08 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/784#discussion_r38897976 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py --- @@ -261,20 +261,9 @@ def set_master(self): self.set_lock()

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138449931 Check on packages.ubuntu.com for the requirements of version/packages. All is good for 12.04/14.04/14.10/15.04. LGTM (Perhaps remove the tomcat6:

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138475391 @resmo Yes, we need to fix that further. It works now on Ubuntu 12.04 and 14.04, but for the LTS coming in 16.04 we need to have this fixed. --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138453360 Hi @jayapalu I will run the test_vpn_users and test_vpc_vpn tests. Will be back to you later. Cheers, Wilder --- If your project is

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-08 Thread vincentbernat
Github user vincentbernat commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138473948 @resmo the only problem with shipping a `/etc/sysctl.d/cloudstack.conf` is that this is more a packaging issue. It has to be handled for all distributions.

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138492302 @wilderrodrigues You can try remote access vpn connection in isolated network VR. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138491884 @wilderrodrigues Configuring remote access vpn and vpn users is success in test cases that covers the code changes. Regarding the remote access vpn connection

RE: [Blocker/Critical] VR related Issues

2015-09-08 Thread Koushik Das
I am also seeing this issue https://issues.apache.org/jira/browse/CLOUDSTACK-8823. This is a blocker for regular isolated network as VR is not coming to 'up' state due to ssh failure. -Original Message- From: Bharat Kumar [mailto:bharat.ku...@citrix.com] Sent: Thursday, 3 September

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138505642 Hi @jayapalu The change LGTM regarding the tests I have executed, which covers almost the whole thing. If you tested it manually, that's fine by

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-138522855 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: CLOUDSTACK-8800 : Improved the listVirtua...

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/780#issuecomment-138524943 LGTM (though not tested in an environment), anyone wants to review @wido @remibergsma @wilderrodrigues ? --- If your project is set up for it, you can reply to

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

2015-09-08 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 326 originally caused by: Started by upstream project "build-4.5" build number 524 originally caused by: Started by

Request for review: Quota feature PR

2015-09-08 Thread Rohit Yadav
Hi all, Please help review the PR: https://github.com/apache/cloudstack/pull/768 The Travis build is having timeout issues on the PR, though all tests are passing when the test groups are swapped (see PR for details). Thanks. Regards, Rohit Yadav Software Architect, ShapeBlue

[GitHub] cloudstack pull request: CLOUDSTACK-8759 - Destroying VPC router r...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/773#issuecomment-138523786 Hi @koushik-das , I will have a look into this. Cheers, Wilder --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Cloudstack 8816 entityuuid missing in som...

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/782#issuecomment-138524209 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: CLOUDSTACK-8799 fixed the defalut routes

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/784#discussion_r38914728 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -95,9 +95,17 @@ def get_control_if(self): return ip

[GitHub] cloudstack pull request: CLOUDSTACK-8800 : Improved the listVirtua...

2015-09-08 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/780#discussion_r38916972 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -2981,11 +2985,16 @@ public VmStatsEntry

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138522610 @wido we needed a specific implementation to set quota alerts so we've the quota specific alert manager for that; if you look at the codebase we have 4 difference

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138525668 @wido @vincentbernat agree --- 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-8800 : Improved the listVirtua...

2015-09-08 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/780#discussion_r38917202 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -4663,7 +4663,7 @@ private

[GitHub] cloudstack pull request: CLOUDSTACK-8821: UI Change

2015-09-08 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/786 CLOUDSTACK-8821: UI Change It provides appropriate message in the UI when configuring the firewall rules in Network page. If the default egress policy is allow, then it says to block

[GitHub] cloudstack pull request: CLOUDSTACK-8821: UI Change

2015-09-08 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/786#discussion_r38911089 --- Diff: ui/scripts/network.js --- @@ -1612,6 +1613,36 @@ });

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138519383 What does the Quota SMTP server do? Why can't we use the existing mechanisms for mailing? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138519872 @wido not a server, just an email client; it takes in quota email template and sends out emails --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8825, CLOUDSTACK-8824 : Fixed ...

2015-09-08 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/787 CLOUDSTACK-8825, CLOUDSTACK-8824 : Fixed issues if vm.allocation.algorithm is set to firstfitleastconsumed Fixed following issues if vm.allocation.algorithm is set to firstfitleastconsumed

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138549740 Hey @bhaisaab , Is there some documentation on the Wiki page on how to test it and what results to expects? Code-wise it's a bit to big to go

[GitHub] cloudstack pull request: config: doc: add default value 10 in desc...

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

Re: [Blocker/Critical] VR related Issues

2015-09-08 Thread Wilder Rodrigues
Hi guys, Thanks for reporting those issues. Concerning the following issues… * Remote Access VPN https://github.com/apache/cloudstack/pull/772 * Default INPUT/Forward policies https://github.com/apache/cloudstack/pull/765 * Default route not configured on VPC

[GitHub] cloudstack pull request: CLOUDSTACK-8688 - default policies for IN...

2015-09-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/765 --- 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-8688 - default policies for IN...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-138529499 Thanks, @bhaisaab ! --- 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-8800 : Improved the listVirtua...

2015-09-08 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/780#discussion_r38917337 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -4751,7 +4760,7 @@ private

RE: Usage Job failed

2015-09-08 Thread Somesh Naidu
The time is reported in milliseconds "144117719" - look up on the internet for an online converter. Regards, Somesh -Original Message- From: mvs babu [mailto:mvsbabu0...@outlook.com] Sent: Monday, September 07, 2015 3:57 AM To: us...@cloudstack.apache.org Cc:

VirtIO disk latency in Wins2012 - repoted wrong

2015-09-08 Thread Andrija Panic
Hi, just wondering if anyone experienced bug - virtio or Win2012 reated, not sure, but I expect drivers bug... In Task Manage you can see average latency for IOs (I was doing FIO disk benchmark inside Windows) - this shows 1000 bigger latency than what can bee seen from Resource Manager (and FIO

Re: [Blocker/Critical] VR related Issues

2015-09-08 Thread Wilder Rodrigues
Hi guys, CLOUDSTACK-8823 is a duplicate of https://issues.apache.org/jira/browse/CLOUDSTACK-8814, created 2 days ago. Cheers, Wilder On 08 Sep 2015, at 13:35, Wilder Rodrigues > wrote: Hi guys, Thanks for reporting those

Re: Heads-up: Serious investigator issue in 4.4

2015-09-08 Thread Pierre-Luc Dion
Hello Remi, Are you sure the async job issue is not fixed in 4.4.4? Because for us upgrading to it did solve that problem so far. Thanks for sharing latest info, Cheers PL On Sep 4, 2015 4:34 PM, "Remi Bergsma" wrote: > Hi all, > > > This brought me some serious

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-09-08 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-138591416 ok, LGTM at least we now have success and failure tests and all checks pass. We can add the more trivial tests for checkForKeyByPublicKey in a later PR ---

[GitHub] cloudstack pull request: CLOUDSTACK-8814 - Refactoring the configu...

2015-09-08 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/788 CLOUDSTACK-8814 - Refactoring the configuration of Routers and VPC routers nics Hi there, I refactored the configureDefaultNics() method in order to split the implementations

[GitHub] cloudstack pull request: CLOUDSTACK-8814 - Refactoring the configu...

2015-09-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/788#issuecomment-138594852 Tests results: Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS === ok Test Deploy Virtual

GUI Issue Around Filtering

2015-09-08 Thread Mike Tutkowski
Hi, Can anyone else confirm what I'm seeing here? http://i.imgur.com/cfl63sY.png If you go to Infrastructure, Primary Storage, go into the details of a primary storage, then go into View Volumes. I was expecting the volumes to be filtered by primary storage, but they're not. Thanks, --

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-09-08 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/685#issuecomment-138771684 :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138774846 There is a FS here https://cwiki.apache.org/confluence/display/CLOUDSTACK/Quota+Service+-+FS --- If your project is set up for it, you can reply to this email and

Re: Dynamic Scalable Template issue

2015-09-08 Thread Todd Pigram
Vadium, After installing XS62ESP1028 via CLI (no reboot on hosts yet) and building an Centos65 instance w/o ‘Dynamic Scalable’ option checked, it showed right in XenCenter. I installed the new tools from (XS62ESP1028) and still good. I created a template from this instance. Deployed said

[GitHub] cloudstack pull request: CLOUDSTACK 8819: Return correct template ...

2015-09-08 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/789 CLOUDSTACK 8819: Return correct template virtual size when using S3. When using S3 the virtual size of the template would equal the actual size. The script wraps around qemu-img,

[GitHub] cloudstack pull request: CLOUDSTACK-8805: Domains become inactive ...

2015-09-08 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/775#issuecomment-138648018 I tested on 4.5. I used mysql 5.6 client. It is working perfectly fine. I think the squirrel client doesn't support the replace function or there may

[GitHub] cloudstack pull request: CLOUDSTACK-8805: Domains become inactive ...

2015-09-08 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/775#issuecomment-138651375 If you tested, that is ok to me. I had not had the opportunity to build and test the ACS with that code in my environment. I just analyzed the code and

[GitHub] cloudstack pull request: CLOUDSTACK-8690:Added remote access vpn a...

2015-09-08 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138793699 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

Re: Heads-up: Serious investigator issue in 4.4

2015-09-08 Thread Remi Bergsma
Yes, I'm sure. Have a look at table async_job_join_map What foreign keys are there? I deleted fk_async_job_join_map__join_job_id, as per 4.5 upgrade script. Regards, Remi On 09 Sep 2015, at 01:44, Pierre-Luc Dion > wrote: Hello Remi, Are you

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-09-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/685 --- 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: Quota

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138788442 @wilderrodrigues testing would require some usage data; and global settings (search for quota in the UI) that are similar to usage ones, there is an enable setting

[GitHub] cloudstack pull request: Quota

2015-09-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-13875 Just to add some notes - a low quota threshold might lock a user account; but won't stop or destroy an existing or already running resource (such as a VM, volumes,