opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-10-25 18:25:59 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #16526 from cdoern/containerHost

Browse Source 
fix port issues for CONTAINER_HOST
This commit is contained in:
OpenShift Merge Robot
2022-11-17 17:47:12 +00:00
committed by GitHub
parent c67a5184dd 14ef6a91bd
commit 44a9014641
2 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
pkg/bindings/connection.go
View File

@ -91,9 +91,12 @@ func NewConnectionWithIdentity(ctx context.Context, uri string, identity string,
var connection Connection var connection Connection
switch _url.Scheme { switch _url.Scheme {
case "ssh": case "ssh":
port, err := strconv.Atoi(_url.Port()) port := 22
if err != nil { if _url.Port() != "" {
return nil, err port, err = strconv.Atoi(_url.Port())
if err != nil {
return nil, err
}
} }
conn, err := ssh.Dial(&ssh.ConnectionDialOptions{ conn, err := ssh.Dial(&ssh.ConnectionDialOptions{
Host: uri, Host: uri,

9
test/e2e/system_connection_test.go
View File

@ -281,6 +281,15 @@ var _ = Describe("podman system connection", func() {
_, err = Start(cmd, GinkgoWriter, GinkgoWriter) _, err = Start(cmd, GinkgoWriter, GinkgoWriter)
Expect(err).ToNot(HaveOccurred()) Expect(err).ToNot(HaveOccurred())
// export the container_host env var and try again
err = os.Setenv("CONTAINER_HOST", fmt.Sprintf("ssh://%s@localhost", u.Username))
Expect(err).ToNot(HaveOccurred())
defer os.Unsetenv("CONTAINER_HOST")
cmd = exec.Command(podmanTest.RemotePodmanBinary, "ps")
_, err = Start(cmd, GinkgoWriter, GinkgoWriter)
Expect(err).ToNot(HaveOccurred())
uri := url.URL{ uri := url.URL{
Scheme: "ssh", Scheme: "ssh",
User: url.User(u.Username), User: url.User(u.Username),