Skip to content
This repository was archived by the owner on Aug 18, 2025. It is now read-only.

Conversation

ericpaulsen
Copy link
Member

adding this chart of all ports related to Coder resources. we've recently sent this to a couple of customers who wanted to hardwire their own network policies, so I thought it'd be helpful to be in docs for future cases.

initially, I intended on creating a markdown table replica of the chart, but had a hard time nesting multiple rows in the larger parent columns... so I just included the screenshot for PR to get this started. if someone could help me make one or add it themselves, I think it would look a bit cleaner.

including @f0ssel & @jawnsy to validate and add context if necessary.

@ericpaulsen ericpaulsen added the docs 📝 Improvements or additions to documentation label Apr 20, 2021
@ericpaulsen ericpaulsen self-assigned this Apr 20, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Apr 20, 2021

✨ Coder.com for PR #284 deployed! It will be updated on every commit.

@khorne3
Copy link
Contributor

khorne3 commented Apr 20, 2021

@ericpaulsen Yeah, markdown tables aren't nice at all. I'll turn your screenshot into a table soon. Thanks!

Copy link
Contributor

@jawnsy jawnsy left a comment

Choose a reason for hiding this comment

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

seems good to me provided that @khorne3 converts the image into a table :)

@ericpaulsen
Copy link
Member Author

@ericpaulsen Yeah, markdown tables aren't nice at all. I'll turn your screenshot into a table soon. Thanks!

Awesome! I'll leave this PR open so you can push a commit with the table under this branch.

</tr>
<tr>
<td rowspan="4">cemanager</td>
<td>loadbalancer</td>
Copy link
Contributor

Choose a reason for hiding this comment

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

@f0ssel should we use LoadBalancer here?

Copy link
Contributor

Choose a reason for hiding this comment

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

I checked cdr/m -- it's super inconsistent, but we should capitalize it the way it appears to the user

@ericpaulsen ericpaulsen merged commit 122c415 into master Apr 21, 2021
@ericpaulsen ericpaulsen deleted the ericpaulsen/ports branch April 21, 2021 19:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs 📝 Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants