Skip to content

Fix Zoekt CI image. Bump to v1.5

Dmitry Gruzd requested to merge fix-zoekt-ci-image into master

What does this MR do and why?

This is a follow-up to Add zoekt-ci-image-1.4 (!751 - merged). It turned out that zoekt-ci-image-1.4 was broken. gitlab-zoekt-indexer binary was failing with Program aborted: open zoekt/test/index/node.uuid: no such file or directory, but the process kept running since we didn't check the status code.

This MR fixes the problem and adds zoekt-ci-image-1.5 with the fixed docker image.

How to test

  1. Checkout master
  2. Build v1.4 
    docker build . -f Dockerfile.zoekt-ci-image-1.4 -t registry.gitlab.com/gitlab-org/gitlab-build-images:zoekt-ci-image-1.4
  3. Run the image 
    docker run -p 6060:6060 -p 6070:6070 --rm -it registry.gitlab.com/gitlab-org/gitlab-build-images:zoekt-ci-image-1.4
  4. Ensure that indexer failed with Program aborted: open zoekt/test/index/node.uuid: no such file or directory
  5. Ensure that indexer is not reachable with 
    $ curl http://localhost:6060/indexer/health
curl: (56) Recv failure: Connection reset by peer
  6. Checkout this MR branch
  7. Build v1.5 
    docker build . -f Dockerfile.zoekt-ci-image-1.5 -t registry.gitlab.com/gitlab-org/gitlab-build-images:zoekt-ci-image-1.5
  8. Run the image 
    docker run -p 6060:6060 -p 6070:6070 --rm -it registry.gitlab.com/gitlab-org/gitlab-build-images:zoekt-ci-image-1.5
  9. You shouldn't see the failure anymore
  10. Verify that containers' healthchecks are reachable 
    $ curl -s -o /dev/null -w "%{http_code}" http://localhost:6060/indexer/health
200
$ curl -s -o /dev/null -w "%{http_code}" http://localhost:6070
200
  11. As an extra check you can run zoekt specs while the container is still running 
    bin/rspec ee/spec/requests/api/admin/search/zoekt_spec.rb

Checklist

Edited by Dmitry Gruzd

Merge request reports