Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: initial Packit configuration #1

Open
wants to merge 7 commits into
base: rhel-9
Choose a base branch
from

Conversation

mrc0mmand
Copy link
Member

@mrc0mmand mrc0mmand commented Aug 25, 2021

Enable a basic set of Packit jobs to build util-linux RPMs from a
RHEL9/C9S specfile - on F34 for now, until we have public C9S images
available in Copr.

Some comments to the sed magic in the yaml configuration file:

  1. We have to create a custom tarball by our own means, since the
    default Packit action doesn't include the necessary .tarball-version
    file. This wouldn't be necessary if we could use the make distcheck
    target, but the default builder chroot is missing some dependencies
    (like bison), so we have to fallback to git archive.

  2. Consequently, we need to call autogen.sh from the specfile (normally
    we'd call it before make distcheck and already have a configure
    binary in the tarball)

  3. Also consequently, explicitly run the check RPM phase during
    rpmbuild (by setting _with_check 1). This would be, also, normally
    executed by make distcheck.

As requested, run the checks on push events into the rhel-9 branch
instead of for each pull request.

This workflow can be later extended to send the built RPMs to the
Testing Farm (TFT)[0] for a more extensive testing.

[0] https://packit.dev/docs/testing-farm/

@mrc0mmand mrc0mmand force-pushed the packit-onboard branch 9 times, most recently from ff73561 to da8d2b4 Compare August 26, 2021 10:15
@mrc0mmand mrc0mmand marked this pull request as ready for review August 26, 2021 10:50
@mrc0mmand mrc0mmand force-pushed the packit-onboard branch 5 times, most recently from 5f78e19 to 9153d05 Compare August 26, 2021 12:05
Enable a basic set of Packit jobs to build util-linux RPMs from a
RHEL9/C9S specfile - on F34 for now, until we have public C9S images
available in Copr.

Some comments to the sed magic in the yaml configuration file:

1) We have to create a custom tarball by our own means, since the
   default Packit action doesn't include the necessary .tarball-version
   file. This wouldn't be necessary if we could use the `make distcheck`
   target, but the default builder chroot is missing some dependencies
   (like bison), so we have to fallback to `git archive`.

2) Consequently, we need to call autogen.sh from the specfile (normally
   we'd call it before `make distcheck` and already have a configure
   binary in the tarball)

3) Also consequently, explicitly run the `check` RPM phase during
   rpmbuild (by setting `_with_check 1`). This would be, also, normally
   executed by `make distcheck`.

As requested, run the checks on push events into the rhel-9 branch
instead of for each pull request.

This workflow can be later extended to send the built RPMs to the
Testing Farm (TFT)[0] for a more extensive testing.

[0] https://packit.dev/docs/testing-farm/
metadata:
branch: rhel-9
targets:
# FIXME: change to CentOS 9 once it's available
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CentOS 9 targets are now available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants