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