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
Add Warnings when Resource Options are applied to Components with no effect #9921
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lint is failing, you need a t.Parallel() inside the cases for loop:
engine/lifeycletest/pulumi_test.go:5365:2: Range statement for test TestComponentOptionWarnings missing the call to method parallel in test Run (paralleltest)
for i, testCase := range cases {
I should probably rebase this after the release. It's probably fine and unlikely to introduce a merge conflict in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
3d3b7f4
to
5f2e819
Compare
Description
This PR is a follow-on to #9863. There are some resource options that have no effect when applied to component resources. This PR will produce a warning each time those options are applied to components.
The initial PR was scoped to emit a warning for the
ignoreChanges
option, while this PR addresses resource optionscustomTimeouts
,additinoalSecretOutputs
,replaceOnChanges
, andretainOnDelete
as well.Fixes #9877
Checklist