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

COPR lost the ability to run fedora-review around 2023-10-13 #2953

Closed
smooge opened this issue Oct 13, 2023 · 3 comments
Closed

COPR lost the ability to run fedora-review around 2023-10-13 #2953

smooge opened this issue Oct 13, 2023 · 3 comments
Assignees

Comments

@smooge
Copy link

smooge commented Oct 13, 2023

I am working on a review for a package and using COPR to do the builds and run fedora-review. On Wednesday (2023-10-11) I was able to do a build and get a fedora-review working. When I tried to do one on Friday (2023-10-13), the build said it completed correctly, but no rpms were in the build and no fedora-review was seen. In the build.log the following message was found:

Last metadata expiration check: 0:04:22 ago on Fri Oct 13 16:37:34 2023.
Last metadata expiration check: 0:04:25 ago on Fri Oct 13 16:37:34 2023.
INFO:  ExclusiveArch dependency checking disabled, enable with EXARCH flag

WARNING: Illegal return from /usr/share/fedora-review/scripts/generic-large-docs.sh, code 80, output: stdout:Documentation size is 0 bytes in 1 files.
 stderr:/usr/share/fedora-review/scripts/generic-large-docs.sh: line 23: bc: command not found

Review template in: /var/lib/copr-rpmbuild/results/composefs/review.txt
fedora-review is automated tool, but *YOU* are responsible for manually
reviewing the results and finishing the review. Do not just copy-paste

https://copr.fedorainfracloud.org/coprs/smooge/composefs/builds/
working review: https://copr.fedorainfracloud.org/coprs/smooge/composefs/build/6517526/
broken review: https://copr.fedorainfracloud.org/coprs/smooge/composefs/build/6523959/

https://download.copr.fedorainfracloud.org/results/smooge/composefs/fedora-rawhide-x86_64/06523959-composefs/ is empty except for build logs.

Talked with praiskup on irc and it looks like some package which required bc no longer does so and so the review failed which stopped the final steps of copying in the rpms and such.

@praiskup
Copy link
Member

applied a patch in prod, does it help?

@praiskup
Copy link
Member

Per @smooge report, the work-around helped.

@praiskup praiskup moved this from Needs triage to In Progress in CPT Kanban Oct 19, 2023
@FrostyX
Copy link
Member

FrostyX commented Oct 21, 2023

Already fixed in the upstream
https://pagure.io/FedoraReview/c/ebfdc2bbbd2374fe7cbf9415051cc9d3f7253bdd

But we need to keep our patch until new fedora-review release.

@FrostyX FrostyX closed this as completed Oct 21, 2023
@praiskup praiskup moved this from In Progress to Done in CPT Kanban Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants