Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package openSUSE-release-tools for 
openSUSE:Factory checked in at 2022-02-22 21:18:10
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/openSUSE-release-tools (Old)
 and      /work/SRC/openSUSE:Factory/.openSUSE-release-tools.new.1958 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "openSUSE-release-tools"

Tue Feb 22 21:18:10 2022 rev:381 rq:956616 version:20220222.d8ef5897

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/openSUSE-release-tools/openSUSE-release-tools.changes
    2022-02-21 17:49:02.291637569 +0100
+++ 
/work/SRC/openSUSE:Factory/.openSUSE-release-tools.new.1958/openSUSE-release-tools.changes
  2022-02-22 21:18:45.134292416 +0100
@@ -1,0 +2,6 @@
+Tue Feb 22 07:43:39 UTC 2022 - opensuse-releaset...@opensuse.org
+
+- Update to version 20220222.d8ef5897:
+  * Remove sentry integration
+
+-------------------------------------------------------------------

Old:
----
  openSUSE-release-tools-20220221.3e14e55f.obscpio

New:
----
  openSUSE-release-tools-20220222.d8ef5897.obscpio

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

Other differences:
------------------
++++++ openSUSE-release-tools.spec ++++++
--- /var/tmp/diff_new_pack.nlXGtl/_old  2022-02-22 21:18:46.014292577 +0100
+++ /var/tmp/diff_new_pack.nlXGtl/_new  2022-02-22 21:18:46.018292578 +0100
@@ -20,7 +20,7 @@
 %define source_dir openSUSE-release-tools
 %define announcer_filename factory-package-news
 Name:           openSUSE-release-tools
-Version:        20220221.3e14e55f
+Version:        20220222.d8ef5897
 Release:        0
 Summary:        Tools to aid in staging and release work for openSUSE/SUSE
 License:        GPL-2.0-or-later AND MIT
@@ -238,8 +238,6 @@
 Requires:       %{name} = %{version}
 Requires:       osc >= 0.165.1
 Requires:       python3-osc
-# internal API change related to accessing DSN in osclib/sentry.py
-Suggests:       python3-sentry-sdk >= 0.11.0
 BuildArch:      noarch
 
 %description -n osclib

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.nlXGtl/_old  2022-02-22 21:18:46.062292586 +0100
+++ /var/tmp/diff_new_pack.nlXGtl/_new  2022-02-22 21:18:46.062292586 +0100
@@ -1,7 +1,7 @@
 <servicedata>
   <service name="tar_scm">
     <param 
name="url">https://github.com/openSUSE/openSUSE-release-tools.git</param>
-    <param 
name="changesrevision">3e14e55fdee2392f011f3b2c90c786263b13bf62</param>
+    <param 
name="changesrevision">d6fb5d680c010d572094a57bdefe3f4fe4f5492b</param>
   </service>
 </servicedata>
 

++++++ openSUSE-release-tools-20220221.3e14e55f.obscpio -> 
openSUSE-release-tools-20220222.d8ef5897.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/ReviewBot.py 
new/openSUSE-release-tools-20220222.d8ef5897/ReviewBot.py
--- old/openSUSE-release-tools-20220221.3e14e55f/ReviewBot.py   2022-02-21 
12:28:02.000000000 +0100
+++ new/openSUSE-release-tools-20220222.d8ef5897/ReviewBot.py   2022-02-22 
08:41:46.000000000 +0100
@@ -18,7 +18,6 @@
 from osclib.core import request_age
 from osclib.memoize import memoize
 from osclib.memoize import memoize_session_reset
-from osclib.sentry import sentry_init
 from osclib.stagingapi import StagingAPI
 import signal
 import datetime
@@ -32,9 +31,6 @@
 
 from itertools import count
 
-# In-case not properly initialized via the CommandLineInterface.
-sentry_sdk = sentry_init()
-
 
 class PackageLookup(object):
     """ helper class to manage 00Meta/lookup.yml
@@ -186,8 +182,6 @@
         for req in self.requests:
             self.logger.info("checking %s" % req.reqid)
             self.request = req
-            with sentry_sdk.configure_scope() as scope:
-                scope.set_extra('request.id', self.request.reqid)
 
             # XXX: this is a hack. Annotating the request with staging_project.
             # OBS itself should provide an API for that but that's currently 
not the case
@@ -202,15 +196,13 @@
 
             try:
                 good = self.check_one_request(req)
-            except Exception as e:
+            except Exception:
                 good = None
 
                 import traceback
                 traceback.print_exc()
                 return_value = 1
 
-                sentry_sdk.capture_exception(e)
-
             if self.review_mode == 'no':
                 good = None
             elif self.review_mode == 'accept':
@@ -439,8 +431,6 @@
             # Store in-case sub-classes need direct access to original values.
             self.action = a
             key = request_action_key(a)
-            with sentry_sdk.configure_scope() as scope:
-                scope.set_extra('action.key', key)
 
             override = self.request_override_check()
             if override is not None:
@@ -897,11 +887,6 @@
         if self.options.fallback_group:
             self.checker.fallback_group = self.options.fallback_group
 
-        sentry_init(conf.config['apiurl'], {
-            'review_bot': self.clazz.__name__,
-            'review_user': self.checker.review_user,
-        })
-
     def setup_checker(self):
         """ reimplement this """
         apiurl = conf.config['apiurl']
@@ -991,8 +976,6 @@
                 self.logger.info("sleeping %d minutes." % interval)
                 time.sleep(interval * 60)
 
-            sentry_sdk.flush()
-
             # Reset all memoize session caches which are designed for single
             # tool run and not extended usage.
             memoize_session_reset()
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/dist/package/openSUSE-release-tools.spec
 
new/openSUSE-release-tools-20220222.d8ef5897/dist/package/openSUSE-release-tools.spec
--- 
old/openSUSE-release-tools-20220221.3e14e55f/dist/package/openSUSE-release-tools.spec
       2022-02-21 12:28:02.000000000 +0100
+++ 
new/openSUSE-release-tools-20220222.d8ef5897/dist/package/openSUSE-release-tools.spec
       2022-02-22 08:41:46.000000000 +0100
@@ -238,8 +238,6 @@
 Requires:       %{name} = %{version}
 Requires:       osc >= 0.165.1
 Requires:       python3-osc
-# internal API change related to accessing DSN in osclib/sentry.py
-Suggests:       python3-sentry-sdk >= 0.11.0
 BuildArch:      noarch
 
 %description -n osclib
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/obs_operator.py 
new/openSUSE-release-tools-20220222.d8ef5897/obs_operator.py
--- old/openSUSE-release-tools-20220221.3e14e55f/obs_operator.py        
2022-02-21 12:28:02.000000000 +0100
+++ new/openSUSE-release-tools-20220222.d8ef5897/obs_operator.py        
2022-02-22 08:41:46.000000000 +0100
@@ -11,10 +11,7 @@
 import json
 import tempfile
 import os
-from osc import conf
 from osclib import common
-from osclib.sentry import sentry_client
-from osclib.sentry import sentry_init
 import subprocess
 import sys
 import time
@@ -27,15 +24,10 @@
 # https://stackoverflow.com/a/47012250, workaround by making EVERYTHING LEGAL!
 http.cookies._is_legal_key = lambda _: True
 
-sentry_sdk = sentry_init()
-
-# Available in python 3.7.
-
 
 class ThreadedHTTPServer(ThreadingMixIn, HTTPServer):
     def handle_error(self, request, client_address):
         super().handle_error(request, client_address)
-        sentry_sdk.capture_exception()
 
 
 class RequestHandler(BaseHTTPRequestHandler):
@@ -179,14 +171,9 @@
         return None
 
     def oscrc_create(self, oscrc_file, apiurl, cookiejar_file, user):
-        sentry_dsn = sentry_client().dsn
-        sentry_environment = sentry_client().options.get('environment')
 
         oscrc_file.write('\n'.join([
             '[general]',
-            # Passthru sentry_sdk options to allow for reporting on 
subcommands.
-            'sentry_sdk.dsn = {}'.format(sentry_dsn) if sentry_dsn else '',
-            'sentry_sdk.environment = {}'.format(sentry_environment) if 
sentry_environment else '',
             'apiurl = {}'.format(apiurl),
             'cookiejar = {}'.format(cookiejar_file.name),
             'staging.color = 0',
@@ -371,9 +358,6 @@
 
 
 def main(args):
-    conf.get_config()  # Allow sentry DSN to be available.
-    sentry_init()
-
     RequestHandler.apiurl = args.apiurl
     RequestHandler.session = args.session
     RequestHandler.debug = args.debug
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/osc-origin.py 
new/openSUSE-release-tools-20220222.d8ef5897/osc-origin.py
--- old/openSUSE-release-tools-20220221.3e14e55f/osc-origin.py  2022-02-21 
12:28:02.000000000 +0100
+++ new/openSUSE-release-tools-20220222.d8ef5897/osc-origin.py  2022-02-22 
08:41:46.000000000 +0100
@@ -25,7 +25,6 @@
 from osclib.origin import origin_updatable
 from osclib.origin import origin_updatable_initial
 from osclib.origin import origin_update
-from osclib.sentry import sentry_init
 from osclib.util import mail_send
 from shutil import copyfile
 import sys
@@ -101,14 +100,8 @@
         if not config:
             raise oscerr.WrongArgs('OSRT:OriginConfig attribute missing from 
{}'.format(opts.project))
 
-    sentry_sdk = sentry_init(apiurl, {'osc_plugin': subcmd})
-    try:
-        function = 'osrt_origin_{}'.format(command)
-        globals()[function](apiurl, opts, *args[1:])
-    except Exception as e:
-        # Capture exception as osc.babysitter will consume any plugin 
exception.
-        sentry_sdk.capture_exception(e)
-        raise e
+    function = 'osrt_origin_{}'.format(command)
+    globals()[function](apiurl, opts, *args[1:])
 
 
 def osrt_origin_config(apiurl, opts, *args):
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/osc-staging.py 
new/openSUSE-release-tools-20220222.d8ef5897/osc-staging.py
--- old/openSUSE-release-tools-20220221.3e14e55f/osc-staging.py 2022-02-21 
12:28:02.000000000 +0100
+++ new/openSUSE-release-tools-20220222.d8ef5897/osc-staging.py 2022-02-22 
08:41:46.000000000 +0100
@@ -32,7 +32,6 @@
 from osclib.repair_command import RepairCommand
 from osclib.rebuild_command import RebuildCommand
 from osclib.request_splitter import RequestSplitter
-from osclib.sentry import sentry_init
 from osclib.supersede_command import SupersedeCommand
 from osclib.prio_command import PrioCommand
 
@@ -392,8 +391,6 @@
             if value:
                 setattr(Fore, name, ansi.code_to_chars(value))
 
-    sentry_init(opts.apiurl, {'osc_plugin': subcmd})
-
     if opts.wipe_cache:
         Cache.delete_all()
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/openSUSE-release-tools-20220221.3e14e55f/osclib/sentry.py 
new/openSUSE-release-tools-20220222.d8ef5897/osclib/sentry.py
--- old/openSUSE-release-tools-20220221.3e14e55f/osclib/sentry.py       
2022-02-21 12:28:02.000000000 +0100
+++ new/openSUSE-release-tools-20220222.d8ef5897/osclib/sentry.py       
1970-01-01 01:00:00.000000000 +0100
@@ -1,60 +0,0 @@
-from osc import conf
-from osc import core
-from osclib.common import VERSION
-
-
-def sentry_init(obs_apiurl=None, tags=None):
-    try:
-        import sentry_sdk
-    except ImportError:
-        sentry_init.client = sentry_client_dummy()
-        return sentry_sdk_dummy()
-
-    sentry_init.client = sentry_sdk.init(
-        conf.config.get('sentry_sdk.dsn'),
-        environment=conf.config.get('sentry_sdk.environment'),
-        release=VERSION)
-
-    with sentry_sdk.configure_scope() as scope:
-        scope.set_tag('osc', core.__version__)
-
-        if obs_apiurl:
-            scope.set_tag('obs_apiurl', obs_apiurl)
-            scope.user = {'username': conf.get_apiurl_usr(obs_apiurl)}
-
-        if tags:
-            for key, value in tags.items():
-                scope.set_tag(key, value)
-
-    return sentry_sdk
-
-
-def sentry_client():
-    return sentry_init.client._client
-
-
-class sentry_sdk_dummy:
-    def configure_scope(*args, **kw):
-        return nop_class()
-
-    def __getattr__(self, _):
-        return nop_func
-
-
-class nop_class:
-    def __enter__(self):
-        return nop_class()
-
-    def __exit__(self, exc_type, exc_val, exc_tb):
-        pass
-
-    def __getattr__(self, _):
-        return nop_func
-
-
-class sentry_client_dummy(nop_class):
-    options = {}
-
-
-def nop_func(*args, **kw):
-    pass

++++++ openSUSE-release-tools.obsinfo ++++++
--- /var/tmp/diff_new_pack.nlXGtl/_old  2022-02-22 21:18:46.678292699 +0100
+++ /var/tmp/diff_new_pack.nlXGtl/_new  2022-02-22 21:18:46.682292699 +0100
@@ -1,5 +1,5 @@
 name: openSUSE-release-tools
-version: 20220221.3e14e55f
-mtime: 1645442882
-commit: 3e14e55fdee2392f011f3b2c90c786263b13bf62
+version: 20220222.d8ef5897
+mtime: 1645515706
+commit: d8ef5897d562c2c855ef1710abab079e3c6393b8
 

Reply via email to