Project

General

Profile

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

Added by cdywan 5 months ago. Updated 2 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
cdywan
Category:
Feature requests
Target version:
Ready
Start date:
2022-10-13
Due date:
% Done:

0%

Estimated time:
Difficulty:

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

Related to openQA Project - action #123556: os-autoinst git cloning of test case repo can fail with auto_review:"fatal.*unable to access.*Connection timed out":retry size:MResolved2023-01-23

Copied to openQA Project - action #118882: Make available wheels more discoverable and share knowledge size:SWorkable2022-10-13

History

#1 Updated by okurz 5 months ago

  • Category set to Feature requests
  • Target version set to Ready

#2 Updated by cdywan 5 months ago

  • Copied to action #118882: Make available wheels more discoverable and share knowledge size:S added

#3 Updated by szarate 5 months ago

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

#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?

#5 Updated by okurz 5 months ago

  • Target version changed from Ready to future

#6 Updated by okurz 3 months ago

  • Target version changed from future to Ready

#7 Updated by cdywan 3 months ago

  • Status changed from Workable to In Progress
  • Assignee set to cdywan

I'll give this a go. Should be pretty straightforward 🤞

#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

#10 Updated by cdywan 2 months ago

  • Status changed from In Progress to Feedback

Code is merged, which should fullfil AC1. okurz is going to announce this in #eng-testing in Slack

#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

Also available in: Atom PDF