clintropolis opened a new pull request #10694: URL: https://github.com/apache/druid/pull/10694
Refactoring of docker-scripts in #10680 broke the 'override' config integration test cluster bring up due to missing environment variables. This PR changes `docker_run_cluster.sh` to check for `DRUID_INTEGRATION_TEST_OVERRIDE_CONFIG_PATH` and only call `getComposeArgs` if it isn't set, otherwise it will call compose in the previous manner. `stop_cluster.sh` was also missing stopping the hadoop container, and like `docker_run_cluster.sh` will also now check for and supply the env variables for override as well when stopping the integration test cluster when the overrides are defined. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
