Page MenuHomePhabricator

Soften a possible cluster search setup fatal
ClosedPublic

Authored by epriestley on Mar 28 2017, 10:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 29, 9:12 AM
Unknown Object (File)
Wed, May 7, 5:27 PM
Unknown Object (File)
Apr 23 2025, 12:00 AM
Unknown Object (File)
Apr 15 2025, 4:09 AM
Unknown Object (File)
Apr 14 2025, 7:02 AM
Unknown Object (File)
Apr 13 2025, 6:58 AM
Unknown Object (File)
Mar 31 2025, 7:36 PM
Unknown Object (File)
Mar 20 2025, 10:57 PM
Subscribers
None

Details

Summary

Ref T12450. The way that config repair and setup issues interact is kind of complicated, and if cluster.search is invalid we may end up using cluster.search before we repair it.

I poked at things for a bit but wasn't confident I could get it to consistently repair before we use it without doing a big messy change.

The only thing that really matters is whether "type" is valid or not, so just put a slightly softer/more-tailored check in for that.

Test Plan
  • With "type": "elastic", loaded setup issues.
  • Before patch: hard fatal.
  • After patch: softer fatal with more useful messaging.

Screen Shot 2017-03-28 at 3.11.24 PM.png (1×1 px, 247 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/infrastructure/cluster/config/PhabricatorClusterSearchConfigOptionType.php
7

Today is not my best day.

chad edited edge metadata.
This revision is now accepted and ready to land.Mar 28 2017, 10:28 PM
This revision was automatically updated to reflect the committed changes.