This message was deleted.

sparse-intern-71089

07/14/2018, 5:42 PM

This message was deleted.

stocky-spoon-28903

07/14/2018, 5:44 PM

For example, every role has the

assume_role_policy

- I have to look up the structure every time.

stocky-spoon-28903

07/14/2018, 5:45 PM

So, something like this might be good: https://github.com/pulumi/pulumi-aws/compare/master...jen20:assume-role-policy-function?expand=1

big-piano-35669

07/14/2018, 5:46 PM

I love it. We have the beginnings of some helpers here, https://github.com/pulumi/pulumi-aws/blob/master/overlays/nodejs/iam/documents.ts, that give you TypeScript schema, so at least you get type-checking. There are three missing pieces, though 1. It's incomplete 2. As you say, factory functions could really cut down on boilerplate 3. The resource types, for the most part, still take JSON serialized strings, so you can't just pass instances directly To fully address 3, we'd need to do https://github.com/pulumi/pulumi-terraform/issues/10, which I know @microscopic-pilot-97530 had looked into at some point but I don't think we ever made progress.

stocky-spoon-28903

07/14/2018, 5:48 PM

I’ll open a pull request for that change then - it’s added to

document

. For now though,

JSON.stringify(assumeRolePolicyForPrincipal("whatever"))

is still an improvement I think.

👍 2

stocky-spoon-28903

07/14/2018, 5:49 PM

I’ve written that function in terms of the

PolicyDocument

and

Principal

types in

document.ts

, so it should be forward-compatible with pulumi-terraform#10 I’d imagine

Open in Slack

Previous Next

Pulumi Community

No matter how you like to participate in developer communities, Pulumi wants to meet you there. If you want to meet other Pulumi users to share use-cases and best practices, contribute code or documentation, see us at an event, or just tell a story about something cool you did with Pulumi, you are part of our community.