Tessera ATs - run Tessera as a process

[pinges]

When we added the Tessera container tests the time to run the acceptance tests increased.

Currently we are starting a docker container for each of the Besu and Tessera nodes for each test that involves Tessera.

1. Reuse the docker instances per test class
2. Make Tessera nodes run in a process/thread like they do in other acceptance tests

Any other ideas?

[macfarla]

Alternative idea - run Tessera as a process (requires some changes to Tessera - Stefan had a PR)

[mark-terry]

Option 1:
testcontainers/testcontainers-java#1781

Container reuse in testcontainers is still WIP and appears to have outstanding work around assigning networks. This does not appear to be viable at this point in time.

[MadelineMurray]

@mark-terry - talking about this in planning today. We're going to do with option 2 and can start the work to make this change (ie, it's not blocked).

@pinges is going to link his previous PR using this approach. The previous PR is a MVP approach so this work is about implementing that approach with production quality code.

[MadelineMurray]

There's going to be two PRs - one in Tessera and one in Besu so leaving this issue in the Besu repo rather than moving to Tessera.

[taccatisid]

Going for "Make Tessera nodes run in a process/thread like they do in other acceptance tests"

[jflo]

Am interested in this, as the PrivacyReorgTest depends on Orion today, and when I swapped it out for Tessera in this pr #3073, I triggered the dependency on testcontainers. That's not a deal breaker per-se, but since it is happening as a unit test instead of an integration/acceptance test, there is no docker daemon running to deploy Tessera to.

[macfarla]

This functionality is now in Tessera. However we need a Tessera release
22.1.1 is current https://github.com/ConsenSys/tessera/releases/tag/tessera-22.1.1
Need 22.1.2 for this

[Gabriel-Trintinalia]

Blocked by Consensys/tessera#1490

[Gabriel-Trintinalia]

Blocked by #4532

Tessera needs Java 17.

[non-fungible-nelson]

@macfarla - anything related to latest tessera AT changes? Close?