Skip to content

Conversation

@seh
Copy link
Contributor

@seh seh commented Jul 22, 2025

Augment the Honeycomb API client in the client package to convey the "dataset_slug" field for triggers. When querying for triggers at the environment level, the API returns all triggers within the environment—including those tied to a particular dataset. Clients can read the populated "dataset_slug" field to distinguish which triggers are tied to which dataset and those that sit at the environment level, above and separate from any particular dataset.

Note that this change does not yet cover the Terraform provider's publishing to or querying of data from the Honeycomb API as exposed through its Terraform resources and data sources.

@seh seh requested a review from a team as a code owner July 22, 2025 18:25
Augment the Honeycomb API client in the "client" package to convey the
"dataset_slug" field for triggers. When querying for triggers at the
environment level, the API returns all triggers within the
environment—including those tied to a particular dataset. Clients can
read the populated "dataset_slug" field to distinguish which triggers
are tied to which dataset and those that sit at the environment level,
above and separate from any particular dataset.

Note that this change does not yet cover the Terraform provider's
publishing to or querying of data from the Honeycomb API as exposed
through its Terraform resources and data sources.
@seh seh force-pushed the include-dataset-slug-in-triggers branch from 11d708e to d0a7492 Compare July 23, 2025 11:57
@jharley jharley added the bug label Jul 23, 2025
Copy link
Collaborator

@jharley jharley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this oversight on our part! ✨

@jharley jharley merged commit 1301112 into honeycombio:main Jul 23, 2025
@seh seh deleted the include-dataset-slug-in-triggers branch July 23, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants