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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade terraform-provider-aws to v5.25.0 #2983

Closed
t0yv0 opened this issue Nov 14, 2023 · 3 comments 路 Fixed by #2998
Closed

Upgrade terraform-provider-aws to v5.25.0 #2983

t0yv0 opened this issue Nov 14, 2023 · 3 comments 路 Fixed by #2998
Assignees
Labels
kind/enhancement Improvements or new features resolution/fixed This issue was fixed

Comments

@t0yv0
Copy link
Member

t0yv0 commented Nov 14, 2023

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Affected area/feature

@t0yv0 t0yv0 added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels Nov 14, 2023
@t0yv0
Copy link
Member Author

t0yv0 commented Nov 14, 2023

Opened to trigger automated update in https://github.com/pulumi/pulumi-aws/actions/runs/6866501017

@t0yv0
Copy link
Member Author

t0yv0 commented Nov 14, 2023

Hm it's not understanding the upstream version.

@t0yv0 t0yv0 self-assigned this Nov 15, 2023
@t0yv0 t0yv0 removed the needs-triage Needs attention from the triage team label Nov 15, 2023
t0yv0 added a commit that referenced this issue Nov 15, 2023
[Upstream v5.25.0 release
notes](https://github.com/hashicorp/terraform-provider-aws/releases/tag/v5.25.0)

Fixes #2983, #2904, #2971, #2900

- [x] Rebuild eks.Cluster patches; upstream moved to AWS SDK v2 for Go,
patches needed updates as well
- [x] Fix pulumi/pulumi-terraform-bridge#1523
in the bridge
- [x] Update bridge to include
pulumi/pulumi-terraform-bridge#1521 and
pulumi/pulumi-terraform-bridge#1520 fixes
affecting P1s in pulumi-aws
- [x] Build a Pulumi test for EKS Cluster add-on removal -> turns out
the property is a no-op, not needed
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Nov 15, 2023
@t0yv0
Copy link
Member Author

t0yv0 commented Nov 15, 2023

Fixed in v6.9.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants