Skip to content

dpservice-exporter: Add file-prefix flag#651

Merged
guvenc merged 1 commit intomainfrom
dpservice-exporter/configurable-socket-path-file-prefix
Mar 14, 2025
Merged

dpservice-exporter: Add file-prefix flag#651
guvenc merged 1 commit intomainfrom
dpservice-exporter/configurable-socket-path-file-prefix

Conversation

@vlorinc
Copy link
Copy Markdown
Contributor

@vlorinc vlorinc commented Mar 11, 2025

Added file-prefix flag to make dpdk telemetry socket more configurable at the start of exporter.
It defaults to "rte" so without this flag it is working as before.
Needed for #643

@vlorinc vlorinc added the enhancement New feature or request label Mar 11, 2025
@vlorinc vlorinc self-assigned this Mar 11, 2025
@vlorinc vlorinc requested a review from a team as a code owner March 11, 2025 11:59
@github-actions github-actions bot added bug Something isn't working size/XS labels Mar 11, 2025
@vlorinc vlorinc marked this pull request as draft March 11, 2025 12:11
@vlorinc vlorinc force-pushed the dpservice-exporter/configurable-socket-path-file-prefix branch from f81022d to b322f04 Compare March 11, 2025 12:26
@github-actions github-actions bot added size/S and removed size/XS labels Mar 11, 2025
Copy link
Copy Markdown
Contributor

@guvenc guvenc left a comment

Choose a reason for hiding this comment

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

LGTM

@vlorinc vlorinc marked this pull request as ready for review March 12, 2025 13:01
@guvenc guvenc merged commit 571f531 into main Mar 14, 2025
6 checks passed
@guvenc guvenc deleted the dpservice-exporter/configurable-socket-path-file-prefix branch March 14, 2025 17:53
@hardikdr hardikdr added this to Roadmap Jun 26, 2025
@hardikdr hardikdr moved this to Done in Roadmap Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking bug Something isn't working enhancement New feature or request size/S

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants