search

ec-europa / joinup-dev

The Joinup project moved to https://git.fpfis.eu/ec-europa/digit-joinup-reference
https://joinup.ec.europa.eu/
European Union Public License 1.2
42 stars 21 forks source link

ISAICP-6722: Retest using previously green commit #2644

Closed pfrenssen closed 2 years ago

pfrenssen commented 3 years ago

DO NOT MERGE

This is a retest of the first commit of #2639. This was green last Friday, but all subsequent builds started failing, both on branches that include this patch and that do not.

It was expected that the main develop branch would start failing after the upstream PR https://github.com/openeuropa/oe_webtools/pull/111 was updated, but the first commit in #2639 was supposed to fix this. Initially it appeared to be successful, but as soon as the latest changes from develop were merged in there it started failing, and ever since that moment all builds are failing, both on CPHP and on the new infrastructure.

Here is the green build for this commit, showing that the patch worked the first time it was tested: https://app.continuousphp.com/git-hub/ec-europa/joinup-dev/build/c7082586-e55e-45cd-b620-386b84f8d64b

Possibly the script is now blocked by the Webtools firewall, but it seems unlikely that this would happen on both test infrastructures at the same time.

pfrenssen commented 3 years ago

Launched test manually on CPHP so that it doesn't fail because of the merge conflict: https://app.continuousphp.com/git-hub/ec-europa/joinup-dev/build/cf2d33ac-80e8-4e4d-a553-b2eb17345d86