ROX-32459: Bump claircore to v1.5.50 to fix node/vm indexing#19422
ROX-32459: Bump claircore to v1.5.50 to fix node/vm indexing#19422
Conversation
|
This change is part of the following stack: Change managed by git-spice. |
|
Skipping CI for Draft Pull Request. |
|
Images are ready for the commit at 2996c00. To use with deploy scripts, first |
Bump ClairCore to v1.5.50 and switch node index layer paths to normalized file:// URIs so VM/node indexing remains compatible with newer ClairCore URI handling and benefits from /proc access robustness plus correct package-scan error propagation. User request: "update to 1.5.50 and implement the full ACS change for nodeIndexer/ROX-32459 context." AI generated the dependency bump, URI conversion, and tests; user validated scope, selected version direction, and reviewed/corrected requirements during implementation.
f710db5 to
2996c00
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #19422 +/- ##
=======================================
Coverage 49.69% 49.70%
=======================================
Files 2702 2702
Lines 203538 203550 +12
=======================================
+ Hits 101155 101166 +11
+ Misses 94856 94855 -1
- Partials 7527 7529 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
AFAICT, these changes don't affect the scenario described in ROX-32459. In any case, I'd like to get this PR merged since I have some dependent work. FWIW, I'm happy to help out with ROX-32459. I agree that 1) we need a way to filter file paths (namely Please let me know how I can help move this PR forward 🙂 |
@BradLugo I will merge it now.
I think they do, because with this version I am not seeing the bug that was easy to reproduce before it. But I agree that we can merge those and if there is any leftover problem it can be then handled in a follow-up. I am happy to accept your support on the issues with walking over too many directories. Let's look at that in a separate PR or in a Slack discussion. I will make the next move, just wanted to keep someone from Scanner team informed of what I found. |
Description
This PR updates node indexing to work with newer ClairCore filesystem URI handling and improves resilience/correctness for the ROX-32459 scenario.
github.com/quay/claircorefromv1.5.44tov1.5.50.file://URIs beforeclaircore.Layer.Init.JSONFormattoFormatJSONto make the code compile (introduced here)Why:
file://URI semantics for filesystem layers.v1.5.44include improvements relevant to ROX-32459 context (including filtering problematic filesystem access and package-scanner error propagation behavior).Acknowledgments:
User-facing documentation
Testing and quality
Automated testing
How I validated my change
Confirming the bug is fixed
✅ The fix is a solution to the issue. However, it makes the indexing pretty slow...
Observing performance
The change seem to significantly increase indexing times. Here few observations (still need more data for a conclusion).
/procis not being searched/proc):In case multiple agents run in parallel (e.g., one daemon, other one-shot), then the runtimes are very long - I killed one run after 10 minutes of waiting for any result. CPU load spikes.
Suggestion: add lock file to prevent more than one agent running in parallel.