[
https://issues.apache.org/jira/browse/METRON-1344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16282002#comment-16282002
]
ASF GitHub Bot commented on METRON-1344:
----------------------------------------
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/858
OK,
After getting the new changes, everything works as advertised. I was able
to run on mac os , and the end to end passed. I ran a few times in a row and
there where no issues, so it looks like setup/teardown works fine.
I think we you have done to this point is great @merrimanr, really great
work. I like the speed of getting compose up and everything.
The idea of doing the namespaces for ES is great too.
I did not run the integration test
To summarize my feedback:
- I would like some scripts that wrap this up into high level usage
- I think we need doc around writing tests that use this framework ( will
we be adding tests one by one + any new tests )?
- I think we should enable debugging on the services in docker on known
ports. Knowing that to @justinleet 's point, any java dev. can get their tool
of choice to work. We can do intellij specific examples on top of that as well
- I think we should have a print out statement, similar to what we have at
the end of vagrant up/ansible that says the ip addresses and ports exposed +
service name. this would include the debugging. I know this helps with
vagrant, it would help here.
> Externalize the infrastructural components using integration tests
> ------------------------------------------------------------------
>
> Key: METRON-1344
> URL: https://issues.apache.org/jira/browse/METRON-1344
> Project: Metron
> Issue Type: Improvement
> Reporter: Ryan Merriman
> Assignee: Ryan Merriman
>
> As we move towards adding the Alerts UI e2e test to our continuous build, we
> need to remove the dependence on vagrant full dev and provide equivalent
> infrastructure within Travis. This will include all the services that the
> e2e tests depend on including Kafka, Zookeeper, Elasticsearch (possibly
> HBase), and Metron REST.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)