Skip to content

Disable ctags for zoekt indexer

John Mason requested to merge jm-disable-zoekt-ctags into master

What does this MR do?

This will disable ctags. The zoekt source code checks this environment variable. If it is an empty string, ctags will not be used: https://github.com/sourcegraph/zoekt/blob/4448a4594e21989eddd12df08a44c82b22f80c74/build/builder.go#L274

See this slack thread for additional context: https://gitlab.slack.com/archives/C3TMLK465/p1711117563711539

Author Check-list

Please read the Contributing document and once you do, complete the following:

  • Check if all of the following apply:
    • Assign to the correct reviewer per the contributing document
    • Apply the correct metadata per the contributing document
    • Link to related MRs for applying the changes on other environments
    • Link to related Chef changes
    • If necessary link to a Criticality 4 Change Request issue

Reviewer Check-list

  • Check if all of the following apply:
    • Reviewed the diff jobs to confirm changes are as expected
    • No changes shown in the diffs not associated with this MR - This may require a rebase or further investigation

Applier Check-list

  • Make sure there is no ongoing deployment for the affected envs before merging (see #announcements slack channel)

Merge request reports