-
Notifications
You must be signed in to change notification settings - Fork 63
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
Packit build failures after merge #2884
Comments
hi @praiskup ! Are all the failures false positives? |
I can see failed builds since fc73a83 perhaps it's due to the edit: I guess |
So it seems like parsed spec file in https://download.copr.fedorainfracloud.org/results/packit/copr/fedora-39-x86_64/06266070-python-copr-common/ and its SRPM is different (look to the spec file in SRPM and spec file inside .tar.gz in SRPM - their version differs) - that's responsible for the majority of errors another bug I found (some time ago) and is responsible for the first fail in https://copr.fedorainfracloud.org/coprs/packit/copr/package/python-copr-common/ is that versions in spec file in SRPM and in tar.gz (inside SRPM) sometimes differ - usually when I submit a PR with new pkg version then it has still its lower version and one more push to PR/new PR to catch up/refresh the version is needed but I have no idea whether this is our or packit/specfile issue |
also, I don't think it's a ci-blocker since we have our CI still running in parallel |
For some reason, Packit generates version
Well I need to have green CI mark in recent commits, otherwise it blocks my brain. :-) |
E.g. here it seems that |
#2956 likely fixes this! |
And all green, finally! |
Thanks @lachmanfrantisek :) |
We have "negative" CI karma for quite some time:
https://github.com/fedora-copr/copr/commits/main
What if we disabled Packit for the time being for pushes?
cc @lachmanfrantisek @lbarcziova
The text was updated successfully, but these errors were encountered: