gnodet-bot commented on code in PR #28554:
URL: https://github.com/apache/camel/pull/28554#discussion_r4386364555
##########
test-infra/camel-test-infra-iggy/src/main/java/org/apache/camel/test/infra/iggy/services/IggyContainer.java:
##########
@@ -44,6 +44,8 @@ public TestInfraIggyContainer() {
addEnv("IGGY_ROOT_PASSWORD", IggyProperties.DEFAULT_PASSWORD);
// Bind to all interfaces so the host can connect to the
container
addEnv("IGGY_TCP_ADDRESS", "0.0.0.0:" +
IggyProperties.DEFAULT_TCP_PORT);
+ // Iggy 0.9.0+ refuses a wildcard bind unless the node
advertises a concrete address (host only, no port)
+ addEnv("IGGY_NODE_ADVERTISED_ADDRESS", "127.0.0.1");
Review Comment:
The official Iggy testing docs
([`iggy.apache.org/docs/server/testing`](https://iggy.apache.org/docs/server/testing))
and the Docker & Helm page recommend setting
`IGGY_SHARDING_CPU_ALLOCATION=all` alongside `IGGY_NODE_ADVERTISED_ADDRESS`
when running Iggy in Docker/CI environments.
Without it, Iggy 0.9.0 exits with:
```
failed to bind shard 0 memory to its NUMA node
```
on Docker Desktop, WSL2, and most CI hosts (which expose only a single NUMA
node).
```suggestion
// Iggy 0.9.0+ refuses a wildcard bind unless the node
advertises a concrete address (host only, no port)
addEnv("IGGY_NODE_ADVERTISED_ADDRESS", "127.0.0.1");
// Disable NUMA pinning — required on single-NUMA hosts
(Docker Desktop, WSL2, CI)
addEnv("IGGY_SHARDING_CPU_ALLOCATION", "all");
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
