Github user anandsubbu closed the pull request at:
https://github.com/apache/incubator-metron/pull/540
---
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
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/678
+1 (non-binding)
Validated that with this fix, Ambari displays the correct status as opposed
to the earlier behavior.
---
If your project is set up for it, you can reply to this email
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/688
I'm +1 (non-binding)
Fired up a full dev and was able to test that `maas_service.sh` starts up
fine. No errors seen in the YARN application logs.
A few observations though
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/680
+1 (non-binding).
Validated the latest fix to be working great. Thank you, @dlyle65535 .
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/680
Sounds great, @dlyle65535 !
---
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
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/680
Hi @dlyle65535 can it be a two step validation as follows:
a) We first verify if we are able to hit the storm URL, if that fails ->
then we mark the service as stopped.
b) If the st
GitHub user anandsubbu opened a pull request:
https://github.com/apache/incubator-metron/pull/583
METRON-949 Change from error to warning message for Metron REST in Ambari
service_advisor
## Contributor Comments
Reduced the level to WARN in service_advisor.py.
Steps
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/583
Could I get some love for 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 project does not have this feature
Github user anandsubbu closed the pull request at:
https://github.com/apache/incubator-metron/pull/583
---
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
GitHub user anandsubbu reopened a pull request:
https://github.com/apache/incubator-metron/pull/583
METRON-949 Change from error to warning message for Metron REST in Ambari
service_advisor
## Contributor Comments
Reduced the level to WARN in service_advisor.py.
Steps
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/583
Thank you, @merrimanr . Can you please help merge this pull request?
---
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 user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/583
@merrimanr , this is because of the following reason:
In the "Assign Masters" page of the Ambari deployment wizard, there is a
check for the Metron REST host to have storm
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/583
@merrimanr, let me know if the above comment answers your question.
---
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
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/604
+1 (non-binding) since this ensures that the Metron UI service starts up
properly
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/759
+1 (non-binding)
Indeed, this was a tricky one. Thank you @nickwallen for your root cause
and fix!! I spun up a 12 node cluster with this patch, kerberized the cluster
and confirmed
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/723
METRON-1141: NameError: metronProfilerHost is not defined in
stack_advisor.py
## Contributor Comments
** Testing Done **
- Modified `service_advisor.py` on the Ambari host with the mpack
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/731
Hi @ottobackwards , I guess you wanted to name the PR title as METRON-1136
instead of METRON-1146?
---
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r138362796
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -506,29 +506,27 @@ public
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r138362926
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -550,4 +548,90 @@ else
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r138362754
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -550,4 +548,90 @@ else
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r138362605
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -550,4 +548,90 @@ else
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r138362883
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -550,4 +548,90 @@ else
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/727
Hi @nickwallen thank you. Sure, I see value in simplifying the naming. How
does `TO_JSON_OBJECT`, `TO_JSON_MAP` and `TO_JSON_LIST` sound?
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/727
Thanks very much @ottobackwards !
@nickwallen , pushed latest commit with the functions renamed. Please have
a look.
---
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/719
METRON-1137: Build RPM for Metron MaaS as a part of rpm-docker packaging
## Contributor Comments
Fixed metron.spec and the pom.xml to now copy and generate
metron-maas-service RPM
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/714#discussion_r135785927
--- Diff: metron-analytics/metron-maas-service/README.md ---
@@ -126,16 +126,20 @@ Now let's install some prerequisites:
Start Squid via `service
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/714#discussion_r135786862
--- Diff: metron-analytics/metron-maas-service/README.md ---
@@ -126,16 +126,20 @@ Now let's install some prerequisites:
Start Squid via `service
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r136954740
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/StringFunctionsTest.java
---
@@ -449,4 +450,68 @@ public
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/727#discussion_r136954647
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
---
@@ -450,4 +454,40 @@ public
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/723
Closing this pull request, since it is the same as
https://github.com/apache/metron/pull/722
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user anandsubbu closed the pull request at:
https://github.com/apache/metron/pull/723
---
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
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/807
METRON-1263: Start Alerts UI service after Metron REST
## Contributor Comments
Currently the Alerts UI service starts up before any of the Metron services
does. Ideally, it should be started
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/841
+1 (non-binding), btw
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/841
As always, very crisp @nickwallen ! My +1 (non-binding) still holds.
---
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/846
METRON-1328: Enhance platform-info.sh script to check if docker daemon is
running
## Contributor Comments
Run the `docker info` command along with checking the version.
**Case
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/846
Hi @ottobackwards , thanks for your review.
I prefixed a 'WARNING' to the message text so it would catch the eye of the
user. Also, I removed the else bit for docker running scenario
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/852
Hi @nickwallen , I found one more reference to quick dev -
[here](https://github.com/nickwallen/metron/blob/7145b06fff9f4ecc8e63d9cdd56ed955a49e676d/metron-interface/metron-rest/src/test/resources
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/852
> I was confused about that README.vm file. Any idea what that is for?
I am not entirely sure, @nickwallen . I could find a lot of similarities
between this README.vm and the main Met
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/852
+1 (non-binding) @nickwallen . This is a much needed fix since it is now
straight-forward to anyone new and wanting to try Metron.
---
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/850
METRON-1335: Install metron-maas-service RPM as a part of the full-dev
deployment
## Contributor Comments
Modified mpack metainfo.xml to install metron-maas-service RPM as well
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/850
bump. Any feedback on this pull request?
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/839
+1 (non-binding) works fine, and a much needed fix as well ! If for some
reason, we were not able to install the templates during the startup, it would
be caught by the service check
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/839
+1 (non-binding). Validated again on full-dev. Works fine, @nickwallen !
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/841
Hi @nickwallen
> I am not sure exactly what the problem is, but the same condition occurs
in master. I would call this a pre-existing condition that we can handle with a
separate
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/841
Hi @nickwallen , tested on the vagrant fastcapa deployment. Deployed fine,
and I was able to see this message as well:
```
TASK [debug
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/829
@nickwallen , for the record full dev spun up fine with the fix. Thank you!
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/829
Hi @nickwallen , thank you for the fix. It is my bad, I should have added
the line `commands = IndexingCommands(params)` before the `try` block in my
previous fix :(.
I am under way
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/850
Sure, that would be great.
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
You're right @nickwallen , it wouldn't work. I reworked on the patch and
here are the latest changes:
- moved the ES template installation back to the Indexing "start" method
-
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
If the install fails the first time due to ES being down, then we WARN and
move on. If ES service comes up later, the admin needs to install the templates
subsequently by using 'Ambari -> Met
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
I have added this to the Elasticsearch README in my recent commit. Please
see here:
https://github.com/apache/metron/pull/817/files#diff-325aea0d364d12c8637eef347ebcfca6
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147681110
--- Diff: metron-platform/metron-elasticsearch/README.md ---
@@ -81,3 +81,13 @@ curl -XPUT
"http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147708307
--- Diff: metron-platform/metron-elasticsearch/README.md ---
@@ -81,3 +81,13 @@ curl -XPUT
"http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147699187
--- Diff: metron-platform/metron-elasticsearch/README.md ---
@@ -81,3 +81,13 @@ curl -XPUT
"http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147714561
--- Diff: metron-platform/metron-elasticsearch/README.md ---
@@ -81,3 +81,13 @@ curl -XPUT
"http://${ELASTICSEARCH}:9200/${SENSOR}_index*/_ma
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147737598
--- Diff: metron-platform/metron-elasticsearch/README.md ---
@@ -86,8 +86,13 @@ rm ${SENSOR}.template
The stock set of Elasticsearch templates
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
Hi @ottobackwards , let me know what you think.
---
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/817#discussion_r147587694
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/864
I did another 12-node deployment on Centos 7 with this PR (bypassed the
kafka issue by installing Kafka broker on all nodes). The fix worked just
perfect. Thanks much @nickwallen !
+1
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/807#discussion_r145701351
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/addon-services/METRON/CURRENT/role_command_order.json
---
@@ -13,16 +13,18
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
Restarting travis
---
GitHub user anandsubbu reopened a pull request:
https://github.com/apache/metron/pull/817
METRON-1283: Install Elasticsearch template as a part of the mpack startup
scripts
## Contributor Comments
For a Metron multi-node deployment using mpack, the Elasticsearch template
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/817
Hi @ottobackwards , made the change to remove the ES template installation
from 'load_web_template' role for vagrant deployments. Fired up full-dev with
the change and it comes up fine. Was able
Github user anandsubbu closed the pull request at:
https://github.com/apache/metron/pull/817
---
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/817
METRON-1283: Install Elasticsearch template as a part of the mpack startup
scripts
## Contributor Comments
For a Metron multi-node deployment using mpack, the Elasticsearch template
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/888
METRON-1389: Zeppelin notebook import does not work with Ambari 2.6
## Contributor Comments
In recent versions of Ambari, [Shiro
Authentication](https://zeppelin.apache.org/docs/0.6.0
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/869#discussion_r159430163
--- Diff: metron-deployment/packaging/ambari/metron-mpack/README.md ---
@@ -0,0 +1,127 @@
+
+
+This provides a Management Pack (MPack
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/869#discussion_r159426522
--- Diff: metron-deployment/packaging/ambari/metron-mpack/README.md ---
@@ -0,0 +1,127 @@
+
+
+This provides a Management Pack (MPack
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/869
+1 (non-binding). Looks good @nickwallen !
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/894
+1 thank you @mmiklavc.
Spun up a 12-node Centos 7 cluster with the latest fix and validated the
following:
* Did 'Stop All Services' -> no issues seen with ES and Kibana servi
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r161735380
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
GitHub user anandsubbu reopened a pull request:
https://github.com/apache/metron/pull/888
METRON-1389: Zeppelin notebook import does not work with Ambari 2.6
## Contributor Comments
In recent versions of Ambari, [Shiro
Authentication](https://zeppelin.apache.org/docs/0.6.0
Github user anandsubbu closed the pull request at:
https://github.com/apache/metron/pull/888
---
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r163022818
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r161955419
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/894
> I'd like to treat this as distinct from the status check if that seems
reasonable to you. I have to admit, I didn't even know the service check piece
existed until now.
Sure, it ma
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161185438
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_slave.py
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161185471
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/KIBANA/5.6.2/package/scripts/kibana_master.py
---
@@ -93,7
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161184707
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_master.py
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/907
Hi @cestella , I did a 12-node deploy on CentOS 7 with this patch.
Post-kerberization, I noticed the following errors in Metron REST. Is this a
related issue or a different one
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/888
@ottobackwards @cestella and @nickwallen - any other feedback on this PR?
The travis failures seems to be in an unrelated piece of code.
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/904
+1 works fine @mmiklavc .
Verified on CentOS 7 multinode cluster. Simulated the problem first and
then ran with the fix.
Here is a sample output with the fix in place
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/907#discussion_r163600738
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/service_check.py
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r160591890
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/907
Hi @cestella , with the latest commit having the normalized producer, I was
able to spin up a 12-node Centos 7 cluster, install with HDP 2.6 base and
kerberize as well without issues. Ran
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/935
METRON-1386: Fix Metron Website Required Links
## Contributor Comments
Made the following changes to the Metron ASF site:
* Fixed the missing entities and made corrections per the Apache
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/932
METRON-1451: On Centos full dev, Metron Indexing shows up as stopped
## Contributor Comments
**Root Cause**
The python-requests module introduced a VendorAlias import machinery
GitHub user anandsubbu reopened a pull request:
https://github.com/apache/metron/pull/932
METRON-1451: On Centos full dev, Metron Indexing shows up as stopped
## Contributor Comments
**Root Cause**
The python-requests module introduced a VendorAlias import machinery
Github user anandsubbu closed the pull request at:
https://github.com/apache/metron/pull/932
---
GitHub user anandsubbu opened a pull request:
https://github.com/apache/metron/pull/938
METRON-1457: Move ASF links to main page in the Metron website
## Contributor Comments
Apache Whimsy checks for the site requisites in the main index.html and not
inside sub-levels. This PR
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/869#discussion_r159361598
--- Diff: metron-deployment/README.md ---
@@ -15,178 +15,134 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
express or implied.
See
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/869#discussion_r159280034
--- Diff: metron-deployment/README.md ---
@@ -15,178 +15,132 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
express or implied.
See
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/864
Hi @nickwallen , I tried this on a 12-node cluster. I validated that
`clusterHostInfo` is populated properly for the alerts_ui, management_ui and
rest_ui hosts.
However, in my case
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/938
Thanks @justinleet for the pointer. I have made a change to use the Powered
By logo instead of the foundation logo with feather, since I thought this
looked better. I also added a direct link
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/938
Thanks all.
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/943
+1
Works as advertised. I used both mpacks to deploy a 12-node CentOS 7
cluster (Ambari 2.6.0.0 and HDP 2.6.3.0). Was able to kerberize the cluster and
get bro indices into ES.
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/938
Thank you @mmiklavc and @nickwallen for your reply on the dev list.
Any feedback on this PR?
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/938
Roger that. Here's the new look. Let me know if this looks ok @justinleet
and @cestella .
![image](https://user-images.githubusercontent.com/20395490/36736657-323cff0a-1bff-11e8-8d13
98 matches
Mail list logo