-
Notifications
You must be signed in to change notification settings - Fork 53
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
Upgrade terraform binary to v1.5.2 #175
Conversation
Fixes upbound#172 Signed-off-by: Yury Tsarev <yury@upbound.io>
Newer zips are failing with ``` unzip: zip flag 8 (streaming) is not supported ``` Signed-off-by: Yury Tsarev <yury@upbound.io>
/test-examples="examples/workspace-inline-aws.yaml" |
Getting this feedback from uptest, it looks like this upgrade will need more love than just the version bump :) |
This is a difference between the aws terraform provider version 4.x and 5.x. What's not obvious to me (but hopefully is to you) is why upgrading the terraform version would automatically update the version of the aws provider. |
If you don't pin the provider version it will often/usually upgrade to the latest provider on a new deployment (cache is empty) as it always pulls the latest version unless you tell it to pull a specific version |
/test-examples="examples/workspace-inline-aws.yaml" |
Signed-off-by: Yury Tsarev <yury@upbound.io>
/test-examples="examples/workspace-inline-aws.yaml" |
@bobh66 it seems that stuff got green, please review :) |
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.
LGTM - thanks!
ENV TF_IN_AUTOMATION=1 | ||
ENV TF_PLUGIN_CACHE_DIR=/tf/plugin-cache | ||
|
||
ADD "bin/${TARGETOS}_${TARGETARCH}/provider" /usr/local/bin/crossplane-terraform-provider | ||
ADD .gitconfig .gitconfig | ||
|
||
RUN curl -s -L https://releases.hashicorp.com/terraform/${TERRAFORM_VERSION}/terraform_${TERRAFORM_VERSION}_${TARGETOS}_${TARGETARCH}.zip | \ | ||
unzip -d /usr/local/bin - \ | ||
RUN curl -s -L https://releases.hashicorp.com/terraform/${TERRAFORM_VERSION}/terraform_${TERRAFORM_VERSION}_${TARGETOS}_${TARGETARCH}.zip -o terraform.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.
Just curious why this change is needed?
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.
unzip from the pipe stream is failing with the new terraform zips
unzip: zip flag 8 (streaming) is not supported
so it breaks the build.
The closest issue upstream I found is richgel999/miniz#180 .
Instead of diving into unzip
specifics, I just decided to unzip it simple file-based way :)
Description of your changes
Fixes #172
I have:
make reviewable
to ensure this PR is ready for review.How has this code been tested
uptest below