Now URIResolver can resolve KReference.Group

[slinkydeveloper]

Signed-off-by: Francesco Guardiani francescoguard@gmail.com

Part of knative/eventing#5086

Changes

• 🎁 Now URIResolver can optionally resolve KReference.Group, as discussed here: Implement KReference.Group resolution for Subscriber.Ref eventing#5440 (comment)

[slinkydeveloper]

cc @n3wscott

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: slinkydeveloper
To complete the pull request process, please assign n3wscott after the PR has been reviewed.
You can assign the PR to them by writing /assign @n3wscott in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• resolver/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[codecov]

Codecov Report

Merging #2179 (29ba44c) into main (51cfaab) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2179   +/-   ##
=======================================
  Coverage   66.05%   66.05%           
=======================================
  Files         224      224           
  Lines        9426     9434    +8     
=======================================
+ Hits         6226     6232    +6     
- Misses       2920     2921    +1     
- Partials      280      281    +1     

Impacted Files	Coverage Δ
resolver/addressable_resolver.go	95.74% <100.00%> (+0.39%)	⬆️
test/gcs/mock/mock.go	90.32% <0.00%> (-1.08%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30f9568...29ba44c. Read the comment docs.

[knative-prow-robot]

@slinkydeveloper: PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[dprotaso]

closing since slinky has moved on from Knative