-
Bug
-
Resolution: Done
-
Major
-
None
-
None
workaround patch for client:
diff --git a/src/main/java/com/openshift/internal/client/ApplicationResource.java b/src/main/java/com/openshift/internal/client/ApplicationResource.java index 60b4bfe..aaebd09 100755 --- a/src/main/java/com/openshift/internal/client/ApplicationResource.java +++ b/src/main/java/com/openshift/internal/client/ApplicationResource.java @@ -616,14 +616,24 @@ public class ApplicationResource extends AbstractOpenShiftResource implements IA throw new OpenShiftSSHOperationException("SSH session for application \"{0}\" is closed or null. Cannot start port forwarding", this.getName()); } for (IApplicationPortForwarding port : ports) { - port.start(session); + try { + port.start(session); + } catch(OpenShiftSSHOperationException oss) { + // ignore for now + // should store this error on the forward to let user know why it could not start/stop + } } return ports; } public List<IApplicationPortForwarding> stopPortForwarding() throws OpenShiftSSHOperationException { for (IApplicationPortForwarding port : ports) { - port.stop(session); + try { + port.stop(session); + } catch(OpenShiftSSHOperationException oss) { + // ignore for now + // should store this error on the forward to let user know why it could not start/stop + } } // make sure port forwarding is stopped by closing session... session.disconnect();
patch in ui:
Index: plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/core/portforward/ApplicationPortForwardingWizardPage.java =================================================================== --- plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/core/portforward/ApplicationPortForwardingWizardPage.java (revision 40574) +++ plugins/org.jboss.tools.openshift.express.ui/src/org/jboss/tools/openshift/express/internal/core/portforward/ApplicationPortForwardingWizardPage.java (working copy) @@ -194,7 +194,7 @@ wizardModel.startPortForwarding(); refreshViewerInput(); } catch (Exception e) { - Logger.error("Failed to start port-forwarding", e); + return OpenShiftUIActivator.createErrorStatus("Failed to start port-forwarding.", e); } return Status.OK_STATUS; } @@ -227,7 +227,7 @@ wizardModel.stopPortForwarding(); refreshViewerInput(); } catch (Exception e) { - Logger.error("Failed to stop Port-forwarding", e); + return OpenShiftUIActivator.createErrorStatus("Failed to stop port-forwarding.", e); } return Status.OK_STATUS; }
- is duplicated by
-
JBIDE-11667 PortForward: When not using 127.0.0.1 I get hard error - should just bring the status into "error"
- Closed