27 Commits

Author SHA1 Message Date
Ludwig Nussel
b6c37df2be Change debug message it info message
for consistency with other messages
2016-07-29 10:29:38 +02:00
Stephan Kulow
cce9cf3c6a Trigger the repo runs independent of the reviews
Just use the generated build number while fetching openqa jobs. This way
we can trigger test runs even if there are no incidents to review to
verify the tests still work on maintained products
2016-07-28 15:26:14 +02:00
Ludwig Nussel
b5bafc4299 Log exception with traceback
Helps to find place where the loop dies
2016-06-17 10:59:29 +02:00
Ludwig Nussel
26ac754a0a Include review message in verbose output 2016-06-10 10:59:36 +02:00
Ludwig Nussel
d16194caea Use decline message also on fallback
In fallback mode the bot would laccept a request on failure. It should also put the failure message in that case.
2016-06-07 15:12:02 +02:00
Ludwig Nussel
e4ab75ce7d Implement default action
The default action can be reimplemented by bots to e.g. auto fail or
auto accept.
2016-06-01 13:59:35 +02:00
Ludwig Nussel
ee9c194301 Fix review-mode
setup_checker gets reimplemented in subclasses. So implement the
review-mode in a way that doesn't require every bot to copy the code
2016-03-17 16:16:36 +01:00
Ludwig Nussel
f6c5d55eeb Fix review concerns 2016-03-10 14:08:38 +01:00
Ludwig Nussel
fcdc95ca16 Implement configurable review mode
The new parameter --review-mode allows to specify how the review bot should handle obs reviews

normal: accept review if check is good, decline if bad
no: don't accept or decline requests
accept: always accept review, even if check fails
fallback-onfail: accept review and set other user or group as review if check fails
fallback-always: always accept review and set other user or group as reviewer afterwards
2016-03-09 14:21:48 +01:00
Ludwig Nussel
1106da20cc move by project function to ReviewBot 2015-11-23 16:09:22 +01:00
Ludwig Nussel
152800dd6a configure level in global logger
when the level is in the global logger other modules that also use
logging inherit it.
2015-11-23 16:08:39 +01:00
Ludwig Nussel
575cfeb9ed reset list of requests on each run 2015-10-05 10:49:45 +02:00
Ludwig Nussel
efbcd75064 move check by project function to ReviewBot 2015-09-29 08:44:30 +02:00
6f7e3f2a02 Fixes the logic error in checkTagNotRequired() 2015-07-09 20:04:31 +08:00
Ludwig Nussel
7cbedd5e19 patchinfos have no release project! 2015-05-28 13:08:36 +02:00
Ludwig Nussel
e69a762f4d runner function for intervals 2015-05-28 13:08:29 +02:00
Ludwig Nussel
8c75e543e1 include request id in log 2015-05-28 13:08:15 +02:00
Ludwig Nussel
237fbd4b71 properly call parent function get_optparser 2015-05-28 13:07:51 +02:00
Ludwig Nussel
47b09f6601 fix maintenance incident request handling 2015-05-28 13:07:11 +02:00
Ludwig Nussel
d5ce84929d introduce DEFAULT_REVIEW_MESSAGES for easier reset to default 2015-05-28 13:06:50 +02:00
Ludwig Nussel
70536d74c7 use memoize to cache source info
used by get_originproject
2015-05-28 13:06:10 +02:00
Ludwig Nussel
a4bf2657d4 refactor _get_verifymd5 to get_sourceinfo 2015-05-28 13:04:08 +02:00
Stephan Kulow
2219c51a30 check group in can_accept_review 2015-03-27 15:12:52 +01:00
Stephan Kulow
8e1bcf066b allow groups to be reviewer too 2015-03-27 15:01:45 +01:00
Ludwig Nussel
39362a03f3 refactor common functions into ReviewBot 2015-03-19 16:42:18 +01:00
Ludwig Nussel
a490d77074 handle multiple reviews by same user
if a review for the maintbot user is added for a second time the old
function would only see the  'accepted' state of the first review and
the bot would not change the review state.
2015-02-20 13:17:47 +01:00
Ludwig Nussel
a590787e33 new review bots factory-source and maintbot
factory-source checks if submissions contain sources already accepted in
Factory. maintbot makes sure maintenance submissions are authored by the
Factory package maintainer.

Both are based on a new generic ReviewBot.py class that can serve as
common framework for review bots.
2014-11-25 14:36:22 +01:00