mirror of https://github.com/pulumi/pulumi.git
98d7246ea8
When displaying the progress of a Pulumi operation to the user, we want the operation being displayed to reflect what is actually happening at that moment in time. Most of the time, this means "just display the operation in question" -- if a `create` is being executed, show "creating", if a `delete` just completed, show "deleted", and so on. However, there are cases where we can do better than just displaying the "raw" operation. Specifically, our "replacement-like" operations comprise a _series_ of steps that must execute for the operation as a whole to make sense. For create-before-replace, we have: * `create replacement` resource * `replace` the old resource * `delete original` resource Other sequences, such as delete-before-replace, are similar (in the case of delete-before-replace, the `delete original` step comes first). While it might make sense to display the underlying steps as the operation progresses, when the series of steps has _completed_, it's (arguably) much clearer to simply render the string `replaced` so that the user knows what has gone on. Similarly, during a preview, it (again arguably) makes more sense for us to state that the intention is to `replace`, rather than any one of `create replacement`/`replace`/`delete original` and so on. Alas, there is a case where this is potentially misleading and thus undesirable behaviour. If an _error_ occurs during execution, the operation will terminate at the next opportunity. In doing so, it will enter a "done" state. At this point, we _do not_ want to rewrite the step that was actually happening before the error interrupted it (e.g. `create replacement`) with the "end" state (e.g. `replaced`), since the error may mean we never reached that desired state. We want the display to be as true to the raw series of steps as possible. This PR implements this change, so that programs which terminate due to errors do not rewrite their steps. This PR addresses some of the confusion in #16270, in which we incorrectly reported that a delete-before-replace resource had been `replaced` when in fact we had only completed the deletion before being interrupted by an error elsewhere. |
||
|---|---|---|
| .. | ||
| display | ||
| diy | ||
| filestate | ||
| httpstate | ||
| state | ||
| apply.go | ||
| apply_test.go | ||
| backend.go | ||
| backend_test.go | ||
| cancellation_scope.go | ||
| doc.go | ||
| errors.go | ||
| inmemoryPersister.go | ||
| mock.go | ||
| policypack.go | ||
| query.go | ||
| snapshot.go | ||
| snapshot_test.go | ||
| stack.go | ||
| updates.go | ||
| watch.go | ||