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

[receiver/statsdreceiver] add ability to customize socket permissions when transport is unixgram (#37807) #38420

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thomas-gouveia
Copy link

Description

This PR introduces a new configuration key socket_permissions used when transport: unixgram to control permissions of the bind socket.

Link to tracking issue

Fixes #37807

Testing

Unit tests

Documentation

Readme updated with new configuration

@thomas-gouveia thomas-gouveia requested review from dmitryax and a team as code owners March 6, 2025 11:01
Copy link

linux-foundation-easycla bot commented Mar 6, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: thomas-gouveia / name: Thomas Gouveia (924e56f)
  • ✅ login: atoulme / name: Antoine Toulme (ae31476)

@github-actions github-actions bot added the receiver/statsd statsd related issues label Mar 6, 2025
@github-actions github-actions bot requested a review from jmacd March 6, 2025 11:02
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch 3 times, most recently from 9a05fdf to 5eda2a8 Compare March 7, 2025 10:27
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch 2 times, most recently from dba7677 to a38c8a6 Compare March 10, 2025 10:04
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from a38c8a6 to 82dbe6a Compare March 11, 2025 15:40
@atoulme
Copy link
Contributor

atoulme commented Mar 11, 2025

@jmacd @dmitryax please review

… when transport is unixgram (open-telemetry#37807)

Signed-off-by: thomas-gouveia <thomas.gouveia@contentsquare.com>
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from 82dbe6a to 924e56f Compare March 11, 2025 17:18
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.

[receiver/statsdreceiver] Add ability to customize permissions of socket when using UDS server
3 participants