Don't fatal in ElasticSearch setup check if no "master" database is configured
ClosedPublic
Actions

Authored by epriestley on Aug 17 2017, 4:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 13, 2:26 AM

	Unknown Object (File)
	Fri, Aug 28, 10:17 PM

	Unknown Object (File)
	Wed, Aug 26, 11:02 PM

	Unknown Object (File)
	Jul 13 2026, 10:02 AM

	Unknown Object (File)
	Jun 17 2026, 11:06 PM

	Unknown Object (File)
	Jun 11 2026, 11:45 AM

	Unknown Object (File)
	Jun 6 2026, 2:48 AM

	Unknown Object (File)
	Mar 17 2026, 2:33 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12965: When no "master" database is configured, the ElasticSearch setup check can fatal

Commits

rPc9986fd5dee6: Don't fatal in ElasticSearch setup check if no "master" database is configured

Summary

Ref T12965. See that task for discussion, and PHI36 for context.

This sweeps the fatal under the rug by skipping it, letting things move forward for now.

Test Plan

Followed instructions in T12965, got a read-only recovery after restart instead of a fatal.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 17 2017, 4:53 PM

Harbormaster completed remote builds in B18093: Diff 44302.Aug 17 2017, 4:54 PM

chad accepted this revision.Aug 17 2017, 5:01 PM

This revision is now accepted and ready to land.Aug 17 2017, 5:01 PM

Closed by commit rPc9986fd5dee6: Don't fatal in ElasticSearch setup check if no "master" database is configured (authored by epriestley). · Explain WhyAug 17 2017, 5:39 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

check/

PhabricatorElasticsearchSetupCheck.php

6 lines

Diff 44304

View Options

src/applications/config/check/PhabricatorElasticsearchSetupCheck.php

Don't fatal in ElasticSearch setup check if no "master" database is configuredClosedPublicActions