-
Notifications
You must be signed in to change notification settings - Fork 592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cache connection required during package discovery process #620
Comments
You could try to add |
@danilopolani, Unfortunately, this does not solve the issue. This is the first thing I tried when I ran into this problem. During package discovery process, the console kernel is loaded, and every manually registered service provider is called, which leads to the same result as if we did not exclude the package from discovery. |
We had the same problem, on our end we fixed it by running our build with a TELESCOPE_ENABLED=false env var. |
@themsaid do you know why this is needed? |
Disabling telescope for builds seems a reasonable solution. |
@taylorotwell While I agree that it makes sense to disable Telescope in this case. |
@taylorotwell Sorry to ask, why is this issue closed? I'm still experiencing this problem. Is there no official way of fixing this? I wanted Telescope to work in my local development environment in Docker Compose. My current mitigation method is to add In my case, I'm not just using Docker for CI/CD, but for my actual dev environment. So it doesn't make sense to disable Telescope in my case. I just found this issue, but I've made a StackOverflow issue a while ago: https://stackoverflow.com/questions/59810019/dockerfile-build-laravel-trying-to-connect-to-redis-service-yet-to-up |
I'm having this issue as well. @themsaid, can Telescope automatically be disabled in this case (package discovery)? |
Latest laravel 7.0 and Telescope 3.3 - still got this error in Docker/Kubernetes context, i am applying |
I was experiencing the same issue and I fixed it by removing the package discovery command completely from the |
i haven't changed much about my environment lately except for bumping to laravel 8.X.... and telescope to 4.x per the upgrade guide. Running into this right now...... was something fixed in 7.x and reverted in 8.x? (or broken in 4.x) |
I figured this out. the change to prepend the scheme (which defaults to tcp) broke me: laravel/framework@aa4e269#diff-89b6287633db08fda2f83951857e18fd I was including the scheme (tls for us) in the REDIS_HOST, causing it to build tcp://tls:// |
I'm NOT using Telescope and I have the same problem. |
* Handle redis connection exception on DumpWatcher register Fixes issue #620 * Cleanup after previous solution removed * Include another case and refactor * Add missing ! * Update src/Telescope.php * Update src/Watchers/DumpWatcher.php * use Exception --------- Co-authored-by: Dries Vints <dries@vints.io>
Description:
I'm using Redis as cache driver for my application.
Also I'm using Docker and CI/CD solution for build and deploy.
As part of my build pipeline I run
composer install
, but because Redis not available during build stage, I receiving following error:After that my pipeline fails.
It seems that the problem occurs here:
telescope/src/Watchers/DumpWatcher.php
Line 43 in 1749dae
I turned off
DumpWatcher
in the config, and the problem has disappeared.Steps To Reproduce:
composer require predis/predis
CACHE_DRIVER
env variable toredis
REDIS_HOST
env variable to any unavailable hostcomposer install
The text was updated successfully, but these errors were encountered: