-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
release: v1.10.2 #531
base: main
Are you sure you want to change the base?
release: v1.10.2 #531
Conversation
along with the release of npm 1.11.0 last week: https://github.com/npm/cli/releases/tag/v11.0.0, we starts to seeing image build failure due to this line: https://github.com/aws/sagemaker-distribution/blame/main/template/v1/Dockerfile#L93. The reason is:
As the fix, I'm replacing the command to |
template/v1/Dockerfile
Outdated
@@ -90,7 +90,7 @@ ARG MAMBA_DOCKERFILE_ACTIVATE=1 | |||
RUN sudo ln -s $(which python3) /usr/bin/python | |||
|
|||
# Update npm version | |||
RUN npm i -g npm | |||
RUN RUN npm update -g npm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicated "RUN" command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I'm making change to reduce RUN
command in both v1.10.2 and v1 template in this PR
This reverts commit 1c7628c.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Staleness Report: 1.10.2(gpu)
Staleness Report: 1.10.2(cpu)
|
Error retrieving image size: name 'STAGING_ACCOUNT' is not defined GPU Total Image Size ReportFailed to retrieve size for target image: 1.10.2-gpu or base image: 1.10.1-gpu CPU Total Image Size ReportFailed to retrieve size for target image: 1.10.2-cpu or base image: 1.10.1-cpu Python Package Size Report (GPU)Target Image Version: 1.10.2 | Base Image Version: 1.10.1Python Packages Total Size Summary
Top-20 Largest Python Packages
Python Package Size DeltaThe total size of newly introduced Python packages is 40.11MB, accounts for 1.13% of the total package size.
Python Package Size Report (CPU)Target Image Version: 1.10.2 | Base Image Version: 1.10.1Python Packages Total Size Summary
Top-20 Largest Python Packages
Python Package Size DeltaThe total size of newly introduced Python packages is 40.80MB, accounts for 3.18% of the total package size.
|
Python Package Dependency ReportTarget Image Version: 1.10.2 | Base Image Version: 1.10.1Image Type: (GPU)
Image Type: (CPU)
Image Type: (GPU)
Image Type: (CPU)
Image Type: (GPU)
Image Type: (CPU)
Image Type: (GPU)
Image Type: (CPU)
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
This pull request was created by GitHub Actions/AWS CodeBuild! Before merging, please do the following: