waiverdb

#194 Allow creating multiple waivers
Merged by lholecek. Opened by lholecek.
lholecek/waiverdb create-multiple-waivers  into  master

Download 194.patch

~~Introduces optional "waivers" POST parameter for "waivers/" which allows
to create multiple waivers - value should be list of dicts.~~

To create multiple waivers, POST list to "waivers/" instead of single waiver ([{...}, ...] instead of {...}).

Fixes #98

/cc @bowlofeggs - this is for you. :)

Oops, the commit message is wrong, there is no "waivers" parameter (that was my original plan). You only need to POST list of waivers ([{...}, ...] instead of {...}).

Let me fix the commit message.

rebased onto e2e83c853edde9564eee185e796b919f56624f72

:+1:, verified with Jenkins! Free to merge.

Hmm, this breaks functional tests in greenwave. Need to figure out why.

rebased onto 27ba71d97cc02bcd1cc1a6db8c24d1042e52908c

Hmm, this breaks functional tests in greenwave. Need to figure out why.

OK, the problem was session.nested_begin() which didn't work well. I replaced it with session.add_all() and session.commit().

:+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.