Bug 547568 - Adding an exposed port via Run Image wizard causes an error

- fix ImageRun.getDockerHostConfig() and getDockerContainerConfig()
  to add a slash between port number and port type when
  setting the configuration port fields

Change-Id: I1837368ac74c9b5074fdd83cb50b5538ff6849f8
Reviewed-on: https://git.eclipse.org/r/142635
Tested-by: Linux Tools Bot <linuxtools-bot@eclipse.org>
Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>

1 files changed, 6 insertions, 3 deletions

diff --git a/containers/org.eclipse.linuxtools.docker.ui/src/org/eclipse/linuxtools/internal/docker/ui/wizards/ImageRun.java b/containers/org.eclipse.linuxtools.docker.ui/src/org/eclipse/linuxtools/internal/docker/ui/wizards/ImageRun.java
index aff0af4370..dd883139eb 100644
--- a/containers/org.eclipse.linuxtools.docker.ui/src/org/eclipse/linuxtools/internal/docker/ui/wizards/ImageRun.java
+++ b/containers/org.eclipse.linuxtools.docker.ui/src/org/eclipse/linuxtools/internal/docker/ui/wizards/ImageRun.java
@@ -145,8 +145,10 @@ public class ImageRun extends Wizard {
 				}
 				final DockerPortBinding portBinding = new DockerPortBinding(exposedPort.getHostAddress(),
 						exposedPort.getHostPort());
-				portBindings.put(exposedPort.getContainerPort() + exposedPort.getPortType(),
-						Arrays.<IDockerPortBinding>asList(portBinding));
+				portBindings.put(
+						exposedPort.getContainerPort()
+								+ "/" + exposedPort.getPortType(), //$NON-NLS-1$
+						Arrays.<IDockerPortBinding> asList(portBinding));
 			}
 			hostConfigBuilder.portBindings(portBindings);
 		}
@@ -255,7 +257,8 @@ public class ImageRun extends Wizard {
 				if (!selectionModel.getSelectedPorts().contains(exposedPort)) {
 					continue;
 				}
-				exposedPorts.add(exposedPort.getContainerPort() + exposedPort.getPortType());
+				exposedPorts.add(exposedPort.getContainerPort()
+						+ "/" + exposedPort.getPortType()); //$NON-NLS-1$
 			}
 			config.exposedPorts(exposedPorts);
 		}