search

pulumi / pulumi-awsx

AWS infrastructure best practices in component form!
https://www.pulumi.com/docs/guides/crosswalk/aws/
Apache License 2.0
227 stars 104 forks source link

Rewrite a warning to avoid printing sub-properties as outputs #1395

Closed mikhailshilkov closed 1 month ago

mikhailshilkov commented 1 month ago

Another fix for https://github.com/pulumi/pulumi-awsx/issues/1372 while https://github.com/pulumi/pulumi-awsx/pull/1374 only partially succeeded.

The previous attempt was successful in resolving the top-level output but child outputs were still unresolved. I was a bit surprised that was the case... outputs are hard!

The new attempt relies on pulumi.jsonStringify which is made exactly for this purpose. This drops JSON formatting but hopefully that not too bad for a small JSON. Example of the result:

 warning: The default subnetStrategy will change from "Legacy" to "Auto" in the next major version. Please specify the subnetStrategy explicitly. The current subnet layout can be specified via "Auto" as:

    [{"cidrMask":20,"type":"Public"}]

Fix #1372

pulumi-bot commented 2 weeks ago

This PR has been shipped in release v2.17.0.