Repository navigation
Conversation
|
Skipping CI for Draft Pull Request. |
Pin the ClairCore fork's retry implementation and enable three retries for repo-to-CPE downloads with the existing HTTP client and timeout. AI-Assisted: Codex, integration change Signed-off-by: Piotr Rygielski <114479+vikin91@users.noreply.github.com>
Contributor
🚀 Build Images ReadyImages are ready for commit 2ef2494. To use with deploy scripts: export MAIN_IMAGE_TAG=5.1.x-277-g2ef2494dfa |
Contributor
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #23436 +/- ##
==========================================
- Coverage 52.00% 51.97% -0.04%
==========================================
Files 2912 2912
Lines 183385 183386 +1
==========================================
- Hits 95372 95314 -58
- Misses 79684 79725 +41
- Partials 8329 8347 +18
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
vikin91
force-pushed
the
piotr/test-claircore-repo2cpe-retries
branch
from
October 15, 2026 11:53
d0c4e64 to
2ef2494
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Pin the ClairCore fork's retry implementation and enable three retries for repo-to-CPE downloads. Keep the existing client and timeout, and cover recovery, exhaustion, permanent errors, and cancellation.
User-facing documentation
Testing and quality
Automated testing
How I validated my change
Verified on six RHCOS nodes manually on a cluster. Recovery produced
503,503,503,200with approximately 1s, 2s, and 4s waits, followed by 572-package index reports reaching Sensor. Persistent 503 responses stopped after four requests; 404 responses were not retried.Prerequisites: OpenShift with RHCOS nodes,
kubectl,curl, Python 3, and a running Podman machine. Run from the StackRox checkout under test, with its CI images already published. These steps assume a fresh ACS deployment./recoveryserves the real mapping after three failures;/exhaustedalways returns 503;/permanentalways returns 404.The Service provides
repo2cpe-smoke.stackrox.svc:8080.Observed:
Recovery delivered index reports containing
572 packages (572 deprecated)to Sensor. Both failure scenarios loggedrhel: unable to create a mappingFile objecton all six nodes.kubectl -n stackrox set env ds/collector --containers=compliance \ ROX_NODE_INDEX_MAPPING_URL- \ ROX_NODE_SCANNING_MAX_INITIAL_WAIT- kubectl -n stackrox rollout status ds/collector --timeout=3m kubectl -n stackrox delete \ deployment/repo2cpe-smoke \ service/repo2cpe-smoke \ configmap/repo2cpe-smoke kubectl -n stackrox get ds collectorCleanup completed; Collector was restored and 6/6 ready.
Limit: Full vulnerability matching was not verified because Scanner V4’s initial vulnerability-data import was still running.