Problem/Motivation

Recipe packaging tests using composer commands have started to fail. Example: https://git.drupalcode.org/issue/drupal-3608303/-/jobs/10722905.

From investigation, the failures can be reproduced locally by updating the composer executable to 2.10.2. The tests do not fail when composer executable version is changed back to 2.9.8.

There was an update to the CI images today, so the issue is likely related to that.

Steps to reproduce

Proposed resolution

There appears to be additional output from composer require commands captured from stdout that includes the line:
No security vulnerability advisories found.

This goes away if the env variable COMPOSER_NO_AUDIT is set to 1.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3608308

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

godotislate created an issue. See original summary.

godotislate’s picture

godotislate
he/him
commented
Issue summary: View changes
godotislate’s picture

godotislate
he/him
commented
Issue summary: View changes

godotislate opened merge request !16214

godotislate’s picture

godotislate
he/him
commented
Priority: Normal » Critical

Bumping to critical because this is blocking builds.

godotislate’s picture

godotislate
he/him
commented
Status: Active » Needs review

MR: https://git.drupalcode.org/project/drupal/-/merge_requests/16214

The PHPUnit Build job is now passing. There's a ignorable warning in the PHPUnit Unit job for PHP 8.6, but that doesn't need to be dealt with here.

macsim’s picture

macsim commented

Adding the NRQI tag but to me it's RTBC
The patch is simple and PHPUnit Build job is now green

macsim’s picture

macsim commented
Issue tags: +Needs Review Queue Initiative

... forgot the tag ...

godotislate’s picture

godotislate
he/him
commented
Issue summary: View changes

  • catch committed 79dbc1b6 on 11.4.x 
    task: #3608308 Composer 2.10 causing Drupal recipe unpacking tests to...

  • catch committed 2d664b39 on main 
    task: #3608308 Composer 2.10 causing Drupal recipe unpacking tests to...

  • catch committed 210f3a57 on 11.x 
    task: #3608308 Composer 2.10 causing Drupal recipe unpacking tests to...

  • catch committed cab80d59 on 11.3.x 
    task: #3608308 Composer 2.10 causing Drupal recipe unpacking tests to...
catch’s picture

catch
he/him
Primary language English
commented
Version: main » 11.3.x-dev
Status: Needs review » Fixed
Related issues: +#3491268: Look into skipping audit of composer operations in package manager tests

Committed/pushed to main, 11.x and 11.4.x and 11.3.x - from needs review since this had a +1 from macsim, it's trivial, and HEAD is broken.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

catch closed merge request !16214

mondrake’s picture

mondrake
Location 🇮🇹
commented

#6

There's a ignorable warning in the PHPUnit Unit job for PHP 8.6

It's due to PHP RFC: Deprecate returning values from __construct() and __destruct(), that was committed in https://github.com/php/php-src/pull/21982.

AFAICS it's an upstream issue for Prophecy, I filed https://github.com/phpspec/prophecy/issues/672

Status: Fixed » Closed (fixed)

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