Bug 159522 - Add linux process subsystem through ssh
Summary: Add linux process subsystem through ssh
Status: CLOSED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 1.0   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: 2.0   Edit
Assignee: Martin Oberhuber CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2006-10-02 17:23 EDT by Yufen Kuo CLA
Modified: 2011-05-25 09:44 EDT (History)
0 users

See Also:


Attachments
org.eclipse.rse.subsystems.processes.ssh.linux plugin zip archive (16.44 KB, application/octet-stream)
2006-10-02 17:25 EDT, Yufen Kuo CLA
no flags Details
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive (16.45 KB, application/octet-stream)
2006-11-16 17:02 EST, Yufen Kuo CLA
no flags Details
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive (16.97 KB, application/octet-stream)
2006-11-16 17:09 EST, Yufen Kuo CLA
no flags Details
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive (16.96 KB, application/octet-stream)
2006-11-16 17:11 EST, Yufen Kuo CLA
no flags Details
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive (16.96 KB, application/octet-stream)
2006-11-16 17:17 EST, Yufen Kuo CLA
no flags Details
Processes subsystem for SSH (16.97 KB, application/zip)
2006-11-16 21:59 EST, David Dykstal CLA
no flags Details
updated code that uses IHostShell instead of Jsch ChannelExec (17.91 KB, application/octet-stream)
2006-11-17 20:53 EST, Yufen Kuo CLA
mober.at+eclipse: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yufen Kuo CLA 2006-10-02 17:23:31 EDT
The attached code addes linux process subsystem using ssh protocol
Comment 1 Yufen Kuo CLA 2006-10-02 17:25:01 EDT
Created attachment 51285 [details]
org.eclipse.rse.subsystems.processes.ssh.linux plugin zip archive

Legal Message: I, Yufen Kuo, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. {I am authorized by my employer to make this contribution under the EPL.}
Comment 2 David Dykstal CLA 2006-10-06 09:41:14 EDT
I have requested DSDP PMC approval for the contribution.
Comment 3 Martin Oberhuber CLA 2006-10-17 07:17:45 EDT
The PMC has approved, vote is recorded in http://dev.eclipse.org/mhonarc/lists/dsdp-pmc/msg00343.html

Dave please go forward, do an initial code review and create a Contribution Questionnaire.
Comment 4 Martin Oberhuber CLA 2006-10-19 03:37:31 EDT
Note that the deadline for all legal documentation for our release has already passed, so we will not be able to put this into RSE 1.0.

Please do start the Contribution Questionnaire process right away, though.
Comment 5 David Dykstal CLA 2006-11-06 21:36:16 EST
Patch was submitted past the IP review cutoff. Will integrate after R1.
Comment 6 Yufen Kuo CLA 2006-11-16 17:02:52 EST
Created attachment 54030 [details]
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive

fixed the copyright
Comment 7 Yufen Kuo CLA 2006-11-16 17:09:32 EST
Created attachment 54032 [details]
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive
Comment 8 Yufen Kuo CLA 2006-11-16 17:11:36 EST
Created attachment 54033 [details]
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive
Comment 9 Yufen Kuo CLA 2006-11-16 17:17:51 EST
Created attachment 54034 [details]
org.eclipse.rse.subsystems.processs.ssh.linux plugin archive
Comment 10 David Dykstal CLA 2006-11-16 21:59:22 EST
Created attachment 54050 [details]
Processes subsystem for SSH

added "and others" to IBM copyright statements.
Comment 11 David Dykstal CLA 2006-11-17 15:22:36 EST
Martin -- I'm reassigning this to you now that its been submitted to the EMO.
Comment 12 Yufen Kuo CLA 2006-11-17 20:53:08 EST
Created attachment 54117 [details]
updated code that uses IHostShell instead of Jsch ChannelExec

This attachment contains the updated code that uses IHostShell to execute the command instead of Jsch ChannelExec.

I tried to use HostShellProcessAdapter.waitFor() function to wait for comand finishes, but the hostShell.isActive() always return true for me.
Comment 13 Martin Oberhuber CLA 2006-11-23 10:23:08 EST
Dave - looks like you are handling this now.
When the ssh processes subsystem is committed, we should look at using the RemoteCommandOperation code for it (see bug 164959)
Comment 14 Martin Oberhuber CLA 2007-02-06 09:32:27 EST
This is now approved as per
https://dev.eclipse.org/ipzilla/show_bug.cgi?id=1060

Going to commit for M5.
I'm going to rename the plugin to
  org.eclipse.rse.subsystems.processes.shell.linux

since it's using any contributed IHostShell now and does not depend on ssh. I'm also going to rename the package to
  org.eclipse.rse.internal.subsystems.processes.shell.linux

and some elements to
  LinuxShellProcessResources.properties
  LinuxShellProcessAdapter.java
  LinuxShellProcessSubSystemConfiguration.java

since I don't think we want to make the implementation official API.

Comments regarding this plan?
Comment 15 Martin Oberhuber CLA 2007-02-21 19:17:55 EST
A first version has been integrated with TM I20070221-1831.

The org.eclipse.rse.subsystems.processes.shell.linux plugin is part of the Core feature. The Subsystem is registered against the "Linux" systemType by default.

Please try it out!
Comment 16 Kushal Munir CLA 2007-02-22 20:59:52 EST
Tried it with both dstore.shells and ssh.shells, and they're both looking good. Both are a bit slow compared to using dstore.processes, with ssh.shells a bit slower than dstore.shells.
Comment 17 Martin Oberhuber CLA 2007-02-23 11:32:00 EST
Verified also with I20070223-0730 on Linux.
This works awsome now, with both dstore and (more important!) ssh.

I can now see the processes on build.eclipse.org through ssh!
It's painfully slow (bug 175300), but it works.
Even killing works.

And what's more, extenders should be easily able to run this over telnet, rlogin or whatever protocol, and use whatever commandline commands they have to get process info.
Comment 18 Martin Oberhuber CLA 2008-08-13 13:16:40 EDT
[target cleanup] 2.0 M5 was the original target milestone for this bug