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

Upstream v5.67.0 #4458

Merged
merged 20 commits into from
Sep 16, 2024
Merged

Upstream v5.67.0 #4458

merged 20 commits into from
Sep 16, 2024

Conversation

t0yv0
Copy link
Member

@t0yv0 t0yv0 commented Sep 13, 2024

This change upgrades upstream to v5.67.0.

Fixes #4435
Fixes #4454

v5.66.0 changes:

Notable upstream additions:

New resources:

  • datazone/environment.Environment

New functions:

  • glue/getRegistry.getRegistry
  • organizations/....getOrganizationalUnitDescendantOrganizationalUnits
  • quicksight/getAnalysis.getAnalysis
  • quicksight/getQuicksightAnalysis.getQuicksightAnalysis

Unfortunately the aws_quicksight_analysis datasource cannot yet be fully supported in the provider because excessively large type hierarchies in Quicksight cause issues for Pulumi SDKs. This is the same problem that makes "definition" fields unavailable for Quicksight resources. A solution is being explored in #4449 but in the meantime the "definition" field is not available yet in Pulumi for the given datasource.

There also was a regression hashicorp/terraform-provider-aws#39320 in v5.66.0 that tripped pulumi-aws testing so the matching pulumi-aws release is skipped.

v5.67.0 changes:

  • cloudformation/stackInstances.StackInstances
  • codebuild/fleet.Fleet
  • computeoptimizer/enrollmentStatus.EnrollmentStatus
  • computeoptimizer/recommendationPreferences.RecommendationPreferences
  • costoptimizationhub/enrollmentStatus.EnrollmentStatus
  • costoptimizationhub/preferences.Preferences
  • datazone/assetType.AssetType
  • lambda/functionRecursionConfig.FunctionRecursionConfig

New functions:

  • codebuild/getFleet.getFleet

Unfortunately one of the upstream features that introduced support for IAM role chaining is projecting as a breaking change to Pulumi types. Since our policy is to prioritize backwards compatibility, this feature has been backed out and is not yet available in the upcoming pulumi-aws release.

See: hashicorp/terraform-provider-aws#22728

Reminder opened to track this in #4459

@t0yv0 t0yv0 mentioned this pull request Sep 13, 2024
Copy link

github-actions bot commented Sep 13, 2024

Does the PR have any schema changes?

Found 6 breaking changes:

Resources

  • 🟢 "aws:rolesanywhere/profile:Profile": required: "roleArns" property is no longer Required

Functions

  • 🟡 "aws:quicksight/getDataSet:getDataSet": inputs: "columnLevelPermissionRules" missing input "columnLevelPermissionRules"

Types

  • 🟢 "aws:appconfig/ExtensionActionPointAction:ExtensionActionPointAction": required: "roleArn" property is no longer Required
  • 🟡 "aws:lex/V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting:V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting": properties: "audioRecognitionSetting" missing
  • 🟡 "aws:networkmanager/getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride:getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride": properties: "edgeSets": items type changed from "string" to "array":
    • 🟡 items had no type but now has &{Type:string Ref: AdditionalProperties: Items: OneOf:[] Discriminator: Plain:false}

New resources:

  • cloudformation/stackInstances.StackInstances
  • codebuild/fleet.Fleet
  • computeoptimizer/enrollmentStatus.EnrollmentStatus
  • computeoptimizer/recommendationPreferences.RecommendationPreferences
  • costoptimizationhub/enrollmentStatus.EnrollmentStatus
  • costoptimizationhub/preferences.Preferences
  • datazone/assetType.AssetType
  • datazone/environment.Environment
  • lambda/functionRecursionConfig.FunctionRecursionConfig

New functions:

  • codebuild/getFleet.getFleet
  • glue/getRegistry.getRegistry
  • organizations/getOrganizationalUnitDescendantOrganizationalUnits.getOrganizationalUnitDescendantOrganizationalUnits
  • quicksight/getAnalysis.getAnalysis
  • quicksight/getQuicksightAnalysis.getQuicksightAnalysis

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

@t0yv0
Copy link
Member Author

t0yv0 commented Sep 13, 2024

Looks like assumeRole is now assumeRoles (plural), which is a breaking change for Pulumi. Looking at options quickly.

SError: ⨯ Unable to compile TypeScript:
Error: index.ts(28,5): error TS2345: Argument of type '{ assumeRole: { roleArn: pulumi.Output<string>; }; }' is not assignable to parameter of type 'ProviderArgs'.
  Object literal may only specify known properties, but 'assumeRole' does not exist in type 'ProviderArgs'. Did you mean to write 'assumeRoles'?

@t0yv0
Copy link
Member Author

t0yv0 commented Sep 13, 2024

Coming from here: hashicorp/terraform-provider-aws@faa9b8e

@t0yv0 t0yv0 marked this pull request as ready for review September 13, 2024 21:52
Copy link
Contributor

@flostadler flostadler left a comment

Choose a reason for hiding this comment

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

This was a wild ride! Good job!

@@ -5,319 +5,508 @@ Subject: [PATCH] Restore legacy bucket


diff --git a/go.mod b/go.mod
index 0d8ccda18d..85b3de0453 100644
index 5b8c85d7d3..a145fa7f16 100644
Copy link
Contributor

Choose a reason for hiding this comment

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

This patch diff is hard to review. Couldn't spot anything wrong though

Copy link
Member Author

Choose a reason for hiding this comment

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

We should probably take upstream/go.mod churn out of patches and try to see if we can accept the unmodified upstream go.mod.

Copy link
Member Author

Choose a reason for hiding this comment

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

@t0yv0 t0yv0 merged commit f4af80f into master Sep 16, 2024
30 checks passed
@t0yv0 t0yv0 deleted the upstream-v5.67.0 branch September 16, 2024 13:23
@pulumi-bot
Copy link
Contributor

This PR has been shipped in release v6.52.0.

lumiere-bot bot referenced this pull request in coolguy1771/home-ops Sep 19, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@pulumi/aws](https://pulumi.io)
([source](https://redirect.github.com/pulumi/pulumi-aws)) | dependencies
| minor | [`6.51.1` ->
`6.52.0`](https://renovatebot.com/diffs/npm/@pulumi%2faws/6.51.1/6.52.0)
|

---

### Release Notes

<details>
<summary>pulumi/pulumi-aws (@&#8203;pulumi/aws)</summary>

###
[`v6.52.0`](https://redirect.github.com/pulumi/pulumi-aws/releases/tag/v6.52.0)

[Compare
Source](https://redirect.github.com/pulumi/pulumi-aws/compare/v6.51.1...v6.52.0)

##### Does the PR have any schema changes?

Found 6 breaking changes:

##### Resources

- `🟢` "aws:rolesanywhere/profile:Profile": required: "roleArns" property
is no longer Required

##### Functions

- `🟡` "aws:quicksight/getDataSet:getDataSet": inputs:
"columnLevelPermissionRules" missing input "columnLevelPermissionRules"

##### Types

- `🟢`
"aws:appconfig/ExtensionActionPointAction:ExtensionActionPointAction":
required: "roleArn" property is no longer Required
- `🟡`
"aws:lex/V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting:V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting":
properties: "audioRecognitionSetting" missing
- `🟡`
"aws:networkmanager/getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride:getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride":
properties: "edgeSets": items type changed from "string" to "array":
- `🟡` items had no type but now has &{Type:string Ref:
AdditionalProperties:<nil> Items:<nil> OneOf:\[] Discriminator:<nil>
Plain:false}

##### New resources:

-   `cloudformation/stackInstances.StackInstances`
-   `codebuild/fleet.Fleet`
-   `computeoptimizer/enrollmentStatus.EnrollmentStatus`
- `computeoptimizer/recommendationPreferences.RecommendationPreferences`
-   `costoptimizationhub/enrollmentStatus.EnrollmentStatus`
-   `costoptimizationhub/preferences.Preferences`
-   `datazone/assetType.AssetType`
-   `datazone/environment.Environment`
-   `lambda/functionRecursionConfig.FunctionRecursionConfig`

##### New functions:

-   `codebuild/getFleet.getFleet`
-   `glue/getRegistry.getRegistry`
-
`organizations/getOrganizationalUnitDescendantOrganizationalUnits.getOrganizationalUnitDescendantOrganizationalUnits`
-   `quicksight/getAnalysis.getAnalysis`
-   `quicksight/getQuicksightAnalysis.getQuicksightAnalysis`

##### What's Changed

- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://redirect.github.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4456](https://redirect.github.com/pulumi/pulumi-aws/pull/4456)
- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://redirect.github.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4464](https://redirect.github.com/pulumi/pulumi-aws/pull/4464)
- Upstream v5.67.0 by [@&#8203;t0yv0](https://redirect.github.com/t0yv0)
in
[https://github.com/pulumi/pulumi-aws/pull/4458](https://redirect.github.com/pulumi/pulumi-aws/pull/4458)

**Full Changelog**:
pulumi/pulumi-aws@v6.51.1...v6.52.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://redirect.github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44NC4wIiwidXBkYXRlZEluVmVyIjoiMzguODQuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsidHlwZS9taW5vciJdfQ==-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
lumiere-bot bot referenced this pull request in coolguy1771/home-ops Sep 19, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@pulumi/aws](https://pulumi.io)
([source](https://redirect.github.com/pulumi/pulumi-aws)) | dependencies
| minor | [`6.51.1` ->
`6.52.0`](https://renovatebot.com/diffs/npm/@pulumi%2faws/6.51.1/6.52.0)
|

---

### Release Notes

<details>
<summary>pulumi/pulumi-aws (@&#8203;pulumi/aws)</summary>

###
[`v6.52.0`](https://redirect.github.com/pulumi/pulumi-aws/releases/tag/v6.52.0)

[Compare
Source](https://redirect.github.com/pulumi/pulumi-aws/compare/v6.51.1...v6.52.0)

##### Does the PR have any schema changes?

Found 6 breaking changes:

##### Resources

- `🟢` "aws:rolesanywhere/profile:Profile": required: "roleArns" property
is no longer Required

##### Functions

- `🟡` "aws:quicksight/getDataSet:getDataSet": inputs:
"columnLevelPermissionRules" missing input "columnLevelPermissionRules"

##### Types

- `🟢`
"aws:appconfig/ExtensionActionPointAction:ExtensionActionPointAction":
required: "roleArn" property is no longer Required
- `🟡`
"aws:lex/V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting:V2modelsSlotTypeValueSelectionSettingAdvancedRecognitionSetting":
properties: "audioRecognitionSetting" missing
- `🟡`
"aws:networkmanager/getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride:getCoreNetworkPolicyDocumentSegmentActionViaWithEdgeOverride":
properties: "edgeSets": items type changed from "string" to "array":
- `🟡` items had no type but now has &{Type:string Ref:
AdditionalProperties:<nil> Items:<nil> OneOf:\[] Discriminator:<nil>
Plain:false}

##### New resources:

-   `cloudformation/stackInstances.StackInstances`
-   `codebuild/fleet.Fleet`
-   `computeoptimizer/enrollmentStatus.EnrollmentStatus`
- `computeoptimizer/recommendationPreferences.RecommendationPreferences`
-   `costoptimizationhub/enrollmentStatus.EnrollmentStatus`
-   `costoptimizationhub/preferences.Preferences`
-   `datazone/assetType.AssetType`
-   `datazone/environment.Environment`
-   `lambda/functionRecursionConfig.FunctionRecursionConfig`

##### New functions:

-   `codebuild/getFleet.getFleet`
-   `glue/getRegistry.getRegistry`
-
`organizations/getOrganizationalUnitDescendantOrganizationalUnits.getOrganizationalUnitDescendantOrganizationalUnits`
-   `quicksight/getAnalysis.getAnalysis`
-   `quicksight/getQuicksightAnalysis.getQuicksightAnalysis`

##### What's Changed

- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://redirect.github.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4456](https://redirect.github.com/pulumi/pulumi-aws/pull/4456)
- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://redirect.github.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4464](https://redirect.github.com/pulumi/pulumi-aws/pull/4464)
- Upstream v5.67.0 by [@&#8203;t0yv0](https://redirect.github.com/t0yv0)
in
[https://github.com/pulumi/pulumi-aws/pull/4458](https://redirect.github.com/pulumi/pulumi-aws/pull/4458)

**Full Changelog**:
pulumi/pulumi-aws@v6.51.1...v6.52.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://redirect.github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44NC4wIiwidXBkYXRlZEluVmVyIjoiMzguODQuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsidHlwZS9taW5vciJdfQ==-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
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.

Upgrade terraform-provider-aws to v5.67.0 Upgrade terraform-provider-aws to v5.66.0
4 participants