Skip to content

Commit

Permalink
address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Ruinong Tian committed Jul 23, 2024
1 parent 9862350 commit 6be4a8a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions build_artifacts/v0/v0.13/v0.13.0/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ RUN sudo ln -s $(which python3) /usr/bin/python
# Enforce dependencies are all installed from conda-forge
RUN conda config --remove channels defaults
RUN micromamba config append channels conda-forge --env
RUN micromamba config set channel_priority strict

# Install glue kernels, and move to shared directory
# Also patching base kernel so Studio background code doesn't start session silently
Expand Down
1 change: 1 addition & 0 deletions template/v0/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ RUN sudo ln -s $(which python3) /usr/bin/python
# Enforce dependencies are all installed from conda-forge
RUN conda config --remove channels defaults
RUN micromamba config append channels conda-forge --env
RUN micromamba config set channel_priority strict

# Install glue kernels, and move to shared directory
# Also patching base kernel so Studio background code doesn't start session silently
Expand Down
2 changes: 1 addition & 1 deletion test/test_artifacts/v0/scripts/run_pysdk_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

# We need to checkout the version of sagemaker-python-sdk that is installed in the mamba environment.

# Sagemaker Python SDK's unit tests requires AWS_DEFAULT_REGION to be set. So, using an arbitrary value of us-east-1
# Sagemaker Python SDK's unit tests requires AWS_DEFAULT_REGION to be set. So, using an arbitrary value of us-west-2
export AWS_DEFAULT_REGION=us-west-2

pysdk_version=$(micromamba list | grep sagemaker-python-sdk | tr -s ' ' | cut -d ' ' -f 3)
Expand Down

0 comments on commit 6be4a8a

Please sign in to comment.