Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow configuring Statsd host and port from env vars #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zorbash
Copy link

@zorbash zorbash commented Jul 10, 2017

This PR changes exometer_report_statsd to be configurable from environmental variables.

It can be configured as follows (Elixir code):

config :exometer,
  report: [
    reporters: [
      {:exometer_report_statsd, [hostname: {:system, "STATSD_HOST"}, port: {:system, "STATSD_PORT"}]}
    ]
  ]

The goal is to build the release artefact using a tool like Distillery just once for multiple environments (staging / production) and supply all the required configuration via env vars.

If this behaviour is decided to be merged, I can also work on the other reporters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant