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

feat: adding support for defining the backend file content #213

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

rodrigorfk
Copy link
Contributor

Description of your changes

This changes introduces the backendFile field in the ProviderConfig, which allows defining the content for the terraform backend configuration file used during terraform init.

Fixes #212

I have:

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

How has this code been tested

  • The unit tests were evolved to cover a case for the mentioned change
  • I have built the provider locally and deployed to a cluster, created a new ProviderConfig defining the backendFile field and test it in a Workspace containing partial backend configuration, I was able to validate the solution is working fine.

@Upbound-CLA
Copy link

Upbound-CLA commented Nov 16, 2023

CLA assistant check
All committers have signed the CLA.

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.

Thanks for the PR! A couple of very small nits and it needs a rebase. Looks good to me.

package/crds/tf.upbound.io_providerconfigs.yaml Outdated Show resolved Hide resolved
apis/v1beta1/types.go Outdated Show resolved Hide resolved
@rodrigorfk
Copy link
Contributor Author

Thanks for the suggestions @bobh66 , the changes were applied and a rebase was performed.

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

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

LGTM
@rodrigorfk thanks a lot for the contribution!

@bobh66 do you have any other suggestion to this PR?

@bobh66
Copy link
Collaborator

bobh66 commented Jan 5, 2024

LGTM - thanks @rodrigorfk

@bobh66 bobh66 merged commit 2a21288 into upbound:main Jan 5, 2024
9 checks passed
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.

Support Partial Backend Configuration
4 participants