Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cross-project-issues-dev] Disabling Mylyn from SimRel + Removing related Orbit workarounds
Hi Alexander,

Sorry that you got caught in the crossfire on this change - the realization that Passage was depending (unintentionally) on a workaround I put in last year for Mylyn made me revisit to see if the cost of fixing Mylyn this way was having unintended consequences. After a year instead of the problem getting better, the problem has become worse.

I appreciate all the analysis done in this thread. One thing to note is this is not a project handbook issue, but simply a release train participation issue. While I am sure Wayne/EMO will facilitate where necessary the rules of engagement are documented in the wiki (out of date) https://wiki.eclipse.org/SimRel/Overview and guided by the Planning Council.  As for the Orbit rule - for 15 years (how old is SimRel?) Orbit did not participate directly. Orbit is not a "normal" project, and the closest reference to the rule I mention is in the FAQ[1] which says "This is analogous to the use of "third party" bundles from Orbit, where the original authors clearly do not "participate" in the release".

PS - It is not 1 day to resolve the issue, but 20 days until M2 and 54 days notification until RC2. If the consensus is that it is too tight, it may be best to introduce this kind of change at the beginning of a release cycle, which means I hold off merging until late March.

[1] https://wiki.eclipse.org/SimRel/Simultaneous_Release_Policy_FAQ

Jonah



~~~
Jonah Graham
Kichwa Coders
www.kichwacoders.com


On Fri, 14 Jan 2022 at 02:43, Alexander Fedorov <alexander.fedorov@xxxxxxxxxx> wrote:
Hi Jonah,

Since Passage was never depending from Mylyn it was surprising to see it disabled. As I understood, it was done because Passage does not mirror all the used Orbit bundles to its p2 site.
Please don't get me wrong, but the gap between notification [1] and action seems a bit tight to me: about 1 day without technical space to fix it.

Perhaps I missed something important during the past years but the rule to always mirror Orbit dependencies to the component p2 site was neither clearly articulated nor enforced previously.

To avoid future misunderstanding I've created a ticket [2] to improve Project Handbook regrading SimRel participation.
@Wayne, @Alexander please invest your time to polish the formulation of this [new?] constraint for Eclipse projects that are willing to participate SimRel.

Thank you,
AF

[1] https://git.eclipse.org/r/c/simrel/org.eclipse.simrel.build/+/189570
[2] https://gitlab.eclipse.org/eclipsefdn/emo-team/emo/-/issues/177

1/13/2022 11:20 PM, Jonah Graham пишет:


On Thu, 13 Jan 2022 at 08:47, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:
On Thu., Jan. 13, 2022, 08:18 Aleksandar Kurtakov, <akurtako@xxxxxxxxxx> wrote:
I would dare to say that as long as the workarounds are in simrel nothing will get fixed - it's time to face reality.
Probably correct, but I don't have the nerve to disable (or knowledge/time to fix) Mylyn. 

Hi folks,

It is time to remove the temporary workarounds. When I had a look today I realised that more and more projects are relying on the temporary workaround initially put in place for Mylyn.

Over a year ago I filed numerous bugs asking projects to fix their contributions, some projects were very responsive and others I have not heard back from.

Therefore for M2 I plan to disable all projects from SimRel that aren't up to date or have otherwise started relying on these workarounds. I will submit the following gerrits[1,2] after 2022-03 M1 is done. Please see the gerrits for what is disabled. I attempted to only disable features where possible and not entire contributions.

The affected projects are (with some comments):
- Mylyn (fully disabled Bug 569078)
- Passage (only one feature, so fully disabled)
- DTP (many features, lots because Lucene 7.x is no longer provided by Eclipse Platform? + Bug 569181)
- WTP (Bug 568136)
- m2e-wtp (JPA related code)
- PDT (Composer feature needs org.apache.commons.exec) 
- soa-bpel (depends on disabled WTP features)

[1] https://git.eclipse.org/r/c/simrel/org.eclipse.simrel.build/+/189612 - Remove the Orbit direct contribution to SimRel workaround

Jonah

~~~
Jonah Graham
Kichwa Coders
www.kichwacoders.com


_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev

_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev

Back to the top