METRON-2347: [BRO-PLUGIN-KAFKA] Use docker compose in end to end tests #42
Conversation
@@ -80,7 +80,10 @@ echo "Running docker_run_create_topic_in_kafka with " | |||
echo "NETWORK_NAME = $NETWORK_NAME" | |||
echo "===================================================" | |||
|
|||
docker run --rm --network "${NETWORK_NAME}" ches/kafka \ | |||
# TODO: Fix this | |||
sleep 2s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ottobackwards Open to suggestions - would prefer not need to go back to using wait-for-it.sh
but that seems to be a rather standard approach to solving this problem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't work that way. Same with healthcheck. If you look in the docker compose file I'm already using depends_on
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
then, why don't you stick with wait_for_it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I may, let me think on it. Was hoping to remove the MIT license and all the overhead but compose isn't as capable as k8s readiness
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a solution for this which is docker-compose
native, just need to find some time to wrap it up.
I am +1 on this. Review and run, seems good to me. The wait for it can be a follow on. Please create a jira. |
Thanks - give me a bit to wrap up responses to your feedback, hopefully I'll get time in the next couple of days. |
Ok, back to you @ottobackwards should have addressed all of your feedback now |
+1 nice work |
Contributor Comments
This is a first step towards
docker-compose
in our end to end tests.Testing
Run
./run_end_to_end.sh
and./finish_end_to_end.sh
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron's Bro kafka writer plugin.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:
For code changes: