Skip to content

Conversation

@ChrisJBurns
Copy link
Collaborator

The filtering code now has been moved to the registry api server further marking this as unrequired in the operator

ref: #2399

this code now has been moved to the registry api server further marking
this as unrequired in the operator

Signed-off-by: ChrisJBurns <[email protected]>
@ChrisJBurns ChrisJBurns changed the title removes regsitry filtering from operator removes registry filtering from operator Nov 13, 2025
@dmartinol
Copy link
Collaborator

and next will be sync and sources I guess. thanks! /lgtm

@ChrisJBurns
Copy link
Collaborator Author

@dmartinol Yep. Although I'm a bit unclear where the boundaries are with those, specifically the sync. The filtering seemed a lot easier to tear out given it's relative standalone pieces of code. I can try data sources next.

@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.73%. Comparing base (cead1c3) to head (d2fbf39).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2566      +/-   ##
==========================================
- Coverage   55.95%   55.73%   -0.22%     
==========================================
  Files         316      313       -3     
  Lines       29766    29605     -161     
==========================================
- Hits        16655    16501     -154     
- Misses      11652    11653       +1     
+ Partials     1459     1451       -8     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ChrisJBurns ChrisJBurns merged commit b58742c into main Nov 13, 2025
30 checks passed
@ChrisJBurns ChrisJBurns deleted the removes-filters branch November 13, 2025 13:57
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