-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
build/builder: install awscli v2 #72907
Conversation
67d8c36
to
c91ee23
Compare
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.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @erikgrinaker, @rail, and @rickystewart)
build/builder/Dockerfile, line 173 at r2 (raw file):
unzip awscliv2.zip && \ ./aws/install && \ rm -rf awscliv2.zip aws
Can we pin the version, like we do for roachprod:
cockroach/pkg/cmd/roachprod/docker/build.sh
Lines 32 to 37 in e1c7d6a
# Debian ships with awscli version 1.x, which is unsupported by roachprod. | |
# Install aws-cli using the official instructions from | |
# https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2-linux.html | |
curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.0.30.zip" -o "awscliv2.zip" | |
sha256sum -c - <<EOF | |
7ee475f22c1b35cc9e53affbf96a9ffce91706e154a9441d0d39cbf8366b718e awscliv2.zip |
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.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @erikgrinaker, @rail, and @rickystewart)
build/builder/Dockerfile, line 173 at r2 (raw file):
Previously, rail (Rail Aliiev) wrote…
Can we pin the version, like we do for roachprod:
cockroach/pkg/cmd/roachprod/docker/build.sh
Lines 32 to 37 in e1c7d6a
# Debian ships with awscli version 1.x, which is unsupported by roachprod. # Install aws-cli using the official instructions from # https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2-linux.html curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.0.30.zip" -o "awscliv2.zip" sha256sum -c - <<EOF 7ee475f22c1b35cc9e53affbf96a9ffce91706e154a9441d0d39cbf8366b718e awscliv2.zip
cockroach/pkg/cmd/roachprod/docker/build.sh
Lines 32 to 41 in e1c7d6a
# Debian ships with awscli version 1.x, which is unsupported by roachprod. | |
# Install aws-cli using the official instructions from | |
# https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2-linux.html | |
curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.0.30.zip" -o "awscliv2.zip" | |
sha256sum -c - <<EOF | |
7ee475f22c1b35cc9e53affbf96a9ffce91706e154a9441d0d39cbf8366b718e awscliv2.zip | |
EOF | |
unzip awscliv2.zip | |
./aws/install | |
rm -rf aws awscliv2.zip |
c91ee23
to
b5df901
Compare
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.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @rail and @rickystewart)
build/builder/Dockerfile, line 173 at r2 (raw file):
Previously, rail (Rail Aliiev) wrote…
rathercockroach/pkg/cmd/roachprod/docker/build.sh
Lines 32 to 41 in e1c7d6a
# Debian ships with awscli version 1.x, which is unsupported by roachprod. # Install aws-cli using the official instructions from # https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2-linux.html curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.0.30.zip" -o "awscliv2.zip" sha256sum -c - <<EOF 7ee475f22c1b35cc9e53affbf96a9ffce91706e154a9441d0d39cbf8366b718e awscliv2.zip EOF unzip awscliv2.zip ./aws/install rm -rf aws awscliv2.zip
Good call, done.
b5df901
to
4ebfbd1
Compare
This breaks the AWS nightly tests. It would be great if we could merge this and push a new image before the next nightly run. |
`roachprod` requires awscli v2, but the `builder` image included v1. Release note: None
4ebfbd1
to
f255b82
Compare
Going to skip CI on this one, image is pushed and bors r=rail |
Build succeeded: |
roachprod
requires awscli v2, but thebuilder
image included v1.Release note: None