[GitHub] cloudstack pull request: CLOUDSTACK-9088: Update the description f...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-161205824 @anshul1886 It is a language independent description of the parameters and applicable to all API client implementations. It can also be view as a more generic/

[GitHub] cloudstack pull request: Update JuniperSrxResource.java

GitHub user wenwenxiong opened a pull request: https://github.com/apache/cloudstack/pull/1148 Update JuniperSrxResource.java fix "action" will not relace in xml if type.equals(SecurityPolicyType.SECURITYPOLICY_EGRESS_DEFAULT) You can merge this pull request into a Git reposit

[GitHub] cloudstack pull request: Revert 1 wenwenxiong patch 1

Github user wenwenxiong closed the pull request at: https://github.com/apache/cloudstack/pull/1147 --- 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: Revert 1 wenwenxiong patch 1

GitHub user wenwenxiong opened a pull request: https://github.com/apache/cloudstack/pull/1147 Revert 1 wenwenxiong patch 1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/wenwenxiong/cloudstack revert-1-wenwenxiong-patch-1 Alte

[GitHub] cloudstack pull request: Strongswan vpn feature

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-161191190 @bhaisaab time is running out, if we want this in 4.7 (and that would be great!) we need to merge on Dec 7 the latest. Can you give this a spin and see if you ca

[GitHub] cloudstack pull request: CLOUDSTACK-9088: Update the description f...

Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-161180226 @DaanHoogland I don't understand how json format is helping here? --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: marvin updates

Please let me know if there are any issues with Marvin? I can look into it. On Tue, Dec 1, 2015 at 5:58 PM, Daan Hoogland wrote: > thanks Abhi, > > On Tue, Dec 1, 2015 at 1:14 PM, Abhinandan Prateek < > abhinandan.prat...@shapeblue.com> wrote: > > > AFAIK, Santhosh and Gaurav were the last few l

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-161117310 reviewed again after update, still 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

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161077855 I ran #942 with a slimmed down version of the test file containing only the expunge test with a factor of 9. I will do some looking into 943 now and edit the te

[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/942#issuecomment-161077181 I had to run one test a couple of times it is bugging us lately: [942.network.results.txt](https://github.com/apache/cloudstack/files/48984/942.network.

Re: [VOTE] Apache CloudStack 4.6.1

+0 from me. I verified that all the sources build properly and the Debian packages look good. Waiting for test results before my final vote comes in. Boris will run tests and based on that result I will vote. Wido On 11/30/2015 11:12 PM, Remi Bergsma wrote: > Hi all, > > I've created a 4.6.1 r

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161066414 That is odd, because the log you sent me earlier, the VM was marked as destroyed, so it was just a matter of executing the expunge thread. Is it possible f

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161060067 @rafaelweingartner I tested #942 with the setting of expunge * 6 but the same test still failed on that PR. I am convinced it has nothing to do with the code of

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

Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-161048814 @miguelaferreira besides fixing these issues and writing marvin test, what else would be needed to include this feature in version 4.7? Would it be possible, or co

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161048740 Oh sorry, I do not know why, but I thought it was perl. Well, if those configurations parameter are being taken from the MS, they are not importan

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161045285 @rafaelweingartner it is python not perl :) it is being read from the ms, and for the proper execution of the test it is not important what exactly it is, is it

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161043712 @DaanHoogland with your explanation I now understand that we want to test if the expunge thread is being executed and if it is doing its job. Therefore, I

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

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46306167 --- Diff: plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/element/NiciraNvpElement.java --- @@ -239,8 +244,51 @@ public boolean imp

Re: deploy dc with marvin

Daan, All of them are possible through marvin. Let me know if you don't figure them out. Thanks, ~Talluri On Tue, Dec 1, 2015 at 8:34 PM, Daan Hoogland wrote: > thanks Patrick, I'll look at those. I found how to enable > zonewideprimarystorage in the meanwhile. My intuition said to just place i

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161027097 @rafaelweingartner I agree with you in general lines. In this case the test had not just succeeded so far, it also started failing a lot since a few days. So so

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161004341 LGTM, tested in a home brew 4.6. AG work in project context. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

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

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46287923 --- Diff: plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/resource/wrapper/NiciraNvpConfigureSharedNetworkUuidCommandWrapper.java ---

Re: deploy dc with marvin

thanks Patrick, I'll look at those. I found how to enable zonewideprimarystorage in the meanwhile. My intuition said to just place it on zone level in the json but it turns out you need to add the scope to it. I am going to make a patch that will deduct this from the place in the json structure. O

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-160990441 @pdube good timing, testing it right now. expect feedback today. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160989059 @remibergsma any update ? @wido yes, #1029 for master, and #1044 is a mistake --- If your project is set up for it, you can reply to this email and have you

Re: deploy dc with marvin

Hey Daahn, Not sure because I haven't tested, but there are files in /setup/dev that give hints. There are advancedsg.cfg and basic.cfg. Both have "securitygroupenabled": "true". Nothing about zone wide primary storage though. Cheers, Patrick On Tue, Dec 1, 2015 at 7:35 AM, Daan Hoogland wrote

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-160978139 @resmo @ustcweizhou @remibergsma Any news on testing? LGTY? --- 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: Quota

Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-160978288 @remibergsma we have fixed a few additional defects and UI design cleanup which are currently being verified. My hope is that we get everything squared away in the

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-160973087 @DaanHoogland problem solved with that class. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

4.6 announcement

The 4.6.0 announcement is out, please take 5 minutes to promote on social media: - NASDAQ GlobeNewswire http://globenewswire.com/news-release/2015/12/01/791835/0/en/The-Apache-Software-Foundation-announces-Apache-CloudStack-v4-6.html - ASF "Foundation" blog http://s.apache.org/VfP - @TheASF Twitt

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-160966699 @DaanHoogland, I checked the logs you sent me. The VMs were marked as destroyed, but it seems that they have not been “destroyed” or remov

Re: Package Repositories

> On Dec 1, 2015, at 9:27 AM, Daan Hoogland wrote: > > On Tue, Dec 1, 2015 at 8:44 AM, Wido den Hollander wrote: > >>> As I mentioned in my initial message, the public ShapeBlue repositories >> [1] are noredist builds of release baselines with no additional patches or >> changes. My understan

deploy dc with marvin

Santosh, Gaurav, I am looking for functionality to deploy a data center with features that I don't see supported (or am to dumb to see). the things I want are: deploy basic zone with security groups deploy advanced zone with security groups deploy a zone with zone wide primary storage I don't th

Re: marvin updates

thanks Abhi, On Tue, Dec 1, 2015 at 1:14 PM, Abhinandan Prateek < abhinandan.prat...@shapeblue.com> wrote: > AFAIK, Santhosh and Gaurav were the last few looking at Marvin framework. > Though I do not see them active as in real active, but once in a while I > see them responding to some issue. >

Re: marvin updates

AFAIK, Santhosh and Gaurav were the last few looking at Marvin framework. Though I do not see them active as in real active, but once in a while I see them responding to some issue. If you know there are some issues to be looked into, just cc that to their email ids, hopefully they will respond o

[GitHub] cloudstack pull request: Quota

Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-160941719 @remibergsma I think all PRs should be squashed. This one is no exception. ;) --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request: CLOUDSTACK-9088: Update the description f...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-160941315 There is two problems to solve here; 1. how to document the api in a concise way. The information you are providing is very useful and I think it makes sens

[GitHub] cloudstack pull request: CLOUDSTACK-9088: Update the description f...

Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-160938583 @DaanHoogland Could you suggest a better way of putting this information in API documents in current scenario? --- If your project is set up for it, you can re

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

Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46265732 --- Diff: api/src/com/cloud/network/guru/NetworkGuruAdditionalFunctions.java --- @@ -0,0 +1,12 @@ +package com.cloud.network.guru; +

[GitHub] cloudstack pull request: CLOUDSTACK-9088: Update the description f...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-160936999 It seems to me this text is only suitable for use with cloudmonkey or maybe python clients. I don't think it makes sense to format it this way in the java anno

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

Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46264185 --- Diff: api/src/com/cloud/network/Networks.java --- @@ -251,6 +252,10 @@ public static URI fromString(String candidate) { i

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

Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46263781 --- Diff: api/src/com/cloud/network/Networks.java --- @@ -251,6 +252,10 @@ public static URI fromString(String candidate) { i

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-160930211 code lgtm but i have really no time to test this anytime soon --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160928383 The integration tests show this test failing (part of test_vm_life_cycle.py): ``` Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge

Re: Billing of deleted projects

Hi, Thanks for sharing Norbert. Since you’ve mentioned the issue affects 4.5.2, we will need to assess if this also applies to 4.6/4.7/master. We could try to fix it before 4.5.3, which would benefit many users. Kishan - has the usage server code changed between 4.5/4.6/master, any comments on

Billing of deleted projects

Hello, we opened a bug which addresses billing of deleted projects: https://issues.apache.org/jira/browse/CLOUDSTACK-9096 Regards Norbert -- www.infosecprojects.net

marvin updates

​Is anybody in the community still maintaining Marvin? (I mean the framework not the tests)​ -- Daan

Billing of deleted projects

Hello, we opened a bug which addresses billing of deleted projects: https://issues.apache.org/jira/browse/CLOUDSTACK-9096 Regards Norbert -- www.infosecprojects.net

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-160925744 Ok, let's discuss sometime outside the scope of this PR. and Let's Get This Merged. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1108#discussion_r46260293 --- Diff: api/src/com/cloud/deploy/DeploymentPlanner.java --- @@ -1,323 +1,306 @@ -// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160922052 @wido cool, it's one of the those that seemed like bugs --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160915928 LGTM based on the code. Seems sane to me. @bhaisaab This seems like a feature, so I don't think it's supposed to go into 4.5? --- If your project is set up f

[GitHub] cloudstack pull request: Strongswan vpn feature

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-160909435 We really need this for 4.7, the VPN feature is practically flaky for users @jayapalu any updates on this, if you're not able to work on this. I can help test/fi

[GitHub] cloudstack pull request: CLOUDSTACK-9092: L10n fix in "Add LDAP Ac...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1139 --- 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-9088: Update the description f...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1126#issuecomment-160908595 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 ena

[GitHub] cloudstack pull request: CLOUDSTACK-9091: Update testpath for para...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1130#issuecomment-160908545 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 ena

[GitHub] cloudstack pull request: CLOUDSTACK-9092: L10n fix in "Add LDAP Ac...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1139#issuecomment-160908402 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 en

[GitHub] cloudstack pull request: CLOUDSTACK-9094: Multiple threads are bei...

Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1140#discussion_r46256253 --- Diff: server/src/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java --- @@ -685,6 +685,16 @@ public void finalizeStop(final Virtua

[GitHub] cloudstack pull request: Show Agent name in Ping response

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1143 --- 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: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160906723 +1 LGTM (had reviewed before) @ustcweizhou If we merge it on 4.6 and fwd-merge on master, we won't need the PR #1029. Lastly, do you think we should put this in

[GitHub] cloudstack pull request: Show Agent name in Ping response

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1143#issuecomment-160906837 LGTM, merging now since we've 2+ 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 pr

RE: Package Repositories

Thanks Daan, So, I believe that the community should produce noredist packages and then ShapeBlue won't need to maintain a repo at all. [ShapeBlue] Paul Angus VP Technology , ShapeBlue d: +44 2036170528 | t: @cloudyangus | m: +4

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160904404 @wido yes https://github.com/apache/cloudstack/pull/1144 , just PR against correct branch now --- If your project is set up for it, you can reply to this email and h

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-160903564 I would swear I saw this code before. Dejavu moment? --- 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: [4.6] CLOUDSTACK-4787 - vmware diskcontro...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1132#issuecomment-160903184 Ping @abhinandanprateek @borisroman @jburwell @pdion891 and others for review --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: Quota

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-160902334 @remibergsma check now, squashed and rebased against latest master now. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack-www pull request: downloads: State packages on download...

Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack-www/pull/20 --- 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 featur

Re: [VOTE] Apache CloudStack 4.6.1

Do we have any RPMs in Jenkins yet? Don't know where to look. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Remi Bergsma" > To: dev@cloudstack.apache.org > Sent: Monday, 30 November, 2015 22:12:00 > Subject: [VOTE] Apache Cl

Re: Package Repositories

On 30-Nov-2015, at 10:02 PM, sebgoa mailto:run...@gmail.com>> wrote: 1-source release (official) 2-community repo (non-official) 3-third party (non-official, with single affiliation of people who have access). ** We just need a simple, acceptable naming for the download page for a transitory p

Re: Package Repositories

On Tue, Dec 1, 2015 at 9:47 AM, Paul Angus wrote: > My experience has been that the vast majority of users want the noredist > version. If the consensus is that the 'community created' packages should > be/must be OSS (something which I think only harms the project) then the > download links and

RE: Package Repositories

We're going in a bit of a circle here. My experience has been that the vast majority of users want the noredist version. If the consensus is that the 'community created' packages should be/must be OSS (something which I think only harms the project) then the download links and documentation shou

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-160895245 [943.vm.results.txt](https://github.com/apache/cloudstack/files/48256/943.vm.results.txt) Here's the test that fails. It failed the second time as well. I wo

Re: Package Repositories

On Tue, Dec 1, 2015 at 8:44 AM, Wido den Hollander wrote: > > As I mentioned in my initial message, the public ShapeBlue repositories > [1] are noredist builds of release baselines with no additional patches or > changes. My understanding is that the project cannot officially distribute > the no