Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

search: replace zoekt-wrapper script sleep hack #3371

Closed
ijt opened this issue Apr 11, 2019 · 0 comments · Fixed by sourcegraph/zoekt#30 or #5784
Closed

search: replace zoekt-wrapper script sleep hack #3371

ijt opened this issue Apr 11, 2019 · 0 comments · Fixed by sourcegraph/zoekt#30 or #5784
Assignees
@keegancsmith
Milestone
3.9

Comments

@ijt
Copy link
Contributor

ijt commented Apr 11, 2019 •
edited
Loading

zoekt-wrapper contains a hack that sleeps five seconds to wait for the frontend to start before starting zoekt-sourcegraph-indexserver. That could be too long, or not long enough. It would be better if zoekt-sourcegraph-indexserver would wait for the frontend to become available before going about its business. Then we could remove the zoekt-wrapper script.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@keegancsmith keegancsmith

Labels
None yet
Projects
None yet
Milestone
3.9
2 participants
@ijt @keegancsmith