Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

METRON-646: Add index templates to metron-docker #441

Closed
wants to merge 6 commits into from

Conversation

kylerichardson
Copy link
Contributor

Enhance metron-docker environment by automatically deploying index templates to elasticsearch as part of the container startup.

I've also included a maven clean goal for the metron-docker project for easier build/rebuild.

Test plan and rebase with master to follow shortly.

@nickwallen
Copy link
Contributor

Hi @kylerichardson - I don't want to throw cold water on your effort, but I am hesitant to create a third deployment code base for metron-docker (in addition to MPack and Ansible.) Do you think that is what this is or would become?

Besides just the index templates, we'd have to add and support a lot of other functionality too. Seems like we should have a goal to move towards a single deployment mechanism that works across multiple platforms (Docker, Metal, etc).

I don't even know if this is feasible, but it may be worth a community discussion. I'll kick something off.

@kylerichardson
Copy link
Contributor Author

kylerichardson commented Feb 6, 2017 via email

@kylerichardson
Copy link
Contributor Author

bump... I read back through the discuss thread from the dev list and we didn't land on any specifics. There seems to be general agreement that metron-docker:

  • fulfills a current need/desire for targeted development and testing (e.g. parsers)
  • could be used to make integration testing quicker and easier (requires exploration)

How do we want to move this forward? I have this and a follow-on PR ready to go for adding HDFS into metron-docker.

@merrimanr
Copy link
Contributor

@kylerichardson I will get this reviewed today

@merrimanr
Copy link
Contributor

Works great. I have just one very small request. Can you update the Usage section of the README (where it lists the UI addresses) to point to the Elasticsearch head plugin instead of just "http://192.168.99.100:9200/"? That will keep someone from having to go research how to access that plugin.

Other than that this gets my +1. Thanks for the contribution.

@kylerichardson
Copy link
Contributor Author

kylerichardson commented Feb 22, 2017 via email

@merrimanr
Copy link
Contributor

+1 from me

@asfgit asfgit closed this in 41fc0dd Feb 27, 2017
asfgit pushed a commit that referenced this pull request Feb 28, 2017
…on via merrimanr) closes #441"

This reverts commit 41fc0dd. Change will be re-commited with full author details.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants