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

Include stacktrace in error message #634

Merged
merged 3 commits into from Nov 11, 2022

Conversation

RobbieMcKinstry
Copy link
Contributor

In TFGen, HCL conversion is allowed to panic. Panics are recovered and converted into errors, which are then displayed to the user.

Currently, these panics don't provide a way to inspect the stacktrace, making it very hard to debug P1s. This PR captures the stacktrace, and logs traces when debug is enabled.

Related

@RobbieMcKinstry RobbieMcKinstry added impact/usability Something that impacts users' ability to use the product easily and intuitively kind/enhancement Improvements or new features area/tfgen Issues in pkg/tgen, excluding docs generation - use area/docsgen for those labels Nov 9, 2022
@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-azuread with merge commit 1a14099

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-random with merge commit 1a14099

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-gcp with merge commit 1a14099

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-azure with merge commit 1a14099

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-azuread with merge commit b27f01e

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-random with merge commit b27f01e

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-gcp with merge commit b27f01e

pkg/tfgen/docs.go Outdated Show resolved Hide resolved
Copy link
Contributor

@viveklak viveklak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nits. LGTM otherwise.

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-azure with merge commit b27f01e

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-aws with merge commit 1a14099

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

Diff for pulumi-aws with merge commit b27f01e

@github-actions
Copy link

Diff for pulumi-azuread with merge commit a77884b

@github-actions
Copy link

Diff for pulumi-random with merge commit a77884b

@github-actions
Copy link

Diff for pulumi-gcp with merge commit a77884b

@github-actions
Copy link

Diff for pulumi-azure with merge commit a77884b

@github-actions
Copy link

Diff for pulumi-aws with merge commit a77884b

@RobbieMcKinstry RobbieMcKinstry merged commit 9fbc56d into master Nov 11, 2022
@RobbieMcKinstry RobbieMcKinstry deleted the mckinstry/expose-stacktrace branch November 11, 2022 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tfgen Issues in pkg/tgen, excluding docs generation - use area/docsgen for those impact/usability Something that impacts users' ability to use the product easily and intuitively kind/enhancement Improvements or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants