Hi Marcus, On 5 March 2018 at 09:23, Marcus Denker <[email protected]> wrote: > > >> On 5 Mar 2018, at 09:16, Alistair Grant <[email protected]> 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
