-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fixes #1013: Add Packit integration to GitHub project for Fedora CI #1014
base: main
Are you sure you want to change the base?
Conversation
@ganeshmurthy @kgiusti "Look, folks, so many CI jobs!" |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1014 +/- ##
=======================================
- Coverage 77.7% 77.7% -0.1%
=======================================
Files 247 247
Lines 63991 63897 -94
Branches 5896 5873 -23
=======================================
- Hits 49775 49697 -78
+ Misses 11532 11524 -8
+ Partials 2684 2676 -8
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There's test issue in all copr jobs with test_80_stats, and also this one |
ef016c4
to
71ba533
Compare
|
71ba533
to
791a607
Compare
That rpm-build:fedora-rawhide-s390x is just a flake, due to #959 |
I finally received Testing Farm API key (https://docs.testing-farm.io/general/0.1/onboarding.html). So that means we (anyone I share the key with) can also trigger the Testing Farm tests from their own machine, without going through the GH PR. Currently the Testing Farm only installs the RPMs.
|
4b33131
to
917e62c
Compare
https://copr.fedorainfracloud.org/coprs/jdanek/skupper-router/build/6086829/ Epel 8 fails on Python dependency, that is to be ignored. There is a segfault in rawhide arm64 test for symbolized backtrace. Then system_tests_tcp_adaptor_tls fails on s390x with I could enable running more tests (subsequent PR), and I guess more arches can be enabled now, besides aarch64, since the compilation failure is now fixed (either subsequent PR, or maybe right here). |
@@ -21,18 +21,51 @@ under the License | |||
|
|||
== rpm | |||
|
|||
The `rpkg` tool expects to have a `spec.rpkg` file at the source root. | |||
In order to build, first link the spec file to the expected location. | |||
The `packit` tool edits the `packaging/skupper-router.srpm` file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The `packit` tool edits the `packaging/skupper-router.srpm` file. | |
The `packit` tool edits the `packaging/skupper-router.spec` file. |
020b97d
to
30ea075
Compare
Note to self: this got stuck on the arm64 crash when libunwind is present |
Regarding builds in Packit (COPR), currently only aarch64 and x86_64 are passing. The s390x and ppc64le are failing on
unrecognized command-line option '-mno-omit-leaf-frame-pointer'
which is from #1074-mno-omit-leaf-frame-pointer
conditionally #1131, will do an RPM rebuild@kgiusti will be surely gladdened to see that libunwind seems to be available on all the architectures, there are no failures building the src.rpm caused by its absence.
https://copr.fedorainfracloud.org/coprs/jdanek/skupper-router/build/6017943/