Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332383 - aggregator doesn't report low level parsing errors in email
Summary: aggregator doesn't report low level parsing errors in email
Status: CLOSED WONTFIX
Alias: None
Product: CBI
Classification: Technology
Component: CBI p2 Repository Aggregator (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: CBI Inbox CLA
QA Contact: David Williams CLA
URL:
Whiteboard:
Keywords:
: 329651 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-12-11 21:14 EST by David Williams CLA
Modified: 2023-09-01 10:05 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2010-12-11 21:14:49 EST
The aggregator reports most errors very well, either in buildlog and usually send eamil on the problem. There is a type of parse error though that does not get reported, the build ends with a "java -13 errror" but the root error can be found in the .log file in the workspace created during the build. 

The root error was the ep.b3aggcontr file had

<features name="org.eclipse.sdk.feature.group" versionRange="v20101118-7Q7xA7DPa2xdN86nbxlooGNr6aWgeOpuz-xq-WrZxV_3Q"/>

instead of 

<features name="org.eclipse.sdk.feature.group" versionRange="3.7.0.v20101118-7Q7xA7DPa2xdN86nbxlooGNr6aWgeOpuz-xq-WrZxV_3Q"/>


The .log file ends with this error message: 

!ENTRY org.eclipse.b3.util 4 -364274674 2010-12-10 17:31:52.341
!MESSAGE file:/shared/indigo/org.eclipse.indigo.build/ep.b3aggrcon: org.eclipse.emf.ecore.xmi.IllegalValueException: Value 'v20101118-7Q7xA7DPa2xdN86nbxlooGNr6aWgeOpuz-xq-WrZxV_3Q' is not legal. (file:/shared/indigo/org.eclipse.indigo.build/ep.b3aggrcon, 7, 124)

!ENTRY org.eclipse.b3.util 4 -364274674 2010-12-10 17:31:52.632
!MESSAGE Build failed! Exception was org.eclipse.core.runtime.CoreException: Not all contributions could be parsed
!STACK 1
org.eclipse.core.runtime.CoreException: Not all contributions could be parsed
        at org.eclipse.b3.util.ExceptionUtils.fromMessage(ExceptionUtils.java:53)
        at org.eclipse.b3.util.ExceptionUtils.fromMessage(ExceptionUtils.java:49)
        at org.eclipse.b3.aggregator.engine.Builder.verifyContributions(Builder.java:1112)
        at org.eclipse.b3.aggregator.engine.Builder.loadModel(Builder.java:940)
        at org.eclipse.b3.aggregator.engine.Builder.run(Builder.java:552)
        at org.eclipse.b3.aggregator.engine.Builder.run(Builder.java:822)
        at org.eclipse.b3.cli.AbstractCommand.run(AbstractCommand.java:119)
        at org.eclipse.b3.cli.Headless.run(Headless.java:130)
        at org.eclipse.b3.cli.Headless.start(Headless.java:197)
        at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
        at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110)
        at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79)
        at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:369)
        at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:79)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:618)
        at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:619)
        at org.eclipse.equinox.launcher.Main.basicRun(Main.java:574)
        at org.eclipse.equinox.launcher.Main.run(Main.java:1407)
!SUBENTRY 1 org.eclipse.b3.util 4 0 2010-12-10 17:31:52.633
!MESSAGE Not all contributions could be parsed



One build log that shows this case is (currently) at 

https://hudson.eclipse.org/hudson/view/Repository%20Aggregation/job/indigo.runAggregator/157/console 

and is fairly normal, except is just ends with 

BUILD FAILED
/opt/public/indigo/org.eclipse.indigo.tools/build.xml:197: exec returned: 13


It would be nice if these parse errors were at least logged in the usual buildlog, and ideally email sent to the contacts in the b3aggrcon file that could not be parsed.
Comment 1 David Williams CLA 2010-12-12 11:49:36 EST
I see now I was incorrect in saying the error message was not logged. 

Looking back today, I do see it in log: 

     [exec] 
     [exec] file:/shared/indigo/org.eclipse.indigo.build/ep.b3aggrcon: org.eclipse.emf.ecore.xmi.IllegalValueException: Value 'v20101118-7Q7xA7DPa2xdN86nbxlooGNr6aWgeOpuz-xq-WrZxV_3Q' is not legal. (file:/shared/indigo/org.eclipse.indigo.build/ep.b3aggrcon, 7, 124)
     [exec] Build failed! Exception was org.eclipse.core.runtime.CoreException: Not all contributions could be parsed
     [exec] Not all contributions could be parsed
     [exec] Eclipse:
     [exec] No exit data available.
     [exec] Eclipse:
     [exec] Java was started but returned exit code=13


I think I missed it because normally I look for error messages by searching for an messages wrapped in something like "email to: "

So, I remember to also look right before "exit" statement ... but, will leave this open, as I think an email should be sent to contacts in offending team. 

Changing to enhancement request. I suspect this is a whole different area of code than the normal email error reporting, and is not a true "bug in code", but a limitation with design. Or something.
Comment 2 Thomas Hallgren CLA 2010-12-12 16:12:02 EST
I think one of the reasons why no email is sent is that this happens very early when the model is parsed. Since the parsing fails, the email address is not yet known. I guess one way to rectify that would be to add a command line option for the top-level email.
Comment 3 Filip Hrbek CLA 2010-12-13 04:39:59 EST
(In reply to comment #2)
> I think one of the reasons why no email is sent is that this happens very early
> when the model is parsed. Since the parsing fails, the email address is not yet
> known. I guess one way to rectify that would be to add a command line option
> for the top-level email.

True, but perhaps we do have a chance to get the buildmaster's email if it's only a contribution (or contributions) what is invalid, in case it is a detached file. It means that the main build file is parseable and becomes invalid only when an attempt to resolve the proxy URI to the contribution is made.

Of course, if the contribution itself if invalid, we can't discover an email belonging to the contribution, but we can send an email to the buildmaster, telling him which contribution URI could not be resolved.
Comment 4 Thomas Hallgren CLA 2011-07-09 09:22:01 EDT
*** Bug 329651 has been marked as a duplicate of this bug. ***
Comment 5 David Williams CLA 2016-09-16 15:42:47 EDT
[Bookkeeping change only. Moving bugs to the new "home" of aggregator, CBI.]
Comment 6 Frederic Gurr CLA 2023-09-01 10:05:27 EDT
If this issue is still relevant, please move it to https://github.com/eclipse-cbi/p2repo-aggregator/issues.