Fixes T8274. That report is very light on details, but I think the issue is that their Elasticsearch is misconfigured and the new setup warning dies a little too hard if the server is just completely dead.
Details
Details
- Reviewers
joshuaspence btrahan - Maniphest Tasks
- T8274: Misconfigured Elasticsearch can fatal the setup issue process
- Commits
- Restricted Diffusion Commit
rPf5a9d1f8d47b: Raise a setup issue for misconfigured Elasticsearch
Set search.elastic.host to an invalid server, got a similar-looking exception (?), applied patch, got a setup warning instead.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- esearch1
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 6152 Build 6173: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
src/applications/config/check/PhabricatorElasticSearchSetupCheck.php | ||
---|---|---|
19–21 | these methods existed already? cool! |