- Issue created by @ktch_my
- First commit to issue fork.
- π³π±Netherlands spokje
The same happened for me.
memcache_admin_post_update_add_service_definitions
invalidated the container, as intended, and afterwards local/development/whatever environments with this module enabled whilst having neither Memcache nor Memcached installed fail with a 500Uncaught PHP Exception Drupal\memcache\MemcacheException: "No Memcache extension found"
. - Status changed to Needs review
4 months ago 9:37am 15 July 2024 - π³π±Netherlands spokje
The MR solved our use case (having
drupal/memcache
enabled, but neither Memcache nor Memcached installed), but is at best "iffy".Not even sure how this situation came to be, since it seems to be not possible to enabled this module without having either Memcache or Memcached installed (See also #3136253: Disable memcache on local or dev environments (Drupal 8) β ), but here we are.
A "proper" solution would include decent tests, probably(?) a dedicated setting and the possibility to install without having either Memcache or Memcached installed.
But foremost, we need a blessing from the maintainer(s) if they want to support such a solution.
Putting this on NR in the hope this will attract that attention.
- πΊπΈUnited States NicholasS
I had the same error one of our environments didn't have a good memecache compiled for its PHP.
BUT the patch did not fix it, so I think it needs work still, same error
- First commit to issue fork.
-
japerry β
committed 31091e16 on 8.x-2.x authored by
Spokje β
Issue #3460944: Local without memcache breaks after update
-
japerry β
committed 31091e16 on 8.x-2.x authored by
Spokje β
- Status changed to Fixed
4 months ago 3:29am 27 July 2024 - πΊπΈUnited States jrglasgow Idaho
Any chance getting a new release with this fix?
- πΊπΈUnited States jrglasgow Idaho
I was having the same issue and ended up tracking it down to this issue... I have a different symptom. After running
composer update
my `drush cr` failed with this message:$ drush cr
In Container.php line 147:
Circular reference detected for service "module_handler", path: "module_handler -> cache.bootstrap -> logger.factory -> logger.syslog -> config.factory -> config.typed".
Failed to run drush cr: exit status 1
the `drush updb` succeeded but had some
[error] MemcachedDriver::set() error
errors.I then put in a breakpoint where the exception was being thrown and traced it back to Memcache and reverted from 8.x-2.6 to 8.x-2.5 and the problem was resolved. Also updated to the latest dev release resolves the issue.
I am putting the specific errors I had in here to make it easier to find when searching.
- πΊπΈUnited States dhansen
Pushing for a full release of this fix. Our sites avoid dev versions as potentially unstable, and this is playing havoc with our testing pipeline and requiring significant workarounds for non-production environments.
- π¦πΊAustralia marknatividad
I am also having a similar circular reference issue after upgrading to Drupal 10.3.1 and downgrading memcache to 2.5.0.
After running
drush cr
I get the following error:In Container.php line 222:
Circular reference detected for service "cache.bootstrap", path: "module_handler -> cache.bootstrap -> cache.backend.memcache -> Drupal\Component\Datetime\TimeInterface -> state -> cache.bootstrap".
- Status changed to Needs work
3 months ago 4:24pm 20 August 2024 - πΊπΈUnited States japerry KVUO
Well shoot. Lets see if there are other reports and then re-escalate as a bug. I'll try to reproduce the issue, but it won't be until next week or early September though.
- π³π±Netherlands spokje
@paulsheldrake What are the symptoms of the issue on 2.7?
Any errors showing in the log, on screen, etc?
- πΊπΈUnited States dhansen
Circling back: the fix on 2.7 seems to work for me. Note that I did need to run drush updatedb which ran add_service_definitions from memcache_admin to clear an error that appeared.
- πΊπΈUnited States wells Seattle, WA
Hmmm same here. I swear this fix worked initially, but now I am seeing the same error as #18 again in 8.x-2.7.
- πΊπΈUnited States wells Seattle, WA
Ah, wait, I see -- in my local environment I have a config that is equivalent to:
$settings['memcache']['servers'] = []; $settings['memcache']['bins'] = [];
But, I did not have that in a remote test environment I was looking at. Hence the confusion. The errors goes away when those settings values are set to empty arrays.
From the discussion it sounds like there is some history to this, but I wonder if there is any way to prevent this error without also requiring that config?
- πΊπΈUnited States seanr
I'll second this - it should fail much more gracefully than it does. When I have time, I'll see if I can work up a patch for that.