Refactor inspectContainer() and add commit(), removeImage() (eclipse-…

…che#4085)

Adds commit() and removeImage() implementations to
OpenShiftConnector. This requires some refactoring of existing
methods (mostly refactoring repeated processes into their own
methods -- e.g. creating an ImageStreamTag and getting image
info from a tag).

Additionally, refactors inspectContainer() method to remove
a call to DockerConnector, instead obtaining the same information
from what's available through the OpenShift API. This fixes an
issue where the IP address of a workspace was unavailable from
the ContainerInfo returned by DockerConnector.

Signed-off-by: Angel Misevski <amisevsk@redhat.com>

plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/DockerInstance.java

@@ -281,6 +281,7 @@ public void destroy() throws MachineException {
         try {
             docker.removeImage(RemoveImageParams.create(image).withForce(false));
         } catch (IOException ignore) {
+            LOG.error("IOException during destroy(). Ignoring.");
         }
     }