Skip to content

Commit

Permalink
[#5152] fix(oceanbase-catalog): OB catalog UTs failed on Mac (#5178)
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

The max user processes in the OB catalog unit tests should be larger
than 120000

### Why are the changes needed?

Fix: #5152

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

./gradlew :catalogs:catalog-jdbc-oceanbase:test -PskipDockerTests=false
  • Loading branch information
featherchen authored Oct 21, 2024
1 parent f69bdaf commit d865df9
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
import com.github.dockerjava.api.DockerClient;
import com.github.dockerjava.api.command.InspectContainerResponse;
import com.github.dockerjava.api.model.ContainerNetwork;
import com.github.dockerjava.api.model.Ulimit;
import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
Expand Down Expand Up @@ -79,8 +80,8 @@ protected BaseContainer(
cmd ->
cmd.getHostConfig()
.withSysctls(
Collections.singletonMap(
"net.ipv4.ip_local_port_range", "20000 40000")));
Collections.singletonMap("net.ipv4.ip_local_port_range", "20000 40000"))
.withUlimits(new Ulimit[] {new Ulimit("nproc", 120000L, 120000L)}));
this.ports = requireNonNull(ports, "ports is null");
this.hostName = requireNonNull(hostName, "hostName is null");
this.extraHosts = extraHosts;
Expand Down

0 comments on commit d865df9

Please sign in to comment.