31 Commits

Author SHA1 Message Date
Stephan Kulow
2de74cfc52 Replace variable name 'l' (x is fine)
https://www.flake8rules.com/rules/E741.html
2022-02-18 14:21:59 +01:00
Stephan Kulow
1875f66a46 legal-auto: Fix regression in product import
the patchinfo handling was actually not from Marcus, it was from
5ff462044534d6fa3b9ecc40c4ab6541fd7538e7
2021-11-22 18:39:29 +01:00
Stephan Kulow
0f9478cbf9 legal-auto: Move the "ignore channels" feature in the proper function
The query_sources function is only used for product scan, which is not
relevant for request review. Fixing it to improve performance on live
patch reviews.
2021-11-22 14:50:50 +01:00
Marcus Meissner
2bd4ea9fd1 try to skip channel files as submitted with kernels and livepatches, as
they cannot be legal reviewed
2021-10-11 16:42:02 +02:00
Stephan Kulow
a8f7146baf Remove E713 offense from flaker 2021-09-21 14:20:08 +02:00
Stephan Kulow
1de383ea96 legal-auto: Reopen obsolete reviews while scanning projects 2021-04-06 08:18:51 +02:00
Stephan Kulow
a9a3247d66 legal: Only wait 2 hours for legal reviews for openSUSE:Factory
The churn rate of openSUSE:Factory is too high for legal to catch up
these days, so we turn the default. Legal reviews for Factory are no longer
blocking, but if we need the same package for another project or
product, the lawyers had some time to look at it.

And if we find a package not acceptable, we'll delete it from Factory
later on. This may get painful, but the current legal situation is
painful too.
2021-03-30 09:03:16 +02:00
Marcus Meissner
7fafa1efc8 fixed a whitespace 2020-06-05 16:47:35 +02:00
Marcus Meissner
268b65ff71 skip SUSE:Channel files harder
isc api /source/home:osalvador:branches:Devel:Kernel:SLE12-SP2-LTSS:Submit/?view=info

shows

  <sourceinfo package="SLE-SERVER_12-SP2-LTSS_x86_64" rev="3" vrev="113" srcmd5="8b9a4eef8f161a41c96b89c7163091a9" lsrcmd5="55f550c22886d610c2e386c727f30817" verifymd5="66e022e70e6e94f85f494918960fc0cb">
      <error>bad build configuration, no build type defined or detected</error>
      <linked project="SUSE:Channels" package="SLE-SERVER_12-SP2-LTSS_x86_64"/>
 </sourceinfo>
2020-06-02 15:36:46 +02:00
Marcus Meissner
222e2ff802 ignore packages that just have _channel inside 2020-05-15 18:18:22 +02:00
Stephan Kulow
e373d8d572 Fix projects import into legaldb
The pickle hash was not reliable - so it took way too long to reimport
2019-05-30 21:16:33 +02:00
Stephan Kulow
0d94a2d85e legal-auto: Port prints to logger calls 2019-05-28 07:01:46 +02:00
Stephan Kulow
a2e1094734 legal-auto: the pickle path needs to be a file 2019-05-28 06:38:49 +02:00
Stephan Kulow
af78e4eb50 Run legal-auto on botmaster as well
Fixes #1959
2019-05-27 18:02:15 +02:00
Stephan Kulow
595e84191e More porting to python3
Travis changed default python to 3.6, which has impact on the way
flake8 checks. So make sure we run fine under flake8 of python 3
2019-04-20 21:16:50 +02:00
Stephan Kulow
f45c0daeb4 legal: Fix crash on declining obsrq#670770 2019-02-22 17:06:41 +01:00
Stephan Kulow
ea9061ccf5 Import urllib2 python agnostic 2018-11-16 09:01:45 +01:00
Jimmy Berry
c47bb4c902 Utilize CacheManager for all existing caches. 2018-09-04 15:19:02 -05:00
Jimmy Berry
6069245350 Remove SUSE copyright, warranty, and license headers.
Distinct copyrights were left as I do not wish to track down commit
history to ensure it properly documents the copyright holders. Also left
non-GPLv2 licenses and left bs_copy untouched as a mirror from OBS.

Already have a mix of with and without headers and even OBS does not place
on majority of files. If SUSE lawyers have an issue it will come up in
legal review for Factory.
2018-08-23 19:18:06 -05:00
Stephan Kulow
b3c1b2bfb0 legal-auto: autopep8 2018-05-15 17:42:58 +02:00
Stephan Kulow
dfef8d8d78 Add retried_GET 2018-05-15 17:42:57 +02:00
Stephan Kulow
a60b10c91b Do not pass md5 but revision
Otherwise cavil can't expand the sources to multibuilds
2018-05-15 15:46:24 +02:00
Stephan Kulow
9122c96505 Use UTC for created 2018-05-15 15:46:24 +02:00
Stephan Kulow
5ff4620445 Add project sub command to legal auto
Scanning an OBS project and marking all legaldb reviews to belong to
that project. All unmarked reviews will disappear over time
2018-05-15 15:46:21 +02:00
Jimmy Berry
202d139743 legal-auto: replace check_action__default() with built-in. 2018-04-16 18:27:58 -05:00
Jimmy Berry
a40ebc1544 ReviewBot: provide comment command override.
For example:

  @leaper override accept
  @repo-checker override decline
2018-03-06 16:42:12 -06:00
Sebastian Riedel
37444ccbd2 fix a typo that currently breaks token authentication 2018-03-01 14:12:49 +01:00
Jimmy Berry
f98a78c1ae Remove Vim config in favor of EditorConfig. 2018-01-23 00:48:00 -06:00
Sebastian Riedel
d376c2efa3 legal-auto: allow for an authentication token to be sent to the legaldb backend with every HTTP request 2017-05-11 10:31:09 +02:00
Stephan Kulow
e79b551ccf Follow the legaldb priority from OBS priority 2017-03-29 17:20:50 +02:00
Stephan Kulow
20d78d2853 Add legal-auto bot talking to the legaldb API 2017-03-28 12:21:09 +02:00