Skip to content

Commit b64c59c

Browse files
committed
update cuda-pjrt env to fix sourcing default envs and remove redundant settings
1 parent 2c7fc89 commit b64c59c

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

ci/envs/build_artifacts/jax-cuda-pjrt.env

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
# limitations under the License.
1414
# ==============================================================================
1515
# Inherit default JAXCI environment variables.
16-
source ci/envs/default
16+
source ci/envs/default.env
1717

1818
# Enable jax-cuda-pjrt build.
1919
export JAXCI_BUILD_PJRT="1"
@@ -35,13 +35,4 @@ fi
3535
# Linux Aarch64 specifc settings
3636
if [[ $os == "linux" ]] && [[ $arch == "aarch64" ]]; then
3737
export JAXCI_DOCKER_IMAGE="us-central1-docker.pkg.dev/tensorflow-sigs/tensorflow/build-arm64:tf-2-18-multi-python"
38-
fi
39-
40-
# When building artifacts for running tests, we need to disable x64 mode and
41-
# clone XLA at HEAD.
42-
if [[ $JAXCI_SETUP_TEST_ENVIRONMENT == 1 ]]; then
43-
# Disable x64 mode
44-
export JAX_ENABLE_X64=0
45-
# Clone XLA at HEAD.
46-
export JAXCI_CLONE_MAIN_XLA=1
47-
fi
38+
fi

ci/utilities/setup_jaxci_envs.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ if [[ -z "$1" ]]; then
2727
fi
2828

2929
# -e: abort script if one command fails
30+
# -u: error if undefined variable used
3031
# -x: log all commands
3132
# -o history: record shell history
3233
# -o allexport: export all functions and variables to be available to subscripts

0 commit comments

Comments
 (0)