[GitHub] [druid] paul-rogers commented on pull request #13774: Query tests migration to new IT framework with TLS

["paul-rogers (via GitHub)" <gi...@apache.org>]

paul-rogers commented on PR #13774:
URL: https://github.com/apache/druid/pull/13774#issuecomment-1441107999

   To help move this along, can we split the PR into parts? Maybe one or two tests per PR?
   
   To do that, create a new PR with one or two tests. Create another with another one or two. These can be done in parallel if there are no common files (the ideal case), or sequentially if there are shared files.
   
   Once all but two of the tests have been done in their own PR, merge master (with those tests) into this branch, leaving the last couple of tests.
   
   In particular, the security-related tests should be their own PR since they are quite complex by themselves.
   
   For extra credit, revise the `docker-compose.yaml` files to us the new templating mechanism. If that is too much of an ask, go ahead and do a PR as-is, and I'll convert the files, since I can do so quickly (having written the generator code...)


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