[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159834948 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: Fix event UUIDS missing on event bus

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159834901 ``` Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS === ok -

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159833976 @ProjectMoon there was one failure in the network tests for the password server. I am rerunning that one to make sure. [.network.results.txt](https://g

Re: Package Repositories

2015-11-25 Thread Daan Hoogland
On Thu, Nov 26, 2015 at 7:52 AM, John Burwell wrote: > All, > ​...​ > In my opinion, favoring one 3rd-party repository over another is > detrimental to the community. ​agree ​ > We should either list all maintained 3rd-party package repositories or > we should list none at all. By maintai

Package Repositories

2015-11-25 Thread John Burwell
All, A conversation emerged on a PR [1] regarding how package repositories should listed on the downloads page [2]. This PR was prompted by a change on the page which removed reference to the ShapeBlue repositories. The PR proposes listing all "3rd-Party Distributions" in a separate section i

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread srikanteswartalluri
Github user srikanteswartalluri commented on the pull request: https://github.com/apache/cloudstack/pull/1123#issuecomment-159821904 @karuturi Reverted the commit on master and 4.6 --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1123#issuecomment-159816344 @srikanteswartalluri 1. no direct commits to master please.. 2. as per the review process, wait for two lgtms before doing a merge (and use the git pr scri

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1123 --- 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: Change references to people.apache.org to...

2015-11-25 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1123#issuecomment-159807931 Thanks Talluri for quickly acting on it. I checked the urls and all are reachable. LGTM !! --- If your project is set up for it, you can reply to this e

[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2015-11-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1124#issuecomment-159743630 I have being wanting to ask you guys about those integration tests. Isn´t it possible for us to run those tests automatically every time a PR is created

[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1124#issuecomment-159743067 LGTM: less is more, an integrations test run on it is of course the minimum. @remibergsma can you queue it? --- If your project is set up for it, you can repl

[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2015-11-25 Thread rafaelweingartner
GitHub user rafaelweingartner opened a pull request: https://github.com/apache/cloudstack/pull/1124 CID-1338387: Deletion of method endPointSelector.selectHypervisorHost Following the discussions and analysis presented on PR#1056 create by @DaanHoogland This PR is intended to pu

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread Srikanteswararao talluri
Moved all the following references in the test code to https://home.apache.org/~talluri/ . github PR https://github.com/apache/cloudstack/pull/1123 http://people.apache.org/~tsp/cloud-set-guest-password http://people.apache.org/~tsp/dummy.iso "http://people.apache.org/~sanjeev/centos63.ova";,

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread srikanteswartalluri
GitHub user srikanteswartalluri opened a pull request: https://github.com/apache/cloudstack/pull/1123 Change references to people.apache.org to home.apache.org in the test… … code Signed-off-by: SrikanteswaraRao Talluri You can merge this pull request into a Git reposi

[GitHub] cloudstack pull request: CLOUDSTACK-9065: fix bug when creating pa...

2015-11-25 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1114#issuecomment-159693036 LGTM, quickly build a RPM collection using this package.sh for centos63 --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159691925 yes, I made a quick test and seams that cloudstack-awsapi never been a dependency in .deb. does anyone would have a different story on this ? --- If your p

[ANNOUNCE] CFP open for ApacheCon North America 2016

2015-11-25 Thread Rich Bowen
Community growth starts by talking with those interested in your project. ApacheCon North America is coming, are you? We are delighted to announce that the Call For Presentations (CFP) is now open for ApacheCon North America. You can submit your proposed sessions at http://events.linuxfoundation.o

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

2015-11-25 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159668471 @bhaisaab can you bring this to the list ? I think it would be best discussed there. --- If your project is set up for it, you can reply to this email and have yo

main docs not on 4.6

2015-11-25 Thread sebgoa
looking at this: http://docs.cloudstack.apache.org/en/latest/# it's still on 4.5 there are no changes to it for 4.6, but it's confusing because there is no 4.6 tag, …..and a 4.5 ref in the header…

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159667765 so on second reading @pdion891 you added removal of awsapi for RPMs but not for Ubuntu... --- If your project is set up for it, you can reply to this email an

[GitHub] cloudstack pull request:

2015-11-25 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/commit/e66dd8c1e00abb9a3b4cbe0b8b50c83b251a728c#commitcomment-14620035 Hi @davidamorimfaria , I had the same issue that @pdion891 had. I added missing quotes as you suggested in PR#1114 but st

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

2015-11-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-159643377 The code LGTM now --- 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: Fix NuageVsp errors for build-master-slow...

2015-11-25 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-159642930 @rafaelweingartner, implemented the test cases as suggested --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

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

2015-11-25 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-159634391 @remibergsma thanks Remi, I rebased to master branch and resolved a few conflicts --- If your project is set up for it, you can reply to this email and have your

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread Daan Hoogland
You are right, Sebastien. @Rajani, do you have an idea on the volume of work, here? On Wed, Nov 25, 2015 at 2:10 PM, sebgoa wrote: > > On Nov 25, 2015, at 2:07 PM, Daan Hoogland > wrote: > > > are those refs expecting particular content returned? (it will be > > redirected) > > > > the way I re

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

2015-11-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-159632142 @nvlivens, I would suggest you to write two test cases instead of one. One test case to test the encode method and another to test the decode meth

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

2015-11-25 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-159629890 @rafaelweingartner, I've added a test for those methods --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-rn/pull/26 --- 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 feat

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159607840 Typo fixed, going to push this PR.. --- 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: Add marvin test to verify if DomainAdmin ...

2015-11-25 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1122 Add marvin test to verify if DomainAdmin is able to delete the tags on public ip address Please go through bug https://issues.apache.org/jira/browse/CLOUDSTACK-9084 for more details.

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

2015-11-25 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159605020 With all respect to shapeblue team, contributions and efforts, -1 with this current PR. Adding "official" in the source section doesn't make sense. This

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread sebgoa
On Nov 25, 2015, at 2:07 PM, Daan Hoogland wrote: > are those refs expecting particular content returned? (it will be > redirected) > the way I read the email, it will be redirected but would give a 404 if you don't move the data yourself :) > On Wed, Nov 25, 2015 at 1:40 PM, Rajani Karuturi

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread Daan Hoogland
are those refs expecting particular content returned? (it will be redirected) On Wed, Nov 25, 2015 at 1:40 PM, Rajani Karuturi wrote: > We have references to people.apache.org in our testdata. > > ~Rajani > > On Wed, Nov 25, 2015 at 5:50 PM, Daniel Gruno > wrote: > > > Hi folks, > > as the subj

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread Rajani Karuturi
We have references to people.apache.org in our testdata. ~Rajani On Wed, Nov 25, 2015 at 5:50 PM, Daniel Gruno wrote: > Hi folks, > as the subject says, people.apache.org is being decommissioned soon, and > personal web space is being moved to a new home, aptly named > home.apache.org ( https:/

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

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159593139 €0,02: I think a great benefit of having shapeblue supplying their own packages is them being able to backort features for their customers. I would like th

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

2015-11-25 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-159592539 @nvazquez Ai, there is a conflict now. Could you please have a look? --- If your project is set up for it, you can reply to this email and have your reply appea

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

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159589449 @abhinandanprateek Please dont get me wrong. This is not a question of good or bad. I am not denying anyone's part in community neither am I saying that x is go

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

2015-11-25 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159586783 @karuturi shapeblue folks are also part of community like citrix or sbp folks. by that definition the shapeblue packages are as good. it is just that sh

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159586201 @karuturi I don't think there is a test for this in the suites we use. I'll them anyway to at least see about regression in some other places. --- If your pro

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

2015-11-25 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159585876 @karuturi there is the issue with noredist under ASF though, how about we move this as a discuss thread to dev@? there are more changes that could be discussed

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

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159583159 well, it got evolved during the 4.0 release and no one objected. May be, first we should start a discuss/vote first on what can we include. community

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

2015-11-25 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159578241 @karuturi reference to the vote/agreement regarding apt-get.eu repositories being agreed as some sort of "official" distribution. I do not recall the community

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

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159576804 @jburwell apt-get is just a temporary spot until we get to packages.cloudstack.apache.org yes, we should mention that these are not official but just the rpms

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

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159576408 @abhinandanprateek I think they were added silently in the first place. We are doing one rpm build as a community for user convenience(which was discusse

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

2015-11-25 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159575622 @karuturi the repos currently listed on the [Downloads page](http://cloudstack.apache.org/downloads.html) are not under community control. Furthermore, the mann

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

2015-11-25 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159573954 As I see it either we do not put any non-official build on the project page. If we are doing it for users convenience we should all such repositories th

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159568824 @remibergsma @DaanHoogland can you take a look at this please? Its a good fix to have in 4.6.1 I didnt run the test suite. But, manually tested this fix. ---

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

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159568400 I agree with @remibergsma -1 for adding more repos which community may not have control. This will definitely confuse users. Since these are build from the s

[GitHub] cloudstack pull request: CLOUDSTACK-9077 Fix injectkeys.sh to work...

2015-11-25 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1109#issuecomment-159564428 @pdion891 Thanks, please have a look. --- 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

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159561086 LGTM, aside for the already mentioned items. Thanks! --- 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-9081 Migration of vm across cl...

2015-11-25 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1118#issuecomment-159555769 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-9083: Add disk serial to kvm v...

2015-11-25 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1121 CLOUDSTACK-9083: Add disk serial to kvm virt xml Adds disk serial ids based on volume uuids to the virt xml. This may be useful for appliances/software that needs some serial ids on the VM

Re: 4.6 release

2015-11-25 Thread Erik Weber
Looks ok aside from a few typos already mentioned in the PR. -- Erik On Wed, Nov 25, 2015 at 10:12 AM, Sebastien Goasguen wrote: > @erik and @nux > > Can you check this: > https://github.com/apache/cloudstack-docs-rn/pull/26 > > let me know if you agree with the upgrade instructions.. > > > O

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9083: Add disk serial to...

2015-11-25 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1120 [4.5] CLOUDSTACK-9083: Add disk serial to kvm virt xml Adds disk serial ids based on volume uuids to the virt xml. This may be useful for appliances/software that needs some serial ids on

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread borisroman
Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#discussion_r45843949 --- Diff: source/global.rst --- @@ -28,7 +28,7 @@ .. |sysvm64-url-ovm|replace:: http://cloudstack.apt-get.eu/systemvm/4.6/systemvm64tem

Re: 4.6 release

2015-11-25 Thread Sebastien Goasguen
@erik and @nux Can you check this: https://github.com/apache/cloudstack-docs-rn/pull/26 let me know if you agree with the upgrade instructions.. > On Nov 23, 2015, at 1:48 PM, Pierre-Luc Dion wrote: > > Hi, > > For cloudstack-awsapi, shouldn't we include is rpm in the 4.6 repo so > ```yum upg

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

2015-11-25 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159543089 According to @ke4qqq we should be able to use packages.apache.org (see https://mail-archives.apache.org/mod_mbox/cloudstack-dev/201506.mbox/%3CCAKprHVYH+m-A2g=cj

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

2015-11-25 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159539414 Let's hear it from @ke4qqq as VP of infra. --- 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-www pull request: downloads: State packages on download...

2015-11-25 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159539257 I think it is quite fair. The source is the official release, and the packages are not official. I don't know that we will ever be able to offer official packag

Build failed in Jenkins: build-4.5 #541

2015-11-25 Thread jenkins
See Changes: [Rohit Yadav] ui: fix instances null check regression from 2f250e2 -- [...truncated 2923 lines...] [debug] execute contextualize [INFO] Using 'UTF-8' encoding to copy filtered resource

[GitHub] cloudstack-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159537788 aside from the typo, LGTM cc/ @remibergsma --- 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-docs-rn pull request: fix pre 4.5 upgrade instruction

2015-11-25 Thread runseb
Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack-docs-rn/pull/26#discussion_r45841371 --- Diff: source/about.rst --- @@ -89,6 +89,21 @@ Link https://hub.docker.com/u/cloudstack/ == =

[GitHub] cloudstack pull request: ui: Fix undefined check in instances.js, ...

2015-11-25 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1119 ui: Fix undefined check in instances.js, regression from 459d638 (fix is already included in the https://github.com/apache/cloudstack/pull/1106 for 4.6 as well) You can merge this pull reques

Jenkins build is still unstable: build-systemvm64-master #683

2015-11-25 Thread jenkins
See

[GitHub] cloudstack pull request: CLOUDSTACK-9065: fix bug when creating pa...

2015-11-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1114#issuecomment-159533493 LGTM on sight, let's not trust on sight as much anymore! I son't consider my lgtm here as good enough even if there are 20. I want to see people reporting havi

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

2015-11-25 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159530377 Stating 3rd party I agree with. I just don't want the confusion of having different company names and people saying on the list they use "the shape blue versi