search

cilium / design-cfps

Repo to store Cilium CFP design docs
Apache License 2.0
29 stars 25 forks source link

CFP-26438: Network Policy Correlation #18

Closed hacktivist123 closed 1 month ago

hacktivist123 commented 7 months ago

This PR stores CFP-26438 in Git to preserve it for future reference.

Author of the CFP: @sypakine

xmulligan commented 1 month ago

@sypakine are you still working on this CFP?

sypakine commented 1 month ago

@sypakine are you still working on this CFP?

This was implemented by @chancez in https://github.com/cilium/cilium/pull/27854.

I actually just merged a PR to bring the implementation closer to that defined in the CFP. At this point however, I feel that the CFP is redundant/obsolete.

xmulligan commented 1 month ago

We recently introduced statuses for CFPs https://github.com/cilium/design-cfps#status. It looks like it would fall under Released now. Would you mind making any minor updates that you think are missing so that we can merge this into the repo to document it?

sypakine commented 1 month ago

Sure thing, i'll work on that early next week if that's OK.

xmulligan commented 1 month ago

Perfect, thank you!

sypakine commented 1 month ago

created a new PR for this (hope that's OK): https://github.com/cilium/design-cfps/pull/51