Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
Right, I hear that! I also really like the idea of a follow on function
that returns the default formats as a great solution to this problem. Should we
do that?
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
I would worry about having to parse the strong argument to determine
whether to use a constant or custom formatter, but I guess my concern is
limited because weâre caching the formatter
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
@ottobackwards wouldn't ISO constant support require us to do a check on
the string before passing to the DateFormat instance, which on something that
is likely to be run A LOT
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
Looks like a very useful gap filler to me. +1 (non-binding) by inspection
from me for sure.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1060
The fix is not necessary in pure Apache Ambari, and is only required in a
non-Apache toolchain, hence from a pure Apache perspective 'works as designed'
(just not very i
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1060
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1060
Turns out this is (in theory) done by the service advisor at
https://github.com/apache/metron/blob/ff1f9cf52b31ae866eb3de1cf8993af97af1790f/metron-deployment/packaging/ambari
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1193
---
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1127
---
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1203
---
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1112
---
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1203
METRON-1755 Create an architecture document for METRON-1663
## Contributor Comments
This adds documentation on the authentication methods used in METRON-1663.
## Pull Request
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1193
METRON-1667 Remove the JPA and MySQL elements
## Contributor Comments
Given that the only use of the JPA elements was for persisting a user
database, these dependencies are no
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1114
---
GitHub user simonellistonball reopened a pull request:
https://github.com/apache/metron/pull/1114
METRON-1667 Remove the JPA and MySQL elements
## Contributor Comments
Given that the only use of the JPA elements was for persisting a user
database, these dependencies are no
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1114
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1186
+1 by inspection, good call to align it with the other tests instead of
leaking the in-memory auth mechanism in.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1112
@mmiklavc the knoxsso is enabled with a toggle in the mpack, to follow the
pattern for NiFi, Ranger and Atlas etc. There is a kind of failover of auth
methods: knoxsso, ldap, nothing in
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/
---
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r213198883
--- Diff: metron-interface/metron-alerts-host/pom.xml ---
@@ -0,0 +1,143 @@
+
--- End diff --
Done, and agreed, it should be a
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r213198753
--- Diff:
metron-interface/metron-alerts-host/src/main/java/org/apache/metron/ui/AlertsApplication.java
---
@@ -0,0 +1,31
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r213198659
--- Diff: metron-interface/metron-ui-host/pom.xml ---
@@ -0,0 +1,142 @@
+
+
+http://maven.apache.org/POM/4.0.0";
+xmln
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r213124491
--- Diff: metron-interface/metron-ui-host/pom.xml ---
@@ -0,0 +1,142 @@
+
+
+http://maven.apache.org/POM/4.0.0";
+xmln
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r213124377
--- Diff:
metron-interface/metron-alerts-host/src/main/scripts/metron-alerts.sh ---
@@ -0,0 +1,55 @@
+#!/usr/bin/env bash
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r212275406
--- Diff:
metron-interface/metron-alerts-host/src/main/scripts/metron-alerts.sh ---
@@ -0,0 +1,55 @@
+#!/usr/bin/env bash
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r212270804
--- Diff: metron-interface/metron-rest-client/bin/pom.xml ---
@@ -0,0 +1,52 @@
+
--- End diff --
Yes. Will remove.
---
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/#discussion_r212225176
--- Diff: metron-interface/metron-ui-host/pom.xml ---
@@ -0,0 +1,142 @@
+
+
+http://maven.apache.org/POM/4.0.0";
+xmln
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/
Thanks for the catches Ryan.
My objective around metron-rest was to try and leave it as little changed
as possible in this effort, since the ticket is more around the ui hosts
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1172#discussion_r211964997
--- Diff:
metron-interface/metron-alerts/src/app/pcap/pcap-filters/pcap-filters.component.ts
---
@@ -15,63 +15,116 @@
* See the License
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1150#discussion_r210596217
--- Diff:
metron-analytics/metron-profiler-spark/src/main/java/org/apache/metron/profiler/spark/function/ProfileBuilderFunction.java
---
@@ -0,0
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1150
Do we have to use groupByKey in the spark implementation, is it not
possible to use reduceByKey to build the profiles, since profilers are by
definition reducible. I've seen group
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1132
+1 to that, PCAP should definitely get its own service, but agreed with
@anandsubbu that should probably be a follow on item.
---
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1127
METRON-1666 Mpack updates for LDAP, SSO config and removal of JPA
## Contributor Comments
MPack updates to go with METRON-1665.
This adds settings for the SSO, LDAP and
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1114
METRON-1667 Remove the JPA and MySQL elements
## Contributor Comments
Given that the only use of the JPA elements was for persisting a user
database, these dependencies are no
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1113
Apologies, re-submitting against the feature branch.
---
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/1113
---
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1113
METRON-1667 Remove the JPA and MySQL elements
## Contributor Comments
Given that the only use of the JPA elements was for persisting a user
database, these dependencies are no
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1112
In the case of non-SSO ldap auth, there is a login form, it's just embedded
in the spring bits instead of the angular bits. It's not as pretty, but we
could add a login.jsp
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1112
And infact through form based auth if sso is off (see the backend issue
METRON-1665). This PR just removes the need to process a spring login form in
the Angular app to lighten the load a
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1112
It would also support ldap authentication, via basic auth.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1112
Patience, patience :) https://issues.apache.org/jira/browse/METRON-1664
will set all this up in ambari to work on full-dev with an embedded ldap
(instead of the current embedded h2) and
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1112
METRON-1668 Remove login services and screens from UIs
## Contributor Comments
Removing the login screens and modules from the UIs since they are
superfluous in the the SSO
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/
METRON-1665 Host UIs on Spring Boot and add SSO authentication
This is part of a move towards using KnoxSSO, and LDAP as an authentication
method for Metron UIs.
The current UIs
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1104#discussion_r202600375
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/DateFunctionsTest.java
---
@@ -182,7 +182,8
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1104
METRON-1670 Stellar WEEK_OF_YEAR test is locale sensitive
## Contributor Comments
Tests were previously not local sensitive.
Used a locale sensitive Calendar instance to
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1096#discussion_r202303847
--- Diff: pom.xml ---
@@ -348,6 +348,16 @@
metron-interface/metron-config README and
NOTICE files
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1063
Should the example target state not be:
`is_alert := is_alert || geo_outlier`
We should probably have a test for that, but I assume the null is falsey
would apply on both
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/1060
METRON-1615 Default endpoint locations based on Ambari service locations
and config
## Contributor Comments
For certain values, the mpacks will now default values for the following
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1021
This is a great feature from my perspective. I've seen a number of people
try and build solutions that require access to the whole message for a bunch of
custom functions, models,
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1020#discussion_r189402062
--- Diff:
metron-platform/metron-solr/src/main/config/schema/error/schema.xml ---
@@ -40,6 +39,8 @@
guid
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1020#discussion_r189398418
--- Diff:
metron-platform/metron-solr/src/main/config/schema/error/schema.xml ---
@@ -40,6 +39,8 @@
guid
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/998
LGTM @merrimanr are you +1 on this now?
---
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1001#discussion_r187058040
--- Diff: metron-deployment/development/centos6/README.md ---
@@ -103,3 +103,8 @@ Tags are listed in the playbooks, some frequently used
tags
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1009
Looks like a typo on the title of the PR... "METRON-1549"
---
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1004#discussion_r184743321
--- Diff:
metron-interface/metron-alerts/e2e/alert-details/alert-status/alert-details-status.e2e-spec.ts
---
@@ -101,7 +104,7 @@ describe
Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/1004#discussion_r184743288
--- Diff:
metron-interface/metron-alerts/e2e/alert-details/alert-status/alert-details-status.e2e-spec.ts
---
@@ -23,64 +23,67 @@ import
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1001
+1 thees are useful additions for anyone getting started.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/942
Do we think we've reached consensus on this? It seems like avoiding the
mixed scenarios is a good thing from a performance perspective, which is key to
something like this which wi
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/993
METRON-1507 Add configuration in development vagrant files to support
vagrant cacâ¦
## Contributor Comments
This is a Vagrant config change to enable the use of vagrant-cachier to
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/979
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/873
Well, if it helped, cool, letâs keep it... was just curious about your
use case and whether we should have a bolder scope or if there is anything like
this that would help elsewhere in
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/873
Did it work out for you to find bottlenecks @ottobackwards ? I would be
more inclined to do multi-run micro-benchmarking (I have a bunch of
multi-sample perfidix runs on all the parser
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/979
METRON-1507 Added support for vagrant cachier
## Contributor Comments
This is a Vagrant config change to enable the use of vagrant-cachier to
reduce build times for the full dev
Github user simonellistonball closed the pull request at:
https://github.com/apache/metron/pull/978
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/978
Ooops... wrong branch. Let me sort out the PR.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/978
The yum stuff only happens in the centos include in ansible, it's not
necessary in the ubuntu version, which uses a different ansible include.
An extra test for the r
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/978
The PR includes ubuntu support, the cachier would just cache debs from apt
instead of rpms from yum. TBH I've tested centos more than ubuntu. The use is
still the same though, shou
GitHub user simonellistonball opened a pull request:
https://github.com/apache/metron/pull/978
METRON-1507 Added support for vagrant cachier
## Contributor Comments
This is a Vagrant config change to enable the use of vagrant-cachier to
reduce build times for the full dev
68 matches
Mail list logo