[GitHub] metron issue #973: METRON-1356: Add a mechanism in Java for discovering serv...
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/973 Going to close this for now until we figure out what our strategy is going to be with UI e2e tests. We may decide not to include those as part of the travis build and instead rely on unit tests. The e2e tests would then be run manually against full dev. ---
[GitHub] metron issue #973: METRON-1356: Add a mechanism in Java for discovering serv...
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/973 - Followed the steps ( we need a docker cheat sheet for deleting existing old machine/containers btw ) - ran the install/integration-test not sure it is related: ```bash [ERROR] Failed to execute goal org.apache.maven.plugins:maven-jar-plugin:3.0.2:jar (default-jar) on project metron-hbase-client: You have to use a classifier to attach supplemental artifacts to the project instead of replacing them. -> [Help 1] [ERROR] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch. [ERROR] Re-run Maven using the -X switch to enable full debug logging. [ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException [ERROR] [ERROR] After correcting the problems, you can resume the build with the command [ERROR] mvn -rf :metron-hbase-client ``` ---
[GitHub] metron issue #973: METRON-1356: Add a mechanism in Java for discovering serv...
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/973 @ottobackwards just updated the README. You previously had to lookup the docker machine ip address and substitute it in source code to get the tests to pass if running locally. The text I removed in the latest commit described how to do that. So for testing this, you should no longer have to do that and the tests should just work. I haven't heard of that before but I will check it out. ---
[GitHub] metron issue #973: METRON-1356: Add a mechanism in Java for discovering serv...
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/973 @merrimanr this looks great. A couple of things: 1. Can you give some steps on how to test this? I would like to keep the FB moving 2. have you ever looked at https://www.testcontainers.org/? ---