Skip to content

chore(sync): Sync terraform provider 1.18.0#19

Merged
displague merged 5 commits into
mainfrom
sync_terraform_provider_1.18.0
Oct 24, 2023
Merged

chore(sync): Sync terraform provider 1.18.0#19
displague merged 5 commits into
mainfrom
sync_terraform_provider_1.18.0

Conversation

@ocobles

@ocobles ocobles commented Oct 19, 2023

Copy link
Copy Markdown
Contributor

fixes #14
fixes #23

bump:

  • go version to 1.21
  • go terraform-provider-equinix to v1.18.0
  • go pulumi-terraform-bridge to v3.62.0
  • go pulumi to v3.90.0

New resources:

  • equinix_fabric_cloud_router
  • equinix_fabric_routing_protocol

New datasources:

  • equinix_fabric_cloud_router
  • equinix_fabric_routing_protocol
  • equinix_metal_devices

@ocobles ocobles requested review from ctreatma and displague October 19, 2023 22:10
@displague

Copy link
Copy Markdown
Member

I have no way to confirm this without #21. My attempt at building locally failed and I don't know if that's my problem or a problem with the PR.

ocobleseqx added 5 commits October 24, 2023 10:20
Signed-off-by: ocobleseqx <oscar.cobles@eu.equinix.com>
Signed-off-by: ocobleseqx <oscar.cobles@eu.equinix.com>
Signed-off-by: ocobleseqx <oscar.cobles@eu.equinix.com>
Signed-off-by: ocobleseqx <oscar.cobles@eu.equinix.com>
Signed-off-by: ocobleseqx <oscar.cobles@eu.equinix.com>
@ocobles ocobles force-pushed the sync_terraform_provider_1.18.0 branch from 8867924 to b219ddd Compare October 24, 2023 08:44
@github-actions

Copy link
Copy Markdown

Does the PR have any schema changes?

Does the PR have any schema changes?

Found 15 breaking changes:

Resources

  • "equinix:fabric/connection:Connection":
    • 🟡 inputs: "additionalInfo": items type changed from "#/types/equinix:fabric/ConnectionAdditionalInfo:ConnectionAdditionalInfo" to "object":
      • 🟡 additional properties had no type but now has &{Type: Ref:pulumi.json#/Any AdditionalProperties: Items: OneOf:[] Discriminator: Plain:false}
    • 🟡 properties: "additionalInfo": items type changed from "#/types/equinix:fabric/ConnectionAdditionalInfo:ConnectionAdditionalInfo" to "object":
      • 🟡 additional properties had no type but now has &{Type: Ref:pulumi.json#/Any AdditionalProperties: Items: OneOf:[] Discriminator: Plain:false}

Types

  • 🔴 "equinix:fabric/ConnectionAdditionalInfo:ConnectionAdditionalInfo" missing
  • 🟢 "equinix:fabric/getConnectionASideAccessPoint:getConnectionASideAccessPoint": required: "routers" property has changed to Required
  • 🟢 "equinix:fabric/getConnectionZSideAccessPoint:getConnectionZSideAccessPoint": required: "routers" property has changed to Required
  • 🔴 "equinix:fabric/getPortLag:getPortLag" missing
  • 🟡 "equinix:fabric/getPortRedundancy:getPortRedundancy": properties: "group" type changed from "integer" to "string"
  • "equinix:fabric/getPortsDatum:getPortsDatum":
    • 🟡 properties: "lags" missing
    • required:
      • 🟢 "lagEnabled" property has changed to Required
      • 🟢 "lags" property is no longer Required
  • 🔴 "equinix:fabric/getPortsDatumLag:getPortsDatumLag" missing
  • 🟡 "equinix:fabric/getPortsDatumRedundancy:getPortsDatumRedundancy": properties: "group" type changed from "integer" to "string"
  • 🟢 "equinix:networkedge/getDeviceSecondaryDevice:getDeviceSecondaryDevice": required: "cloudInitFileId" property has changed to Required

New resources:

  • fabric/cloudRouter.CloudRouter
  • fabric/routingProtocol.RoutingProtocol

New functions:

  • fabric/getCloudRouter.getCloudRouter
  • fabric/getRoutingProtocol.getRoutingProtocol
  • metal/getDevices.getDevices

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

@displague

Copy link
Copy Markdown
Member

Thanks @ocobles! This is big update and the benefits of #21 are clearly shown here.

@displague displague changed the title Sync terraform provider 1.18.0 chore(sync): Sync terraform provider 1.18.0 Oct 24, 2023
@displague displague merged commit ba8962c into main Oct 24, 2023
@displague displague deleted the sync_terraform_provider_1.18.0 branch October 24, 2023 13:59
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.

Go get: module declares its path as: fake_java_module

2 participants