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

RFC 0011: Allow Intrinsic Functions and Pseudo-Parameter References in DeletionPolicy and UpdateReplacePolicy RFC #74

Merged

Conversation

MalikAtalla-AWS
Copy link
Contributor

Language Enhancement Request For Comment

This is a request for comments about allowing Intrinsic Functions and Pseudo-Parameter References in DeletionPolicy and UpdateReplacePolicy Resource Attributes RFC. See the tracking issue (#11) for additional details.


Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

jlhood
jlhood previously approved these changes May 13, 2022

# Limitation

Does not support expressions containing references to *[Resource properties](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-template-resource-type-ref.html)* or *certain pseudo parameters*

Choose a reason for hiding this comment

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

Why does it have this limitation?

Copy link
Contributor Author

@MalikAtalla-AWS MalikAtalla-AWS May 13, 2022

Choose a reason for hiding this comment

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

Thanks for the question. These are short-term limitations due to underlying implementation constraints. In the fullness of time, these limitations should be removed.

lejiati
lejiati previously approved these changes May 17, 2022
jlhood
jlhood previously approved these changes May 17, 2022
@MalikAtalla-AWS MalikAtalla-AWS added the approved RFC PR approved by CFN engineers label May 19, 2022
@MalikAtalla-AWS MalikAtalla-AWS dismissed stale reviews from jlhood and lejiati via 036c20a May 20, 2022 15:48
@MalikAtalla-AWS MalikAtalla-AWS removed the approved RFC PR approved by CFN engineers label May 20, 2022
@MalikAtalla-AWS MalikAtalla-AWS added the approved RFC PR approved by CFN engineers label May 31, 2022
@MalikAtalla-AWS MalikAtalla-AWS merged commit 6cd0d63 into aws-cloudformation:main May 31, 2022
@MalikAtalla-AWS MalikAtalla-AWS deleted the deletionPolicy branch May 31, 2022 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved RFC PR approved by CFN engineers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants