Closed rahulii closed 2 days ago
Hi @rahulii. Thanks for your PR.
I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
Hello @rahulii, thanks for your work so far on this. Do you need any more details to continue here? You can also check on #8014, how we solved this for the Channel (particularly on https://github.com/knative/eventing/pull/8014/commits/b7ff773d20de19d1e7a278103f4a6a14d7e06f9b and https://github.com/knative/eventing/pull/8014/commits/789113828e15b3a27434667a74e98e04a9aefa8d). Please let us know if you have any questions :)
@creydr thanks for the references, will get this done.
Hello @rahulii, any updates on this?
Hello @rahulii, any updates on this?
hey @creydr yes I was trying to understand the changes on your PR - particularly this func - https://github.com/knative/eventing/pull/8014/files#diff-118d54e11363ce17cd0de2c082c234f0dcd47f4d1078f58d1d6b0866a091886bR128
hey @creydr yes I was trying to understand the changes on your PR - particularly this func - https://github.com/knative/eventing/pull/8014/files#diff-118d54e11363ce17cd0de2c082c234f0dcd47f4d1078f58d1d6b0866a091886bR128
Hey @rahulii, this was only specific to the Channel implementation, as we needed to copy the policy also to the backing channel. This is not needed for the Sequence. I guess you can focus mostly on something like https://github.com/knative/eventing/pull/8014/commits/b7ff773d20de19d1e7a278103f4a6a14d7e06f9b and https://github.com/knative/eventing/pull/8014/commits/789113828e15b3a27434667a74e98e04a9aefa8d
/ok-to-test
hey @creydr yes I was trying to understand the changes on your PR - particularly this func - https://github.com/knative/eventing/pull/8014/files#diff-118d54e11363ce17cd0de2c082c234f0dcd47f4d1078f58d1d6b0866a091886bR128
Hey @rahulii, this was only specific to the Channel implementation, as we needed to copy the policy also to the backing channel. This is not needed for the Sequence. I guess you can focus mostly on something like b7ff773 and 7891138
ah gotcha!
Attention: Patch coverage is 76.92308%
with 9 lines
in your changes missing coverage. Please review.
Project coverage is 67.92%. Comparing base (
3b1bfb4
) to head (8b2ede3
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: creydr, rahulii
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Fixes #7978
Proposed Changes
Pre-review Checklist
Release Note
Docs