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

Add statsd_port config option #968

Merged
merged 2 commits into from Jun 2, 2023
Merged

Add statsd_port config option #968

merged 2 commits into from Jun 2, 2023

Conversation

tombruijn
Copy link
Member

Add statsd_port config option

Allow configuration of the agent's statsd server port through the statsd_port option.

Configure the Puma plugin using env vars. It cannot access the normal AppSignal config, because the gem isn't loaded in the main process.

Requires an agent update from https://github.com/appsignal/appsignal-agent/pull/995

Bump agent to fd8ee9e

  • Rely on APPSIGNAL_RUNNING_IN_CONTAINER config option value before
    other environment factors to determine if the app is running in a
    container.
  • Fix container detection for hosts running Docker itself.
  • Add APPSIGNAL_STATSD_PORT config option.

Allow configuration of the agent's statsd server port through the
statsd_port option.

Configure the Puma plugin using env vars. It cannot access the normal
AppSignal config, because the gem isn't loaded in the main process.

Requires an agent update from
appsignal/appsignal-agent#995
- Rely on APPSIGNAL_RUNNING_IN_CONTAINER config option value before
  other environment factors to determine if the app is running in a
  container.
- Fix container detection for hosts running Docker itself.
- Add APPSIGNAL_STATSD_PORT config option.
@tombruijn tombruijn merged commit 34562a1 into main Jun 2, 2023
12 checks passed
@tombruijn tombruijn deleted the configure-statsd-port branch June 2, 2023 06:49
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.

None yet

3 participants