[Epic] Q2 Bridge Quality

[t0yv0]

Internal Design Doc

Timeline: Q2

• Scan kind/bug issues in pulumi-terraform-bridge, adopt some of them in the epic
• Confusing provider configuration error message #343
• error.go Error() ignores AttributeError Leading to ambiguous error messages #906
• panic previewing or updating an AWS EKS ManagedNodeGroup hit an assertion failure #570
• panic Cryptic error when providing invalid AppSettings for AppService #940
• bug: Terraform Bridge does not reliably perform Terraform state upgrades #923
• Provide missing key names in error messages #651
• Scan bridged Tier 1 providers for issues that might belong here
  • Schedule calls with interested parties (@kpitzen interested)
  • Place to start https://metabase.corp.pulumi.com/question/1717-top-issues-tier-1-providers
  • Also ditto Airtable
  • Quite possibly issue we find there pre-empt the list below
• [3] constant diff from wafv2 with no changes pulumi-aws#1423
• [1] random.RandomPassword API docs are broken pulumi-random#286
• [3] Re-enable CheckConfig / DiffConfig #244
• [3] ID attributes need to respect providers that expect numerical IDs #352
• [2] [tfgen] Computed nested properties should not be included on input interfaces? #493
• [1] State is updated even if the API call fails with an HTTP 400 (gcp.cloudscheduler.Job) #496
• [1] Add the ability to support default aliases in the bridge #607
• [1] Duplicate underscore panic #62
• [1] Terraform deprecation warning in Pulumi output #6
• [1] Automatically enable Terraform logging at a certain log level #97
• [1] Let the user know if a TF provider repo is not found during docs generation #528
• [1] tfgen defaults optional provider args that can come from the environment to empty values, rather than nil #635 try to repro likely closed
• [1] Lambda Alias AdditionalVersionWeights not set correctly #335 try to repro likely closed

To triage:

• pulumi-docker.Container docs mention terraform #1048

[iwahbe]

For testing, should we add these?

• Add documentation cleanup test #898
• write test for edge case examples generation #877 (might be addressed soon)

[iwahbe]

We have agreed to address pulumi/pulumi-cloudflare#366 as part of bridge work in Q2. @t0yv0 can I add it to the epic.

This works in TF, so the problem is the bridge treating empty objects differently then TF.

[AaronFriel]

Another candidate issue: pulumi/pulumi-random#286

[t0yv0]

I've missed #723 because I was scanning mostly bugs. I feel like it can fit here with #651 same area.

[iuliancristea]

Hi @t0yv0 - For #635, the issue is very much still there and causing problems.

[t0yv0]

Relevant label: https://github.com/pulumi/pulumi-aws/issues?q=is%3Aissue+is%3Aopen+label%3Abug%2Fdiff+

[phillipedwards]

@t0yv0 just curious, is there any timeline for #906? Would be a nice QOL improvement for helping users debug issues.

[t0yv0]

That's a great idea! Yes, very valuable.

[t0yv0]

#1118 came up as a real QOL improvement opportunity for the providers team; current workarounds include dropping fields entirely in providers such as azure-native to get past the limit on file size.

[AaronFriel]

@t0yv0 should #540 be a candidate here as well? It looks like this affects multiple providers.

[t0yv0]

Possibly pulumi/pulumi-aws#1655 - highly visible.

[t0yv0]

The quarters are changing on us, closing this epic. We've made some progress here. I will open a new epic for Q3 once we have a chance to replan. AS always issue upvotes are highly appreciated as they inform priorities for maintainers.

[pulumi-bot]

Cannot close issue without required labels: resolution/