Seems like the execute_tests.sh hasn't been updated to remove setting the
environment variables after we removed usage of dockerized tests plugin.

Sai

On Mon, 23 Jan 2023 at 21:49, Sai Boorlagadda <sai.boorlaga...@gmail.com>
wrote:

> Could someone provide context on how integration & dunit tests are run in
> CI?
>
> I remember geode had dockerized test plugin that isolates test execution.
> But looking at this PR[1] it seems we removed usage of that plugin and have
> a customized mechanism to allocate ports so that we don't need to run tests
> in docker containers.
>
> But the CI source (execute_tests.sh) seems to still pass docker
> image -PdunitDockerUser=geode -PdunitDockerImage=\$(docker images. So do we
> need to run these tests in docker containers?
>
> 1. https://github.com/apache/geode/pull/6720
>
> Sai
>

Reply via email to