test(test-tooling): fix BesuTestLedger start cfg: publish all ports

1. Previously we specified the publish all ports flag of the Docker Engine incorrectly and this lead to the ports not actually being published on randomized ports. 2. This broke the supply chain app example when we tried to migrate it to use 2 separate besu test ledger instances in tandem because they were conflicting on the ports due to the lack of randomization. Signed-off-by: Peter Somogyvari <[email protected]>
hyperledger-cacti · Jul 8, 2024 · ad08788 · ad08788
1 parent 9970352
commit ad08788
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/packages/cactus-test-tooling/src/main/typescript/besu/besu-test-ledger.ts b/packages/cactus-test-tooling/src/main/typescript/besu/besu-test-ledger.ts
@@ -261,10 +261,9 @@ export class BesuTestLedger implements ITestLedger {
             Retries: 299,
             StartPeriod: 3000000000, // 1 second
           },
-          // This is a workaround needed for macOS which has issues with routing
-          // to docker container's IP addresses directly...
-          // https://stackoverflow.com/a/39217691
-          PublishAllPorts: true,
+          HostConfig: {
+            PublishAllPorts: true,
+          },
           Env: this.envVars,
         },
         {},