Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package python-openqa_review for 
openSUSE:Factory checked in at 2021-06-24 18:22:58
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
 and      /work/SRC/openSUSE:Factory/.python-openqa_review.new.2625 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "python-openqa_review"

Thu Jun 24 18:22:58 2021 rev:39 rq:901745 version:1.23.0

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes    
    2021-06-23 17:38:20.396480390 +0200
+++ 
/work/SRC/openSUSE:Factory/.python-openqa_review.new.2625/python-openqa_review.changes
      2021-06-24 18:23:16.804963201 +0200
@@ -1,0 +2,14 @@
+Thu Jun 24 13:09:18 UTC 2021 - ok...@suse.com
+
+- Update to version 1.23.0:
+  * Fix handling of softfail bugrefs
+  * Issue: Consistently Use self.bugref instead of bugref from arg
+
+-------------------------------------------------------------------
+Thu Jun 24 11:38:16 UTC 2021 - ok...@suse.com
+
+- Update to version 1.22.0:
+  * Also exclude newly introduced "EOL" job groups
+  * Specify Loader=yaml.SafeLoader explicitly
+
+-------------------------------------------------------------------

Old:
----
  python-openqa_review-1.21.2.obscpio

New:
----
  python-openqa_review-1.23.0.obscpio

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

Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.Z8KRjd/_old  2021-06-24 18:23:17.300963750 +0200
+++ /var/tmp/diff_new_pack.Z8KRjd/_new  2021-06-24 18:23:17.300963750 +0200
@@ -31,7 +31,7 @@
 %define         binaries openqa-review openqa-review-daily-email 
openqa-review-sles-ha tumblesle-release openqa-review-functional_yast_concise
 %define         oldpython python
 Name:           python-%{short_name}%{?name_ext}
-Version:        1.21.2
+Version:        1.23.0
 Release:        0
 Summary:        A review helper script for openQA
 License:        MIT

++++++ python-openqa_review-1.21.2.obscpio -> 
python-openqa_review-1.23.0.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/python-openqa_review-1.21.2/bin/openqa-review-daily-email 
new/python-openqa_review-1.23.0/bin/openqa-review-daily-email
--- old/python-openqa_review-1.21.2/bin/openqa-review-daily-email       
2021-06-23 11:11:22.000000000 +0200
+++ new/python-openqa_review-1.23.0/bin/openqa-review-daily-email       
2021-06-24 15:05:31.000000000 +0200
@@ -9,7 +9,7 @@
 recv="${recv:-"openqa-suse-sta...@suse.de"}"
 html_target_file="${html_target_file:-"openqa_suse_de_status.html"}"
 html_target="${html_target:-"/suse/okurz/Export/${html_target_file}"}"
-openqa_review_args="${openqa_review_args:-"--host ${openqa_host} -n -r -T 
--query-issue-status --no-empty-sections --include-softfails 
--running-threshold=2 --exclude-job-groups ^(Released|Development|old)"}"
+openqa_review_args="${openqa_review_args:-"--host ${openqa_host} -n -r -T 
--query-issue-status --no-empty-sections --include-softfails 
--running-threshold=2 --exclude-job-groups ^(Released|Development|old|EOL)"}"
 load_args="${load_args:-"--load --load-dir=${tmp}"}"
 openqa_review_email_args="${openqa_review_email_args:-"${load_args}"}"
 openqa_review_html_args="${openqa_review_html_args:-"${load_args}"}"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/python-openqa_review-1.21.2/openqa_review/openqa_review.py 
new/python-openqa_review-1.23.0/openqa_review/openqa_review.py
--- old/python-openqa_review-1.21.2/openqa_review/openqa_review.py      
2021-06-23 11:11:22.000000000 +0200
+++ new/python-openqa_review-1.23.0/openqa_review/openqa_review.py      
2021-06-24 15:05:31.000000000 +0200
@@ -204,7 +204,7 @@
 
 To prevent further reminder comments one of the following options should be 
followed:
 1. The test scenario is fixed by applying the bug fix to the tested product or 
the test is adjusted
-2. The openQA job group is moved to "Released"
+2. The openQA job group is moved to "Released" or "EOL" (End-of-Life)
 3. The label in the openQA scenario is removed
 """
 )
@@ -694,16 +694,16 @@
                     log.debug("#0 ticket id reference found")
                     self.msg = "NOTE: boo#0/bsc#0/poo#0 label used, please 
review. Consider creating progress ticket for the investigation"
                     return
-                elif bugref.startswith("poo#"):
+                elif self.bugref.startswith("poo#"):
                     log.debug("Test issue discovered, looking on progress")
                     self.issue_type = "redmine"
-                    self.json = progress_browser.get_json(bugref_href + 
".json")["issue"]
+                    self.json = progress_browser.get_json(self.bugref_href + 
".json")["issue"]
                     self.status = self.json["status"]["name"]
                     self.assignee = self.json["assigned_to"]["name"] if 
"assigned_to" in self.json else "None"
                     self.subject = self.json["subject"]
                     self.priority = self.json["priority"]["name"]
                     self.last_comment_date = 
datetime.datetime.strptime(self.json["updated_on"], "%Y-%m-%dT%H:%M:%SZ")
-                elif bugref.startswith(("boo#", "bsc#", "bgo#")):
+                elif self.bugref.startswith(("boo#", "bsc#", "bgo#")):
                     log.debug("Product bug discovered, looking on bugzilla")
                     self.issue_type = "bugzilla"
                     self.json = bugzilla_browser.json_rpc_get("/jsonrpc.cgi", 
"Bug.get", {"ids": [self.bugid]})[
@@ -716,14 +716,16 @@
                     self.subject = self.json["summary"]
                     self.priority = self.json["priority"].split(" ")[0] + "/" 
+ self.json["severity"]
                 else:
-                    log.debug('No valid bugref found. Bugref found: "%s"' % 
bugref)
+                    log.debug('No valid bugref found. Bugref found: "%s"' % 
self.bugref)
                 self.queried = True
             except DownloadError as e:  # pragma: no cover
-                log.info("A download error has been encountered for bugref %s 
(%s): %s" % (bugref, bugref_href, e))
+                log.info(
+                    "A download error has been encountered for bugref %s (%s): 
%s" % (self.bugref, self.bugref_href, e)
+                )
                 self.msg = str(e)
                 self.error = True
             except TypeError as e:
-                log.error("Error retrieving details for bugref %s (%s): %s" % 
(bugref, bugref_href, e))
+                log.error("Error retrieving details for bugref %s (%s): %s" % 
(self.bugref, self.bugref_href, e))
                 self.msg = "Ticket not found"
                 self.error = True
 
@@ -959,6 +961,7 @@
                 if not match:  # pragma: no cover
                     log.info("Could not find bug reference in text '%s', 
skipping." % v["bugref"])
                     continue
+                v["bugref"] = match.group(0)
                 bugref, bug_id = match.group(1), match.group(2)
                 assert bugref, "No bugref found for %s" % v
                 assert bug_id, "No bug_id found for %s" % v
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/python-openqa_review-1.21.2/openqa_review/tumblesle_release.py 
new/python-openqa_review-1.23.0/openqa_review/tumblesle_release.py
--- old/python-openqa_review-1.21.2/openqa_review/tumblesle_release.py  
2021-06-23 11:11:22.000000000 +0200
+++ new/python-openqa_review-1.23.0/openqa_review/tumblesle_release.py  
2021-06-24 15:05:31.000000000 +0200
@@ -250,7 +250,7 @@
             raise NotImplementedError("tag check not implemented")
         elif self.args.check_against_build == "release_info":
             with open(self.release_info_path, "r") as release_info_file:
-                release_info = yaml.load(release_info_file)
+                release_info = yaml.load(release_info_file, 
Loader=yaml.SafeLoader)
                 build["released"] = release_info[self.args.product]["build"]
         else:
             build["released"] = self.args.check_against_build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/python-openqa_review-1.21.2/tests/tags_labels/:tests:384330:file:select_patterns_and_packages-125.txt
 
new/python-openqa_review-1.23.0/tests/tags_labels/:tests:384330:file:select_patterns_and_packages-125.txt
--- 
old/python-openqa_review-1.21.2/tests/tags_labels/:tests:384330:file:select_patterns_and_packages-125.txt
   2021-06-23 11:11:22.000000000 +0200
+++ 
new/python-openqa_review-1.23.0/tests/tags_labels/:tests:384330:file:select_patterns_and_packages-125.txt
   2021-06-24 15:05:31.000000000 +0200
@@ -1,2 +1,2 @@
 # Soft Failure:
-bsc#1029660
+Some text to make parsing more interesting - bsc#1029660

++++++ python-openqa_review.obsinfo ++++++
--- /var/tmp/diff_new_pack.Z8KRjd/_old  2021-06-24 18:23:17.684964177 +0200
+++ /var/tmp/diff_new_pack.Z8KRjd/_new  2021-06-24 18:23:17.684964177 +0200
@@ -1,5 +1,5 @@
 name: python-openqa_review
-version: 1.21.2
-mtime: 1624439482
-commit: a478590cb222c60bef066a542116df38f46ec823
+version: 1.23.0
+mtime: 1624539931
+commit: bbeba7956e219beb48d19059607a2d157351663b
 

Reply via email to