-
Notifications
You must be signed in to change notification settings - Fork 901
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
Docs: fix CombineFromComposite
patch example
#3354
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.
Thanks @ytsarev! One small nitpick
docs/reference/composition.md
Outdated
@@ -438,14 +438,14 @@ composed resource field. | |||
|
|||
```yaml | |||
# Patch from the XR's spec.parameters.location field and the | |||
# metadata.annotations[crossplane.io/claim-name] annotation to the composed | |||
# metadata.labels[crossplane.io/claim-name] annotation to the composed |
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.
# metadata.labels[crossplane.io/claim-name] annotation to the composed | |
# metadata.labels[crossplane.io/claim-name] label to the composed |
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.
@hasheddan amended, thanks a lot for the catch!
`crossplane.io/claim-name` is a label, not an annotation Signed-off-by: Yury Tsarev <yury@upbound.io>
8526c62
to
35a8649
Compare
Successfully created backport PR #3357 for |
Successfully created backport PR #3358 for |
Successfully created backport PR #3359 for |
Description of your changes
crossplane.io/claim-name
is a label, not an annotationSigned-off-by: Yury Tsarev yury@upbound.io
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested