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 provider version to v3.110.0 #825

Merged
merged 9 commits into from
Oct 4, 2024

Conversation

turkenf
Copy link
Collaborator

@turkenf turkenf commented Sep 24, 2024

Description of your changes

In this PR:

  • Upgraded Terraform provider version to v3.110.0
  • Added reference configurations to fields that were removed due to the change of examples in the Terraform registry and caused issues during the generate type ShareDirectoryParameters has no field or method ShareNameRef
  • Added new fields manually to v1beta1 APIs
  • Made secret name field sensitive for ContainerApp to avoid breaking change.
  • Fixed example manifests for the resources; KubernetesCluster and SQLFunction

I have:

  • Read and followed Crossplane'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

Breaking CRD schema changes

  • package/crds/azurestackhci.azure.upbound.io_clusters.yaml =>
NAME                                             SYNCED   READY   EXTERNAL-NAME   AGE
cluster.azurestackhci.azure.upbound.io/example   True     True    example         150m
NAME                                                                     SYNCED   READY   EXTERNAL-NAME   AGE
computeinstance.machinelearningservices.azure.upbound.io/exampleci0001   True     True    exampleci0001   32m

Native schema version changes

NAME                                                       SYNCED   READY   EXTERNAL-NAME
                                                  AGE
blobinventorypolicy.storage.azure.upbound.io/example-bip   True     True    /subscriptions/038f2b7c-3265-43b8-8624-c9ad5da610a8/resourceGroups/example-rg-bip/providers/Mi
crosoft.Storage/storageAccounts/examplergb34552   29m

Random Tests

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/cosmosdb/v1beta1/sqlfunction.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/dataprotection/v1beta1/backuppolicyblobstorage.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/machinelearningservices/v1beta1/computeinstance.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/network/v1beta1/bastionhost.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/certificateregistration/v1beta1/appservicecertificateorder.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/insights/v1beta1/applicationinsightsapikey.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/insights/v1beta1/applicationinsightssmartdetectionrule.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 26, 2024

/test-examples="examples/network/v1beta1/subnet.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 27, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Sep 27, 2024

/test-examples="examples/containerservice/v1beta1/kubernetescluster.yaml"

1 similar comment
@turkenf
Copy link
Collaborator Author

turkenf commented Sep 27, 2024

/test-examples="examples/containerservice/v1beta1/kubernetescluster.yaml"

Signed-off-by: Fatih Türken <turkenf@gmail.com>
Signed-off-by: Fatih Türken <turkenf@gmail.com>
Signed-off-by: Fatih Türken <turkenf@gmail.com>
@turkenf turkenf force-pushed the bump-tf-3.110.0 branch 3 times, most recently from 7573b4f to 4ef6af4 Compare October 1, 2024 12:54
Signed-off-by: Fatih Türken <turkenf@gmail.com>
@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/network/v1beta1/subnet.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/insights/v1beta1/applicationinsightssmartdetectionrule.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/insights/v1beta1/applicationinsightsapikey.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/certificateregistration/v1beta1/appservicecertificateorder.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/network/v1beta1/bastionhost.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/machinelearningservices/v1beta1/computeinstance.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/dataprotection/v1beta1/backuppolicyblobstorage.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/cosmosdb/v1beta1/sqlfunction.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

1 similar comment
@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

Signed-off-by: Fatih Türken <turkenf@gmail.com>
@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/cosmosdb/v1beta1/sqlfunction.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/certificateregistration/v1beta1/appservicecertificateorder.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 1, 2024

/test-examples="examples/machinelearningservices/v1beta1/computeinstance.yaml"

Signed-off-by: Fatih Türken <turkenf@gmail.com>
@turkenf
Copy link
Collaborator Author

turkenf commented Oct 2, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

https://github.com/crossplane-contrib/provider-upjet-azure/actions/runs/11144688742

Signed-off-by: Fatih Türken <turkenf@gmail.com>
@turkenf
Copy link
Collaborator Author

turkenf commented Oct 2, 2024

/test-examples="examples/containerservice/v1beta1/kubernetescluster.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 2, 2024

/test-examples="examples/network/v1beta1/subnet.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 2, 2024

/test-examples="examples/cdn/v1beta2/endpoint.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Oct 2, 2024

/test-examples="examples/network/v1beta2/routefilter.yaml"

@turkenf turkenf marked this pull request as ready for review October 2, 2024 15:46
@turkenf turkenf requested a review from mergenci October 2, 2024 15:56
Copy link
Collaborator

@mergenci mergenci left a comment

Choose a reason for hiding this comment

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

Using my bespoke Go command, I verified that the new 24 singleton list fields were backported successfully. I'll do a few more checks for other fields in the morning and complete my review then. Please see one question I had below.

Signed-off-by: Cem Mergenci <cmergenci@gmail.com>
Signed-off-by: Cem Mergenci <cmergenci@gmail.com>
@mergenci
Copy link
Collaborator

mergenci commented Oct 4, 2024

/test-examples="examples/containerapp/v1beta1/containerapp.yaml"

@mergenci
Copy link
Collaborator

mergenci commented Oct 4, 2024

/test-examples="examples/cosmosdb/v1beta1/sqlfunction.yaml"

@mergenci
Copy link
Collaborator

mergenci commented Oct 4, 2024

/test-examples="examples/network/v1beta1/subnet.yaml"

Copy link
Collaborator

@mergenci mergenci 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 your great effort @turkenf 🙏 We still have a lot to do before we could version bump Terraform providers automatically.

I Uptested some of the resources, to InitParameters of which I added SecretRefs. I edited PR description with links to the latest Uptest results.

@turkenf turkenf merged commit 580e8a4 into crossplane-contrib:main Oct 4, 2024
14 checks passed
@turkenf turkenf deleted the bump-tf-3.110.0 branch October 4, 2024 11:23
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