greenwave

#204 Fix checking policies for specific item
Merged by lholecek. Opened by lholecek.
lholecek/greenwave fix-policy-checking  into  master

Download 204.patch

Results and waivers that Policy.check() takes as arguments should all be
related to given item.

This was broken in 6e23596 by passing results and waivers for all items.

I broke making decision for multiple items/packages - broken in greenwave-0.7.1. :(

I need to write some tests for this.

Hmmm. PR#184 effectively undoes commit 6e23596 too, I think. So we could just go with that instead...?

But certainly more test cases would not hurt.

Regardless, :+1: for this if we need to urgently roll out a fix.

:+1:

Pull-Request has been merged by lholecek
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.