[release/1.7] Add support for HPC port forwarding#10008
[release/1.7] Add support for HPC port forwarding#10008fuweid merged 1 commit intocontainerd:release/1.7from
Conversation
|
@cpuguy83 @MikeZappa87 @fuweid @kevpar could you please take a look when you have some time please? Its a partial backport of the port forwarding changes that were checked into the containerd/main branch yesterday. Thanks! |
kiashok
changed the title
kiashok
force-pushed
the
supportHPC-portFwd-1.7
branch
from
ba893ef to
4d648f9
Compare
kiashok
force-pushed
the
supportHPC-portFwd-1.7
branch
from
4d648f9 to
4e00d82
Compare
|
We do not backport a new feature unless there is an exceptional reason |
@AkihiroSuda We get constant requests from customers to fix this. Unfortunately, we missed to check in port forwarding support for HPC therefore we need to fix this in 1.7. What is the process to get this approved? |
@AkihiroSuda is this ok to have? |
|
cc @margichen |
If this is a bug fix, yes |
|
@fuweid @MikeZappa87 could you please take a look? Thanks! |
kiashok
force-pushed
the
supportHPC-portFwd-1.7
branch
from
4e00d82 to
e991643
Compare
kiashok
force-pushed
the
supportHPC-portFwd-1.7
branch
from
e991643 to
f649e51
Compare
Signed-off-by: Kirtana Ashok <[email protected]>
kiashok
force-pushed
the
supportHPC-portFwd-1.7
branch
from
f649e51 to
3df5d44
Compare
|
@cpuguy83 @MikeZappa87 could you please take a look? :) Thanks! |
7 participants
This PR is a partial change from commit b97ef91fb7212f512487fc1e28106050ad2f5766
This change only adds support for port forwarding on Windows host process containers by establishing connection to localhost at the requested port since HPCs share the network namespace of the host.
This PR does not change any existing behavior for process isolated or hyperV containers since we'd like to limit changes on a release branch.