Project

General

Profile

Actions

action #153340

closed

OBS CI checks for os-autoinst subpackages size:M

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

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2024-01-10
Due date:
% Done:

0%

Estimated time:

Description

Motivation

https://github.com/os-autoinst/os-autoinst/pull/2430#discussion_r1447407616

os-autoinst OBS CI checks only check the main package, not the openvswitch subpackage unlike openQA (see e.g. OBS checks in os-autoinst/openQA#5352) showing OBS: openQA:openQA-client-test and OBS: openQA:openQA-test and such.

Acceptance criteria

  • AC1: os-autoinst pull requests trigger OBS CI checks for each os-autoinst subpackage

Suggestions


Related issues 1 (0 open1 closed)

Related to openQA Infrastructure - action #153544: os-autoinst:os-autoinst-openvswitch-test not building on Leap_15.5 x86_64Resolvedjbaier_cz2024-01-15

Actions
Actions #1

Updated by livdywan 9 months ago

  • Subject changed from OBS CI checks for os-autoinst subpackages to OBS CI checks for os-autoinst subpackages size:M
  • Description updated (diff)
  • Status changed from New to Workable
Actions #2

Updated by jbaier_cz 9 months ago

  • Assignee set to jbaier_cz
Actions #3

Updated by jbaier_cz 9 months ago

  • Status changed from Workable to Feedback

With https://github.com/os-autoinst/os-autoinst/pull/2439 already merged, I added the newly created spec file into service file for os-autoinst and also added them into multibuild configuration. Now we need to wait for build.

Actions #4

Updated by jbaier_cz 9 months ago

  • Related to action #153544: os-autoinst:os-autoinst-openvswitch-test not building on Leap_15.5 x86_64 added
Actions #5

Updated by jbaier_cz 9 months ago

  • Status changed from Feedback to In Progress
Actions #6

Updated by jbaier_cz 9 months ago

The test sub-package is pulling latest successful version of the package, which can yield unexpected results (like successful build test for package which cannot be build at all). https://github.com/os-autoinst/os-autoinst/pull/2441 should fix that.

Actions #7

Updated by livdywan 9 months ago

  • Status changed from In Progress to Feedback

jbaier_cz wrote in #note-6:

The test sub-package is pulling latest successful version of the package, which can yield unexpected results (like successful build test for package which cannot be build at all). https://github.com/os-autoinst/os-autoinst/pull/2441 should fix that.

The branch got merged.

For some reason this is commented out in
https://github.com/os-autoinst/os-autoinst/blob/master/dist/rpm/os-autoinst-test.spec#L8

Is this q from the PR still relevant?

Actions #8

Updated by jbaier_cz 9 months ago

livdywan wrote in #note-7:

For some reason this is commented out in
https://github.com/os-autoinst/os-autoinst/blob/master/dist/rpm/os-autoinst-test.spec#L8

Is this q from the PR still relevant?

Yeah, I was also interested about the reason. The git blame suggested it was like this forever, so the question might not be that important.

Actions #9

Updated by okurz 9 months ago

  • Status changed from Feedback to Resolved

Well, we don't remember why we had it but it works now properly requiring the specific version so we should be good.

Actions

Also available in: Atom PDF