-
Notifications
You must be signed in to change notification settings - Fork 216
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
chore(docs): Add docs about architecture and fix structure #1229
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Able to run Build
Serve
|
|
||
### Plugin Lifecycle | ||
|
||
The Plugin Manager is in charge of starting up all of the plugins, and the Watcher Manager - which in turn starts the watchers. It can also reconcile plugins, which will regenerate the eBPF code and the BPF object. |
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.
Adding the watcher information here makes it confusing. Describe it in a later paragraph. Also, mention why watchers are required.
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.
cb835a6 I moved the watchers to a new paragraph and added some new detail. Please correct me if I am wrong. Admittedly I'm not super familiar with the watchers :)
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
Signed-off-by: Kamil <[email protected]>
|
||
Retina **collects customizable telemetry**, which can be exported to **multiple storage options** (such as Prometheus, Azure Monitor, etc.) and **visualized in a variety of ways** (like Grafana, Azure Log Analytics, etc.). | ||
|
||
![High Level Architecture](./img/Retina%20Arch.png "High Level Architecture") |
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.
this image doesn't load correctly, might be because of the space in the image's name might messing up the path or it's a Github issue
Description
The main goal of this PR was to document the Retina architecture, namely the data plane and the available control planes. While working on this, I also made some smaller changes to improve our consistency.
What this PR does:
Smaller changes
Architecture docs
.png
and the source for the diagram.excalidraw
are included so that the diagrams can be versioned.retina.sh
in terms of the feature descriptions. Also added a "What is Hubble?" subsection with a brief description.Related Issue
#1055
There is also another PR opened for Hubble installation - #1223
Checklist
git commit -S -s ...
). See this documentation on signing commits.Screenshots (if applicable) or Testing Completed
I think its a bit redundant to put the Architecture Diagrams in here, as the files themselves are included. So check those out directly. Or open up a preview of the markdown page itself in GitHub.
Retina favicon image
Example of new file structure - showcasing that the heading now opens rather than having a landing page when clicked on.
Parity of feature highlight between
retina.sh
and "What is Retina?" page.Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.