Page MenuHomePhorge
Differential D4208

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

Authored by mollekopf on Mar 27 2023, 10:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 16, 3:55 AM
Unknown Object (File)
Mon, Oct 14, 7:58 PM
Unknown Object (File)
Mon, Oct 14, 4:27 AM
Unknown Object (File)
Fri, Oct 11, 7:42 AM
Unknown Object (File)
Fri, Oct 4, 8:05 PM
Unknown Object (File)
Sep 28 2024, 9:52 PM
Unknown Object (File)
Sep 17 2024, 1:20 AM
Unknown Object (File)
Sep 13 2024, 8:40 AM
View All Files
Subscribers
machniak

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
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf requested review of this revision.Mar 27 2023, 10:30 AM
mollekopf created this revision.
Harbormaster completed remote builds in B42164: Diff 12056.Mar 27 2023, 10:30 AM
mollekopf updated this revision to Diff 12062.Mar 27 2023, 10:32 AM

Fixup

Harbormaster completed remote builds in B42167: Diff 12062.Mar 27 2023, 10:32 AM
mollekopf added a reviewer: Restricted Project.Mar 27 2023, 11:33 AM
machniak accepted this revision.Mar 27 2023, 11:49 AM
machniak subscribed.

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

Harbormaster completed remote builds in B42185: Diff 12080.Mar 27 2023, 12:03 PM
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 mollekopf). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
app/
Console/
Commands/
Status/
22 lines

Diff 12101

View Options

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

Loading...