Skip to content

Commit

Permalink
Allow to run tests with leak detection enabled. (#8323)
Browse files Browse the repository at this point in the history
Motivation:

We should add some command to be able to run all tests with leak detection enabled. This will then be used on the CI during PR builds.

Modifications:

Add new docker-compose config to run with leak-detection enabled.

Result:

Easy way to enable leak detection while running tests via docker.
  • Loading branch information
normanmaurer authored Sep 27, 2018
1 parent c546ab2 commit 9acd927
Show file tree
Hide file tree
Showing 9 changed files with 27 additions and 0 deletions.
2 changes: 2 additions & 0 deletions docker/docker-compose.centos-6.110.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ services:

test:
image: netty:centos-6-1.10
test-leak:
image: netty:centos-6-1.10

test-boringssl-static:
image: netty:centos-6-1.10
Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-6.111.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-6-1.11

test-leak:
image: netty:centos-6-1.11

test-boringssl-static:
image: netty:centos-6-1.11

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-6.18.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-6-1.8

test-leak:
image: netty:centos-6-1.8

test-boringssl-static:
image: netty:centos-6-1.8

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-6.19.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-6-1.9

test-leak:
image: netty:centos-6-1.9

test-boringssl-static:
image: netty:centos-6-1.9

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-7.110.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-7-1.10

test-leak:
image: netty:centos-7-1.10

test-boringssl-static:
image: netty:centos-7-1.10

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-7.111.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-7-1.11

test-leak:
image: netty:centos-7-1.11

test-boringssl-static:
image: netty:centos-7-1.11

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-7.18.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-7-1.8

test-leak:
image: netty:centos-7-1.8

test-boringssl-static:
image: netty:centos-7-1.8

Expand Down
3 changes: 3 additions & 0 deletions docker/docker-compose.centos-7.19.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ services:
test:
image: netty:centos-7-1.9

test-leak:
image: netty:centos-7-1.9

test-boringssl-static:
image: netty:centos-7-1.9

Expand Down
4 changes: 4 additions & 0 deletions docker/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@ services:
- ..:/code
working_dir: /code

test-leak:
<<: *common
command: /bin/bash -cl "./mvnw -Pleak clean install -Dio.netty.testsuite.badHost=netty.io"

test:
<<: *common
command: /bin/bash -cl "./mvnw clean install -Dio.netty.testsuite.badHost=netty.io"
Expand Down

0 comments on commit 9acd927

Please sign in to comment.