Project

General

Profile

Actions

action #133349

closed

openQA-in-openQA tests fail in zypper command with 'Maximum (6) redirects followed' auto_review:"Test died.*retry -s 30.*zypper.*nginx":retry size:S

Added by okurz 9 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Feature requests
Target version:
Start date:
2023-07-26
Due date:
% Done:

0%

Estimated time:

Description

Observation

https://openqa.opensuse.org/tests/3455846#step/openqa_webui/13 shows a problem in zypper

Error code: Curl error 47
Error message: Maximum (6) redirects followed

Acceptance criteria

  • AC1: No more failed redirects
  • AC2: All network-related retry calls use exponential backoff

Suggestions

We already use retry as visible in
https://openqa.opensuse.org/tests/3455846/video?filename=video.webm&t=5.5,6
but we don't use exponential backoff. Maybe just add that?

Workaround

Retrigger the job

Actions #1

Updated by okurz 9 months ago

  • Tags set to openqa-in-openqa, retry, zypper, download, test
  • Subject changed from openQA-in-openQA tests fail in zypper command with "Maximum (6) redirects followed" to openQA-in-openQA tests fail in zypper command with 'Maximum (6) redirects followed' auto_review:"Test died.*retry -s 30.*zypper.*nginx":retry
  • Description updated (diff)
  • Priority changed from Normal to High
Actions #2

Updated by okurz 9 months ago

  • Subject changed from openQA-in-openQA tests fail in zypper command with 'Maximum (6) redirects followed' auto_review:"Test died.*retry -s 30.*zypper.*nginx":retry to openQA-in-openQA tests fail in zypper command with 'Maximum (6) redirects followed' auto_review:"Test died.*retry -s 30.*zypper.*nginx":retry size:S
  • Description updated (diff)
  • Status changed from New to Workable
Actions #3

Updated by mkittler 9 months ago

  • Assignee set to mkittler
Actions #4

Updated by mkittler 9 months ago

  • Status changed from Workable to Feedback

PR for adding exponential backoff (AC2): https://github.com/os-autoinst/os-autoinst-distri-openQA/pull/139

Whether it helps with the redirection problem I can't say but I suppose just waiting longer between the retry attempts is the best we can do.

Actions #5

Updated by okurz 9 months ago

  • Status changed from Feedback to Resolved

No problems seen after merge, this should be good enough

Actions #6

Updated by mkittler 8 months ago

  • Status changed from Resolved to Feedback

The PR hasn't been merged yet.

Actions #7

Updated by mkittler 8 months ago

  • Status changed from Feedback to Resolved

Now the PR has been merged for real.

Actions

Also available in: Atom PDF