[
https://issues.apache.org/jira/browse/METRON-1434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16425864#comment-16425864
]
ASF GitHub Bot commented on METRON-1434:
----------------------------------------
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/916
@as22323 Let me first say that this is a nice piece of work. You've
followed all the existing patterns and your contribution looks solid.
But ultimately I am +0 on this. I want others to speak up should they see
value in this, but I'd argue that this should not be merged. Please let me
explain my reasoning.
**Support Burden**
IMHO, the 'surface area' to support in Metron has always been too large.
We as a community spend too much time supporting non-core functionality, which
impedes our progress in delivering features targeted toward the cybersecurity
use case.
One key example of this are the multiple different deployment targets that
we support. This has been a difficult and time consuming support task over the
history of the project. I've done plenty of work on this myself.
Coalescing Metron installation around the MPack was a major step forward
for us. This was intended to help remove some of that support burden. With
the MPack, much of that burden is shifted to Ambari.
For example, with everything in Metron today, you can stand-up a single
node in AWS and use the Mpack to install Metron. It is not as "push button"
simple as your contribution here, but it is "good enough" considering the
resources we have in the community today.
**Uncommon Use Case**
We should also considering that running Metron on a single node is a recipe
for a horrible user experience. It should only be run on a single node for
development purposes, which is something that we already do support. I would
not recommend that anyone run Metron on a single node for any other purpose.
--
I'd prefer not too add to our ongoing support burden by merging this PR.
What you've done is a great contribution and I'd love to see you publicly share
and support this as a separate project, outside of Apache Metron, going forward.
If others agree or disagree with my reasoning, please do feel free to share.
> Ability to deploy Metron full dev as a single node (centos6) to AWS with
> vagrant and AMI
> ----------------------------------------------------------------------------------------
>
> Key: METRON-1434
> URL: https://issues.apache.org/jira/browse/METRON-1434
> Project: Metron
> Issue Type: Improvement
> Affects Versions: 0.4.3
> Environment: i) AWS Image: Centos6 - AWS t2.t2xlarge [32 GB Memory, 8
> Cores, 64 bit] based on metron-deployment/development/centos6/
> ii) AWS Single Node Cluster Deployment Using Vagrant: MacOS
> iii) AMI "GCR-Xetron Demo"/"ami-93cb4ff7" - AWS t2.t2xlarge
> Reporter: ashah
> Assignee: ashah
> Priority: Minor
> Labels: AWS, EC2, vagrant
> Fix For: 0.4.3
>
>
> The pull request has two potential solutions for deploying to Metron to AWS.
> i) AWS Single Node Cluster Deployment Using Vagrant
> ii) AWS Single Node Cluster Deployment Using an AMI
> *i) AWS Single Node Cluster Deployment Using Vagrant:*
> I was able to deploy Metron as a single node to AWS using Vagrant on MacOS.
> It was tested using the latest Metron code (as of 2018-01-30 - 8:00am UTC)
> that was in git. I'm assuming the latest that was in git is 0.4.3 that is
> progress.
> _Instructions:_
> [https://github.com/as22323/metron/blob/master/metron-deployment/README.md#aws-single-node-cluster-deployment-using-vagrant]
>
> *ii) AWS Single Node Cluster Deployment Using an AMI:*
> In addition to including instructions for deploying 0.4.3 using Vagrant on a
> Mac I also included instructions for deploying to AWS using a preexisting AMI
> (I made the following AMI public as a proof-of-concept: "GCR-Xetron
> Demo"/"ami-93cb4ff7"). This might be the quicket option for deployment since
> no preinstall requirments would be needed.
> _Instructions:_
> [https://github.com/as22323/metron/blob/master/metron-deployment/README.md#aws-single-node-cluster-deployment-using-an-ami]
>
> ****PULL REQUEST CHECKLIST ITEMS****
> *"For documentation related changes:"*
> Running cd site-book, mvn site produced "BUILD SUCCESS"
>
> *"For code changes:"*
> Was not sure where to run "mvn -q clean integration-test install &&
> dev-utilities/build-utils/verify_licenses.sh ". Core Metron Code was not
> changed. For i) the following code were newly added/modified:
> [Vagrant to work with aws]
> [https://github.com/as22323/metron/blob/master/metron-deployment/development/aws-centos6/Vagrantfile]
> [changing eth1 to eth0]
> [https://github.com/as22323/metron/blob/master/metron-deployment/development/aws-centos6/inventory/group_vars/all]
>
>
> *"How the change may be verified: "*
> You should be able to see Ambari after following instructions
> (http://<elasticip>:8080)
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
