fix: Prevent a deadlock on provider error in py automation api

[elthariel]

Description

The rationale behind this change is explained in the linked issue. I haven't worked on the Checklist below, as I'd like to get an eye with more pulumi knownledge on the issue beforehand.

Also, reproducing deadlocks in unit tests is always quite tricky, so I'd rather not do it until I'm sure this is the proper approach

Fixes #11594

Checklist

• I have added tests that prove my fix is effective or that my feature works

• I have run make changelog and committed the changelog/pending/<file> documenting my change

• Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[pulumi-bot]

Changelog

[uncommitted] (2022-12-09)

Bug Fixes

• [sdk/python] Fix a deadlock on provider-side error with automation api

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[justinvp]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[justinvp]

LGTM. Thanks, @elthariel! I'll get it merged after the acceptance tests have run.

[elthariel]

I've noticed in the tests that there was a lint error, so I've updated my PR to fix the issue. I hope it doesn't mess with the acceptance tests

[justinvp]

No worries, I'll re-run

[justinvp]

/run-acceptance-tests

[elthariel]

That's weird. It doesn't seems like it ran the tests again. At least I don't see anything in the list of workflows

[justinvp]

That's weird. It doesn't seems like it ran the tests again.

Strange. Not sure why. (We've been tweaking how our CI is setup, so there may be some kinks to work out still). In any case, I ran the linter locally on your branch and it LGTM.

[justinvp]

bors merge

[bors]

Build succeeded:

• bors-ok