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

Document using function-patch-and-transform #582

Open
Tracked by #4951
negz opened this issue Oct 25, 2023 · 2 comments
Open
Tracked by #4951

Document using function-patch-and-transform #582

negz opened this issue Oct 25, 2023 · 2 comments
Assignees

Comments

@negz
Copy link
Member

negz commented Oct 25, 2023

What's Missing?

I'd like to start gently encouraging folks to use http://github.com/crossplane-contrib/function-patch-and-transform rather than "native" P&T. Some time after v1.15 I'd like us to consider deprecating native P&T per crossplane/crossplane#4746.

There's several pages in the docs that cover Composition that we could update, including:

https://docs.crossplane.io/master/concepts/compositions/
https://docs.crossplane.io/master/concepts/patch-and-transform/
https://docs.crossplane.io/master/getting-started/provider-aws-part-2/
https://docs.crossplane.io/master/getting-started/provider-azure-part-2/
https://docs.crossplane.io/master/getting-started/provider-gcp-part-2/

@negz
Copy link
Member Author

negz commented Oct 25, 2023

Part of me is eager to draw people's attention to the new feature and wants to update every example of P&T to have a function-patch-and-transform equivalent (maybe in a tab). That feels like something we should consider when Functions are GA, though. For now I think it would be a good candidate to update the "Composition Functions" entry on the Compositions concepts page.

@jbw976
Copy link
Member

jbw976 commented Jul 3, 2024

Ha, I found this one just after opening #791. The new issue is a bit more aggressive and focused on functions in general, as opposed to this older issue focusing on function-patch-and-transform. Perhaps this one can be closed and we rally around #791?

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

No branches or pull requests

2 participants