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

Bug 470627

Summary:	mirrors url not returning xml?
Product:	Community	Reporter:	David Williams <david_williams>
Component:	Servers	Assignee:	Eclipse Webmaster <webmaster>
Status:	RESOLVED FIXED	QA Contact:
Severity:	blocker
Priority:	P3	CC:	denis.roy, dennis.huebner, Ed.Merks
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Linux
See Also:	https://git.eclipse.org/r/50563 https://git.eclipse.org/c/www.eclipse.org/downloads.git/commit/?id=c4dae9663c6a46472e9b4da4b09f49f5021a6878
Whiteboard:

Description David Williams

2015-06-19 15:51:45 EDT

Please see the log at 

https://hudson.eclipse.org/simrel/job/simrel.mars.runaggregator.VALIDATE/420/consoleFull

I am not actually sure what the problem is, but off-hand appears to be that the format=xml is not being honored, and returning HTML? 

Perhaps it's a case when an empty list should be returned, but is returning some kind of "not found" HTML instead? 

For the aggregation build, I will try disabling mirror look up (I think that used to be automatic "no mirrors"  used when running in infrastructure .... right?

Comment 1 David Williams

2015-06-19 16:02:07 EDT

Dennis, and Ed, 

Perhaps you should look at the log too. It seems most of the errors involved "emf <something>"

And, the string for mirrors URL looks odd ... like part of it is repeated? ... is that your input? 

 http://www.eclipse.org/downloads/download.php?file=/modeling/emft/b3/updates-4.4/&protocol=http&format=xml&countryCode=us&timeZone=-5&format=xml

Comment 2 David Williams

2015-06-19 16:07:36 EDT

I think I should look too ... I think these are coming from the b3 aggregator repo itself! I'll check.

Comment 3 David Williams

2015-06-19 16:11:08 EDT

(In reply to David Williams from comment #2)
> I think I should look too ... I think these are coming from the b3
> aggregator repo itself! I'll check.

No, they appear correct in repo ... I think this is something infrastructure is adding, and resolving to the actual "section page" (instead of returning xml). 

    <property name='p2.mirrorsURL' value='http://www.eclipse.org/downloads/download.php?file=/modeling/emft/b3/updates-4.4/&amp;protocol=http&amp;format=xml'/>

Comment 4 Eclipse Genie

2015-06-19 16:13:02 EDT

New Gerrit change created: https://git.eclipse.org/r/50563

Comment 5 Denis Roy

2015-06-19 16:14:34 EDT

This is a regression.  We're eager to redirect you to a mirror if there are no mirrors.  While the mirror list looks fine from the outside world, it is empty for builds running on HIPP.

The attached patch should fix that.

Comment 6 Eclipse Genie

2015-06-19 16:15:14 EDT

Gerrit change https://git.eclipse.org/r/50563 was merged to [master].
Commit: http://git.eclipse.org/c/www.eclipse.org/downloads.git/commit/?id=c4dae9663c6a46472e9b4da4b09f49f5021a6878

Comment 7 Denis Roy

2015-06-19 16:26:04 EDT

This should be fixed now.  Can you re-run your job?

Comment 8 David Williams

2015-06-19 17:47:30 EDT

Yes, fixed now. I re-removed the eclipse.p2.mirrors=false, and no exceptions in the log. 

https://hudson.eclipse.org/simrel/job/simrel.mars.runaggregator.VALIDATE/423/consoleFull