Hello community,

here is the log from the commit of package openSUSE-release-tools for 
openSUSE:Factory checked in at 2018-08-18 00:03:45
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/openSUSE-release-tools (Old)
 and      /work/SRC/openSUSE:Factory/.openSUSE-release-tools.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "openSUSE-release-tools"

Sat Aug 18 00:03:45 2018 rev:120 rq:629468 version:20180815.cec5fa6

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/openSUSE-release-tools/openSUSE-release-tools.changes
    2018-08-15 10:39:04.160365493 +0200
+++ 
/work/SRC/openSUSE:Factory/.openSUSE-release-tools.new/openSUSE-release-tools.changes
       2018-08-18 00:03:47.331019087 +0200
@@ -1,0 +2,14 @@
+Wed Aug 15 21:00:52 UTC 2018 - [email protected]
+
+- Update to version 20180815.cec5fa6:
+  * repo_checker: whitelist_clean(): correct attribute_value_save().
+
+-------------------------------------------------------------------
+Tue Aug 14 20:32:04 UTC 2018 - [email protected]
+
+- Update to version 20180814.f629ed8:
+  * ReviewBot: check_one_request(): correct logic to return "lowest" result.
+  * leaper: remove only_one_action limitation.
+  * leaper: provide maintenance_incident support.
+
+-------------------------------------------------------------------

Old:
----
  openSUSE-release-tools-20180814.17dddb4.obscpio

New:
----
  openSUSE-release-tools-20180815.cec5fa6.obscpio

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ openSUSE-release-tools.spec ++++++
--- /var/tmp/diff_new_pack.N96Uuz/_old  2018-08-18 00:03:49.027024156 +0200
+++ /var/tmp/diff_new_pack.N96Uuz/_new  2018-08-18 00:03:49.051024228 +0200
@@ -20,7 +20,7 @@
 %define source_dir openSUSE-release-tools
 %define announcer_filename factory-package-news
 Name:           openSUSE-release-tools
-Version:        20180814.17dddb4
+Version:        20180815.cec5fa6
 Release:        0
 Summary:        Tools to aid in staging and release work for openSUSE/SUSE
 License:        GPL-2.0-or-later AND MIT

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.N96Uuz/_old  2018-08-18 00:03:49.087024335 +0200
+++ /var/tmp/diff_new_pack.N96Uuz/_new  2018-08-18 00:03:49.091024347 +0200
@@ -1,6 +1,6 @@
 <servicedata>
   <service name="tar_scm">
     <param 
name="url">https://github.com/openSUSE/openSUSE-release-tools.git</param>
-    <param 
name="changesrevision">e46d9b0910ccef84a96223929008389dd196c2b6</param>
+    <param 
name="changesrevision">07bc6c031a0783b17e5cfefd3e0c0db323ee6c3b</param>
   </service>
 </servicedata>

++++++ openSUSE-release-tools-20180814.17dddb4.obscpio -> 
openSUSE-release-tools-20180815.cec5fa6.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/openSUSE-release-tools-20180814.17dddb4/ReviewBot.py 
new/openSUSE-release-tools-20180815.cec5fa6/ReviewBot.py
--- old/openSUSE-release-tools-20180814.17dddb4/ReviewBot.py    2018-08-14 
22:16:44.000000000 +0200
+++ new/openSUSE-release-tools-20180815.cec5fa6/ReviewBot.py    2018-08-15 
22:56:03.000000000 +0200
@@ -330,7 +330,7 @@
         if self.comment_handler is not False:
             self.comment_handler_add()
 
-        overall = None
+        overall = True
         for a in req.actions:
             # Store in-case sub-classes need direct access to original values.
             self.action = a
@@ -340,8 +340,14 @@
                 fn = 'check_action__default'
             func = getattr(self, fn)
             ret = func(req, a)
-            if ret == False or overall is None and ret is not None:
-                overall = ret
+
+            # In the case of multiple actions take the "lowest" result where 
the
+            # order from lowest to highest is: False, None, True.
+            if overall is not False:
+                if ((overall is True and ret is not True) or
+                    (overall is None and ret is False)):
+                    overall = ret
+
         return overall
 
     @staticmethod
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/openSUSE-release-tools-20180814.17dddb4/leaper.py 
new/openSUSE-release-tools-20180815.cec5fa6/leaper.py
--- old/openSUSE-release-tools-20180814.17dddb4/leaper.py       2018-08-14 
22:16:44.000000000 +0200
+++ new/openSUSE-release-tools-20180815.cec5fa6/leaper.py       2018-08-15 
22:56:03.000000000 +0200
@@ -48,7 +48,6 @@
         ReviewBot.ReviewBot.__init__(self, *args, **kwargs)
 
         # ReviewBot options.
-        self.only_one_action = True
         self.request_default_return = True
         self.comment_handler = True
 
@@ -225,6 +224,27 @@
 
             return review_result
 
+        if target_project.endswith(':Update'):
+            self.logger.info("expected origin is '%s' (%s)", origin,
+                             "unchanged" if origin_same else "changed")
+
+            if origin_same:
+                return True
+
+            good = self._check_matching_srcmd5(origin, target_package, 
src_srcinfo.verifymd5)
+            if good:
+                self.logger.info('submission source found in origin 
({})'.format(origin))
+                return good
+            good = self.factory._check_requests(origin, target_package, 
src_srcinfo.verifymd5)
+            if good or good == None:
+                self.logger.info('found pending submission against origin 
({})'.format(origin))
+                return good
+
+            return None
+        elif self.action.type == 'maintenance_incident':
+            self.logger.debug('unhandled incident pattern (targetting non 
:Update project)')
+            return True
+
         # obviously
         if src_project in ('openSUSE:Factory', 'openSUSE:Factory:NonFree'):
             self.source_in_factory = True
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20180814.17dddb4/repo_checker.py 
new/openSUSE-release-tools-20180815.cec5fa6/repo_checker.py
--- old/openSUSE-release-tools-20180814.17dddb4/repo_checker.py 2018-08-14 
22:16:44.000000000 +0200
+++ new/openSUSE-release-tools-20180815.cec5fa6/repo_checker.py 2018-08-15 
22:56:03.000000000 +0200
@@ -594,7 +594,7 @@
         else:
             print('y')
 
-        api.attribute_value_save('Config', ''.join(config_new), 'repo_checker 
whitelist clean')
+        api.attribute_value_save('Config', ''.join(config_new))
 
     def whitelist_clean_set(self, config, key, value):
         # Unfortunately even OscConfigParser does not preserve empty lines.

++++++ openSUSE-release-tools.obsinfo ++++++
--- /var/tmp/diff_new_pack.N96Uuz/_old  2018-08-18 00:03:49.995027049 +0200
+++ /var/tmp/diff_new_pack.N96Uuz/_new  2018-08-18 00:03:49.999027061 +0200
@@ -1,5 +1,5 @@
 name: openSUSE-release-tools
-version: 20180814.17dddb4
-mtime: 1534277804
-commit: 17dddb4d4d3b8e332df22af2b1b6b3a15291e09b
+version: 20180815.cec5fa6
+mtime: 1534366563
+commit: cec5fa69297b5c3221001e91a182f3e0a6ea2c66
 


Reply via email to