Problem/Motivation

We are noticing automated tests being unable to run due to composer errors. The errors started occurring some time in the start of 2023. The errors were encountered even when tests were run without any code changes, the same tests that were passing earlier.

https://www.drupal.org/project/lingotek/issues/3261217

https://www.drupal.org/project/lingotek/issues/3338717

Steps to reproduce

Run the failing automated tests here

https://www.drupal.org/project/lingotek/issues/3261217
https://www.drupal.org/project/lingotek/issues/3338717

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

jahanzaib_at_straker created an issue. See original summary.

jahanzaib_at_straker’s picture

testing this patch to address the composer error noticed in the dispatcher.

npaudyal001’s picture

StatusFileSize
new778 bytes

Testing a new fix

npaudyal001’s picture

StatusFileSize
new1003 bytes
npaudyal001’s picture

StatusFileSize
new969 bytes

Saw no effect on composure.json changes. Testing the drupalci.yml commands.

npaudyal001’s picture

StatusFileSize
new1.34 KB

  • npaudyal001 authored 1d648d6d on 3.9.x
    Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...
npaudyal001’s picture

StatusFileSize
new408 bytes

  • npaudyal001 authored 29678c47 on 3.9.x
    Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...

  • npaudyal001 authored 6e506ee4 on 3.9.x
    Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...

  • npaudyal001 authored 08f488ca on 3.9.x
    Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...
damienmckenna’s picture

Status: Active » Needs work

Did you mean to commit the changes to the 3.9.x branch and not the 4.0.x branch?

jeromeroberts’s picture

StatusFileSize
new2.09 KB

Attempting a patch to update the drupalci build file. - start with 3.9.x as not 100% sure this will work.
During investigation of Nick's patch to set allow-plugin true for cweagans, it was noted that the build was failing before even attempting to set allow-plugin true.
I've found some references that suggest we may be able to add the entire build file with unset default values which will enable us to move Nick's commands further up the build file and thus running within the assemble_codebase step before composer install.
Creating this build file on my local drupalci instance resolved the cweagan error.
I found once the testing commenced, there were some failing tests though - likely unrelated.

jeromeroberts’s picture

Yeah - so cannot patch the drupalci.yml file. Because Jenkins loads this file before the patch is applied and therefor not honoured by the Jenkins test.
To test this change, you'd need to apply the changes directly to repo branch rather than a patch. Rollback/reset the commit if it does not work.

  • jeromeroberts authored 850f9697 on 3.9.x
    Issue #3344857 by npaudyal001, jahanzaib_at_straker, jeromeroberts:...
jeromeroberts’s picture

StatusFileSize
new2.62 KB

Trying a different approach to last time.
cweagans is included via the acquia/cohesion package. This change instead attempts to allow-plugin for for cweagans and then install the dev/require-dev package after.
If this works, it would mean acquia/cohesion composer dev dependency will need to be managed through drupalci.yml rather than composer.json.
Just try 3.9.x branch first.

damienmckenna’s picture

All current work is going into the 4.0.x branch, shouldn't you focus your efforts there instead of 3.9.

jeromeroberts’s picture

StatusFileSize
new1.16 KB

Okay - now we know that moved things along. Here's the same patch changes for the latest 4.0.x branch.

  • jeromeroberts authored 721fd050 on 4.0.x
    Issue #3344857 by npaudyal001, jeromeroberts, jahanzaib_at_straker:...
npaudyal001’s picture

@jeromeroberts, With the patch it seems to build fine, but after the commit tests on other patches are throwing composer config error. I am reverting this commit to test if the error is coming from here.

  • jeromeroberts authored 1bf20847 on 4.0.x
    Issue #3344857 by npaudyal001, jeromeroberts, jahanzaib_at_straker:...
npaudyal001’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.