Jimmy Berry
8ebf07b139
unselect: suggest marking a request ignored if eligible to restage.
2017-02-15 21:46:28 -06:00
Jimmy Berry
87cb5c2ab0
ReviewBot: drop duplicate import namedtuple.
2017-02-15 10:01:36 -06:00
Jimmy Berry
748fce38cc
ReviewBot & leaper: provide deduplicate method for leaper comment.
...
Otherwise, leaper will indicate an unhandled request twice.
2017-02-15 10:01:20 -06:00
Jimmy Berry
dcf435430c
leaper: utilize ReviewBot check_action__default() message.
2017-02-15 09:45:18 -06:00
Jimmy Berry
23d4a7a6a5
ReviewBot: check_action__default() should return None not set ret.
2017-02-15 09:44:31 -06:00
Jimmy Berry
b0f4a5fe2b
leaper: drop extra definition of check_action__default().
...
The second definition being the current behavior and added later would seem
to be the desired implementation.
2017-02-15 09:05:45 -06:00
Jimmy Berry
8e842e507c
check_tags: grammatical corrections.
2017-02-15 08:12:15 -06:00
Jimmy Berry
dfd98dab30
check_tags: warn when issue references are removed.
2017-02-15 08:12:15 -06:00
Jimmy Berry
7ddd4287e1
check_tags: use len() instead of list() since only count is of interest.
2017-02-15 08:12:15 -06:00
Jimmy Berry
c1fde90c64
check_tags: correct backwards diff.
...
Issues added in a request show as removed and visa versa.
2017-02-15 08:12:15 -06:00
Jimmy Berry
1ab3f846a5
check_tags: handle failed to load issue diff for existing package.
...
The previous code would crash on ET.parse(f) when the request failed
and the package existed.
2017-02-15 08:12:13 -06:00
Jimmy Berry
de9f94cae5
ReviewBot: extract __class__.__name__ as default for self.bot_name.
...
Allow future bots to change the name used to represent the bot.
2017-02-15 07:55:35 -06:00
f45fb1fd62
Do not add the package exists in the previous version
2017-02-15 19:08:46 +08:00
Ludwig Nussel
b3129e9353
Merge pull request #690 from nilxam/adjust_fcc_423
...
Adjust FCC_Submitter for 42.3
2017-02-15 10:02:19 +01:00
Ludwig Nussel
5b61b92808
Merge pull request #684 from jberry-suse/leaper-comment-to-reviewbot
...
ReviewBot: refactor leaper comment from log functionality.
2017-02-15 09:59:32 +01:00
Ludwig Nussel
10d4abb077
Merge pull request #683 from jberry-suse/update_status_comments-dashboard
...
stagingapi: update_status_comments() include link to dashboard.
2017-02-15 09:56:32 +01:00
Ludwig Nussel
7502dbfff8
Merge pull request #689 from lnussel/ttm
...
Only output timestamps when run on tty
2017-02-15 09:54:07 +01:00
b1e781436c
Adjust FCC_Submitter for 42.3
2017-02-14 21:03:03 +08:00
Ludwig Nussel
924d05b198
Merge pull request #657 from DimStar77/statedir
...
Announcer: put the state of the last version into /var/lib
2017-02-14 13:44:29 +01:00
Ludwig Nussel
e5c5a07c3b
Merge pull request #686 from jberry-suse/document-staging-commands
...
osc-staging: add missing documentation and correct/clean existing
2017-02-14 13:40:02 +01:00
Ludwig Nussel
125eb4ddf3
Only output timestamps when run on tty
...
If not connected to a tty output probably goes to systemd's journal.
2017-02-14 13:30:14 +01:00
Ludwig Nussel
a5df36aaab
Merge pull request #685 from jberry-suse/ReviewBot-super
...
ReviewBot: use super().check_source_submission() in subclasses.
2017-02-14 09:21:16 +01:00
Ludwig Nussel
a58a063c0f
Merge pull request #682 from jberry-suse/check_source-self-submission
...
check_source: allow self-submission.
2017-02-14 09:20:18 +01:00
5289d80a18
Merge pull request #688 from nilxam/proper_status_comment
...
adi: Do not update status comment per request but project
2017-02-13 19:39:03 +08:00
9a322d712e
adi: Do not update status comment per request but project
2017-02-13 18:11:20 +08:00
0f188daf6a
Merge pull request #681 from jberry-suse/adi-move
...
adi: map wanted_requests to str() before passing to splitter.
2017-02-13 16:02:21 +08:00
Jimmy Berry
7307b8ec57
osc-staging: drop extraneous space in freeze method call.
2017-02-10 17:23:40 -06:00
Jimmy Berry
b37fd2e094
osc-staging: adi command min should be 0 for consistency.
2017-02-10 17:23:40 -06:00
Jimmy Berry
64ef24a8a7
osc-staging: check command does not support 2 arguments.
2017-02-10 17:23:40 -06:00
Jimmy Berry
2a562b89af
osc-staging: add missing documentation and correct/clean existing.
2017-02-10 17:23:32 -06:00
Jimmy Berry
92a5f60369
ReviewBot: use super().check_source_submission() in subclasses.
...
No need to duplicate the log message in multiple places.
2017-02-10 16:09:51 -06:00
Jimmy Berry
015d59cdc7
ReviewBot: refactor leaper comment from log functionality.
...
Simple interface:
- comment_handler_add(): Add handler to start recording log messages
for comment.
- comment_write(): Write comment from log messages if not similar to
previous comment.
See leaper.py for example usage.
2017-02-10 15:58:57 -06:00
Jimmy Berry
55c4382248
stagingapi: update_status_comments() include link to dashboard.
2017-02-10 11:48:59 -06:00
Jimmy Berry
34f5006ab5
check_source: allow self-submission.
2017-02-10 11:19:25 -06:00
Jimmy Berry
997f929ad4
adi: map wanted_requests to str() before passing to splitter.
2017-02-10 10:05:38 -06:00
6eefb429d3
Merge pull request #675 from DimStar77/CaaSP
...
check_source: declare devel:CaaSP as devel project
2017-02-10 15:46:04 +01:00
Jimmy Berry
4c5ebffe98
Merge pull request #670 from jberry-suse/issue-diff-author-and-text
...
issue-diff: use bugowner or maintainer as assignee for bug (and small changes)
2017-02-09 12:07:23 -06:00
8cab152ce3
Merge pull request #676 from jberry-suse/sle-list
...
request_splitter: house keeping (and fix for SLE [non-ring] workflow)
2017-02-09 16:12:26 +08:00
Jimmy Berry
d44ea40707
issue-diff: update description to include a short reason and link.
2017-02-08 16:48:43 -06:00
Jimmy Berry
5da8f897a1
request_splitter: s/available/remaining/ for a more accurate label.
2017-02-08 16:45:39 -06:00
Jimmy Berry
ef5de87c5d
request_splitter: delete requests should always be considered in a ring.
2017-02-08 13:54:01 -06:00
Jimmy Berry
08f2218574
request_splitter: list: remove case for delete request without ring.
...
This case should no longer exist as non-ring requests will have already
be separated out of this loop.
2017-02-08 13:20:36 -06:00
Jimmy Berry
9f28923ba5
request_splitter: non-ring projects consider all requests in rings.
2017-02-08 13:19:08 -06:00
Jimmy Berry
3de3cae13a
request_splitter: utilize ring attribute rather then lookup again.
2017-02-08 13:18:21 -06:00
Jimmy Berry
404a3efe58
request_splitter: rather than an empty group name use 00.
...
Sorts to the top and is consistent with old behavior.
2017-02-08 13:15:52 -06:00
Stephan Kulow
c41e042306
Fix typo in openqa bot
2017-02-08 19:58:28 +01:00
cc09fe3d0b
check_source: declare devel:CaaSP as devel project
2017-02-08 19:44:45 +01:00
1bd141a928
Merge pull request #673 from jberry-suse/source-checker-empty-changes
...
source-checker: handle add/remove changes files for patch diff check.
2017-02-08 19:01:14 +01:00
Jimmy Berry
a1ef0c1498
source-checker: handle add/remove changes files for patch diff check.
...
See sr#455547 which now functions properly.
2017-02-08 11:51:57 -06:00
Ludwig Nussel
a782d74a81
Merge pull request #672 from coolo/no_testing_notification
...
maintenance: Do not send notifications for *starting* to test
2017-02-08 16:52:59 +01:00
