This message was deleted.

sparse-intern-71089

04/26/2023, 8:22 AM

This message was deleted.

careful-bird-79707

04/26/2023, 10:54 AM

Just FYI, there's

config.Require

if you want to err when a key is missing

polite-sandwich-68547

04/26/2023, 2:17 PM

thank you @careful-bird-79707, I'll amend my code. Appreciate your input.

🙌 1

polite-sandwich-68547

04/26/2023, 2:18 PM

looks better now, simpler 🙂

Copy code

var crtArn = config.Require(ctx, "certificate:arn")
		var vpcID = config.Require(ctx, "vpc:id")

careful-bird-79707

04/26/2023, 2:20 PM

Definitely 🙂 I'm usually very stringent about not panic'ing but in this case it ain't too bad

polite-sandwich-68547

04/26/2023, 2:24 PM

In this case I actually find it brilliant to panic

salmon-account-74572

04/26/2023, 5:00 PM

A pattern you’ll see in some of our architecture templates is this:

Copy code

minClusterSize, err := cfg.TryInt("minClusterSize")
if err != nil {
    minClusterSize = 3
}

This has the effect of setting a default value if the configuration value isn’t set. If you prefer to have the program error out, then

Require

is the way to go IMO.

polite-sandwich-68547

04/26/2023, 6:28 PM

thank you @salmon-account-74572, I have in mind some use cases for this approach as well!

salmon-account-74572

04/26/2023, 6:29 PM

Happy to help!

❤️ 1

5 Views

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.