Skip to content

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

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

Conversation

thomas-gouveia
Copy link
Contributor

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 (25a05ec)

@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

@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from 82dbe6a to 924e56f Compare March 11, 2025 17:18
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from ae31476 to 028a80b Compare March 21, 2025 15:41
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from 028a80b to b621bf7 Compare March 24, 2025 10:35
@thomas-gouveia
Copy link
Contributor Author

@jmacd @dmitryax could you review this please?

@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from b621bf7 to a02fa84 Compare April 1, 2025 08:40
@thomas-gouveia thomas-gouveia force-pushed the feat/37807/statsdreceiver/flexibility-socket-permissions branch from a02fa84 to 25a05ec Compare April 7, 2025 08:02
@atoulme atoulme merged commit 8f40000 into open-telemetry:main Apr 19, 2025
171 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 19, 2025
akshays-19 pushed a commit to akshays-19/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2025
… when transport is unixgram (open-telemetry#37807) (open-telemetry#38420)

#### 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
open-telemetry#37807

#### Testing

Unit tests

#### Documentation

Readme updated with new configuration

Signed-off-by: thomas-gouveia <[email protected]>
Fiery-Fenix pushed a commit to Fiery-Fenix/opentelemetry-collector-contrib that referenced this pull request Apr 24, 2025
… when transport is unixgram (open-telemetry#37807) (open-telemetry#38420)

#### 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
open-telemetry#37807

#### Testing

Unit tests

#### Documentation

Readme updated with new configuration

Signed-off-by: thomas-gouveia <[email protected]>
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
4 participants