Project

General

Profile

Actions

action #153421

open

[spike][timeboxed:10h] Replace scriptgen with executing rsync from python

Added by livdywan 12 months ago. Updated 10 months ago.

Status:
New
Priority:
Low
Assignee:
-
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Observation

In #112871 we found that it's very difficult to follow the logic, and part of that is the fact that scriptgen.py is used to generate several scripts using variables, configured commands and hard-coded logic to replace literal strings found in either of those.

What if the logic could be changed to run rsync instead of generating scripts? Let's try out how feasible that is.

Acceptance criteria

  • AC1: Sync can be executed directly without generated code

Suggestions

  • Change scriptgen.py to execute rsync, possibly use https://github.com/gchamon/sysrsync
  • Reduce the magical replacements in the "p" functions
  • Error handling within Python code

Out of scope

  • Complete rewrite of everything
  • Changing the plugin completely. This is a spike solution

Related issues 2 (0 open2 closed)

Related to openQA Project (public) - action #161879: hook script support for obs-sync plugin - single hook script before "isos post" size:MResolvedmkittler2024-06-04

Actions
Copied from QA (public) - action #112871: obs_rsync_run Minion tasks fail with no error message size:MResolvedlivdywan

Actions
Actions #1

Updated by livdywan 12 months ago

  • Copied from action #112871: obs_rsync_run Minion tasks fail with no error message size:M added
Actions #2

Updated by livdywan 12 months ago

  • Target version deleted (Ready)

I didn't mean to put it in Ready. This came up as a follow-up idea while discussing the related ticket.

Actions #3

Updated by okurz 12 months ago

  • Subject changed from [spike] Replace scriptgen with executing rsync from python to [spike][timeboxed:10h] Replace scriptgen with executing rsync from python
  • Target version set to Ready

As a 10h timebox we can do that in Ready

Actions #4

Updated by tinita 12 months ago

  • Description updated (diff)
Actions #5

Updated by okurz 12 months ago

  • Priority changed from Normal to Low
Actions #6

Updated by okurz 12 months ago

  • Target version changed from Ready to Tools - Next
Actions #7

Updated by okurz 10 months ago

  • Target version changed from Tools - Next to future
Actions #8

Updated by livdywan 7 months ago

  • Related to action #161879: hook script support for obs-sync plugin - single hook script before "isos post" size:M added
Actions

Also available in: Atom PDF