[GitHub] metron issue #862: METRON-1343: Swagger UI for User Controller needs request...

Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/862 Thanks @ottobackwards . Updated the manual verification steps. ---

Re: [DISCUSS] Community Meetings

Not sure about posting the recordings - you will need to check and make sure that doesn't violate anything. Just a friendly reminder... It is important that meetings have notes and a summary that is sent out describing topics to be decided on the mailing list. No decisions can be made in the

New Travis Build Image

The new Trusty image in travis that just landed breaks the build for one of my PR’s. I am not sure why, but I verified that going back to the old image resolves the problem. If you see a build fail, and it works locally or it is failing in a strange way, be wary. ottO

Re: [DISCUSS] Upcoming Release

So I was looking at some of the docs and saw that Upgrading.md has a dangling section that maybe should be removed. Link , link Jon On Tue, Dec 12, 2017 at 2:21 PM Matt Foley

[GitHub] metron issue #857: METRON-1340: Improve e2e tests for metron alerts

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/857 OK, I was able to get the tests to run. First run I get a single error. @merrimanr and I are talking through it now. ``` [INFO] Test spec for login page [INFO] ✗ should

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 The new Trusty image breaks my build. I need to figure out why exactly, so I'll be trying a few things. ---

[GitHub] metron issue #857: METRON-1340: Improve e2e tests for metron alerts

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/857 Should we also run the tests using the new Maven profile? `mvn test -Pe2e` The README should probably be updated to reflect this as well. ---

[GitHub] metron issue #857: METRON-1340: Improve e2e tests for metron alerts

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/857 A couple notes for the community - if your node/npm versions are out of date and you've already built the project, you'll run into trouble with versions not matching against the expected node

[GitHub] metron pull request #834: METRON-1306: When index template install fails, we...

Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/834 ---

[GitHub] metron issue #857: METRON-1340: Improve e2e tests for metron alerts

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/857 Seems that my NodeJS and npm versions got mangled since last time I ran this. Perhaps migrating laptops, upgrading to High Sierra, and switching from MacPorts to HomeBrew might have something to do

[GitHub] metron issue #834: METRON-1306: When index template install fails, we should...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/834 I'm good +1 ---

“777” Feature Branch Redux

I have created a new feature branch feature/METRON-1211-extensions-parsers-gradual to track the parser extension work and have rebased https://github.com/apache/metron/pull/774 on to that. I have also updated confluence and jira :

[GitHub] metron issue #774: METRON-1212 The bundle system and maven plugin

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/774 I have created a new feature branch feature/METRON-1211-extensions-parsers-gradual to track this. and have rebased this PR on to that. I have also updated confluence and jira :

[GitHub] metron issue #834: METRON-1306: When index template install fails, we should...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/834 I expect to address this in the "777" feature branch/ parser effort. ---

[GitHub] metron issue #834: METRON-1306: When index template install fails, we should...

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/834 After reading this whole thread I agree with @ottobackwards. If we installed and started all services and THEN installed our parsers separately this whole issue goes away. If a parser's template

Re: [DISCUSS] Community Meetings

Yes, I do. I suspect the best bet will be to post recordings somewhere on the apache.org metron site. Simon > On 12 Dec 2017, at 18:36, Otto Fowler wrote: > > Excellent, do you have the > 40 min + record option? > > > On December 12, 2017 at

Re: [DISCUSS] Community Meetings

Excellent, do you have the > 40 min + record option? On December 12, 2017 at 13:19:55, Simon Elliston Ball ( si...@simonellistonball.com) wrote: Happy to volunteer a zoom room. That seems to have worked for most in the past. Simon > On 12 Dec 2017, at 18:09, Otto Fowler

Re: [DISCUSS] Community Meetings

Happy to volunteer a zoom room. That seems to have worked for most in the past. Simon > On 12 Dec 2017, at 18:09, Otto Fowler wrote: > > Thanks! I think I’d like something hosted though. > > > On December 12, 2017 at 11:18:52, Ahmed Shah

Re: [DISCUSS] Community Meetings

Thanks! I think I’d like something hosted though. On December 12, 2017 at 11:18:52, Ahmed Shah (ahmeds...@cmail.carleton.ca) wrote: Hello, wrt "- How are we going to host it"... I've used BigBlueButton as an end user at our University. It is LGPL open source. https://bigbluebutton.org/

Re: [DISCUSS] Community Meetings

Hello, wrt "- How are we going to host it"... I've used BigBlueButton as an end user at our University. It is LGPL open source. https://bigbluebutton.org/ https://bigbluebutton.org/developers/ -Ahmed ___ Ahmed Shah (PMP, M. Eng.)

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/853 Ambari does not manage this config. It is only included in the base application.yml as a default setting. I don't feel like this setting should be in Ambari for a couple reasons: changing it

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/853 What I get to was, the setting should be exposed in ambari if ambari manages the config. ---

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

GitHub user merrimanr reopened a pull request: https://github.com/apache/metron/pull/858 METRON-1344: Externalize the infrastructural components using integration tests ## Contributor Comments This PR will add infrastructure to our Travis build that will allow the Alerts UI

[GitHub] metron pull request #858: METRON-1344: Externalize the infrastructural compo...

Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/858 ---

[GitHub] metron issue #858: METRON-1344: Externalize the infrastructural components u...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/858 @merrimanr, I have created labels in jira that are applicable to feature branch Jiras, This will allow for tracking these jiras in confluence as I have done here :

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/853 Each environment will have it's own application.yaml. Full dev has one, our testing environment has one, Ambari ships one etc. There is also a base application.yaml that has the defaults

[GitHub] metron issue #579: METRON-941 fix PaloAltoParser

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/579 @ctramnitz is there any update on this PR? Is there something we can do to help? ---

[GitHub] metron issue #834: METRON-1306: When index template install fails, we should...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/834 Where are we at with this? ---

[GitHub] metron issue #853: METRON-1337: List of facets should not be hardcoded

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/853 Does ambari own the application.yaml currently? ---

[GitHub] metron issue #862: METRON-1343: Swagger UI for User Controller needs request...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/862 Like, - Spin up Full Dev - Go to Swagger at http://node1:4200(?) - You should see X now ---

[GitHub] metron issue #862: METRON-1343: Swagger UI for User Controller needs request...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/862 Great, can you list that as the manual verification please? ---

[GitHub] metron issue #854: Experimental Improvements - Feedback Only - Do Not Merge

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/854 based on @justinleet 's comment, where do we stand with this ---

[GitHub] metron issue #856: METRON-1339 Stellar Shell functionality to verify stored ...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/856 @nickwallen any feedback, does the annotated approach match what you imagined? ---

[GitHub] metron pull request #859: METRON-1345: Update EC2 README for custom Ansible ...

Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/859#discussion_r156374333 --- Diff: metron-deployment/roles/ambari_config/vars/small_cluster.yml --- @@ -87,6 +87,8 @@ configurations: topology.classpath: '{{

[GitHub] metron issue #862: METRON-1343: Swagger UI for User Controller needs request...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/862 Hi! Thanks for the contribution. How can we test these changes? Do we need new tests in the code for this change ? Please consider the items in the PR checklist ( listed

[GitHub] metron pull request #863: METRON-1347: Indexing Topology should fail tuples ...

Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/863#discussion_r156372312 --- Diff: metron-platform/metron-indexing/README.md --- @@ -15,6 +15,12 @@ Indices are written in batch and the batch size and batch timeout are

[GitHub] metron pull request #863: METRON-1347: Indexing Topology should fail tuples ...

Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/863#discussion_r156372868 --- Diff: metron-platform/metron-writer/src/main/java/org/apache/metron/writer/bolt/BulkMessageWriterBolt.java --- @@ -229,17 +239,30 @@ public void

[GitHub] metron issue #859: METRON-1345: Update EC2 README for custom Ansible tags

Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/859 Nice. I'm glad you uncovered the issue with the REST API. Reaffirming +1 ---

Re: [DISCUSS] Upcoming Release

RC1 is looking good to me. I validated the MD5s, built Metron, built the Bro plugin and reviewed the other artifacts like release notes. Running the RAT check on a 'clean' Metron does not produce any errors for me. It is only after building Metron, which pulls in additional Node dependencies,