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, May 25, 12:25 PM

	Unknown Object (File)
	Tue, May 20, 3:33 AM

	Unknown Object (File)
	Apr 25 2025, 3:06 PM

	Unknown Object (File)
	Apr 24 2025, 5:29 AM

	Unknown Object (File)
	Apr 19 2025, 11:29 AM

	Unknown Object (File)
	Apr 18 2025, 3:01 AM

	Unknown Object (File)
	Apr 12 2025, 9:19 AM

	Unknown Object (File)
	Apr 10 2025, 8:16 PM

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