pulumi/pkg/resource
Thomas Gummerer 804c9bc7f3
fix hang when continue-on-error is used with import resource option (#16572)
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
2024-07-03 10:24:26 +00:00
..
analyzer Clean up uses of .Error() (#14965) 2023-12-20 15:54:06 +00:00
deploy fix hang when continue-on-error is used with import resource option (#16572) 2024-07-03 10:24:26 +00:00
edit Rewrite `DeletedWith` properties when renaming stacks (#16216) 2024-05-16 15:09:32 +00:00
graph implement skeleton command for `pulumi state move` (#16493) 2024-06-28 09:43:19 +00:00
provider turn on the golangci-lint exhaustive linter (#15028) 2024-01-17 16:50:41 +00:00
stack implement skeleton command for `pulumi state move` (#16493) 2024-06-28 09:43:19 +00:00
testdata ci: radical idea - what if slow tests & no stdout makes GH consider runner dead? 2022-03-06 14:52:13 -08:00