Problem/Motivation

After #3572280: Remove Migrate Drupal we have a lot less kernel tests to run.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3576180

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

catch created an issue. See original summary.

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Postponed

Postponed on #3572280: Remove Migrate Drupal

catch’s picture

catch
he/him
Primary language English
commented
Status: Postponed » Needs review

No longer postponed.

catch opened merge request !14964

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Needs review » Reviewed & tested by the community

LGTM

catch’s picture

catch
he/him
Primary language English
commented

I've been seeing a lot of very slow pipelines recently, but if you look at https://git.drupalcode.org/project/drupal/-/pipelines/758280/builds you can see that while the Kernel jobs take about 5 minutes, functional and functional js jobs take about 10 minutes, so this definitely doesn't turn kernel tests into a problem on the slower runs. On faster runs I'd hope it also doesn't turn them into a problem but we'd actually need to get one to find out.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

These two might help a bit with overall functional test times:

#3572055: Fix skip for unsupported methods on REST tests
#3575888: Combine HTTP method tests in EntityResourceTestBase

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 589e3dd712a to main. Thanks!

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.

  • longwave committed 589e3dd7 on main 
    test: #3576180 Tweak kernel test running

By: catch

longwave closed merge request !14964

Status: Fixed » Closed (fixed)

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