action #118633
coordination #80142: [saga][epic] Scale out: Redundant/load-balancing deployments of openQA, easy containers, containers on kubernetes
coordination #108527: [epic] os-autoinst wheels for scalable code reuse of helper functions and segmented test distributions
coordination #117673: [epic][tools] sporadic "Unable to clone Git repository" for wheels
Re-try on cloning of wheel repositories size:M
0%
Description
Motivation¶
Sometimes wheels (or other repos) can't be cloned on the first attempt, see #117673
https://openqa.suse.de/tests/9668977#
[2022-10-06T02:01:46.761898+02:00] [info] ::: OpenQA::Isotovideo::Utils::checkout_git_repo_and_branch: Cloning git URL 'https://github.com/Zaoliang/functional_wheel' [2022-10-06T02:01:52.973889+02:00] [debug] Cloning into 'functional_wheel'... fatal: unable to access 'https://github.com/Zaoliang/functional_wheel/': OpenSSL SSL_connect: Connection reset by peer in connection to github.com:443
Acceptance criteria¶
- AC1: Wheels cloning is retried
Suggestions¶
- A reasonable default is used to retry when cloning fails
- Crosscheck how we handle cloning test case and needle repos
- Optional: a maximum retry of N times can be configured in the wheels.yaml or just in test variables
Related issues
History
#2
Updated by cdywan 5 months ago
- Copied to action #118882: Make available wheels more discoverable and share knowledge size:S added
#4
Updated by okurz 5 months ago
szarate wrote:
Optional: a maximum retry of N times can be configured in the wheels.yaml or just in test variables
This should not be optional, and also not be on test variables but on the wheels.yaml
We commonly define such settings in openQA test variables. Why do you need it in wheels.yaml?
#8
Updated by openqa_review 3 months ago
- Due date set to 2023-01-18
Setting due date based on mean cycle time of SUSE QE Tools
#11
Updated by okurz 2 months ago
- Due date deleted (
2023-01-18) - Status changed from Feedback to Resolved
https://github.com/os-autoinst/os-autoinst/pull/2238 merged. I pinged about that change in https://suse.slack.com/archives/C02CANHLANP/p1673517402966779
As we don't currently use wheels in os-autoinst-distri-opensuse we shouldn't need to await any verification in production.
#12
Updated by cdywan 2 months ago
okurz wrote:
As we don't currently use wheels in os-autoinst-distri-opensuse we shouldn't need to await any verification in production.
Follow-up https://github.com/os-autoinst/os-autoinst/pull/2253 as per feedback in https://suse.slack.com/archives/C02CANHLANP/p1674482162461009
#13
Updated by cdywan 2 months ago
- Related to action #123556: os-autoinst git cloning of test case repo can fail with auto_review:"fatal.*unable to access.*Connection timed out":retry size:M added