Skip to content

Upgrade terraform-provider-equinix to v1.36.4#93

Merged
ocobles merged 4 commits into
mainfrom
upgrade-terraform-provider-equinix-to-v1.36.4
May 22, 2024
Merged

Upgrade terraform-provider-equinix to v1.36.4#93
ocobles merged 4 commits into
mainfrom
upgrade-terraform-provider-equinix-to-v1.36.4

Conversation

@srushti-patl

@srushti-patl srushti-patl commented May 16, 2024

Copy link
Copy Markdown
Contributor

This PR was generated via $ upgrade-provider equinix/pulumi-equinix --upstream-provider-name terraform-provider-equinix.


  • Updating Java Gen version from 0.10.0 to 0.11.0.
  • Upgrading terraform-provider-equinix from 1.36.3 to 1.36.4.
  • Upgrading pulumi-terraform-bridge from v3.81.0 to v3.82.0.
  • Upgrading pulumi-terraform-bridge/pf from v0.34.0 to v0.35.0.
  • Added authorization_key schema to Fabric Connection access_point

@srushti-patl srushti-patl self-assigned this May 16, 2024
@srushti-patl srushti-patl requested a review from a team May 16, 2024 00:03
@github-actions

Copy link
Copy Markdown

Does the PR have any schema changes?

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

@srushti-patl srushti-patl marked this pull request as draft May 20, 2024 19:56
@srushti-patl srushti-patl marked this pull request as ready for review May 21, 2024 18:45
Comment thread provider/resources.go Outdated
"account": {
MaxItemsOne: tfbridge.True(),
},
"authentication_key": {

@ocobles ocobles May 22, 2024

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

did you add this manually or was it the upgrade-provider? Here we only define the fields that must be modified/overrided, like using AltTypes to use Enums, or MaxItemsOne when we use a list/set in terraform as an object and it does not have MaxItems:1 specified

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay got it. I have added it manually but removed it from the PR.

@srushti-patl srushti-patl requested a review from ocobles May 22, 2024 18:22
@ocobles ocobles merged commit ca4f153 into main May 22, 2024
@ocobles ocobles deleted the upgrade-terraform-provider-equinix-to-v1.36.4 branch May 22, 2024 21:18
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