Make APCu requirements errors precise and explanatory

Created on 27 April 2020, about 5 years ago
Updated 23 January 2023, over 2 years ago

On the status page, I'm seeing:

Errors found:
PHP APCu caching:
Enabled (96 MB)
Memory available: 8.96 MB.

Does this mean that there's only 8.96 MB left of the 96 MB that's enabled (which is too little), or that it's not possible to enable the full 96 MB because only 8.96 MB can be allocated?

If it's the former, there should be a message saying something like this:

Your APCu cache is too small; it needs to be made larger.

Otherwise:

You don't have enough RAM/storage/whatever to enable the full APCu cache you're enabled in your Web server settings.

So:

  1. It's not clear what the error means, and
  2. It's not clear how to fix it.

There's no helpful messaging here whatsoever.

📌 Task
Status

Needs review

Version

10.1

Component
Base 

Last updated about 13 hours ago

Created by

🇨🇦Canada colan Toronto 🇨🇦

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • Status changed to Needs work over 2 years ago
  • 🇺🇸United States smustgrave

    Changes caused a CI issue

  • 🇮🇳India pooja saraah Chennai

    Fixed failed commands on #19
    Attached patch against Drupal 10.1.x

  • 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
  • 🇺🇸United States smustgrave

    The change/work is being done in the MR switching to a patch just adds noise. So moving to NW to get the work in the MR

  • Status changed to Needs review over 2 years ago
  • Status changed to RTBC over 2 years ago
  • 🇺🇸United States smustgrave

    Thank you for updating the MR.

    Tested this manually and verified the message on the status report page.
    For testing I tweaked the numbers to make sure a new message appeared and confirmed that as well.

    Looks good to me

  • 🇺🇸United States xjm

    This is a definite improvement, thank you!

    Adding credit for @colan for the original report, @cilefen, @Chi, and @smustgrave for review and manual testing, @markdorison for work on the MR, @solideogloria for confirming this is still an issue at the reviewer's request, and @pooja saraah for helping fix the PHPCS error (but agreed that next time the change should be added to the MR directly.

    As this is a string and render array change, I committed it to 10.1.x only.

  • Status changed to Fixed over 2 years ago
    • xjm committed a8edc1b8 on 10.1.x
      Issue #3131358 by smustgrave, markdorison, pooja saraah, colan, cilefen...
  • Automatically closed - issue fixed for 2 weeks with no activity.

  • Status changed to Fixed about 2 years ago
  • 🇬🇧United Kingdom juc1

    I am getting this error in Drupal 10.

    What happens if the PHP APCu caching memory becomes exhausted - is it just the page loads slowly = not too bad, or somethine like white screen of death = very bad?
    .

  • 🇫🇷France papijo

    My main grievance is that this APCu message is categorised as an error. It is certainly not an error, as it does not prevent my Drupal site from functioning normally. At least it should be re-categorized as a warning, not an error.
    Plus of course, make the message more explanatory.

  • 🇫🇷France dqd London | N.Y.C | Paris | Hamburg | Berlin

    The last comment here is the reason I searched for an existing issue. My vote for "warning" instead of "error".

  • 🇺🇸United States xjm

    Bot ate my credit for committing this apparently. Fixing that.

    To those posting on the closed issue, you are better off filing new issues in the queue. People don't look at closed issues.

Production build 0.71.5 2024