sourcegraph / zoekt

Fast trigram based code search
Apache License 2.0
736 stars 83 forks source link

docker: remove scip-ctags from Dockerfile #802

Closed keegancsmith closed 3 months ago

keegancsmith commented 3 months ago

This is an optional dependency that I don't believe OSS users of docker use. We used to use these dockerfiles in Sourcegraph, but have since moved to building it in our own CI with scip-ctags.

Test Plan: built locally

docker build -t zoekt .
docker build -t zoekt-indexserver . -f Dockerfile.indexserver
docker build -t zoekt-webserver   . -f Dockerfile.webserver
keegancsmith commented 3 months ago

@sourcegraph/search-platform we should consider making the indexserver docker image instead the zoekt-indexserver binary since this image is no longer used by Sourcegraph. That would be a better service to our OSS users.