search

ovn-org / ovn-kubernetes

A robust Kubernetes networking platform
https://ovn-kubernetes.io/
Apache License 2.0
767 stars 333 forks source link

Reformatted AdminNetworkPolicy to align with new feature format #4469

Closed dtzhou2 closed 1 day ago

dtzhou2 commented 4 days ago

…d added additional examples to AdminNetworkPolicy moved baseline admin network policy to seperate section.

What this PR does and why is it needed

This PR enhances the documentation for AdminNetworkPolicy, bringing it in-line with the new feature documentation template. It also separates out AdminNetworkPolicy and BaselineAdminNetworkPolicy, as I believe they are separate features that can be documented separately.

Which issue(s) this PR fixes

Fixes #

Special notes for reviewers

How to verify it

Details to documentation updates

Added many more examples and user facing documentation in AdminNetworkPolicy, bringing it in line with the new feature documentation template. Clarified the motivation and introduction behind the feature as well as moving around some of the developer focused documentation. Broke up the AdminNetworkPolicy and BaselineAdminNetworkPolicy sections into two separate pieces of documentation, BaselineAdminNetworkPolicy will be reformatted in another pull request.

Description for the changelog

Does this PR introduce a user-facing change?

tssurya commented 4 days ago

thanks for the PR! But please see https://github.com/ovn-org/ovn-kubernetes/pull/4431 which is already taking a stab at this (I think this PR should be closed in favour of that which has had a couple of round of reviews and I plan to finish addressing remaining comments soon)

tssurya commented 1 day ago

closing this as spoken offline