Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lack of Diagnostic Information when Bug present in Upstream Provider #2413

Open
zbuchheit opened this issue Sep 12, 2024 · 2 comments
Open
Labels
impact/accessibility Something that is difficult or impossible for some people to use kind/enhancement Improvements or new features

Comments

@zbuchheit
Copy link

What happened?

When an upstream provider has a bug that triggers an inconsistent result of apply error, our downstream providers displays this behavior as a spurious diff without any error or diagnostics. I would expect the behavior to display a log or show what is causing the spurious diff.

Example

Example Error

2024-09-11T17:09:09.291-0700 [ERROR] vertex "mist_device_gateway.gateway_one" error: Provider produced inconsistent result after apply
╷
│ Error: Provider produced inconsistent result after apply
│ 
│ When applying changes to mist_device_gateway.gateway_one, provider
│ "provider[\"registry.terraform.io/juniper/mist\"]" produced an unexpected new value:
│ .additional_config_cmds: was cty.ListValEmpty(cty.String), but now null.
│ 
│ This is a bug in the provider, which should be reported in the provider's own issue
│ tracker.

When I translate the code to pulumi in a bridged provider instead of an error I get a spurious diff and doesn't appear in the logs even when TF_LOG=debug is used.

Output of pulumi about

N/A

Additional context

I think the behavior itself is actually acceptable in that it doesn't throw an error and is an upstream bug that needs to be fixed, but I would expect a diagnostic log to show up to help troubleshoot this or some sort of warning.

Currently it is very difficult to figure out what is causing the spurious diff if the resource has a complex schema, whereas in TF the error points you to the problem immediately

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@zbuchheit zbuchheit added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Sep 12, 2024
@guineveresaenger guineveresaenger added kind/enhancement Improvements or new features impact/accessibility Something that is difficult or impossible for some people to use and removed needs-triage Needs attention from the triage team labels Sep 12, 2024
@guineveresaenger
Copy link
Contributor

Thank you for filing this bug. I've added kind/enhancement as well since this is a subcategory of inconsistencies in diff display that we may have a better fix for soon.

cc @iwahbe

@zbuchheit
Copy link
Author

If I can be of any help to provide logs etc, just let me know. Thanks! 😄

@iwahbe iwahbe removed the kind/bug Some behavior is incorrect or out of spec label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/accessibility Something that is difficult or impossible for some people to use kind/enhancement Improvements or new features
Projects
None yet
Development

No branches or pull requests

3 participants