Skip to content

Conversation

@zivnevo
Copy link
Collaborator

@zivnevo zivnevo commented Jul 25, 2024

This PR allows users to set Peer labels (attributes) from the clusterlink deploy peer CLI.
These labels propagate thru platform.Config, the Instance CR and the control-plane CLI, all the way to the authz manager, where they are added to the "source" attributes.

This is a first step towards solving issue #291 .

Delayed to phase 2:
Each Peer should keep track of attributes of other Peers (possibly by slightly abusing the health check mechanism).
This will allow adding Peer attributes to the "destination" attributes, so Peer attributes can be used in both from and to sections of AccessPolicies.

zivnevo added 2 commits July 25, 2024 12:50
Attributes are passed to the controlplane thru its cli,
and are added to the set of client attributes

Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
@zivnevo zivnevo added the policies Issues related to policies and policy engine label Jul 25, 2024
@zivnevo zivnevo marked this pull request as draft July 25, 2024 13:29
zivnevo added 3 commits July 25, 2024 17:52
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jul 29, 2024
zivnevo and others added 7 commits July 29, 2024 12:17
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
Signed-off-by: Ziv Nevo <nevo@il.ibm.com>
@zivnevo zivnevo marked this pull request as ready for review July 30, 2024 05:52
Copy link
Collaborator

@kfirtoledo kfirtoledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zivnevo zivnevo merged commit 6d46d56 into clusterlink-net:main Jul 31, 2024
@zivnevo zivnevo deleted the peer-attrs branch July 31, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

policies Issues related to policies and policy engine size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants