Repository navigation
Conversation
|
@roosterfish @elijahgreenstein a small docs addition, could you take a look when you have a moment? |
elijahgreenstein
left a comment
There was a problem hiding this comment.
@wak327 thank you for this docs update. A few thoughts on adding some cross references to relevant docs pages.
ffd8b5e to
ce26092
Compare
|
@roosterfish I addressed @elijahgreenstein's review: the ports section now links to the MicroCeph and MicroOVN docs for their additional ports, and the Ceph dashboard note points to the Ceph documentation instead of giving a command. Could you take a look? |
ce26092 to
09f027c
Compare
|
@roosterfish @elijahgreenstein I applied the latest suggestions. Could you take another look? |
roosterfish
left a comment
There was a problem hiding this comment.
Thanks for your contribution, please see my comments.
09f027c to
2277a7d
Compare
|
@roosterfish thanks for the review, I've addressed your comments. Could you take another look? |
Signed-off-by: Waleed Khalid <wak327@gmail.com>
2277a7d to
d69a156
Compare
|
@elijahgreenstein @roosterfish I applied the latest suggestions. Could you take another look? |
Fixes #1433
The requirements reference doesn't say which ports MicroCloud uses, so users can run into port conflicts, for example with the Ceph dashboard, without knowing why.
This adds a "Network ports" section under "Networking requirements" listing the ports from
service/service_handler.go:It also explains that the TCP ports must be reachable between all cluster members, while 9444 only needs to be reachable on the initiator when using multicast discovery. For the additional ports used by Ceph and OVN, it links to the MicroCeph and MicroOVN documentation.
Checklist