3206 Commits

Author SHA1 Message Date
Jimmy Berry
a8919a516a cycle: cycles() rename group parameter to staging to better reflect usage. 2017-08-24 21:48:00 -05:00
Jimmy Berry
a07728fe39 repo_checker: rework to handle multiple stagings per group (ie :DVD). 2017-08-24 21:47:56 -05:00
Jimmy Berry
ffb81c445e core: add memoize(session=True) to package_binary_list(). 2017-08-24 21:47:09 -05:00
Jimmy Berry
cfc21469fd stagingapi: provide staging_walk() to cleanly handle :DVD subproject. 2017-08-24 21:47:05 -05:00
Jimmy Berry
acced3a66c osclib/conf: set staging-dvd-archs to empty for SUSE:* projects.
Currently none of the SUSE:* projects have :DVD subprojects and there is
no way to know short of checking if the project exists. This allows for
len(staging-dvd-archs) to be used to know.
2017-08-24 21:45:11 -05:00
Jimmy Berry
2345965455 repo_checker: replace separate ignore methods with new algorithm. 2017-08-24 21:44:45 -05:00
Jimmy Berry
3e4c0ecebd repo_checker.pl: support multiple directories to check. 2017-08-24 21:44:45 -05:00
Jimmy Berry
80983fd0b0 repo_checker: move result_comment() headers into respective check methods.
This will be necessary for proper context in future commits and seems to
make more sense.
2017-08-24 21:44:45 -05:00
Jimmy Berry
1a51818ab5 repo_checker: only bother to parse if --post-comments. 2017-08-24 21:44:45 -05:00
Jimmy Berry
2aae21d3ed Merge pull request #1095 from jberry-suse/repo_checker-explicit-build-requirement
repo_checker: delete: explicitly indicate build requirement.
2017-08-24 21:13:23 -05:00
Jimmy Berry
a0a461fb2b Merge pull request #1094 from jberry-suse/repo_checker-delete-ignore
repo_checker: delete: do not decline instead ignore.
2017-08-24 21:06:45 -05:00
Jimmy Berry
c0ede24144 Merge pull request #1092 from jberry-suse/repo_checker-pre
repo_checker: use <pre> instead of ``` to avoid html entity escaping.
2017-08-24 20:51:54 -05:00
Jimmy Berry
55bfb95d8f Merge pull request #1091 from jberry-suse/repo_checker.pl-safer-parse
repo_checker.pl: ensure that parser does not bleed over to other sections.
2017-08-24 20:51:36 -05:00
Jimmy Berry
fc1b0dc124 Merge pull request #1090 from jberry-suse/cycle-drop-ignore
cycle: remove ugly ignore code from _get_builddepinfo_graph().
2017-08-24 20:51:01 -05:00
Jimmy Berry
1862f76891 Merge pull request #1089 from jberry-suse/commentapi-truncate
commentapi: provide truncate() and apply during add_comment() (and use in repo_checker).
2017-08-24 20:50:16 -05:00
Jimmy Berry
c71e075396 Merge pull request #1088 from jberry-suse/ReviewBot-only_replace
repo_checker: post passed comment only if previous failed comment (and debug improvements).
2017-08-24 20:48:59 -05:00
Jimmy Berry
83382871c1 Merge pull request #1087 from jberry-suse/ReviewBot-logger-format
ReviewBot: simplify logger format.
2017-08-24 20:44:01 -05:00
Ludwig Nussel
7bbd005248 Merge pull request #1097 from lnussel/master
Use configured staging group for adi
2017-08-24 13:14:37 +02:00
Ludwig Nussel
adeb4c4dfe Merge pull request #1080 from nilxam/freeze_adi
Freeze Cmd: Implement freezing adi staging project
2017-08-24 12:54:43 +02:00
Ludwig Nussel
8cbb51891d Use configured staging group for adi 2017-08-24 12:01:13 +02:00
Jimmy Berry
9f78c818cd repo_checker: delete: explicitly indicate build requirement. 2017-08-23 17:17:37 -05:00
Jimmy Berry
05dd5c9130 repo_checker: delete: do not decline instead ignore. 2017-08-23 17:12:08 -05:00
Jimmy Berry
49c0c5b19f repo_checker: use <pre> instead of ``` to avoid html entity escaping. 2017-08-23 16:43:25 -05:00
Jimmy Berry
7f0c3262fe repo_checker.pl: ensure that parser does not bleed over to other sections. 2017-08-23 16:34:16 -05:00
Jimmy Berry
8ec224169c cycle: remove ugly ignore code from _get_builddepinfo_graph().
With the new repo_checker utilizing the staging process none of this
should be necessary.
2017-08-23 16:02:48 -05:00
Jimmy Berry
8dfc27d912 repo_checker: package_comments(): drop truncation in favor of commentapi. 2017-08-23 15:59:27 -05:00
Jimmy Berry
bf2832aaf2 commentapi: provide truncate() and apply during add_comment(). 2017-08-23 15:59:22 -05:00
Jimmy Berry
ad9f76bf46 repo_checker: post passed comment only if previous failed comment. 2017-08-23 15:30:08 -05:00
Jimmy Berry
ebd2f580ec ReviewBot: comment_write(): provide only_replace option. 2017-08-23 15:19:36 -05:00
Jimmy Berry
ca13855d2e ReviewBot: comment_write(): pull dryrun protected bit out individually. 2017-08-23 15:17:44 -05:00
Jimmy Berry
757d31cf35 ReviewBot: comment_write(): place debug_key at the end for consistency. 2017-08-23 15:15:02 -05:00
Jimmy Berry
7502d6ab74 Merge pull request #1082 from jberry-suse/stagingapi-update_status_comments-no-blank
stagingapi: avoid making an empty package-diff comment.
2017-08-23 15:05:47 -05:00
Jimmy Berry
f88e8a9297 ReviewBot: simplify logger format.
The output is rather verbose, especially inside journals, and not terribly
helpful.
2017-08-23 15:00:33 -05:00
773ce3904b Freeze Cmd: Implement freezing adi staging project
Since we have introduced the source project expand function in adi staging
project, a new function to freeze a adi staging project is needed as well.
2017-08-23 17:09:42 +08:00
Jimmy Berry
55aeefacb4 Merge pull request #1073 from jberry-suse/check_source-remove-diff
check_source: remove DIFFCOUNT algorithm and instead always add review_team.
2017-08-22 17:06:59 -05:00
Jimmy Berry
2873a27ef1 Merge pull request #1075 from jberry-suse/owner_fallback-startswith
core.owner_fallback(): check if startswith() instead of equal.
2017-08-22 17:02:20 -05:00
Jimmy Berry
65d0de2eca stagingapi: avoid making an empty package-diff comment. 2017-08-22 15:33:03 -05:00
09b046f852 Merge pull request #1076 from jberry-suse/ReviewBot-get_devel_project-raise
ReviewBot: get_devel_project(): correct error handling to only allow 404.
2017-08-22 18:24:30 +08:00
9bd2789d08 Merge pull request #1077 from nilxam/links_only_valid
stagingapi: Ensure the links is valid before processing
2017-08-22 17:04:47 +08:00
940edfea4b Merge pull request #1078 from Vogtinator/staging-d
Adjust for staging :D
2017-08-22 17:00:55 +08:00
2ad8e87c9b Remove sw_management_kde pattern from test-2 DVD
Merged into kde_yast.
2017-08-22 10:07:35 +02:00
05f20c2fbd Add amarok to test-2 DVD
It's installed by the test runs so needs to be available.
2017-08-22 10:06:24 +02:00
29939067bf stagingapi: Ensure the links is valid before processing
Make sure the links of package in Rings must are valid before processing,
otherwise someone had package linked to the Ring package in somewhere else than
Rings will breaks select/delete/rm_to_prj, ie. _link will be created. Therefore,
only return the candidate if it has proper linked from another ring and have
different package name.
2017-08-22 04:52:30 +08:00
Jimmy Berry
164b04903c ReviewBot: add_review(): remove left-over print() line. 2017-08-18 17:35:05 -05:00
Jimmy Berry
d565f46123 ReviewBot: get_devel_project(): correct error handling to only allow 404.
The original intent was clearly to allow 404 (package does not exist for
new packages), but crash on anything else. Instead it consumes all
exceptions and does nothing different even with the e.code condition.
2017-08-18 17:34:32 -05:00
Jimmy Berry
702f6a6243 core.owner_fallback(): check if startswith() instead of equal.
This allows for cases where the requested project was something like
openSUSE:Leap:42.3:Update and the returned project was openSUSE:Leap:42.3.
With this change the fallback will be triggered and the global maintainer
will then be used.
2017-08-18 16:03:04 -05:00
Jimmy Berry
6afbc550ad check_source: rename perl and asciidoc to match name. 2017-08-18 15:18:04 -05:00
Jimmy Berry
0980989206 check_source: remove DIFFCOUNT algorithm and instead always add review_team. 2017-08-18 15:15:26 -05:00
Jimmy Berry
f230438237 Merge pull request #1072 from jberry-suse/dist-dockerfile-dup
dist/ci/Dockerfile: include `dup` before `in` to handle major base changes.
2017-08-18 15:06:08 -05:00
Jimmy Berry
4c8c478fca dist/ci/Dockerfile: include dup before in to handle major base changes. 2017-08-18 15:00:40 -05:00