Skip to content

Commit

Permalink
packit: Build PRs into default packit COPRs
Browse files Browse the repository at this point in the history
Building all PRs of all container projects into the same COPR does not
properly isolate PRs from each other.

To avoid that, change the copr_build configuration to use the packit
default COPRs, which are specific to the particular PR, and disappear
after a few weeks. Depending projects should only run against what
landed in gvisor-tap-vsock/main i.e. the podman-next COPR.

Unmerging the two copr jobs allows us the redundant targets: list for the
"main" builds into podman-next. Specifying the target list is not necessary
any more since packit/packit-service#1499 got fixed.

Signed-off-by: Martin Pitt <mpitt@redhat.com>
  • Loading branch information
martinpitt authored and cfergeau committed Aug 16, 2023
1 parent b0d9740 commit 5e3626c
Showing 1 changed file with 15 additions and 32 deletions.
47 changes: 15 additions & 32 deletions .packit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,52 +9,35 @@
specfile_path: rpm/gvisor-tap-vsock.spec
upstream_tag_template: v{version}

srpm_build_deps:
- make

jobs:
- &copr
job: copr_build
- job: copr_build
trigger: pull_request
owner: rhcontainerbot
project: packit-builds
enable_net: true
srpm_build_deps:
- make
# keep in sync with https://copr.fedorainfracloud.org/coprs/rhcontainerbot/podman-next
targets:
- fedora-rawhide-aarch64
- fedora-rawhide-x86_64
- fedora-eln-aarch64
- fedora-eln-x86_64
- fedora-38-aarch64
- fedora-38-x86_64
- centos-stream-9-aarch64
- centos-stream-9-x86_64
- centos-stream-8-aarch64
- centos-stream-8-x86_64
- centos-stream+epel-next-8-x86_64
- centos-stream+epel-next-8-aarch64
- centos-stream+epel-next-9-x86_64
- centos-stream+epel-next-9-aarch64
additional_repos:
- "copr://rhcontainerbot/podman-next"

- <<: *copr
# Run on commit to main branch
# Run on commit to main branch
- job: copr_build
trigger: commit
enable_net: true
branch: main
owner: rhcontainerbot
project: podman-next
targets:
- fedora-rawhide-aarch64
- fedora-rawhide-ppc64le
- fedora-rawhide-s390x
- fedora-rawhide-x86_64
- fedora-eln-aarch64
- fedora-eln-ppc64le
- fedora-eln-s390x
- fedora-eln-x86_64
- fedora-38-aarch64
- fedora-38-ppc64le
- fedora-38-s390x
- fedora-38-x86_64
- centos-stream+epel-next-9-aarch64
- centos-stream+epel-next-9-ppc64le
- centos-stream+epel-next-9-s390x
- centos-stream+epel-next-9-x86_64
- centos-stream+epel-next-8-aarch64
- centos-stream+epel-next-8-ppc64le
- centos-stream+epel-next-8-x86_64

- job: propose_downstream
trigger: release
Expand Down

0 comments on commit 5e3626c

Please sign in to comment.