Skip to content

Add option to use DogStatsD as telemetry provider - #377

Open
yzoug wants to merge 2 commits into
ansible-community:masterfrom
yzoug:patch-1
Open

Add option to use DogStatsD as telemetry provider#377
yzoug wants to merge 2 commits into
ansible-community:masterfrom
yzoug:patch-1

Conversation

@yzoug

@yzoug yzoug commented Jul 21, 2025

Copy link
Copy Markdown
Contributor

Add a new variable, vault_dogstatsd_address, to use the DogStatsD telemetry provider. It's documented here: https://developer.hashicorp.com/vault/docs/configuration/telemetry#dogstatsd_addr

The PR also updates the role variable documentation section on telemetry.

yzoug added 2 commits July 21, 2025 10:06
Add the option to set a DogStatsD address in the Vault main configuration, in the telemetry section, if the user defines the new variable `vault_dogstatsd_address`.

Cf. https://developer.hashicorp.com/vault/docs/configuration/telemetry#dogstatsd_addr

@lakrass lakrass left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The code itself looks good!

Can you please also implement the variable dogstatsd_tags for the sake of completeness?

Also please use conventional commits and rebase your branch when finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants