tree c57f11ca17a2b46b0e8ae8d6b4f4375e319cfcfb
parent 4c0b35020775896c1eddf548490b6f5b7404a69f
author Jeff Johnston <jjohnstn@redhat.com> 1590009913 -0400
committer Jeff Johnston <jjohnstn@redhat.com> 1590013596 -0400

Bug 563412 - Port Settings inconsistently saved/restored for Docker Run

- add new IRunDockerImageLaunchConfigurationConstants.UNUSED_PORTS
- modify ImageRun.getDockerHostConfig() to take a list argument
  that will be modified to contain all unselected ports
- fix ExposedPortModel.fromString() method to not include the
  slash separator as part of the type
- modify LaunchConfigurationUtils.createRunImageLaunchConfiguration
  method to accept new unused ports list argument
- modify RunDockerImageLaunchConfigurationDelegate to pass an
  empty unused ports list to RunImageCommandHandler.runImage()
- modify RunImageCommandHandler.execute() to use new API for
  getting host config and setting unused ports list
- modify RunImagePortsTab.initializeFrom() method to get unused ports
  from launch configuration
- fix Docker ui tests to pass empty unused ports list
- remove checkAllElements() in ImageRunSelectionPage and let
  selection be determined dynamically from settings
- fix ImageRunSelectionPage.setDefaultValues() to restore
  unused ports

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