Skip to content

Commit 56c109c

Browse files
committed
Move comment
1 parent f1a9fa8 commit 56c109c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,15 +87,17 @@ fi
8787
# By default, TensorFlow maps nearly all of the GPU memory visible to the process.
8888
# This is causing issue when other libraries are trying to run tests using a GPU.
8989
# See: https://www.tensorflow.org/guide/gpu#allowing_gpu_memory_growth
90+
#
91+
# Note about `--hostname localhost` (b/158137436)
92+
# hostname defaults to the container name which fails DNS name
93+
# resolution with --net=none (required to keep tests hermetic). See details in bug.
9094
docker run --rm -t --read-only --net=none \
9195
-e HOME=/tmp -e KAGGLE_DATA_PROXY_TOKEN=test-key \
9296
-e KAGGLE_USER_SECRETS_TOKEN_KEY=test-secrets-key \
9397
-e KAGGLE_URL_BASE=http://127.0.0.1:8001 \
9498
-e KAGGLE_DATA_PROXY_URL=http://127.0.0.1:8000 \
9599
-e KAGGLE_DATA_PROXY_PROJECT=test \
96100
-e TF_FORCE_GPU_ALLOW_GROWTH=true \
97-
# b/158137436: hostname defaults to the container name which fails DNS name
98-
# resolution with --net=none (required to keep tests hermetic). See details in bug.
99101
--hostname localhost \
100102
--shm-size=2g \
101103
-v $PWD:/input:ro -v /tmp/python-build/working:/working \

0 commit comments

Comments
 (0)