#240 RFE: Build/test a PR in given side tag
Opened 3 years ago by churchyard. Modified a year ago

Hello.

When we build interconnected packages, we want to build them in side tags. But in order to run the CI on the "second" package, we need to build the "first" package in rawhide. Sometimes, that is fine, sometimes we cannot do that. In the ideal world, we would CI test both pull requests together, but for build dependencies, that doesn't really work yet. As an alternative, would it be possible to introduce a keyword to the Pull Request, like:

Build-Target: f35-build-side-38414

The Ci would work as-is except it would:

  • scratch build stuff in f35-build-side-38414 instead of regular target
  • add the Koji repo f35-build-side-38414 to the tested system before installing any packages

WDYT?


@churchyard for all PR workflows we are aiming to use Zuul, which should already make it possible to have multiple PRs build together. Isn't this workflow kinda supported there already?

We are working on adding support for Zuul to be able to run then also tmt based tests, but we are quite overdue with that :( but it is coming. Anyway I assume you now have STI tests now?

for all PR workflows we are aiming to use Zuul, which should already make it possible to have multiple PRs build together. Isn't this workflow kinda supported there already?

Not for BuildRequires. It uses Koji scratch builds.

We are working on adding support for Zuul to be able to run then also tmt based tests, but we are quite overdue with that :( but it is coming.

Awesome.

Anyway I assume you now have STI tests now?

Mostly yes.

I've just seen this in c9s: https://gitlab.com/redhat/centos-stream/rpms/python3.9/-/merge_requests/16

It has side-tag: c9s-build-side-394-stack-gate in the initial comment and it works.

Does this work in Fedora as well? If so, that's awesome!

Metadata Update from @fbo:
- Issue tagged with: Zuul CI

2 years ago

Apparently, it does not work in Fedora, only in c9s.

@fbo is this something we can also enable in Fedora if it works somehow in Cs9?

Login to comment on this ticket.

Metadata