Skip to content

Conversation

@rbtr
Copy link
Collaborator

@rbtr rbtr commented Jul 28, 2023

Reason for Change:

Issue Fixed:

Requirements:

Notes:

@rbtr rbtr force-pushed the design/cns-swift-v2 branch 2 times, most recently from 2e1521d to 18d8e3d Compare July 28, 2023 19:28
@rbtr rbtr marked this pull request as ready for review August 1, 2023 17:55
@rbtr rbtr requested a review from a team as a code owner August 1, 2023 17:55
rbtr added 2 commits August 4, 2023 21:11
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the design/cns-swift-v2 branch from c3d72b6 to 2c754b8 Compare August 4, 2023 21:11
@rbtr rbtr self-assigned this Aug 4, 2023
@rbtr rbtr added cns Related to CNS. docs Documentation only labels Aug 4, 2023
@rbtr rbtr enabled auto-merge (squash) August 4, 2023 21:12
Copy link
Member

@nddq nddq left a comment

Choose a reason for hiding this comment

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

lgtm

The control boundaries for CNS are its CRDs and IPAM API which the CNI communicates over.
CRDs are:
- PodNetwork (represents a subnet delegation/registration)
- PodNetworkInstance (represents a reservation set)
Copy link
Collaborator

Choose a reason for hiding this comment

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

name is probably changing but not super important

@rbtr rbtr disabled auto-merge August 8, 2023 19:37
@rbtr rbtr merged commit 764d89e into Azure:master Aug 8, 2023
@rbtr rbtr deleted the design/cns-swift-v2 branch August 8, 2023 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cns Related to CNS. docs Documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants