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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add descriptive error message for SSA Patch delete for required fields #2107

Closed
lblackstone opened this issue Jul 28, 2022 · 0 comments 路 Fixed by #2111
Closed

Add descriptive error message for SSA Patch delete for required fields #2107

lblackstone opened this issue Jul 28, 2022 · 0 comments 路 Fixed by #2111
Assignees
Labels
area/server-side-apply kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Milestone

Comments

@lblackstone
Copy link
Member

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Based on finding from #2104

In that bug report, the user was unable to delete a DeploymentPatch resource that had force-overwritten a required field. While this is expected behavior for SSA resources, it is likely to be a common point of misunderstanding that we could address with a descriptive error message that explains how to resolve the problem.

There are a couple options, depending on the use case:

  1. Set the retainOnDelete resource option, which will leave the resource unchanged and drop it from Pulumi management during a pulumi destroy operation.
  2. Transfer ownership of the field to another controller prior to the pulumi destroy operation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/server-side-apply kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants