Re: [keycloak-dev] All PR's failing

Requested Travis assistance as well [1]. There are now several timeouts caused by compilation of keycloak taking too long, e.g. [2] [1] https://travis-ci.community/t/travis-builds-fail-due-to-slowness/4571 [2] https://travis-ci.org/keycloak/keycloak/jobs/568226589 On Tue, Aug 6, 2019 at 9:56 AM Hynek Mlnarik <hmlnarik(a)redhat.com <mailto:hmlnarik@redhat.com>> wrote: Travis seems inconsistent re the times so better safe than sorry. Hence I've reopened Bruno's PR [1] for splitting adapter tests, but at the same time, I'd rather excluded authorization tests completely since they take most of the time of adapter tests, and only included those when appropriate [2]. However I'm not convinced that any of these PRs would help. I have just cancelled a build stuck in the very early stage of the build for almost 20 minutes (git clone, specifically "remote: Counting objects: 100% (14222/14222), done." before build) which points to infrastructure rather than testsuite issue. --Hynek [1] https://github.com/keycloak/keycloak/pull/6221 [2] https://github.com/keycloak/keycloak/pull/6222 On Mon, Aug 5, 2019 at 10:38 PM Marek Posolda <mposolda(a)redhat.com <mailto:mposolda@redhat.com>> wrote: On 05. 08. 19 19:13, Stan Silvert wrote: > On 8/5/2019 3:43 AM, Hynek Mlnarik wrote: >> The PRs are blocked by two issues: >> 1) Adapter tests sometimes take too much time: This should be >> addressed by (now merged) [1] >> 2) OpenShiftTokenReviewEndpointTest being unstable, addressed by [2] >> >> IIRC, we had issues with Travis caches [3, 4], but if there is time it >> might be worth checking whether the issues persist. >> >> Once [2] is merged, Travis should be stable again for rebased PRs. > Do we know when this is going to happen? PR is approved by Hynek and me, but travis didn't pass for the PR due the adapter tests exceed the 50 minutes limit. I am retrying one more time. I agree with what Bruno's suggested to split adapter tests into more groups. Or fix adapter tests to not require realm re-import after each test as that take significant amount of time. See [1]. But that will probably require some more changes and may be probably few days of work... [1] https://github.com/keycloak/keycloak/blob/master/testsuite/integration-ar... Marek > >> --Hynek >> >> [1] https://github.com/keycloak/keycloak/pull/6209 >> [2] https://github.com/keycloak/keycloak/pull/6207 >> [3] https://issues.jboss.org/browse/KEYCLOAK-5124 >> [4] https://issues.jboss.org/browse/KEYCLOAK-7285 >> >> On Sat, Aug 3, 2019 at 12:52 PM Bruno Oliveira <bruno(a)abstractj.org <mailto:bruno@abstractj.org> >> <mailto:bruno@abstractj.org <mailto:bruno@abstractj.org>>> wrote: >> >>      Before answer this e-mail I was doing some experiments and there are >>      some things that I identified we can do about this. >> >>      1. Enable cache again on Travis >> >>      That won't fix the issue, but may help >> >>      2. Split the adapter tests >> >>      I did some test with the following PR: >> https://github.com/keycloak/keycloak/pull/6221/files >> >>      Here's the output: >> https://travis-ci.org/keycloak/keycloak/builds/567252894 >> >>      3. Review the tests we run on Travis >> >>      I'm not so sure if everything that we have running on Travis today is >>      really required. >> >> >>      For now if others agree on item 2, I can polish my PR and work to >>      get it merged. >> >>      On 2019-08-02, Stan Silvert wrote: >>      > The adapter tests are timing out on all new PR's. >>      > >>      > Hynek, I'm told that you might be able to help? >>      > >>      > Stan >>      > >>      > _______________________________________________ >>      > keycloak-dev mailing list >>      > keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> <mailto:keycloak-dev@lists.jboss.org <mailto:keycloak-dev@lists.jboss.org>> >>      > https://lists.jboss.org/mailman/listinfo/keycloak-dev >> >>      -- >> >>      abstractj >> > _______________________________________________ > keycloak-dev mailing list > keycloak-dev(a)lists.jboss.org <mailto:keycloak-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/keycloak-dev