[jira] [Commented] (METRON-465) Automatically set storm-site topology.classpath

2016-10-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/METRON-465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15609116#comment-15609116
 ] 

ASF GitHub Bot commented on METRON-465:
---

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-metron/pull/314


> Automatically set storm-site topology.classpath
> ---
>
> Key: METRON-465
> URL: https://issues.apache.org/jira/browse/METRON-465
> Project: Metron
>  Issue Type: Sub-task
>Reporter: David M. Lyle
>Assignee: David M. Lyle
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (METRON-465) Automatically set storm-site topology.classpath

2016-10-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/METRON-465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15608501#comment-15608501
 ] 

ASF GitHub Bot commented on METRON-465:
---

Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/314
  
+1 by inspection


> Automatically set storm-site topology.classpath
> ---
>
> Key: METRON-465
> URL: https://issues.apache.org/jira/browse/METRON-465
> Project: Metron
>  Issue Type: Sub-task
>Reporter: David M. Lyle
>Assignee: David M. Lyle
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (METRON-465) Automatically set storm-site topology.classpath

2016-10-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/METRON-465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15605475#comment-15605475
 ] 

ASF GitHub Bot commented on METRON-465:
---

Github user dlyle65535 commented on the issue:

https://github.com/apache/incubator-metron/pull/314
  
Bump?


> Automatically set storm-site topology.classpath
> ---
>
> Key: METRON-465
> URL: https://issues.apache.org/jira/browse/METRON-465
> Project: Metron
>  Issue Type: Sub-task
>Reporter: David M. Lyle
>Assignee: David M. Lyle
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (METRON-465) Automatically set storm-site topology.classpath

2016-10-24 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/METRON-465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15601480#comment-15601480
 ] 

ASF GitHub Bot commented on METRON-465:
---

Github user nsinha17 commented on the issue:

https://github.com/apache/incubator-metron/pull/314
  
+1
I tested this out in the openstack environment and the fix looks good to me.
Steps followed for testing:-
1. Built the Mpack and RPMs by pulling out the code from 
dlyle65535:METRON-465
2. Installed Metron and related services on the 12 node openstack cluster.
3. The storm configuration property :-"topology.classpath"  got auto 
populated with the value :-
 /etc/hbase/conf:/etc/hadoop/conf
Attached is the image :-
https://cloud.githubusercontent.com/assets/19325569/19640781/e278bb66-99fb-11e6-9a8b-8381960c8533.png";>



> Automatically set storm-site topology.classpath
> ---
>
> Key: METRON-465
> URL: https://issues.apache.org/jira/browse/METRON-465
> Project: Metron
>  Issue Type: Sub-task
>Reporter: David M. Lyle
>Assignee: David M. Lyle
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (METRON-465) Automatically set storm-site topology.classpath

2016-10-17 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/METRON-465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15582724#comment-15582724
 ] 

ASF GitHub Bot commented on METRON-465:
---

GitHub user dlyle65535 opened a pull request:

https://github.com/apache/incubator-metron/pull/314

METRON-465: Automatically set storm-site topology.classpath

1) Set storm-site topology.classpath to /etc/hbase/conf:/etc/hadoop/conf as 
recommendation
2) Display error when storm-site topology.classpath doesn't not contain 
/etc/hbase/conf or /etc/hadoop/conf

Tested on Docker cluster following procedure here: [Metron 
MPack](https://www.evernote.com/l/AhLFVR-9CsFIYYnOnF43BlxSsT4F856qwaY) for 
following cases.

1) Adds topology.classpath to storm-site configuration and sets it to 
/etc/hbase/conf:/etc/hadoop/conf when it does not exist.
2) Appends /etc/hbase/conf:/etc/hadoop/conf when topology.classpath does 
not contain both paths
3) Append appropriate path when only one is missing. 


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dlyle65535/incubator-metron METRON-465

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-metron/pull/314.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #314


commit 622d1d5a2f0f89a35d672c5cfdae1ca23c214609
Author: David Lyle 
Date:   2016-10-17T16:40:54Z

METRON-465: Automatically set storm-site topology.classpath




> Automatically set storm-site topology.classpath
> ---
>
> Key: METRON-465
> URL: https://issues.apache.org/jira/browse/METRON-465
> Project: Metron
>  Issue Type: Sub-task
>Reporter: David M. Lyle
>Assignee: David M. Lyle
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)