Skip to content

[injector] feat(backend): add dump config (#4386) #132

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
Merged
camrrx merged 11 commits into from
Jan 20, 2026
Merged

[injector] feat(backend): add dump config (#4386) #132

camrrx merged 11 commits into from
Jan 20, 2026

Conversation

@camrrx
Copy link
Member

@camrrx camrrx commented Dec 30, 2025 •
edited
Loading

Proposed changes

  • Add a shared --dump-config-schema argument in injector_common.
  • Use it in all injectors to print the config schema
  • Add injector configuration files.
  • Add collector metadata to pyproject.toml under [tool.cmw].
  • Add a JSON metadata file.

Testing Instructions

  1. Run an injector with --dump-config-schema.
  2. Check the schema is printed

Related issues

  • #4386

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

@camrrx camrrx changed the title [injectors] Add dump config [injectors] (feat): add dump config (#4386) Dec 30, 2025
@camrrx camrrx changed the title [injectors] (feat): add dump config (#4386) [injector all] feat: add dump config (#4386) Dec 30, 2025
@camrrx camrrx changed the title [injector all] feat: add dump config (#4386) [injector all] feat(backend): add dump config (#4386) Dec 30, 2025
@camrrx camrrx changed the title [injector all] feat(backend): add dump config (#4386) [injector] feat(backend): add dump config (#4386) Dec 30, 2025
camrrx and others added 7 commits December 30, 2025 15:22
@camrrx
[injector all] generator manifest - adapt all injectors
7e0d5f5
@camrrx
[injector all] format code
eb60e79
@camrrx
[injector all] format code
b11e8c2
@antoinemzs
housekeeping
a8020d0 
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
@antoinemzs
ignoring poetry lock
24f40a7 
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
@antoinemzs
add injector_common to docker builds
6c1cff5 
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
@antoinemzs
enable injector_common in docker builds for aws and http
e420afb 
Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>
antoinemzs
antoinemzs approved these changes Jan 15, 2026
View reviewed changes
Copy link
Member

@antoinemzs antoinemzs left a comment

Choose a reason for hiding this comment

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

Need to merge the base branch to resolve conflicts

camrrx added 2 commits January 20, 2026 11:30
@camrrx
[injector all] metadatas
7f65ff2
@camrrx
Merge branch 'release/current' into add_dump_config
8ddcdf3 
# Conflicts:
#	aws/poetry.lock
#	injector_common/pyproject.toml
@camrrx camrrx merged commit b3e4d9e into release/current Jan 20, 2026
5 checks passed
@camrrx camrrx deleted the add_dump_config branch January 20, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antoinemzs antoinemzs antoinemzs approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@camrrx @antoinemzs