Skip to content

fix(ci): OSCI image prefetch issues [4.10]#21534

Merged
ovalenti merged 1 commit into
release-4.10from
ovalenti/osci-fetch-issues
Jul 3, 2026
Merged

fix(ci): OSCI image prefetch issues [4.10]#21534
ovalenti merged 1 commit into
release-4.10from
ovalenti/osci-fetch-issues

Conversation

@ovalenti

@ovalenti ovalenti commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Description

We try to apply some of the changes attempted to workaround some image prefetch issues.

#19287 is the inspiration.

The OSCI failure looks like :

{orchestratormanager.OrchestratorManagerException: The deployment did not start or reach replica ready state - if this job uses image prefetch check that this image is in the jobs prefetch list e.g. qa-tests-backend/scripts/images-to-prefetch.txt - quay.io/rhacs-eng/qa-multi-arch:nginx-1.12@sha256:72daaf46f11cc753c4eab981cbf869919bd1fee3d2170a2adeac12400f494728 orchestratormanager.OrchestratorManagerException orchestratormanager.OrchestratorManagerException: The deployment did not start or reach replica ready state - if this job uses image prefetch check that this image is in the jobs prefetch list e.g. qa-tests-backend/scripts/images-to-prefetch.txt - quay.io/rhacs-eng/qa-multi-arch:nginx-1.12@sha256:72daaf46f11cc753c4eab981cbf869919bd1fee3d2170a2adeac12400f494728
	at app//orchestratormanager.Kubernetes.waitForDeploymentAndPopulateInfo(Kubernetes.groovy:2081)
	at app//orchestratormanager.Kubernetes.batchCreateDeployments(Kubernetes.groovy:239)
	at app//K8sEventDetectionTest.setupSpec(K8sEventDetectionTest.groovy:43)
	Suppressed: java.lang.NullPointerException: Cannot invoke "io.stackrox.proto.storage.PolicyOuterClass$ListPolicy.getId()" because the return value of "org.codehaus.groovy.runtime.DefaultGroovyMethods.find(java.util.Collection, groovy.lang.Closure)" is null
		at Services.getPolicyByName(Services.groovy:98)
		at K8sEventDetectionTest.cleanupSpec(K8sEventDetectionTest.groovy:72)
}

@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ovalenti ovalenti self-assigned this Jul 2, 2026
@github-actions github-actions Bot added the backport PR to backport changes from master to release branch label Jul 2, 2026
@ovalenti

ovalenti commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

/test all

@ovalenti ovalenti changed the title fix(ci): OSCI image prefetch issues fix(ci): OSCI image prefetch issues [4.10] Jul 2, 2026
@rhacs-bot

Copy link
Copy Markdown
Contributor

Images are ready for the commit at cc51ee0.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.10.5-rc.1-1-gcc51ee0746.

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.43%. Comparing base (d8f1b23) to head (cc51ee0).

Additional details and impacted files
@@               Coverage Diff                @@
##           release-4.10   #21534      +/-   ##
================================================
- Coverage         49.43%   49.43%   -0.01%     
================================================
  Files              2662     2662              
  Lines            201163   201163              
================================================
- Hits              99446    99436      -10     
- Misses            94273    94280       +7     
- Partials           7444     7447       +3     
Flag Coverage Δ
go-unit-tests 49.43% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ovalenti ovalenti requested a review from porridge July 2, 2026 16:05
@ovalenti ovalenti marked this pull request as ready for review July 2, 2026 16:05
@ovalenti ovalenti requested a review from janisz as a code owner July 2, 2026 16:05
@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown

@ovalenti: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ocp-dev-preview-nongroovy-e2e-tests cc51ee0 link false /test ocp-dev-preview-nongroovy-e2e-tests

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@ovalenti ovalenti merged commit bea37cb into release-4.10 Jul 3, 2026
117 of 122 checks passed
@ovalenti ovalenti deleted the ovalenti/osci-fetch-issues branch July 3, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport PR to backport changes from master to release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants