Page MenuHomePhabricator

Soften a possible cluster search setup fatal
ClosedPublic

Authored by epriestley on Mar 28 2017, 10:18 PM.
Tags
None
Referenced Files
F15505576: D17576.id42285.diff
Tue, Apr 15, 4:09 AM
F15503124: D17576.id.diff
Mon, Apr 14, 7:02 AM
F15496493: D17576.diff
Sun, Apr 13, 6:58 AM
F15460046: D17576.id42284.diff
Mon, Mar 31, 7:36 PM
F15418381: D17576.id42284.diff
Thu, Mar 20, 10:57 PM
F15391923: D17576.id42285.diff
Mar 15 2025, 12:23 PM
F15391432: D17576.id.diff
Mar 15 2025, 9:50 AM
Unknown Object (File)
Feb 26 2025, 6:31 AM
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.