-
Notifications
You must be signed in to change notification settings - Fork 156
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
RFC: Exclusive Relationship Management Resources Support #4772
Labels
kind/engineering
Work that is not visible to an external user
kind/enhancement
Improvements or new features
Comments
flostadler
added
kind/enhancement
Improvements or new features
kind/engineering
Work that is not visible to an external user
and removed
needs-triage
Needs attention from the triage team
labels
Nov 14, 2024
t0yv0
changed the title
RFC: Exclusive Management Resource Support
RFC: Exclusive Relationship Management Resources Support
Nov 14, 2024
t0yv0
added a commit
that referenced
this issue
Nov 14, 2024
Exclusive Relationship Management Resources do not currently work as you would expect in Pulumi but require `pulumi up --refresh`. A NOTE is added with this information to the affected resources docs pages so that users are made aware of the problem. More context in: #4772
t0yv0
added a commit
that referenced
this issue
Nov 19, 2024
Exclusive Relationship Management Resources do not currently work as you would expect in Pulumi but require `pulumi up --refresh`. A NOTE is added with this information to the affected resources docs pages so that users are made aware of the problem. More context in: #4772
t0yv0
added a commit
that referenced
this issue
Nov 20, 2024
Exclusive Relationship Management Resources do not currently work as you would expect in Pulumi but require `pulumi up --refresh`. A NOTE is added with this information to the affected resources docs pages so that users are made aware of the problem. More context in: #4772
t0yv0
added a commit
that referenced
this issue
Nov 21, 2024
Exclusive Relationship Management Resources do not currently work as you would expect in Pulumi but require `pulumi up --refresh`. A NOTE is added with this information to the affected resources docs pages so that users are made aware of the problem. More context in: #4772
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
kind/engineering
Work that is not visible to an external user
kind/enhancement
Improvements or new features
Pulumi AWS provider should withhold input property deprecations introduced by the Terraform AWS provider until Pulumi-specific issues on the intended upgrade path are resolved.
The affected properties include (see 39376 for a full list):
aws.iam.Role
managedPolicyArns
aws.iam.Role
inlinePolicies
The latest iteration of Terraform AWS provider design thinking is introducing exclusive relationship management resources. To illustrate on the
aws.iam.Role
resource:inline_policy
repeated argument in favor of N relationship resources such asaws_iam_role_policy
aws_iam_role_policy
resources in Terraform does not guarantee that the associatedaws_iam_role
has no policies in the cloud that are not being tracked, something thatinline_policy
used to guaranteeaws_iam_role_policies_exclusive
is introduced: if users list allaws_iam_role_policy
inaws_iam_role_policies_exclusive
then Terraform will proactively detect when the actual set of policies in the cloud does not match the one specified byaws_iam_role_policies_exclusive
Unfortunately when testing this upgrade scenario in the Pulumi provider we found that
aws.iam.RolePoliciesExclusive
does not detect role policy drift due to specifics of how Pulumi maps Terraform providers (#4766).Because of this issue, users trying to act on the deprecation notice in Pulumi will not be able to rely on
aws.iam.RolePoliciesExclusive
to ensure all inline policies are tracked in Pulumi. Other property deprecations are likely similarly affected.The proposal going forward is as follows:
References
The text was updated successfully, but these errors were encountered: