[3.0] Adding horizon:status command

[dbpolito]

Adding horizon:status command to be able to check via cli the status of Horizon.

[driesvints]

@dbpolito I believe this is for #468 ?

It's always better to be very thorough in your PR description so it has a higher chance of being accepted.

[dbpolito]

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.

[driesvints]

The addition of public function currentStatus(); on the MasterSupervisorRepository interface is a breaking change in this release. I think it might be better if we add this in the next one. Didn't check this on the previous PR, sorry.

[dbpolito]

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.

[driesvints]

Yeah, that might be better for now.

[dbpolito]

Done

[driesvints]

@dbpolito I've merged this into master. You can send the changes you did at first there if you want.

[delmicio]

@dbpolito How can I use it programmatically without Artisan::call function?

[dbpolito]

@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

[delmicio]

@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?

[delmicio]

found it on another package if some is interested:
https://github.com/antonioribeiro/health/blob/master/src/Checkers/Horizon.php