From af9f83f11c9b92ea806b33b75337de7e5d93592d Mon Sep 17 00:00:00 2001
From: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
Date: Thu, 23 Aug 2018 14:50:38 +0900
Subject: exposes tcp port only if no proto specified.

Also it fix the issue of exposing both tc/udp port even if
only one proto specified.

Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>

Closes: #1325
Approved by: mheon
---
 pkg/spec/createconfig.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

(limited to 'pkg')

diff --git a/pkg/spec/createconfig.go b/pkg/spec/createconfig.go
index 6a7ddc1ae..41030bf26 100644
--- a/pkg/spec/createconfig.go
+++ b/pkg/spec/createconfig.go
@@ -492,10 +492,7 @@ func (c *CreateConfig) CreatePortBindings() ([]ocicni.PortMapping, error) {
 			}
 
 			pm.HostPort = int32(hostPort)
-			// CNI requires us to make both udp and tcp structs
-			pm.Protocol = "udp"
-			portBindings = append(portBindings, pm)
-			pm.Protocol = "tcp"
+			pm.Protocol = containerPb.Proto()
 			portBindings = append(portBindings, pm)
 		}
 	}
-- 
cgit v1.2.3-54-g00ecf