[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 proj

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 fea

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]: Clou

[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 GitHu

[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 ca

[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 you

[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 an

[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 o

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 mailto:rajani.karut...@citrix.com>> 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 CLOUDSTACK-SECRET-KEY 127.0.0.1 - - [07/

[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 th

[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: 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 r

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

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

[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: 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: https://travis-ci.org/apa

[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 r

[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 enab

[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 for

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 defect(

[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 so

[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 Rout

[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. I

[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 n

Missing classes on master branch, project “cloud-plugin-network-vcs”

2015-09-07 Thread Rafael Weingärtner
Hi folks, I tried to build the master branch today, and I got two missing classes: *com.cloud.network.schema*.showvcs.Output; *com.cloud.network.schema*.showvcs.VcsNodeInfo; They are used on “com.cloud.network.resource.BrocadeVcsResource”, does anyone know those classes? -- Rafael Weingärtn

[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 = _snapshotFactory.getSnapshot(sna

[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 an

[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 https://github.com

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

2015-09-07 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: exists

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

2015-09-07 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 -> https:/

[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-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 s