Skip to content

honey: add HONEYCOMB_SUFFIX and HONEYCOMB_DISABLE

Administrator requested to merge k/honey-suffix into main

Created by: keegancsmith

I want to enable honeycomb on dogfood. Honeycomb's official documentation recommends creating a new dataset rather than adding a column for to match against. In particular they give examples of "-dev" suffix. So this commit adds support for a suffix for each dataset name via the envvar HONEYCOMB_SUFFIX.

In particular I want to log our search core debug dataset. This doesn't respect honey.Enabled, but rather its own envvar. So rather than enabling everything, we add HONEYCOMB_DISABLE so Enabled returns false. This way at first we will just get our debug data.

This commit also removes the unused "honey.Builder".

Merge request reports