Skip to content

fix(ci): increase timeout processing RH OVAL data #1883

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

Merged
merged 2 commits into from
May 8, 2025

Conversation

BradLugo
Copy link
Contributor

@BradLugo BradLugo commented May 8, 2025

Description

Increases the timeout for the HTTP client that requests Red Hat OVAL feeds. We previously did this for the v2 feed in this PR: #1820. However, our "v1" OVAL updater also uses the v2 feed URLs:

bulkOVALURI = ovalURI + "v2/"

bulkRHSAXMLBZ2URLs = map[string]string{
bulkOVALURI + "RHEL6/rhel-6-including-unpatched.oval.xml.bz2": "6",
bulkOVALURI + "RHEL7/rhel-7-including-unpatched.oval.xml.bz2": "7",
bulkOVALURI + "RHEL8/rhel-8-including-unpatched.oval.xml.bz2": "8",
}

Moreover, these changes will keep the timeout consistent between reading from the "v1" and v2 Red Hat OVAL feeds.

Example of the error these changes should fix: https://github.com/stackrox/scanner/actions/runs/14848990112/job/41689019123

@BradLugo BradLugo added the generate-dumps-on-pr Generates the image based on dumps from the PR label May 8, 2025
Copy link

openshift-ci bot commented May 8, 2025

@BradLugo: 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/e2e-tests 7c7ceb7 link false /test e2e-tests

Full PR test history. Your PR dashboard.

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.

@dcaravel dcaravel self-requested a review May 8, 2025 14:19
Copy link
Contributor

@dcaravel dcaravel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, LGTM

@BradLugo BradLugo merged commit 4a55542 into master May 8, 2025
28 of 29 checks passed
@BradLugo BradLugo deleted the blugo/rh-oval-timeout branch May 8, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
generate-dumps-on-pr Generates the image based on dumps from the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants