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

Remove Resource.SchemaFunc references for compatibility with the older Terraform Plugin SDK versions #348

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

ulucinar
Copy link
Collaborator

@ulucinar ulucinar commented Feb 15, 2024

Description of your changes

In upjet main HEAD, we've bumped the Terraform plugin SDK version to v2.30.0, which has the Resource.SchemaFunc API. However, there are still providers that depend on the Terraform plugin SDK versions where this API is not yet available. This PR removes the dependency on Resource.SchemaFunc so that those providers can now consume [email protected].

I have:

  • Read and followed Upjet's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Tested in the context of crossplane-contrib/provider-upjet-aws#1160 via the uptest run:
https://github.com/upbound/provider-aws/actions/runs/7921053846

…r Terraform Plugin SDK versions

Signed-off-by: Alper Rifat Ulucinar <[email protected]>
@ulucinar ulucinar marked this pull request as draft February 15, 2024 18:59
@ulucinar ulucinar marked this pull request as ready for review February 16, 2024 11:41
Copy link
Member

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

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

Thanks @ulucinar LGTM!

@ulucinar ulucinar merged commit cf9c9ad into crossplane:release-1.1 Feb 16, 2024
7 checks passed
@ulucinar ulucinar deleted the old-sdk branch February 16, 2024 11:42
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.

2 participants