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

Fix provider Delete #16441

Merged
merged 2 commits into from
Jun 21, 2024
Merged

Fix provider Delete #16441

merged 2 commits into from
Jun 21, 2024

Conversation

lunaris
Copy link
Contributor

@lunaris lunaris commented Jun 21, 2024

#16302 (78c4820) introduced a regression whereby outputs are mixed up with inputs for provider Deletes. This commit fixes the behaviour so that it matches that before the refactoring.

Fixes #16440

#16302
(78c4820) introduced a regression
whereby outputs are mixed up with inputs for provider `Delete`s. This
commit fixes the behaviour so that it matches that before the
refactoring.

Fixes #16440
@lunaris lunaris requested a review from a team as a code owner June 21, 2024 13:42
@pulumi-bot
Copy link
Contributor

pulumi-bot commented Jun 21, 2024

Changelog

[uncommitted] (2024-06-21)

Bug Fixes

  • [engine] Fix provider Deletes
    #16441

Copy link
Contributor

@julienp julienp left a comment

Choose a reason for hiding this comment

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

Oh dear

…en-by-16302.yaml

Co-authored-by: Thomas Gummerer <t.gummerer@gmail.com>
@lunaris lunaris enabled auto-merge June 21, 2024 14:33
@lunaris lunaris added this pull request to the merge queue Jun 21, 2024
@justinvp justinvp mentioned this pull request Jun 21, 2024
Merged via the queue into master with commit 283bbd2 Jun 21, 2024
49 of 50 checks passed
@lunaris lunaris deleted the wjones/delete-inputs branch June 21, 2024 18:55
github-merge-queue bot pushed a commit that referenced this pull request Jun 22, 2024
Waiting on #16441, but there are
also a few other fixes in the queue that I'll opportunistically wait on
as well.

Tentative changelog:

### Features

- [engine] Resolve provider in the engine before passing it to
transforms
  [#16409](#16409)

- [sdk/go] Suggest valid attributes with similar names to unrecognised
ones when validating project definitions
  [#16097](#16097)

- [cli/new] Allow passing runtime options as args in pulumi new
  [#16346](#16346)

- [cli/new] Query language runtime for options during “pulumi new”
  [#16346](#16346)

- [cli/new] Add packagemanager prompt to pulumi new for nodejs
  [#16417](#16417)

- [sdk/nodejs] Detect pnpm workspaces when running pulumi install
  [#15525](#15525)

- [sdk/nodejs] Add options to Workspace::removeStack()
  [#16333](#16333)

- [sdk/python] Automatically convert requirements.txt to pyproject.toml
when using Poetry
  [#16346](#16346)

- [sdkgen/python] Generate TypedDict types for inputs
  [#15957](#15957)


### Bug Fixes

- [engine] Fix a panic when ignoring wildcard values with arrays of
different length
  [#16406](#16406)

- [cli/engine] Fix --continue-on-error running indefinitely when a
resource fails to be created or updated
  [#16371](#16371)

- [sdk/nodejs] Avoid an unhandled error when `dependencies` is missing
from `package.json` during closure serialization
  [#16433](#16433)

- [cli/plugin] Fix plugin install command when plugin type is tool
  [#16407](#16407)

- [sdk/python] Fix Python SDK docs by escaping the trailing underscore
in a docstring
  [#14866](#14866)

- [sdk/python] Don't incorrectly emit deprecation warnings for
non-deprecated properties
  [#16400](#16400)

- [sdk/python] Handle extra CLI arguments passed policy packs plugins
  [#16402](#16402)

- [sdk/python] Add VIRTUAL_ENV environment variable when running inside
a virtual environment
  [#16425](#16425)

- [sdk/python] Don't lift dunder attributes on `Output`s


### Miscellaneous

- [sdk/{go,nodejs,python}] Add register resource transform alias for
register stack transform

- [cli/new] Instruct the user to use 'pulumi install' when using
--generate-only
  [#16411](#16411)
@pulumi-bot
Copy link
Contributor

This PR has been shipped in release v3.121.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Delete sends old state in place of old inputs
4 participants