pulumi-aws icon indicating copy to clipboard operation
pulumi-aws copied to clipboard

Published 20 hours ago • verified publisher icon pulumi

PATCH 0007-Workaround-SSM-Parameter-tier-bug.patch

Open t0yv0 opened this issue 2 years ago • 2 comments

Tracking context on upstream patch 0007-Workaround-SSM-Parameter-tier-bug.patch here.

t0yv0 avatar Aug 28 '23 15:08 t0yv0

Change schema, remove Computed, add Default

t0yv0 avatar Aug 28 '23 15:08 t0yv0

Reviewing this patch lightly again today as @corymhall mentioned it's part of the friction in a scheduled upstream upgrade in https://github.com/pulumi/pulumi-aws/pull/3959

I think GetRawConfig() used to incorrectly return null in the bridge. To workaround the application of default tier got moved from inside the provider implementation reading GetRawConfig to a bridge-level default application.

The missing test program would be to provision an ssm.Parameter with various values of tier (including undefined) and verify it works as expected.

t0yv0 avatar May 20 '24 20:05 t0yv0