Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 369828

Summary: Test failures on JRE 7
Product: [Eclipse Project] Platform Reporter: Dani Megert <daniel_megert>
Component: User AssistanceAssignee: Dani Megert <daniel_megert>
Status: RESOLVED WORKSFORME QA Contact:
Severity: normal    
Priority: P3 CC: cgold, daniel_megert, david_williams, kim.moir
Version: 4.2   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Shot at a possible fix
none
Revert previous change and add project specific settings
none
Clean up none

Description Dani Megert CLA 2012-01-26 11:33:20 EST
On JRE 7 we have 6 tests failing with:
Connection refused: connect
Comment 1 Dani Megert CLA 2012-01-26 11:36:05 EST
Created attachment 210130 [details]
Shot at a possible fix
Comment 2 Dani Megert CLA 2012-01-26 11:37:11 EST
At a first glance it doesn't look like the usual test order issue we've seen on JRE 7. A possible problem could be a timeout.

The attached patch increases the timeout for two of the tests.
Comment 3 Kim Moir CLA 2012-01-26 11:58:02 EST
Patch released to master and merged into integration.
Comment 4 Dani Megert CLA 2012-01-30 04:44:27 EST
> A possible problem could be a timeout.
> The attached patch increases the timeout for two of the tests.

This did not help. We'd have to debug this directly on the machine.


Chris any idea what could be wrong or what's the best approach to debug this?
Comment 5 Dani Megert CLA 2012-01-30 09:15:48 EST
Kim, did you change something on the test machine? The tests passed on the latest N-build:

http://download.eclipse.org/eclipse/downloads/drops/N20120129-2000/testresults/html/org.eclipse.ua.tests_win32.win32.x86_7.0.html
Comment 6 Kim Moir CLA 2012-01-30 10:12:54 EST
No.  However, the build machines were rebooted over the weekend, due to the power outage.
Comment 7 Chris Goldthorpe CLA 2012-01-30 14:39:15 EST
I don't have a good idea as to why those particular 7 tests would be failing.
Comment 8 Dani Megert CLA 2012-01-31 06:37:31 EST
(In reply to comment #5)
> Kim, did you change something on the test machine? The tests passed on the
> latest N-build:
> 
> http://download.eclipse.org/eclipse/downloads/drops/N20120129-2000/testresults/html/org.eclipse.ua.tests_win32.win32.x86_7.0.html

And failed again in N20120130-2000.

I don't see any other way than debugging this directly on the build machine.
Comment 9 Dani Megert CLA 2012-02-05 05:00:52 EST
(In reply to comment #8)
> (In reply to comment #5)
> > Kim, did you change something on the test machine? The tests passed on the
> > latest N-build:
> > 
> > http://download.eclipse.org/eclipse/downloads/drops/N20120129-2000/testresults/html/org.eclipse.ua.tests_win32.win32.x86_7.0.html
> 
> And failed again in N20120130-2000.
> 
> I don't see any other way than debugging this directly on the build machine.

Kim, is there a way to debug this on the build machine?
Comment 10 Dani Megert CLA 2012-02-05 05:01:55 EST
Created attachment 210546 [details]
Revert previous change and add project specific settings
Comment 11 Kim Moir CLA 2012-02-05 13:35:08 EST
patch released to master and integration
Comment 12 Dani Megert CLA 2012-02-06 03:53:16 EST
(In reply to comment #11)
> patch released to master and integration

It looks like applying patches is causing troubles for you every time ;-)
My patch contained changes to two files but only one ended up in 'master':

http://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=78942201284c85d2b95ee73cb689a76e91f0d567

Can you apply the next patch (also patches two files)? Thanks!
Comment 13 Dani Megert CLA 2012-02-06 03:54:23 EST
Created attachment 210555 [details]
Clean up
Comment 14 Dani Megert CLA 2012-02-06 03:55:01 EST
(In reply to comment #13)
> Created attachment 210555 [details] [diff]
> Fix

Note that just cleans up things. It's not "the" fix.
Comment 15 Kim Moir CLA 2012-02-07 08:48:52 EST
Okay, this has been released to master and integration.
Comment 16 Dani Megert CLA 2012-02-07 09:33:19 EST
(In reply to comment #15)
> Okay, this has been released to master and integration.

It looks like it's only in 'master' (last change in 'integration' is almost two days old):
http://git.eclipse.org/c/platform/eclipse.platform.ua.git/log/?h=integration
Comment 17 Dani Megert CLA 2012-02-07 09:44:00 EST
Kim, you still did not commit and push all files. The patch contains:

org.eclipse.ua.tests/.settings/org.eclipse.pde.prefs

I can't see this in 'master':

http://git.eclipse.org/c/platform/eclipse.platform.ua.git/tree/org.eclipse.ua.tests/.settings
Comment 18 Dani Megert CLA 2012-02-07 09:47:40 EST
You probably get hit by bug 345099.
Comment 19 Kim Moir CLA 2012-02-07 10:50:18 EST
okay I tried again
Comment 20 Dani Megert CLA 2012-02-09 03:54:24 EST
(In reply to comment #19)
> okay I tried again

Looks good now, thanks!

How do we proceed with the connection timeout failures?
Comment 21 Dani Megert CLA 2012-08-10 03:40:00 EDT
Looks like the Hudson install is now correct and all test pass.