Skip to content
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

docs: table update #1172

Merged
merged 1 commit into from
Mar 11, 2023
Merged

docs: table update #1172

merged 1 commit into from
Mar 11, 2023

Conversation

karl-cardenas-coding
Copy link
Contributor

Describe the Change

This PR updates the architecture table by adding outbound 443 for workload clusters.

Review Changes

💻 Add Preview URL

🎫 Jira Ticket

@spectro-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: karl-cardenas-coding
To complete the pull request process, please assign
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@@ -33,15 +33,17 @@ The following ports must be reachable from a network perspective for Palette to

|Port |Direction|Purpose |
|:---------------|:---------|:-----------------------|
|HTTPS (tcp/443) |INBOUND |Browser/API access to Management Platform|
|HTTPS (tcp/443) |INBOUND |Browser/API access to management platform|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'https' instead of 'HTTPS'.


## Workload Cluster


|Port |Direction | Purpose|
|:---------------|:---------|:--------------|
|HTTPS (tcp/443) |OUTBOUND | API access to management platform|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'https' instead of 'HTTPS'.

@@ -57,7 +59,7 @@ The following ports must be reachable from a network perspective for Palette to

|Port |Direction|Purpose |
|:---------------|:---------|:-----------------------|
|HTTPS (tcp/443) |INBOUND |Browser/API access to Management Platform|
|HTTPS (tcp/443) |INBOUND |Browser/API access to management platform|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'https' instead of 'HTTPS'.

@@ -69,5 +71,6 @@ The following ports must be reachable from a network perspective for Palette to

|Port |Direction | Purpose|
|:---------------|:---------|:--------------|
|NATS (tcp/4222) |OUTBOUND |Agent communication via Message Bus |
|HTTPS (tcp/443) |OUTBOUND | API access to management platform|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'https' instead of 'HTTPS'.

@netlify
Copy link

netlify bot commented Mar 11, 2023

Deploy Preview for docs-spectrocloud ready!

Name Link
🔨 Latest commit 31d4d75
🔍 Latest deploy log https://app.netlify.com/sites/docs-spectrocloud/deploys/640bc7b751694100087345d0
😎 Deploy Preview https://deploy-preview-1172--docs-spectrocloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants