Project

General

Profile

Actions

action #162095

open

openQA Tests - coordination #96596: [qe-core][CI] CI/CD and Coding style improvements

os-autoinst-distri-opensuse uses os-autoinst-common for perltidy

Added by tinita 6 months ago. Updated 6 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Feature requests
Target version:
Start date:
2024-06-11
Due date:
% Done:

0%

Estimated time:

Description

Motivation

In #155062 we moved os-autoinst to tidyall from os-autoinst-common via a subrepo.

os-autoinst-distri-opensuse is still using tools/tidy which is a symlink to the old tools/tidy in the "manually" cloned os-autoinst repo as a subdirectory.

It should use the new tidyall script by using os-autoinst-common as a subrepo or submodule.
The perltidy version should go into os-autoinst-distri-opensuse/cpanfile directly instead of using the one from os-autoinst.
Updating the subrepo or submodule should happen via a make target.

Also it should be checked if cloning os-autoinst in os-autoinst-distri-opensuse is still necessary. make prepare installs some perl modules from os-autoinst's cpanfile https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/Makefile#L11 but that shouldn't be needed as we have rpm packages for that.

  • AC1: os-autoinst-distri-opensuse uses tidyall from os-autoinst-common and maintains its own Perl::Tidy version in cpanfile
  • AC2: Cloning os-autoinst in os-autoinst-distri-opensuse is obsolete (if possible)

Suggestions

  • If we use a subrepo, a make subrepo-update should simply do a subrepo clone --force to avoid any conflicts which can sometimes happen due to a restriction of git-subrepo. Also, updates of os-autoinst-common is probably not necessary very often.
    • Advantage: for users the subrepo is just there, as it is a simple subdirectory in git with a special status file, so they don't need to clone any submodules
    • Disadvantage: os-autoinst-common now contains quite some files and they would only be needed for development. A submodule would only clone them on request

Related issues 1 (1 open0 closed)

Related to openQA Project - coordination #154777: [saga][epic] Shareable os-autoinst and test distribution pluginsNew2021-01-08

Actions
Actions #1

Updated by tinita 6 months ago

  • Tracker changed from coordination to action
Actions #2

Updated by tinita 6 months ago

  • Related to coordination #154777: [saga][epic] Shareable os-autoinst and test distribution plugins added
Actions #3

Updated by tinita 6 months ago

  • Description updated (diff)
Actions #4

Updated by okurz 6 months ago

  • Target version set to future
Actions

Also available in: Atom PDF