Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 370922 - Create continuous integration for search console
Summary: Create continuous integration for search console
Status: CLOSED WONTFIX
Alias: None
Product: e4
Classification: Eclipse Project
Component: Search (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
Depends on: 370707
Blocks:
  Show dependency tree
 
Reported: 2012-02-08 04:21 EST by Dimitar Georgiev CLA
Modified: 2019-09-04 03:12 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitar Georgiev CLA 2012-02-08 04:21:12 EST
The current state of the Search Console continuous integration is suboptimal.

- Committers of the component are unfamiliar with how the e4 build works, and therefore do not know how to ensure it isn't broken by commits in search;
- Breaking of the build usually occurs outside European working hours, and we are unable to react in a timely manner
- the build framework enforces that we make undesired changes to our code, as per Bug 370631
- if we want to make enhancements of the build specific to Search, e.g. Findbugs/Emma/Architecture 101, etc, we have to rely on platform/ui releng to provide those

It is therefore worth discussing whether to set up dedicated CI of Search Console on hudson.eclipse.org.

- It would be based on Maven and Tycho, which we have the expertise to maintain;
- Tycho is known to handle the cyclic dependencies problem described in Bug 370631
- The whole build would be trigerred on each submit in the development branch, and we would have notification in case of failure;
- Search console would still be built (without its tests being run) by the e4 build, in order to have Search Console in the e4 updatesite.
 
Ideally, the development can happen in a development branch, e.g. "dev", and successful build would automatically merge changes in "master", which is currently autotagged in e4 build. This way, only stable content would end up on e4 update site.
This is prio 2, however.

Please comment whether you see any problems with such an approach?

Regards, Dimitar
Comment 1 Paul Webster CLA 2012-02-08 09:10:07 EST
re: maven and tycho.  If you want to maintain a maven/tycho build there's no problem with that, p2 does the same thing (on the side).  Other projects are using maven 3.0.3 and tycho 0.13 (and the soon to be released 0.14).  They still must consume their 3rd party deps from Orbit, but tycho doesn't mind.  There are problems, though, around qualifiers (which tycho can't read from maps and aren't supposed to change if your plugin hasn't changed).  I don't know how you are going to overcome that.

We can also get a job set up on Hudson, although it won't support auto-tagging (hudson can't do that).

But this seems to be a workaround for a problem that doesn't exist (the reasons to use fragments for tests are not valid within the Eclipse Project).

It would be nice to be able to run the e4build with your maven poms as well, but your maven build would need to be able to:

1) build a p2 repo and sign it when run as the correct user (e4Build) on build.eclipse.org, as mentioned in http://wiki.eclipse.org/Minerva#Signing

2) find the correct version of 4.2 to run against

3) provision the tests correctly in the 4.2 eclipse and run the tests

This still won't remove the requirement that to graduate for use in the Eclipse SDK that Search Console needs the tests  to be built by PDE build and run by the automated test framework, and the bugs that are standing in the way you would have to fix yourself.

PW
Comment 2 Dimitar Georgiev CLA 2012-02-09 03:51:37 EST
As Paul has pointed out, our code must be compatible with PDE build anyways - Dani is working on that at the moment.
This is a side activity, so I am marking it as an enhancement.
Comment 3 Dani Megert CLA 2012-02-09 03:55:36 EST
(In reply to comment #2)
> As Paul has pointed out, our code must be compatible with PDE build anyways -
> Dani is working on that at the moment.

You don't refer to me here, do you?
Comment 4 Dimitar Georgiev CLA 2012-02-09 03:57:11 EST
No, 'Dani' reads 'Danail' here. Sorry for that :)
Comment 5 Lars Vogel CLA 2019-09-04 03:12:18 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it and remove the stalebug whiteboard tag. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--