Skip to content

[pkg/stanza] cache attributes extracted from file name #42388

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 23 commits into
base: main
Choose a base branch
from
Draft

[pkg/stanza] cache attributes extracted from file name #42388

wants to merge 23 commits into from

Conversation

bacherfl
Copy link
Contributor

@bacherfl bacherfl commented Sep 1, 2025 •
edited
Loading

Description

This is a PoC for a possible solution for #41672 - Extensive tests and documentation are missing, but this should illustrate the idea of how the regex based metadata extraction from the file path could be done more efficiently

Link to tracking issue

Related to #41672

Testing

Added unit tests

Documentation

Added docs in the readme explaining how to configure the metadata extraction

@bacherfl bacherfl requested review from andrzej-stencel and a team as code owners September 1, 2025 12:47
@bacherfl bacherfl changed the title Feat/41672/cache parsed regex results [pkg/stanza] cache attributes extracted from file name Sep 1, 2025
@bacherfl bacherfl mentioned this pull request Sep 2, 2025
Open
@bacherfl bacherfl marked this pull request as draft September 2, 2025 12:01
@bacherfl
fix dependency issues
75e13a6 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
fix linting
e64b917 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
add tests for lru cache
1badbf0 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
cleanup, add more tests
1719979 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
add changelog entry
2434e8f 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
fix linting
76ed2d0 
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl
add license header
1359f4c 
Signed-off-by: Florian Bacher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrzej-stencel andrzej-stencel Awaiting requested review from andrzej-stencel andrzej-stencel is a code owner

@pjanotti pjanotti Awaiting requested review from pjanotti pjanotti will be requested when the pull request is marked ready for review pjanotti is a code owner

@dmitryax dmitryax Awaiting requested review from dmitryax dmitryax will be requested when the pull request is marked ready for review dmitryax is a code owner

@crobert-1 crobert-1 Awaiting requested review from crobert-1 crobert-1 will be requested when the pull request is marked ready for review crobert-1 is a code owner

@atoulme atoulme Awaiting requested review from atoulme atoulme will be requested when the pull request is marked ready for review atoulme is a code owner

@dehaansa dehaansa Awaiting requested review from dehaansa dehaansa will be requested when the pull request is marked ready for review dehaansa is a code owner

@evan-bradley evan-bradley Awaiting requested review from evan-bradley evan-bradley will be requested when the pull request is marked ready for review evan-bradley is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@codeboten codeboten

Labels
internal/common pkg/stanza/fileconsumer pkg/stanza
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bacherfl @codeboten