-
Notifications
You must be signed in to change notification settings - Fork 60
NGF: Update architecture overview #450
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
base: ngf-feature-cp-dp-split
Are you sure you want to change the base?
NGF: Update architecture overview #450
Conversation
✅ All required contributors have signed the F5 CLA for this PR. Thank you! |
I have hereby read the F5 CLA and agree to its terms |
3e366c8
to
d41b315
Compare
8319e75
to
e95d5f2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like there might be some rebase/merging issues with other files included in the pr, but the gateway-architecture looks good to me! 🚀
Co-authored-by: Saylor Berman <s.berman@f5.com>
d41b315
to
c78fee7
Compare
Proposed changes
Problem: The Gateway architecture document does not reflect the new control plane/ data plane split
Solution: Update the architecture document to reflect the new architecture. This PR also converts the existing diagrams into mermaid from PNG files.
Closes nginx/nginx-gateway-fabric#3234
Checklist
Before merging a pull request, run through this checklist and mark each as complete.
README.md
andCHANGELOG.md
Footnotes
Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content. ↩