action #69157: Improve package and base OS version support: Update contribution hints or review checklists for package impact - openQA Project - openSUSE Project Management Tool

Actions

Copy link

action #69157

closed

action #68350: Improve package and base OS version support

Improve package and base OS version support: Update contribution hints or review checklists for package impact

Added by okurz almost 4 years ago. Updated over 3 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

okurz

Category:

Organisational

Target version:

Ready

Start date:

2020-07-21

Due date:

% Done:

Estimated time:

Description

Motivation¶

When we introduce new dependencies or require a new version of our dependencies we should be careful that the base operating systems still provide all what is needed.

Acceptance criteria¶

AC1: Our review process ensures that package impact is considered (human review or automatic checks)

Suggestions¶

Update contribution hints or review checklists, e.g. to look out for package builds, make sure packages and updated packages are submitted to openSUSE Factory and whatever current Leap version is in development
Optional: Automatic check of (new) package dependencies against what is available in openSUSE Factory and/or Tumbleweed
Optional: As above but for current Leap version in development to ensure new dependencies are also submitted to Leap if not already in

Actions

Copy link

Updated by mkittler over 3 years ago

We have checks for the OBS build when a PR is submitted and it actually fulfills AC1.

However, so far only @coolo knows how it works and it is unclear where to find documentation. Hence I don't like to mark the ticket as resolved. Maybe @coolo can enlighten us how the OBS checks work? Then someone could also look into the optional ideas mentioned in the suggestions.

Actions

Copy link

Updated by okurz over 3 years ago

The check is just https://github.com/openSUSE/obs-pullrequest-builder (I think) running somewhere. We tried to run the OBS trigger in a CI pipeline itself but we did not find a way to store necessary OBS credentials properly so we use the current way. However the automatic check is not the same. It helps but does not cover all. We should state what we commonly follow, as I suggested in "Suggestions", e.g. that we only use packages that are in Factory and need to submit new packages to Factory otherwise.

Actions

Copy link