Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Bump Terraform native provider to v2.83.0 #196

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

ulucinar
Copy link
Collaborator

@ulucinar ulucinar commented Jul 18, 2022

Description of your changes

Fixes #197
Bumps native provider version to v2.83.0.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Successfully testing by importing an existing resource group and by provisioning/updating/deleting a KubernetesCluster.

Signed-off-by: Alper Rifat Ulucinar <[email protected]>
@ytsarev
Copy link
Collaborator

ytsarev commented Jul 18, 2022

Thanks a lot @ulucinar !

I've tested this PR with the following example to verify that it is also fixing #197

apiVersion: dbforpostgresql.azure.jet.crossplane.io/v1alpha2
kind: FlexibleServer
metadata:
  name: example-flexible-psqlserver-memory-optimized
spec:
  forProvider:
    resourceGroupNameRef:
      name: example-flex-rg
    location: "East US"
    version: "12"
    administratorLogin: "psqladminun"
    administratorPasswordSecretRef:
      name: example-secret
      namespace: upbound-system
      key: password
    storageMb: 32768
    skuName: "MO_Standard_E2ds_v4" # new V4 sku name
 k get -f examples/postgresql/flexiblepostgresqlserver-mo-v4.yaml
NAME                                                                                                  READY   SYNCED   EXTERNAL-NAME                                  AGE
flexibleserver.dbforpostgresql.azure.jet.crossplane.io/example-flexible-psqlserver-memory-optimized   True    True     example-flexible-psqlserver-memory-optimized   10m

Everything looks great, thanks a ton. 👍

@ulucinar ulucinar merged commit f207b07 into crossplane-contrib:main Jul 18, 2022
@ulucinar ulucinar deleted the bump-2.83 branch July 18, 2022 12:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for _v4 sku names
3 participants