Hi Marcus, On 5 March 2018 at 09:23, Marcus Denker <marcus.denker@inria.fr> wrote:
On 5 Mar 2018, at 09:16, Alistair Grant <akgrant0710@gmail.com> wrote:
Hi Esteban & Marcus,
I'm getting repeated validation failures for:
https://pharo.manuscript.com/f/cases/21431
It's the same set of tests that fail each time, and as far as I can tell they have nothing to do with the patch I submitted.
Do you know if this is happening on other tests?
I saw that Saturday but decided to wait till Monday (weekends are important..).
So: no, I have *no* idea what happened. From one CI run to the next, suddenly around 160 tests related to Calypso started failing due to a missing method.
Now starting from sometime today, this problem stoped. The last failing PR checks fail due to different reasonsâ¦
And I have no idea why.
(And yes, we al know that 1) the PR checks need more compute power, too slow 2) we *need* to track down the reason why still *a lot* of times the PR fails even though it should not.
The problem is that just keeping a build alive of this kind is a full time job.. that we have nobody doing, so many many people do as much as they can and we hope it will get betterâ¦.)
Thanks for the update. I took a look at the failures and it appears that BehaviorTest>>testBehaviorRespectsPolymorphismWithTraitBehavior ClassDescriptionTest>>testClassDescriptionRespectsPolymorphismWithTraitDescription ClassTest>>testClassRespectsPolymorphismWithTrait are all failing due to changes in Fuel - methods were changed from traits to local methods. I'll try and track down when the change was introduced and by whom, but hopefully it is just a matter of updating the tests. Cheers, Alistair