Bulk update Drupal core 7.x issues after Drupal 7 EOL

Created on 7 January 2025, 2 months ago

Problem/Motivation

Drupal 7 security and bugfix support has ended as of 5 January 2025. Drupal core issues filed against Drupal 7 should be either closed or update to a different branch. If we want to close the issues, they may be done with such simple comment as:

Drupal 7 security and bugfix support has ended as of 5 January 2025. Read more at https://www.drupal.org/about/drupal-7/d7eol

A script used to do this is available in #2631820-25: [policy, then infra] Bulk update core issues for relevant minor versions β†’ .

Let's discuss!

πŸ“Œ Task
Status

Active

Component

Other

Created by

πŸ‡­πŸ‡ΊHungary GΓ‘bor Hojtsy Hungary

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @GΓ‘bor Hojtsy
  • πŸ‡³πŸ‡ΏNew Zealand quietone

    Yes, the simple comment is clear. What will be the status of the issue?

    However, there are D7 issues that are open and have a commit for Drupal 8+. Those issues need to be identified so that credit can be applied before they are closed. Perhaps this should be a child issue with a list of all the issues in the summary?

  • πŸ‡­πŸ‡ΊHungary GΓ‘bor Hojtsy Hungary

    Of the 21260 total open core issues, 4680 are against 7.x: https://www.drupal.org/project/issues/drupal?text=&status=Open&prioritie... β†’

    How would we go about figuring out which ones are backports? Do we mean the 66 issues that are explicitly marked "Patch (to be ported"? https://www.drupal.org/project/issues/drupal?text=&status=15&priorities=... β†’ (I think this is a good dynamic link, why do we need to manually list them in an issue?)

  • πŸ‡¬πŸ‡§United Kingdom catch

    For the status, I would probably go for 'closed (outdated)'.

    Many open Drupal 7 issues have been fixed in Drupal 10/11 in a different d.o issue, so it's not that we took an explicit decision to reject the issue, it's that it didn't get completed in Drupal 7 before the EOL. In some cases they'll reference the fix that went into 8+ but not always (like a lot of duplicates).

  • πŸ‡ΈπŸ‡°Slovakia poker10

    According to this Slack discussion https://drupal.slack.com/archives/C079NQPQUEN/p1734040568387179 @quietone somehow filtered the list of open D7 issues which already seems to have commit and a 'needs backport to D7' tag to "just" 113:

    Did some work on this and I've got 113 open issues that have a commit message. That is D7 issues tagged with 'needs backport to D7'.

    I think that it is a good idea to review at least these issues for missing credits and close them properly as Fixed. Maybe d.a. can help with a custom query to verify this number if needed? Otherwise I agree with closing all others with status Closed (outdated). They will be still indexed by search engines, so if someone runs into an issue which is still relevant for D11+, it can be reopened. But I expect most of them to be not relevant anymore.

  • πŸ‡³πŸ‡ΏNew Zealand quietone

    Should the search be for all open issues that have a commit? The 'patch to be ported' issues will be part of that group, but not all. For example, #534594: [meta] system info cache writing and registry rebuilding is broken without a full bootstrap β†’ , was committed and re-opened for tests and it had many status changes since then. And if so, would credit be applied up to the commit, but not after?

  • πŸ‡ΈπŸ‡°Slovakia poker10

    Should the search be for all open issues that have a commit?

    I think that ideally yes, but hard to say if that will be doable, without knowing the number of issues we will need to review/close manually in such case.

Production build 0.71.5 2024