[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[anandsubbu]

Github user anandsubbu commented on the issue:

https://github.com/apache/metron/pull/1201
  
Hi @nickwallen , I would like to create a fresh pull request based on these 
changes. Could you go ahead and merge this please ?


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
I want to hold off on the merge until we get the pcap properties change 
sorted on this. @anandsubbu  is opening a re-opening a PR on that.


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
+1 Looks great.  Thanks a bunch for the contribution.


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
> @MohanDV ... pull request 1132. It will be a duplicate effort here.

Gotcha.  I was not aware of the other PR.  That will work.


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[MohanDV]

Github user MohanDV commented on the issue:

https://github.com/apache/metron/pull/1201
  
@nickwallen having separate panel for PCAP configs is been addressed in the 
pull request  [1132](https://github.com/apache/metron/pull/1132). It will be a 
duplicate effort here. 


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
The configuration elements for PCAP are still under Metron > Configs > 
REST.  Now that you've added stop/start, I think it would make sense to pull 
those out into its own component/panel, like the other topologies.

![screen shot 2018-09-19 at 2 09 18 
pm](https://user-images.githubusercontent.com/2475409/45772417-a26a2300-bc15-11e8-8ae7-71c545244a01.png)

I was able to spin it up, start the pcap topology, throw some pcap at it, 
and see it get archived in HDFS, then search for that pcap in the UI.  So ya 
for that!


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
Can you merge this with master?  It is hard to tell what is part of this PR.


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[MohanDV]

Github user MohanDV commented on the issue:

https://github.com/apache/metron/pull/1201
  
@nickwallen It's my bad I dint run this up in Full Dev, I verified my 
scripts on a 12 node cluster. I did the required changes to support the Full 
dev deploymwnt and updated the pull request.


---

[GitHub] metron issue #1201: METRON-1709 : Added the required scripts to start / stop...

[nickwallen]

Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/1201
  
I am not able to spin-up the Full Dev environment with this.  It keeps 
failing here.  Have you run this up in Full Dev?
```
TASK [ambari_config : Deploy cluster with Ambari; http://node1:8080] 
***
fatal: [node1]: FAILED! => {"changed": false, "failed": true, "msg": 
"Ambari client exception occurred: Could not create blueprint: request code 
400, request message {\n  \"status\" : 400,\n  \"message\" 
: \"Cluster Topology validation failed.  Invalid service component count: 
[METRON_PCAP(actual=0, required=1)].  To disable topology validation and create 
the blueprint, add the following to the end of the url: 
'?validate_topology=false'\"\n}"}
to retry, use: --limit 
@/Users/nallen/tmp/metron-pr1201/metron-deployment/development/centos6/ansible/playbook.retry

PLAY RECAP 
*
node1  : ok=76   changed=42   unreachable=0failed=1

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.
```


---