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

2015-09-07 Thread manuiiit
GitHub user manuiiit opened a pull request: https://github.com/apache/cloudstack/pull/780 CLOUDSTACK-8800 : Improved the listVirtualMachines API call to include memory utilization information for a VM for xenserver,hyperv and for vmware. You can merge this pull request into a Git

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

2015-09-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138244755 LGTM. We do not need a persistent file since this PY file will be called during runtime. --- If your project is set up for it, you can reply to this email and have

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

2015-09-07 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-138250804 @karuturi Wilder will add marvin test for this PR, I will run that and post the results --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Quota

2015-09-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138252027 Travis passed for all test except for first one due to timeout issue. --- If your project is set up for it, you can reply to this email and have your reply appear

Re: Weird Eclipse Issues around com.cloud.utils

2015-09-07 Thread Daan Hoogland
Mike, Boris, all, Re-importing the project is probably the easiest way but I only set the source path anew and that worked as well. On Fri, Sep 4, 2015 at 9:38 AM, Boris Schrijver wrote: > Hi Mike, > > I had the same problem a couple of days ago, it was due to a refactor of

[GitHub] cloudstack pull request: Fixing findbugs error due to PR #755 merg...

2015-09-07 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/779#issuecomment-138214857 Thanks everyone for the quick merge :) --- 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

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

2015-09-07 Thread Rajani Karuturi
Its successful now. Thanks for the help. $ ec2stack-register http://localhost:5000 CLOUDSTACK-API-KEY CLOUDSTACK-SECRET-KEY 127.0.0.1 - - [07/Sep/2015 13:52:24] "POST / HTTP/1.1" 200 - Successfully Registered! $ ec2stack-configure EC2Stack bind address [localhost]: EC2Stack bind port [5000]:

[RESULT][VOTE] Apache CloudStack EC2stack 0.8.0

2015-09-07 Thread sebgoa
Hi all, After 72 hours, the vote for CloudStack EC2stack 0.8.0 *passes* with 4 PMC + 1 votes +1 (PMC / binding) * Rajani * Rohit * Sebastien (implicit) * David Thanks to everyone participating. I will now prepare the release announcement to go out after 24 hours to give the mirrors time to

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

2015-09-07 Thread Daan Hoogland
Rafael, and other non committing contributors, committers are responsible for getting code merged and reverted if it breaks things in hindsight but not the only and sometimes not even the best people to judge code on its merits. Your opinion is very welcome to us all. thanks for the valuable

Jenkins build is back to normal : build-master-slowbuild #2247

2015-09-07 Thread jenkins
See

[GitHub] cloudstack pull request: Quota

2015-09-07 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138230799 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

[GitHub] cloudstack pull request: Fixing findbugs error due to PR #755 merg...

2015-09-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/779#issuecomment-138197048 LGTM no. 3 ;) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

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

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

2015-09-07 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-138238373 @miguelaferreira @wilderrodrigues waiting for the PR merge :) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Fixing findbugs error due to PR #755 merg...

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

CS 4.5.2: all hosts reboot after 3 days at production

2015-09-07 Thread Vadim Kimlaychuk
Hello all, I have experienced accidental cluster reboot 3 days after update to CS 4.5.2. Cluster is XenServer 6.5 with SP1. Reboot has been started from slave node and then - master. Syslog on slave shows only this: Sep 6 00:47:05 xcp2 last message repeated 3 times Sep 6 00:47:15

[GitHub] cloudstack pull request: Fixed issue: UI labels are display with k...

2015-09-07 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/781#issuecomment-138270743 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: Fixed issue: UI labels are display with k...

2015-09-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/781 --- 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 issue: UI labels are display with k...

2015-09-07 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/781 Fixed issue: UI labels are display with key. ex: label.yes due to missing comma in the jsp, all the labels below it are broken leading to showing keys instead of strings in UI You can merge

Re: [VOTE] Release Apache CloudStack EC2stack 0.8.0

2015-09-07 Thread Rohit Yadav
On 07-Sep-2015, at 2:03 pm, Rajani Karuturi > wrote: Its successful now. Thanks for the help. Great, I guess we’ve enough votes for doing the release now :) $ ec2stack-register http://localhost:5000 CLOUDSTACK-API-KEY

[GitHub] cloudstack pull request: Quota

2015-09-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-138305391 Tested why Travis was failing, the issue is with one of the Travis machines, test1 seems to be failing irrespective of the actual test:

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

2015-09-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/777#issuecomment-138310467 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: sysctl: don't modify /etc/sysctl.conf

2015-09-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138336997 If we do not need to persist a file, LGTM --- 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-8690:Added remote access vpn a...

2015-09-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138310582 Hi @jayapalu Could you have a look at the smoke/test_vpc_vpn.py test and let me know if that covers your changes? There is also a test

New Defects reported by Coverity Scan for cloudstack

2015-09-07 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 1 new defect(s) introduced to cloudstack found with Coverity Scan. 1 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New

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

2015-09-07 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/783 debian: Updated dependencies for Packages A few dependencies have been updated to their latest version and some have been removed. The ordering for some dependencies has been changed

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

2015-09-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-138310084 While looks good, I'll wait for comments from @remibergsma @miguelaferreira Ian and others on it was done this way? --- If your project is set up for it, you can

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

2015-09-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/765#issuecomment-138321248 @miguelaferreira @remibergsma @karuturi @DaanHoogland The test is done! Results: Test iptables default INPUT/FORWARD policy on

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

2015-09-07 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/772#issuecomment-138436271 @miguelaferreira Configuration on the vpc router also applied. So the tests should get passed. --- If your project is set up for it, you can reply to this email

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

2015-09-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/775#issuecomment-138383672 Tested on 4.3.2, and in that version this problem happens. I would say that calling a domain “%” sees odd, but in some cases I guess that might be

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

2015-09-07 Thread bvbharatk
GitHub user bvbharatk opened a pull request: https://github.com/apache/cloudstack/pull/784 CLOUDSTACK-8799 fixed the defalut routes CLOUDSTACK-8799 made changes to fix CsRedundant.py You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: CLOUDSTACK-5863: revert volume snapshot f...

2015-09-07 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/732#issuecomment-138426187 @ustcweizhou this caused a new coverity issue. Can you check? ``` 411 SnapshotInfo snapshotOnPrimaryStore =

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

2015-09-07 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/782 Cloudstack 8816 entityuuid missing in some of the events In some of the events generated, entity uuid was missing making it difficult to find the entity. Fixed the same. Tested it on