Project

General

Profile

action #57233

Properly handle transport errors in ObsRsync plugin

Added by andriinikitin over 1 year ago. Updated 6 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
Target version:
Start date:
2019-09-23
Due date:
% Done:

0%

Estimated time:

Description

After hotfix https://github.com/os-autoinst/openQA/pull/2346 the plugin will ignore transport errors and proceed syncing the project.
That is not fully correct behavior and this ticket should provide proper fix. (E.g. add parameter retry_count_on_exception to ObsRsync plugin)


Related issues

Related to openQA Infrastructure - action #54338: rsync.pl on OSD is unable to access IBS: "@ERROR: access denied to trigger-repos-sync from UNKNOWN (2620:113:80c0:8080:5054:ff:fe49:b768)"Rejected2019-07-17

History

#1 Updated by andriinikitin over 1 year ago

Actually after the hotfix plugin will retry checking OBS status indefinitely, which is probably incorrect approach as well

#2 Updated by okurz over 1 year ago

  • Blocks action #54338: rsync.pl on OSD is unable to access IBS: "@ERROR: access denied to trigger-repos-sync from UNKNOWN (2620:113:80c0:8080:5054:ff:fe49:b768)" added

#3 Updated by okurz over 1 year ago

  • Related to action #52967: rsync.pl Lots of errors about not existant media for SLE12SP5 which only apply for SLE15 added

#4 Updated by andriinikitin about 1 year ago

  • Blocks deleted (action #54338: rsync.pl on OSD is unable to access IBS: "@ERROR: access denied to trigger-repos-sync from UNKNOWN (2620:113:80c0:8080:5054:ff:fe49:b768)")

#5 Updated by andriinikitin about 1 year ago

  • Related to deleted (action #52967: rsync.pl Lots of errors about not existant media for SLE12SP5 which only apply for SLE15)

#6 Updated by okurz about 1 year ago

  • Related to action #54338: rsync.pl on OSD is unable to access IBS: "@ERROR: access denied to trigger-repos-sync from UNKNOWN (2620:113:80c0:8080:5054:ff:fe49:b768)" added

#7 Updated by okurz 6 months ago

  • Status changed from New to Rejected
  • Assignee set to okurz

This change might have been implemented by now already. We do not have more details in neither comments nor ticket description so I do not think this ticket helps us further

Also available in: Atom PDF