-
Notifications
You must be signed in to change notification settings - Fork 2
Hub-LVol Port Adjustment in Port Tables #53
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
base: main
Are you sure you want to change the base?
Conversation
🤖: Deployment available as https://docstest.simplyblock.io/hub-lvol-ports |
bbbb9c1
to
c117b16
Compare
🤖: Deployment available as https://docstest.simplyblock.io/hub-lvol-ports |
🤖: Deployment available as https://docstest.simplyblock.io/hub-lvol-ports |
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.
Please find my inline comments.
@@ -51,7 +51,9 @@ demo@worker-1 ~> sudo modprobe nbd | |||
| bdts | ingress | storage | 4420 | TCP | | |||
| Cluster Control | ingress | control | 5000 | TCP | | |||
| spdk-http-proxy | ingress | storage, control | 8080 | TCP | | |||
| lvol-proxy | ingress | storage, control | 9090-9900 | TCP | | |||
| secondary forwarding | ingress | storage, control | 9060-9098 | TCP | |
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.
name should be "node nvmf port"
| lvol-proxy | ingress | storage, control | 9090-9900 | TCP | | ||
| secondary forwarding | ingress | storage, control | 9060-9098 | TCP | | ||
| hub-lvol | ingress | storage, control | 9099 | TCP | | ||
| lvol-proxy | ingress | storage, control | 9100-9900 | TCP | |
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.
could be:
lvol-nvmf-port
or just
lvol-nvmf
@@ -51,7 +51,9 @@ demo@worker-1 ~> sudo modprobe nbd | |||
| bdts | ingress | storage | 4420 | TCP | |
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.
"bdts"
this is not needed anymore, we can remove it
@@ -51,7 +51,9 @@ demo@worker-1 ~> sudo modprobe nbd | |||
| bdts | ingress | storage | 4420 | TCP | | |||
| Cluster Control | ingress | control | 5000 | TCP | |
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.
rename to:
storage node api
Source / Target Network:
storage, control
@@ -51,7 +51,9 @@ demo@worker-1 ~> sudo modprobe nbd | |||
| bdts | ingress | storage | 4420 | TCP | | |||
| Cluster Control | ingress | control | 5000 | TCP | | |||
| spdk-http-proxy | ingress | storage, control | 8080 | TCP | |
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 is actually a range: 8080-8180
🤖: Deployment available as https://docstest.simplyblock.io/hub-lvol-ports |
Updated storage node port table for hub-lvols