Skip to content

Add CTAGS_COMMAND to zoekt indexer environment variables

John Mason requested to merge johnmason-master-patch-33347 into master

What does this MR do?

Configures zoekt to use universal ctags in staging. I verified that /usr/bin/ctags is indeed universal ctags.

➜  gitlab-build-images git:(jm-add-zoekt-ctags) docker run --rm -it registry.gitlab.com/gitlab-org/build/cng/gitlab-zoekt-indexer:v16.9.1 /bin/sh
$ type ctags
ctags is /usr/bin/ctags
$ ctags --version
Universal Ctags 5.9.0, Copyright (C) 2015 Universal Ctags Team
Universal Ctags is derived from Exuberant Ctags.
Exuberant Ctags 5.8, Copyright (C) 1996-2009 Darren Hiebert
  Compiled: Sep  3 2021, 18:12:18
  URL: https://ctags.io/
  Optional compiled features: +wildcards, +regex, +gnulib_regex, +iconv, +option-directory, +xpath, +json, +interactive, +sandbox, +yaml, +packcc, +optscript
$

Related issues

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion

Required

  • Merge Request Title, and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com
  • When ready for review, MR is labeled "~workflow::ready for review" per the Distribution MR workflow

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes
  • Documentation created/updated
  • Integration tests added to GitLab QA
  • The impact any change in container size has should be evaluated
  • New dependencies are managed with dependencies.io
Edited by Jason Plum

Merge request reports