mirror of https://github.com/pulumi/pulumi.git
804c9bc7f3
This is a very similar fix to https://github.com/pulumi/pulumi/pull/16371, but for imports instead of create and updates. When using the import resource option with continue-on-error, and there is a diff in the import, the import fais, but we still also return the completion function from the `Apply` call. This results in the engine trying to call `Done` twice, which in turn results in it running indefinitely, trying to write the result to a channel that's no longer being read from. Fix this by not returning the completion function from `Apply` for the ImportStep when there is an error. (I went through the other steps as well to double check we don't need a similar fix there, and it looks like they are all fine). Fixes: https://github.com/pulumi/pulumi/issues/16570 |
||
|---|---|---|
| .. | ||
| testdata | ||
| alias_test.go | ||
| analyzer_test.go | ||
| continue_on_error_test.go | ||
| delete_before_replace_test.go | ||
| golang_sdk_test.go | ||
| import_test.go | ||
| parameterized_test.go | ||
| pending_delete_test.go | ||
| pending_replace_test.go | ||
| provider_test.go | ||
| pulumi_test.go | ||
| refresh_legacy_diff_test.go | ||
| refresh_test.go | ||
| resource_reference_test.go | ||
| retain_on_delete_test.go | ||
| source_query_test.go | ||
| step_generator_test.go | ||
| target_test.go | ||
| test_plan.go | ||
| transformation_test.go | ||
| update_plan_test.go | ||