Skip to content
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

Bump Terraform CLI to v1.5.5 #187

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Bump Terraform CLI to v1.5.5 #187

merged 1 commit into from
Aug 28, 2023

Conversation

ulucinar
Copy link
Contributor

@ulucinar ulucinar commented Aug 28, 2023

Description of your changes

This PR bumps the Terraform CLI version to v1.5.5 and consumes it from https://github.com/upbound/terraform/releases.

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Manually tested the provider using Terraform v1.5.5 using the provider example examples/workspace-inline-aws.yaml:

❯ k get managed
NAME                                    READY   SYNCED   AGE
workspace.tf.upbound.io/sample-inline   True    True     21s

Also verified via the the uptest run here.

@ulucinar
Copy link
Contributor Author

/test-examples="examples/workspace-inline-aws.yaml"

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 -o terraform.zip \
RUN curl -s -L https://github.com/upbound/terraform/releases/download/v${TERRAFORM_VERSION}/terraform_${TERRAFORM_VERSION}_${TARGETOS}_${TARGETARCH}.zip -o terraform.zip \
Copy link
Contributor

@jeanduplessis jeanduplessis Aug 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bobh66 @ytsarev: Take note that we should not change the location of where we pull the Terraform binary from due to the Terraform BSL change. When the time comes where there is a need for a newer version we should evaluate our options before simply upgrading to a new TF version.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add a comment to that effect before the RUN command.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @bobh66, added a comment explaining the situation.

Copy link
Collaborator

@bobh66 bobh66 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest adding a comment regarding the reason we're pulling from the private repo and why, otherwise someone will change it without realizing.

- Consume the Terraform CLI from https://github.com/upbound/terraform/releases

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
@ulucinar
Copy link
Contributor Author

/test-examples="examples/workspace-inline-aws.yaml"

@ulucinar ulucinar merged commit fd4f716 into upbound:main Aug 28, 2023
10 checks passed
@ulucinar ulucinar deleted the tf-1.5.5 branch August 28, 2023 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants