-
Notifications
You must be signed in to change notification settings - Fork 590
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
List applying policies in mt-brokers .status.policies
#7976
Comments
/assign |
@asr2003 are you still working on this issue? |
@EraKin575, this issue is still in status "Backlog", as not all the dependent issues (e.g. #7985 (which should be done soon)) are resolved yet. So @asr2003 didn't have a chance to work on it yet. |
Yes, waiting to solve upstream issue
…On Mon, 17 Jun, 2024, 5:08 pm Christoph Stäbler, ***@***.***> wrote:
@asr2003 <https://github.com/asr2003> are you still working on this issue?
@EraKin575 <https://github.com/EraKin575>, this issue is still in status
"Backlog", as not all the dependent issues (e.g. #7985
<#7985> (which should be done
soon)) are resolved yet. So @asr2003 <https://github.com/asr2003> didn't
have a chance to work on it yet.
—
Reply to this email directly, view it on GitHub
<#7976 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGXZB6GJH5EGZX2YDZ4XPHDZH3DCFAVCNFSM6AAAAABJB2H6HGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTGE2TINJQGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Yeah! Thanks, will fix soon. I have ongoing exams now. Will submit a PR
within a week :)
…On Mon, 24 Jun, 2024, 8:29 pm Christoph Stäbler, ***@***.***> wrote:
Hey @asr2003 <https://github.com/asr2003>,
This issue is ready to be worked on. You can also check on #8014
<#8014>, how we solved this for
the Channel (particularly on b7ff773
<b7ff773>
and 7891138
<7891138>
).
Please let us know if you have any questions :)
—
Reply to this email directly, view it on GitHub
<#7976 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGXZB6FS2FKROCWQQKZBXZLZJAX5FAVCNFSM6AAAAABJB2H6HGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBWG44DKNJSGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hello @asr2003, |
@creydr Will unassign this and will reassign if any other contributor not pickup the issue till my exams are done |
As the Eventing Authorization feature track describes, target resources of
EventPolicies
, will reflect in their status, whichEventPolicy
is applied to them.For example:
As the mt-broker should support
EventPolicies
, wee need to do the following:.status.policies
and set the BrokersEventPoliciesReady
condition. Also watch EventPolicies for changes.EventPoliciesReady
condition to true, in case no policy applies, and setmessage
according to thedefault-authorization-mode
(e.g.Default authorization mode is 'allow-all'
). Check the Default Authorization Mode section in the feature track for additional information.Hint:
AppliedEventPoliciesStatus
to the Broker is done in AddAppliedEventPoliciesStatus
type #7975Prerequisites:
AppliedEventPoliciesStatus
type #7975Additional hints for new contributors before starting with this issue:
Draft
status, the issue is subject to change and thus should not be started to be worked on/assign
). Please be aware that we might unassign you, if we don't see any progress from your side to give other contributors also a chance to work on this issue.The text was updated successfully, but these errors were encountered: