[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-01-19 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/220 Can we close this then? --- 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: [GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread Otto Fowler
Ryan, I owe you an apology, that discussion is more towards the web app pr and not the rest api which would still be consumed by an ambari view or a custom app. I confused the threads. On January 19, 2017 at 09:29:29, merrimanr (g...@git.apache.org) wrote: Github user merrimanr commented on

[GitHub] incubator-metron pull request #420: METRON-668: Remove the "tickUpdate" prof...

2017-01-19 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/incubator-metron/pull/420 METRON-668: Remove the "tickUpdate" profile config and make the "init" phase not reset variables Please see description at [METRON-668](https://issues.apache.org/jira/browse/METRON-668)

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-01-19 Thread 2xyo
Github user 2xyo commented on the issue: https://github.com/apache/incubator-metron/pull/220 Hi, sorry for not getting back to you. I am no longer able to try this setup (new job, no more server at work for this setup and not enough ram at home :/) --- If your project is set up

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread Justin Leet
Yeah, this looks like a huge leap forward, and I'm thrilled that you made such good progress. Great job, Matt. On Thu, Jan 19, 2017 at 1:55 PM, Michael Miklavcic < michael.miklav...@gmail.com> wrote: > Agreed! Matt, thanks for taking this on and glad I could help. > > M > > On Thu, Jan 19, 2017

Build failing

2017-01-19 Thread zeo...@gmail.com
The build has been showing as failing for a little while now. I know we recently updated the language around Merge Requirements , but if I recall properly our current issue is

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96933025 --- Diff: metron-interface/metron-rest/README.md --- @@ -0,0 +1,458 @@ +# Metron REST and Configuration UI + +This UI exposes and aids

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96929829 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/GrokValidation.java --- @@ -0,0 +1,51 @@ +/** +

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96934260 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/GrokConfig.java --- @@ -0,0 +1,36 @@ +/** + * Licensed

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96930729 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/SensorParserConfigVersion.java --- @@ -0,0 +1,51 @@

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96936537 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/GlobalConfigController.java --- @@ -0,0 +1,74 @@ +/**

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96930355 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/GrokValidation.java --- @@ -0,0 +1,51 @@ +/**

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96933666 --- Diff: metron-interface/metron-rest/pom.xml --- @@ -0,0 +1,357 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96936912 --- Diff: metron-interface/metron-rest-client/pom.xml --- @@ -0,0 +1,54 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96936044 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/ZookeeperConfig.java --- @@ -0,0 +1,47 @@ +/** + *

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96930979 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/StellarFunctionDescription.java --- @@ -0,0 +1,49 @@

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96933249 --- Diff: metron-interface/metron-rest/pom.xml --- @@ -0,0 +1,357 @@ + + + +http://maven.apache.org/POM/4.0.0;

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96931769 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/TopologySummary.java --- @@ -0,0 +1,31 @@ +/** +

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96931516 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/TopologyResponse.java --- @@ -0,0 +1,42 @@ +/**

[GitHub] incubator-metron pull request #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/316#discussion_r96930477 --- Diff: metron-interface/metron-rest-client/src/main/java/org/apache/metron/rest/model/KafkaTopic.java --- @@ -0,0 +1,60 @@ +/** ---

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread Michael Miklavcic
Agreed! Matt, thanks for taking this on and glad I could help. M On Thu, Jan 19, 2017 at 11:42 AM, Casey Stella wrote: > Oh wow, I really like the looks of that. I was skeptical before, but if > you got that far in a couple of days, I think this is a worth-while >

[GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/316 The primary driver as I see it for a REST API is to facilitate building user interfaces for various interactions with Metron. At some point (I would argue now) we will want/need to

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread Matt Foley
Thanks, Jon! I’m working on characterizing exactly how to fix the two main issues. I think I’ve got a script that will auto-fix the triple-backtick problem. The bullet list problem will require hand-editing, so I want to make sure I’ve got the right recommendation. The larger issue is going

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread zeo...@gmail.com
Looking at the screenshot, that would be an incredible improvement on what we currently have. I'd be happy to help out with any markdown modifications and documentation cleanup, if necessary, to fix the problems you outlined above. Jon On Thu, Jan 19, 2017 at 11:22 AM Matt Foley

[GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/316 I would very much prefer to not wait. There are other endpoints outside of configuration management that we need, a REST API isn't limited to just that. If we feel we should remove the

[GitHub] incubator-metron issue #408: METRON-608 Mpack to install a single-node test ...

2017-01-19 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/408 This is great stuff. Can you talk a bit about why you split it off instead of just updated the existing MPack to support 1+ nodes? I'd prefer not to maintain two if we don't have to.

[GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on the issue: https://github.com/apache/incubator-metron/pull/316 @ottobackwards that's a good point. To me it still may makes sense to have this API (which the Ambari view would use). I think it would be useful for clusters not using Ambari. But I

[GitHub] incubator-metron issue #408: METRON-608 Mpack to install a single-node test ...

2017-01-19 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/incubator-metron/pull/408 Is "MySQL should not be installed on a node already running a MySQL instance" still accurate wrt Ambari MPack? I thought this was fixed. I see it noted as the 2nd bullet point

[GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/316 Also, the point that this may be better managed from within ambari as a view may need consideration. On January 19, 2017 at 08:58:51, JJ Meyer

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread Otto Fowler
Not seeing the attachment, is it attached to a jira? On January 19, 2017 at 04:19:02, Matt Foley (ma...@apache.org) wrote: Here’s a screen shot, attached :-) On 1/19/17, 1:04 AM, "Matt Foley" wrote: Hi all, I’ve put together a prototype doc book, along the lines we

Re: [PROPOSAL] Reduce Reliance on Ansible for Deployment

2017-01-19 Thread David Lyle
For the first increment, I am planning on using the Ambari Metron topology service that currently exists in the MPack. Handling the side loading isn't in the scope of this proposal. We'll need to design that separately. -D... On Thu, Jan 19, 2017 at 8:48 AM, Otto Fowler

[GitHub] incubator-metron issue #316: METRON-503: Metron REST API

2017-01-19 Thread jjmeyer0
Github user jjmeyer0 commented on the issue: https://github.com/apache/incubator-metron/pull/316 @merrimanr @cestella does it make sense to hold off on this for a little while? It looks like all configuration may be managed by Ambari going forward. I am getting this from the dev

Re: [PROPOSAL] Reduce Reliance on Ansible for Deployment

2017-01-19 Thread Otto Fowler
So - my prototype was adding a new parser type and completely integrating it with the install, all the way down to monit templates. All of that work was configuration work in ansible. I think I’m asking about changes to something that wasn’t documented anyways, as I think you are pointing out by

Re: [PROPOSAL] up-to-date versioned documentation

2017-01-19 Thread Matt Foley
Hi all, I’ve put together a prototype doc book, along the lines we discussed, and it looks pretty worthwhile. Many thanks to Mike M. who whipped the pom.xml file into shape, and helped me find the right site.xml file to imitate. If you’re interested, please do a single-branch clone as follows: