[GitHub] [druid] abhishekagarwal87 commented on a change in pull request #11487: Fix integration test credential resource path handling

[GitBox <gi...@apache.org>]

abhishekagarwal87 commented on a change in pull request #11487:
URL: https://github.com/apache/druid/pull/11487#discussion_r675470721



##########
File path: integration-tests/script/copy_resources.sh
##########
@@ -14,13 +14,20 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+echo "Copying integration test resources."
+
 set -e
 
 # setup client keystore
 ./docker/tls/generate-client-certs-and-keystores.sh
 rm -rf docker/client_tls
 cp -r client_tls docker/client_tls
 
+# install druid jars
+rm -rf $SHARED_DIR/docker

Review comment:
       CI failures look legit. 
   We also need to run `mkdir -p $SHARED_DIR` so that $SHARED_DIR is present. It's probably already created on everyone's machine who has ever run IT tests :) 




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org