Skip to content

Commit

Permalink
Unset default value for FAIL_ON_TEST_FAILURE
Browse files Browse the repository at this point in the history
as unsetting it via --build-arg seems unreliable
docker/compose#3608
  • Loading branch information
ruffsl committed Mar 10, 2023
1 parent 2cd7f21 commit c5c8502
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ RUN sed --in-place \

# test overlay build
ARG RUN_TESTS
ARG FAIL_ON_TEST_FAILURE=True
ARG FAIL_ON_TEST_FAILURE
RUN if [ -n "$RUN_TESTS" ]; then \
. install/setup.sh && \
colcon test && \
Expand Down
2 changes: 1 addition & 1 deletion tools/distro.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ RUN sed --in-place \

# test overlay build
ARG RUN_TESTS
ARG FAIL_ON_TEST_FAILURE=True
ARG FAIL_ON_TEST_FAILURE
RUN if [ -n "$RUN_TESTS" ]; then \
. install/setup.sh && \
colcon test && \
Expand Down
6 changes: 3 additions & 3 deletions tools/source.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ FROM ros2_builder AS ros2_tester

# test overlay build
ARG RUN_TESTS
ARG FAIL_ON_TEST_FAILURE=True
ARG FAIL_ON_TEST_FAILURE
RUN if [ -n "$RUN_TESTS" ]; then \
. install/setup.sh && \
colcon test && \
Expand Down Expand Up @@ -165,7 +165,7 @@ FROM underlay_builder AS underlay_tester

# test overlay build
ARG RUN_TESTS
ARG FAIL_ON_TEST_FAILURE=True
ARG FAIL_ON_TEST_FAILURE
RUN if [ -n "$RUN_TESTS" ]; then \
. install/setup.sh && \
colcon test && \
Expand Down Expand Up @@ -216,7 +216,7 @@ FROM overlay_builder AS overlay_tester

# test overlay build
ARG RUN_TESTS
ARG FAIL_ON_TEST_FAILURE=True
ARG FAIL_ON_TEST_FAILURE
RUN if [ -n "$RUN_TESTS" ]; then \
. install/setup.sh && \
colcon test && \
Expand Down

0 comments on commit c5c8502

Please sign in to comment.