Skip to content

Add packit CI#5

Open
LecrisUT wants to merge 1 commit into
airsim:mainfrom
LecrisUT:ci/packit
Open

Add packit CI#5
LecrisUT wants to merge 1 commit into
airsim:mainfrom
LecrisUT:ci/packit

Conversation

@LecrisUT

@LecrisUT LecrisUT commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

Add a packit workflow to test the builds in Fedora against the latest packages. This CI does:

  • builds on every PR, commit to main and release in their own environments
  • create a PR on rawhide after a release
  • (downstream CI) trigger a build when a packit PR is merged

Some additional improvements that can be made:

  • Build for epel10 also (maybe it is missing some dependencies). This can be used to gather the missing dependencies and request epel10 branching all at once
  • Add tests for post packaging, e.g. testing that the -devel files work appropriately or tests that require internet connection

Requires:

Originally described here

Signed-off-by: Cristian Le <git@lecris.dev>
@LecrisUT LecrisUT marked this pull request as draft June 2, 2025 11:51
@LecrisUT

LecrisUT commented Jun 2, 2025

Copy link
Copy Markdown
Contributor Author

Marking as draft to prevent accidental merge until the CI is installed

@LecrisUT LecrisUT mentioned this pull request Jun 2, 2025
@LecrisUT

Copy link
Copy Markdown
Contributor Author

Oh I guess I haven't announced it here, the airsim copr group is created 1. Let me know who I should add to manage it if needed. @da115115 want to move this one forward?

Footnotes

  1. https://copr.fedorainfracloud.org/groups/g/airsim/coprs/

@LecrisUT LecrisUT marked this pull request as ready for review August 13, 2025 14:05
@da115115

da115115 commented Aug 13, 2025

Copy link
Copy Markdown
Member

Oh I guess I haven't announced it here, the airsim copr group is created 1. Let me know who I should add to manage it if needed. @da115115 want to move this one forward?

Footnotes

  1. https://copr.fedorainfracloud.org/groups/g/airsim/coprs/

Yes, sure, @LecrisUT , you can go forward by yourself. Do not hesitate if you need anything from my side (I think you have both accesses to upstream and to Fedora packaging by now)

@LecrisUT

Copy link
Copy Markdown
Contributor Author

Requires:

* [Packit onboarding](https://packit.dev/docs/guide)

This part I am unable to do.

@da115115

Copy link
Copy Markdown
Member

Requires:

* [Packit onboarding](https://packit.dev/docs/guide)

This part I am unable to do.

Neither do I.
If it helps, I can (briefly) onboard you on the project overall. Because, so far, the more I read PackIt information, the less I understand. I guess it is just of matter of "clicking", and it still does not click for me.

@LecrisUT

Copy link
Copy Markdown
Contributor Author

/packit build

Hmm let me check the current state. It should be just enable the app, and respond to a ping on https://github.com/packit/notifications/issues if it requires

@LecrisUT

Copy link
Copy Markdown
Contributor Author

/packit build

It didn't pick it up so, yeah, seems like it's not setup (should show 👀 reaction).

If it helps, I can (briefly) onboard you on the project overall. Because, so far, the more I read PackIt information, the less I understand. I guess it is just of matter of "clicking", and it still does not click for me.

Sure thing, I can help out

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.

2 participants