Skip to content

Commit

Permalink
ci: Run cockpit tests in PRs
Browse files Browse the repository at this point in the history
See https://cockpit-project.org/blog/tmt-cross-project-testing.html

Drop the install-only tests, as TF only runs the default "install check"
test if there are no plans, but now we have one. That will cover the
installation/upgrade as preparation step.
  • Loading branch information
martinpitt committed Sep 25, 2023
1 parent 76f54f9 commit 3bab283
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 0 deletions.
1 change: 1 addition & 0 deletions .fmf/version
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
1
13 changes: 13 additions & 0 deletions packit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,21 @@ jobs:
- fedora-development
- fedora-latest-stable

# run tests for packages which test SELinux policy well, see plans/ with `revdeps == yes`
- job: tests
identifier: revdeps
trigger: pull_request
notifications:
failure_comment:
message: "Cockpit tests failed for commit {commit_sha}. @martinpitt, @jelly, @mvollmer please check."
targets:
- fedora-development
- fedora-latest-stable
tf_extra_params:
environments:
- artifacts:
- type: repository-file
id: https://copr.fedorainfracloud.org/coprs/g/cockpit/main-builds/repo/fedora-$releasever/group_cockpit-main-builds-fedora-$releasever.repo
tmt:
context:
revdeps: "yes"
30 changes: 30 additions & 0 deletions plans/cockpit.fmf
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
# reverse dependency test for https://github.com/cockpit-project/cockpit
# packit should automatically notify the cockpit maintainers on failures.
# For questions, please contact @martinpitt, @jelly, @mvollmer

enabled: false
adjust+:
when: revdeps == yes
enabled: true

discover:
how: fmf
url: https://github.com/cockpit-project/cockpit
ref: main
execute:
how: tmt

/basic:
summary: Run tests for basic packages
discover+:
test: /test/browser/basic

/network:
summary: Run tests for cockpit-networkmanager
discover+:
test: /test/browser/network

/optional:
summary: Run tests for optional packages
discover+:
test: /test/browser/optional

0 comments on commit 3bab283

Please sign in to comment.