You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by dlyle65535 <gi...@git.apache.org> on 2016/03/21 14:44:08 UTC
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
GitHub user dlyle65535 opened a pull request:
https://github.com/apache/incubator-metron/pull/53
METRON-77: Refactor Deployment Scripts to Accept Changes from METRON-72
PCAP topology filename is now remote.yaml.
YAF indexing can sometimes auto-code risn as a number.
Metron-UI copy needs to be an archive rather than recurse directories.
The Kafka client is installed as a side-effect of other tasks. Make it a dependency of each task that requires it.
The single-node deployment can only run 4 topologies. I recommend running Bro, Snort, Yaf and Enrichment by default. It can be overridden should one desire.
This has been run on both single-node Vagrant and EC2.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dlyle65535/incubator-metron METRON-77
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/53.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 #53
----
commit f22eba602b1699cebe1b58acd78de9f47216e719
Author: David Lyle <dl...@gmail.com>
Date: 2016-03-20T15:16:22Z
METRON-77: Refactor Deployment Scripts to Accept Changes from METRON-72
PCAP topology filename is now remote.yaml.
YAF indexing can sometimes auto-code risn as a number.
Metron-UI copy needs to be an archive rather than recurse directories.
The Kafka client is installed as a side-effect of other tasks. Make it a dependency of each task that requires it.
The single-node deployment can only run 4 topologies. I recommend running Bro, Snort, Yaf and Enrichment by default. It can be overridden should one desire.
----
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the pull request:
https://github.com/apache/incubator-metron/pull/53#issuecomment-199292568
This looks good. I especially like the single-node deployment only running bro, snort, yaf and enrichment. I think we could even trim it down to just bro and yaf to give us some headroom.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
Posted by dlyle65535 <gi...@git.apache.org>.
Github user dlyle65535 commented on the pull request:
https://github.com/apache/incubator-metron/pull/53#issuecomment-199327208
No thanks @merrimanr. I had that wrong. The regexes match on the lhs and replace the rhs. It works like a champ.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the pull request:
https://github.com/apache/incubator-metron/pull/53#issuecomment-199339197
+1 from me now. I believe the travis error is due to travis not being set up yet, which will happen external to 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
Posted by merrimanr <gi...@git.apache.org>.
Github user merrimanr commented on the pull request:
https://github.com/apache/incubator-metron/pull/53#issuecomment-199326697
Dave commented in Metron-72 that we should revert /incubator-metron/metron-streaming/Metron-Topologies/src/main/resources/Metron_Configs/etc/env/config.properties back to use zkpr[1-3] and kfka1. I agree with him and think we should but don't see it in this PR. Can we include that change as well?
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request: METRON-77: Refactor Deployment Scri...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/53
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---