Page MenuHomekolab.org
Differential D4208

./artisan status:health: check every backend instead of just one
ClosedPublic
Actions

Authored by mollekopf on Mar 27 2023, 10:30 AM.

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rKdb54b0f95107: ./artisan status:health: check every backend instead of just one

Diff Detail

Repository
rK kolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

mollekopf requested review of this revision.Mar 27 2023, 10:30 AM
mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Mar 27 2023, 11:33 AM
machniak accepted this revision.Mar 27 2023, 11:49 AM
machniak added a subscriber: machniak.

An improvement would be to print which backend failed. Also, we shouldn't var_export the whole response object, error code/message would be enough.

This revision is now accepted and ready to land.Mar 27 2023, 11:49 AM
mollekopf updated this revision to Diff 12080.Mar 27 2023, 12:03 PM

Only print the status code and reason

This revision was landed with ongoing or failed builds.Mar 27 2023, 2:43 PM
Closed by commit rKdb54b0f95107: ./artisan status:health: check every backend instead of just one (authored by Christian Mollekopf <mollekopf@apheleia-it.ch>). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 12101

View Options

src/app/Console/Commands/Status/Health.php

Loading...