[GitHub] metron pull request #1171: METRON-1740 make parser support CONFIG and SYSTEM...

2018-08-29 Thread liuy-tnz
Github user liuy-tnz commented on a diff in the pull request: https://github.com/apache/metron/pull/1171#discussion_r213836879 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/paloalto/README.md --- @@ -0,0 +1,15 @@ +#

[jira] [Commented] (METRON-1751) Storm Profiler dies when consuming null message

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596729#comment-16596729 ] ASF GitHub Bot commented on METRON-1751: Github user asfgit closed the pull request at:

[GitHub] metron pull request #1176: METRON-1751 Storm Profiler dies when consuming nu...

2018-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1176 ---

[GitHub] metron issue #1176: METRON-1751 Storm Profiler dies when consuming null mess...

2018-08-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1176 Thanks for the review @cestella ! I ran this up in Full Dev and everything looked proper. ---

[GitHub] metron pull request #1178: METRON-1757 Storm Profiler Serialization Exceptio...

2018-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1178 ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596567#comment-16596567 ] ASF GitHub Bot commented on METRON-1757: Github user JonZeolla commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/1178 Yeah that would explain why I wasn't hitting it in full-dev, but I did hit it in my test deployment. ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596486#comment-16596486 ] ASF GitHub Bot commented on METRON-1757: Github user nickwallen commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1178 FYI - I tested this on a multi-node cluster. I was experiencing the problem and the topology could not write out any profile measurements. I updated the Profiler lib with these changes

[jira] [Commented] (METRON-1696) Pcap parser fails to write pacap sequence file to hdfs on kerberized cluster

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1696?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596454#comment-16596454 ] ASF GitHub Bot commented on METRON-1696: Github user MohanDV commented on the issue:

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-29 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 Closed as it is addressed in METRON-1738 ---

[jira] [Commented] (METRON-1696) Pcap parser fails to write pacap sequence file to hdfs on kerberized cluster

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1696?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596452#comment-16596452 ] ASF GitHub Bot commented on METRON-1696: Github user MohanDV commented on the issue:

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-29 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/1134 closing as it addressed in METRON-1738 ---

[GitHub] metron pull request #1134: METRON-1696: Create the HDFS directory for pcap s...

2018-08-29 Thread MohanDV
Github user MohanDV closed the pull request at: https://github.com/apache/metron/pull/1134 ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596431#comment-16596431 ] ASF GitHub Bot commented on METRON-1757: Github user ottobackwards commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/1178 The work-around to this issue, and some documentation of it to the extent you feel necessary should go out to the users list. ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596426#comment-16596426 ] ASF GitHub Bot commented on METRON-1757: Github user nickwallen commented on the issue:

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596425#comment-16596425 ] ASF GitHub Bot commented on METRON-1757: Github user JonZeolla commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1178 Here is one other detail that I forgot to add to the description. This may be important if you are trying to validate this fix on a multi-node cluster. * This serialization error will

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/1178 This looks good to me +1 thanks @nickwallen ---

[jira] [Commented] (METRON-1750) Create Parser for Syslog RFC 5424 Messages

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596422#comment-16596422 ] ASF GitHub Bot commented on METRON-1750: Github user ottobackwards commented on a diff in the

[GitHub] metron pull request #1175: METRON-1750 Metron Parser for valid RFC 5424 Sysl...

2018-08-29 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1175#discussion_r213706134 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/syslog/Syslog5424Parser.java --- @@ -0,0 +1,75 @@ +/* + *

[jira] [Commented] (METRON-1740) Improve Palo Alto parser to handle CONFIG and SYSTEM syslog messages

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596419#comment-16596419 ] ASF GitHub Bot commented on METRON-1740: Github user cestella commented on a diff in the pull

[GitHub] metron pull request #1171: METRON-1740 make parser support CONFIG and SYSTEM...

2018-08-29 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1171#discussion_r213704549 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/paloalto/README.md --- @@ -0,0 +1,15 @@ +#

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596413#comment-16596413 ] ASF GitHub Bot commented on METRON-1757: Github user cestella commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1178 This is really good work, @nickwallen. I'm +1 by inspection here, pending @JonZeolla's +1 ---

[jira] [Commented] (METRON-1751) Storm Profiler dies when consuming null message

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596408#comment-16596408 ] ASF GitHub Bot commented on METRON-1751: Github user cestella commented on the issue:

[GitHub] metron issue #1176: METRON-1751 Storm Profiler dies when consuming null mess...

2018-08-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1176 +1 by inspection pending full-dev validation. ---

[jira] [Commented] (METRON-1696) Pcap parser fails to write pacap sequence file to hdfs on kerberized cluster

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1696?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596405#comment-16596405 ] ASF GitHub Bot commented on METRON-1696: Github user cestella commented on the issue:

[GitHub] metron issue #1134: METRON-1696: Create the HDFS directory for pcap sequence...

2018-08-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1134 Should this be closed out? ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596394#comment-16596394 ] ASF GitHub Bot commented on METRON-1757: Github user JonZeolla commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/1178 Looks good. I wasn't able to finish my testing last night but I am going to take a stab at validating. Based on your comments above though this was great digging, thanks. ---

[jira] [Commented] (METRON-1757) Storm Profiler Serialization Exception

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596384#comment-16596384 ] ASF GitHub Bot commented on METRON-1757: Github user nickwallen commented on the issue:

[GitHub] metron issue #1178: METRON-1757 Storm Profiler Serialization Exception

2018-08-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1178 Sure. Done. ---

[jira] [Commented] (METRON-1743) CEF testPaloAltoCEF test using a confusing variable name

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596343#comment-16596343 ] ASF GitHub Bot commented on METRON-1743: Github user asfgit closed the pull request at:

[GitHub] metron pull request #1173: METRON-1743: CEF test confusing URL

2018-08-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1173 ---

[jira] [Commented] (METRON-1476) Update angular

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596327#comment-16596327 ] ASF GitHub Bot commented on METRON-1476: Github user justinleet commented on the issue:

[GitHub] metron issue #1096: METRON-1476: Update angular

2018-08-29 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/1096 @sardell To get this running in full dev, I had to change a line in the metron.spec file for the RPMs to build properly. `%attr(0644,root,root) %{metron_home}/web/alerts-ui/*.bundle.css`

[jira] [Commented] (METRON-1750) Create Parser for Syslog RFC 5424 Messages

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596310#comment-16596310 ] ASF GitHub Bot commented on METRON-1750: Github user cestella commented on a diff in the pull

[jira] [Commented] (METRON-1476) Update angular

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596309#comment-16596309 ] ASF GitHub Bot commented on METRON-1476: Github user ruffle1986 commented on the issue:

[GitHub] metron pull request #1175: METRON-1750 Metron Parser for valid RFC 5424 Sysl...

2018-08-29 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1175#discussion_r213670531 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/syslog/Syslog5424Parser.java --- @@ -0,0 +1,75 @@ +/* + *

[GitHub] metron issue #1096: METRON-1476: Update angular

2018-08-29 Thread ruffle1986
Github user ruffle1986 commented on the issue: https://github.com/apache/metron/pull/1096 It has turned out that Angular's changed the way how it renders templates by default in version 6. Earlier, it preserved whitespaces by default but in version 6 the `preservedWhitespace` option

[GitHub] metron pull request #1175: METRON-1750 Metron Parser for valid RFC 5424 Sysl...

2018-08-29 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1175#discussion_r213669448 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/syslog/Syslog5424Parser.java --- @@ -0,0 +1,75 @@ +/* + *

[jira] [Commented] (METRON-1750) Create Parser for Syslog RFC 5424 Messages

2018-08-29 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/METRON-1750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596304#comment-16596304 ] ASF GitHub Bot commented on METRON-1750: Github user cestella commented on a diff in the pull