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

New multiplexor engine #175

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

joselsegura
Copy link
Collaborator

@joselsegura joselsegura commented Mar 21, 2024

Description

Adding a "multiplexor" engine. It will take a filter as a configuration parameter. Then, it will open the provided archive from the path argument, open the tarball find and seek files with a relative path specified in the filters dictionary, marking each archive with 1 or more "marks" to be processed by an enabled publisher.

Fixes #CCXDEV-12640

Type of change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Unit tests (no changes in the code)

Testing steps

Tested locally with provided unit tests + local deployment using Minio and Kafka to perform some operations

Checklist

  • pre-commit run -a passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.94%. Comparing base (c62ccc9) to head (aa0b258).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #175      +/-   ##
==========================================
+ Coverage   82.59%   82.94%   +0.34%     
==========================================
  Files          30       31       +1     
  Lines         931      950      +19     
==========================================
+ Hits          769      788      +19     
  Misses        162      162              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joselsegura joselsegura merged commit c1e8226 into RedHatInsights:main Mar 21, 2024
5 checks passed
@joselsegura joselsegura deleted the multiplexor-engine branch March 21, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants