-
Notifications
You must be signed in to change notification settings - Fork 37
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
gRPC diagrams #1712
gRPC diagrams #1712
Conversation
✅ Deploy Preview for docs-spectrocloud ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Just some small fixes to address - table headers need boldface, and I saw the convention in these tables is no period. I removed a couple of remaining periods for consistency in this doc.
@@ -182,12 +200,22 @@ The following ports must be reachable from a network perspective for Palette sel | |||
|NATS (tcp/4222) |OUTBOUND |Agent communication via message bus | | |||
|HTTPS (tcp/443) |OUTBOUND |vSphere vCenter API, Registry (packs, integrations), Pack containers, Application updates. |
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.
|HTTPS (tcp/443) |OUTBOUND |vSphere vCenter API, Registry (packs, integrations), Pack containers, Application updates. | |
|HTTPS (tcp/443) |OUTBOUND |vSphere vCenter API, Registry (packs, integrations), Pack containers, Application updates |
Co-authored-by: Rita Watson <[email protected]>
* docs: fixed missing grpc images * docs: updated tabs order * Apply suggestions from code review Co-authored-by: Rita Watson <[email protected]> --------- Co-authored-by: Rita Watson <[email protected]> (cherry picked from commit fe87664)
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
* docs: fixed missing grpc images * docs: updated tabs order * Apply suggestions from code review Co-authored-by: Rita Watson <[email protected]> --------- Co-authored-by: Rita Watson <[email protected]> (cherry picked from commit fe87664) Co-authored-by: Karl Cardenas <[email protected]>
* docs: fixed missing grpc images * docs: updated tabs order * Apply suggestions from code review Co-authored-by: Rita Watson <[email protected]> --------- Co-authored-by: Rita Watson <[email protected]>
Describe the Change
This PR adds the missing gRPC images reported in the #docs channel by @JPedro2. A quick fix to the release notes is also added. The architecture page's structure has also been changed. Instead of a single tab group that displays the gRPC or the NATS architecture. Each section now contains its own tab to help the reader follow along easier.
Review Changes
💻 Preview URL