Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351477 - When running a systemtap plugin to draw a graph it displays the graph wizard even if the connection fails.
Summary: When running a systemtap plugin to draw a graph it displays the graph wizard ...
Status: CLOSED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: Systemtap (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 minor (vote)
Target Milestone: 0.9.0   Edit
Assignee: anithra CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-07 13:17 EDT by Otavio Pontes CLA
Modified: 2012-07-04 11:48 EDT (History)
4 users (show)

See Also:


Attachments
Patch to fix the reported problem (3.59 KB, patch)
2011-07-07 13:19 EDT, Otavio Pontes CLA
no flags Details | Diff
Improved patch to fix the problem (3.43 KB, patch)
2011-07-07 13:23 EDT, Otavio Pontes CLA
anithra: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Otavio Pontes CLA 2011-07-07 13:17:35 EDT
Build Identifier: M20110210-1200

When I run a systemtap plugin with 'run w/Graph' option and the connection fails It displays the wizard to continue the process.

Reproducible: Always

Steps to Reproduce:
1. Open the Systemtap IDE
2. Load a stap script
3. Click in 'Run w/Chart' Button
4. A form to enter machine name, user and password is displayed
5. Fill the form with an incorrect machine name or user/password
6. A message saying that the connection failed will be displayed
7. The wizard is displayed and after filling it, it fails.
Comment 1 Otavio Pontes CLA 2011-07-07 13:19:19 EDT
Created attachment 199272 [details]
Patch to fix the reported problem

This patch aborts the running processes if the connection to the remote machine fails.
Comment 2 Otavio Pontes CLA 2011-07-07 13:23:16 EDT
Created attachment 199275 [details]
Improved patch to fix the problem

The patch I sent in previous comment has 2 incorrect lines. Sorry.
Comment 3 anithra CLA 2011-09-19 05:57:39 EDT
A small change in the patch - the continueRun is modified in  buildScript() so it needs to be checked again after buildScript() is invoked. Looks fine otherwise...

http://git.eclipse.org/c/linuxtools/org.eclipse.linuxtools.git/commit/?id=104be8caedb2f54a28a716d86803beb817101ea1
Comment 4 Wainer dos Santos Moschetta CLA 2011-09-19 10:34:20 EDT
(In reply to comment #3)
> A small change in the patch - the continueRun is modified in  buildScript() so
> it needs to be checked again after buildScript() is invoked. Looks fine
> otherwise...
> 
> http://git.eclipse.org/c/linuxtools/org.eclipse.linuxtools.git/commit/?id=104be8caedb2f54a28a716d86803beb817101ea1

Did you apply on Git master branch? Will it be available on Linux Tools 0.8.1?
Comment 5 Andrew Overholt CLA 2011-09-30 15:44:40 EDT
This was committed too late for 0.8.1 (the deadline we discussed on the mailing list was 1 September).

If you're interested in seeing whether or not something is on a particular branch in git, you can use the EGit history view and look for the "Branches" list.  I put up a screenshot here:

http://fedorapeople.org/~overholt/egit-historyview.png

HTH,

Andrew
Comment 6 Otavio Pontes CLA 2012-07-04 11:48:13 EDT
Verified in last release.