-
Notifications
You must be signed in to change notification settings - Fork 645
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
[3.0] Adding horizon:status command #545
Conversation
Yes, that's my use case, this will solve the problem too. Sorry, i didn't look for related tickets. I agree that would be better. |
The addition of |
Well, this is not required. I can duplicate the logic on the command for now so we can have it. It will be definitely useful to people, not sure this has to wait a new version. I did that to reuse the logic. |
Yeah, that might be better for now. |
Done |
@dbpolito I've merged this into master. You can send the changes you did at first there if you want. |
@dbpolito How can I use it programmatically without Artisan::call function? |
@delmicio that depends on your use case or what you want to do... in my case i'm using as readiness/livinessProbe on kubernetes with bash script |
@dbpolito I see there is a protected function called currentStatus on this PR. Any chance to do what that function does on a laravel controller for example? |
found it on another package if some is interested: |
Adding
horizon:status
command to be able to check via cli the status of Horizon.