Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2018-02-10 17:58:21
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Sat Feb 10 17:58:21 2018 rev:20 rq:574664 version:1.10.0
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2018-01-20 11:27:47.177359463 +0100
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2018-02-10 17:58:21.863703907 +0100
@@ -1,0 +2,26 @@
+Fri Feb 09 13:31:37 UTC 2018 - [email protected]
+
+- Update to version 1.10.0:
+ * Add helper script for SLE15 functional+autoyast concise report
+ * Reference new review wrapper script in setup.py
+
+-------------------------------------------------------------------
+Thu Feb 8 09:26:55 UTC 2018 - [email protected]
+
+- Adapt license file location according to new suggestion around %license
+
+-------------------------------------------------------------------
+Wed Feb 07 16:21:34 UTC 2018 - [email protected]
+
+- Update to version 1.9.0:
+ * openqa-review-daily-email: Fix passing of additional options with
whitespace
+ * Add two options to further abbreviate condensed reports
+ * Include difference in numbers on verbosity level above 3
+ * Correct optional code for updating reference reports
+ * Encode severity together with priority for ticket references
+ * Mark closed tickets as red
+ * setup.py: Fix flake8 reported style issues with explicit except
+ * setup.py: Catch errors as reported on no version.py from OBS build
+ * setup.py: python2 backward cruft/craft
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.8.3.tar.gz
New:
----
python-openqa_review-1.10.0.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.BGprGo/_old 2018-02-10 17:58:22.527679871 +0100
+++ /var/tmp/diff_new_pack.BGprGo/_new 2018-02-10 17:58:22.531679726 +0100
@@ -19,17 +19,19 @@
# Define just "test" as a package in _multibuild file to distinguish test
# instructions here
%if "@BUILD_FLAVOR@" == ""
-%bcond_with test
+%define _test 0
+%define name_ext %nil
%else
-%bcond_without test
+%define _test 1
+%define name_ext -test
%endif
%{?!python_module:%define python_module() python-%{**} python3-%{**}}
%define short_name openqa_review
-%define binaries openqa-review openqa-review-daily-email
openqa-review-sles-ha tumblesle-release
+%define binaries openqa-review openqa-review-daily-email
openqa-review-sles-ha tumblesle-release
openqa-review-functional_autoyast_concise
%define oldpython python
Name: python-%{short_name}%{?name_ext}
-Version: 1.8.3
+Version: 1.10.0
Release: 0
Summary: A review helper script for openQA
License: MIT
@@ -37,7 +39,7 @@
Source:
https://github.com/okurz/%{short_name}/archive/%{version}.tar.gz#/python-%{short_name}-%{version}.tar.gz
Url: https://github.com/okurz/%{short_name}
BuildRequires: python-rpm-macros
-%if %{with test}
+%if 0%{?_test}
BuildRequires: %{oldpython}-%{short_name}
%else
BuildRequires: %{python_module setuptools}
@@ -73,7 +75,7 @@
A review helper script for openQA. For more details look into the README file.
%prep
-%if %{with test}
+%if 0%{?_test}
# workaround to prevent post/install failing assuming this file for whatever
# reason
touch %{_sourcedir}/%{short_name}
@@ -84,14 +86,16 @@
%endif
%build
-%if %{with test}
+%if 0%{?_test}
openqa-review --help
tumblesle-release --help
%else
%python_build
%endif
-%if %{with test}
+%if 0%{?_test}
+# disable debug packages in package test to prevent error about missing files
+%define debug_package %{nil}
%else
%install
%python_install
@@ -110,16 +114,15 @@
%files %{python_files}
%defattr(-,root,root,-)
-%doc LICENSE README.md
+%license LICENSE
+%doc README.md
%{python_sitelib}
%python_alternative %{_bindir}/openqa-review
%python_alternative %{_bindir}/openqa-review-sles-ha
%python_alternative %{_bindir}/openqa-review-daily-email
+%python_alternative %{_bindir}/openqa-review-functional_autoyast_concise
%python_alternative %{_bindir}/tumblesle-release
-%else
-# disable debug packages in package test to prevent error about missing files
-%define debug_package %{nil}
%endif
%changelog
++++++ _service ++++++
--- /var/tmp/diff_new_pack.BGprGo/_old 2018-02-10 17:58:22.567678423 +0100
+++ /var/tmp/diff_new_pack.BGprGo/_new 2018-02-10 17:58:22.571678278 +0100
@@ -1,7 +1,7 @@
<services>
<service name="tar_scm" mode="disabled">
<param name="filename">python-openqa_review</param>
- <param name="versionformat">@PARENT_TAG@+@TAG_OFFSET@.%h</param>
+ <param name="versionformat">@PARENT_TAG@</param>
<param name="url">git://github.com/okurz/openqa_review.git</param>
<param name="scm">git</param>
<param name="revision">master</param>
++++++ python-openqa_review-1.8.3.tar.gz -> python-openqa_review-1.10.0.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.8.3/bin/openqa-review-daily-email
new/openqa_review-1.10.0/bin/openqa-review-daily-email
--- old/openqa_review-1.8.3/bin/openqa-review-daily-email 2018-01-19
08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/bin/openqa-review-daily-email 2018-02-09
14:34:50.000000000 +0100
@@ -7,7 +7,7 @@
recv="${recv:-"[email protected]"}"
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)"}"
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}
--report-links"}"
@@ -18,15 +18,16 @@
openqa_review_save_args="${openqa_review_save_args:-"--report-links
--reminder-comment-on-issues --save --save-dir ${tmp}"}"
openqa_review="${openqa_review:-"$(which openqa-review)"}"
TPL="${TPL:-"dashboard_files/dashboard.html.in"}"
-save_report="$(${openqa_review} $openqa_review_args $openqa_review_save_args)"
-email_report="$(${openqa_review} $openqa_review_args
$openqa_review_email_args)"
+# We have to preserve the '$@' until here to prevent too early evaluation of
parameters with spaces
+save_report="$(${openqa_review} ${openqa_review_args}
${openqa_review_save_args} "$@")"
+email_report="$(${openqa_review} $openqa_review_args $openqa_review_email_args
"$@")"
(echo -e "This is an automated message generated by 'openqa-review', see
https://github.com/okurz/openqa_review for details. An HTML version of this
report is available on https://w3.nue.suse.com/~okurz/${html_target_file}.
Status of tests and builds on ${openqa_host} as of $(date
--iso-8601='seconds'):\n" && \
echo "$email_report" && \
echo -e "\nAny feedback regarding the script processing or contact can be
communicated using issues on the github repo or directly to
[email protected]\n\nRegards,\nYour openqa_review") | mutt -s "Daily status from
${openqa_host}" -e 'my_hdr From: openqa-review <[email protected]>' $recv
-html_report="$(${openqa_review} $openqa_review_args $openqa_review_html_args |
decrease_header | markdown)"
-html_report_closed="$(${openqa_review} $openqa_review_args $load_args -f
closed | decrease_header | markdown)"
-html_report_unassigned="$(${openqa_review} $openqa_review_args $load_args -f
unassigned | decrease_header | markdown)"
+html_report="$(${openqa_review} $openqa_review_args $openqa_review_html_args
"$@" | decrease_header | markdown)"
+html_report_closed="$(${openqa_review} $openqa_review_args $load_args -f
closed "$@" | decrease_header | markdown)"
+html_report_unassigned="$(${openqa_review} $openqa_review_args $load_args -f
unassigned "$@" | decrease_header | markdown)"
TPL_OPENQA_HOST=${openqa_host}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/bin/openqa-review-functional_autoyast_concise
new/openqa_review-1.10.0/bin/openqa-review-functional_autoyast_concise
--- old/openqa_review-1.8.3/bin/openqa-review-functional_autoyast_concise
1970-01-01 01:00:00.000000000 +0100
+++ new/openqa_review-1.10.0/bin/openqa-review-functional_autoyast_concise
2018-02-09 14:34:50.000000000 +0100
@@ -0,0 +1,6 @@
+#!/bin/sh -e
+new="${1:?"Need build id of 'new' build, e.g. 451.2"}"
+old="${2:?"Need build id of 'old' build, e.g. 438.1"}"
+for group in Functional Autoyast; do
+ openqa-review --host http://openqa.suse.de -vvvv -s -n -T --bugrefs
--no-empty-sections --include-softfails --query-issue-status -j "SLE 15 /
$group" --short-failure-str --abbreviate-test-issues --builds $new,$old $args |
tee $(date +%F)_${group,,}_$new.md
+done
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.8.3/openqa_review/openqa_review.py
new/openqa_review-1.10.0/openqa_review/openqa_review.py
--- old/openqa_review-1.8.3/openqa_review/openqa_review.py 2018-01-19
08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/openqa_review/openqa_review.py 2018-02-09
14:34:50.000000000 +0100
@@ -613,7 +613,7 @@
self.resolution = self.json['resolution']
self.assignee = self.json['assigned_to'] if 'assigned_to'
in self.json else 'None'
self.subject = self.json['summary']
- self.priority = self.json['priority'].split(' ')[0]
+ self.priority = self.json['priority'].split(' ')[0] + '/'
+ self.json['severity']
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))
@@ -680,7 +680,9 @@
status = self.status
if self.resolution:
status += ' (%s)' % self.resolution
- msg = 'Ticket status: %s, prio: %s, assignee: %s' % (status,
self.priority, self.assignee)
+ if status.startswith('VERIFIED') or status.startswith('Resolved'):
+ status = '<span style="color: red;">%s</span>' % status
+ msg = 'Ticket status: %s, prio/severity: %s, assignee: %s' %
(status, self.priority, self.assignee)
else:
msg = None
return '[%s](%s%s)%s' % (
@@ -729,11 +731,15 @@
def __str__(self):
"""Return as markdown."""
- return '* %s%s%s\n' % (
- 'soft fails: ' if self.soft else '',
+ test_bug_str = '%s%s' % (
', '.join(map(self._format_failure, self.failures)),
' -> %s' % self.bug if self.bug else ''
)
+ short_str = self.bug if self.bug else ', '.join(i['name'] for i in
self.failures)
+ return '* %s%s\n' % (
+ 'soft fails: ' if self.soft else '',
+ short_str if self.args.short_failure_str else test_bug_str
+ )
@classmethod
def for_each(cls, args, root_url, failures, test_browser):
@@ -850,22 +856,29 @@
log.error('Could not find any soft failure reference within
details of soft-failed job \'%s\'. Could be deleted workaround needle?.' %
absolute_url(self.root_url, result_item))
- def __str__(self):
- """Return as markdown."""
+ def _todo_issues_str(self):
+ if self.args.abbreviate_test_issues:
+ return issue_listing('### Test issues',
self.issues['new']['openqa'] + self.issues['existing']['openqa'] +
self.issues['new']['todo'] +
+ self.issues['existing']['todo'],
self.args.show_empty)
todo_issues = todo_review_template.substitute({
'new_issues': issue_listing('***new issues***',
self.issues['new']['todo'], self.args.show_empty),
'existing_issues': issue_listing('***existing issues***',
self.issues['existing']['todo'], self.args.show_empty),
})
+ return todo_issues if (self.issues['new']['todo'] or
self.issues['existing']['todo']) else ''
+
+ def __str__(self):
+ """Return as markdown."""
+ abbrev = self.args.abbreviate_test_issues
return openqa_review_report_arch_template.substitute({
'arch': self.arch,
'status_badge': status_badge_str[self.status_badge],
# everything that is 'NEW_ISSUE' should be product issue but if
tests have changed content, then probably openqa issues
# For now we can just not easily decide unless we use the
'bugrefs' mode
- 'new_openqa_issues': issue_listing('**New openQA-issues:**',
self.issues['new']['openqa'], self.args.show_empty),
- 'existing_openqa_issues': issue_listing('**Existing
openQA-issues:**', self.issues['existing']['openqa'], self.args.show_empty),
+ 'new_openqa_issues': '' if abbrev else issue_listing('**New
openQA-issues:**', self.issues['new']['openqa'], self.args.show_empty),
+ 'existing_openqa_issues': '' if abbrev else
issue_listing('**Existing openQA-issues:**', self.issues['existing']['openqa'],
self.args.show_empty),
'new_product_issues': issue_listing('**New Product bugs:**',
self.issues['new']['product'], self.args.show_empty),
'existing_product_issues': issue_listing('**Existing Product
bugs:**', self.issues['existing']['product'], self.args.show_empty),
- 'todo_issues': todo_issues if (self.issues['new']['todo'] or
self.issues['existing']['todo']) else '',
+ 'todo_issues': self._todo_issues_str(),
})
@@ -889,8 +902,9 @@
current_summary = parse_summary(current_details)
previous_summary = parse_summary(previous_details)
- changes = {k: v - previous_summary.get(k, 0) for k, v in
iteritems(current_summary) if k != 'none' and k != 'incomplete'}
- log.info("Changes since last build:\n\t%s" % '\n\t'.join("%s: %s" %
(k, v) for k, v in iteritems(changes)))
+ changes = SortedDict({k: v - previous_summary.get(k, 0) for k, v in
iteritems(current_summary) if k != 'none'})
+ self.changes_str = '***Changes since reference build***\n\n* ' + '\n*
'.join("%s: %s" % (k, v) for k, v in iteritems(changes)) + '\n'
+ log.info("%s" % self.changes_str)
self.build = get_build_nr(current_url)
self.ref_build = get_build_nr(previous_url)
@@ -922,6 +936,8 @@
build_str = self.build
if self.args.verbose_test and self.args.verbose_test > 1:
build_str += ' (reference %s)' % self.ref_build
+ if self.args.verbose_test and self.args.verbose_test > 3:
+ build_str += '\n\n' + self.changes_str
openqa_review_report_product =
openqa_review_report_product_template.substitute({
'now': now_str,
@@ -975,6 +991,12 @@
help="""Parse \'bugrefs\' from test results comments
and triage issues accordingly.
See
https://progress.opensuse.org/projects/openqav3/wiki/Wiki#Show-bug-or-label-icon-on-overview-if-labeled-gh550
for details about bugrefs in openQA""")
+ parser.add_argument('--short-failure-str', action='store_true',
+ help="""Instead of the default long failure
description use a short version only outputting the referenced bug or the
failing test in
+ case of failure without bugref.""")
+ parser.add_argument('--abbreviate-test-issues', action='store_true',
+ help="""If requested abbreviate all 'test issue'
related reporting sections and instead only show a short list of these next to
the
+ full-detail product issues. Useful to focus on
product-related reports.""")
parser.add_argument('-R', '--query-issue-status', action='store_true',
help="""Query issue trackers for the issues found and
report on their status and assignee. Implies "-r/--bugrefs" and
needs configuration file {} with credentials, see
'--query-issue-status-help'.""".format(CONFIG_PATH))
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.8.3/setup.py
new/openqa_review-1.10.0/setup.py
--- old/openqa_review-1.8.3/setup.py 2018-01-19 08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/setup.py 2018-02-09 14:34:50.000000000 +0100
@@ -1,6 +1,6 @@
import os
import sys
-from subprocess import check_output
+from subprocess import check_output, CalledProcessError
from setuptools import setup
@@ -9,16 +9,22 @@
# Also, when git is not available (PyPi package), use stored version.py.
version_py = os.path.join(os.path.dirname(__file__), 'version.py')
+# python2 backwards craft
+try:
+ FileNotFoundError
+except NameError:
+ FileNotFoundError = IOError
+
try:
# This will not generate PEP440 compliant version strings for any commit
# that is not on the tag itself. setuptools/dist will give a warning.
# Still, this is good enough for now. A (big) alternative would be
# gh:warner/python-versioneer
version_git = check_output(["git", "describe",
"--tags"]).rstrip().decode('ascii')
-except:
+except (CalledProcessError, OSError):
try:
version_git =
open(version_py).read().strip().split('=')[-1].replace('\'', '').strip()
- except:
+ except FileNotFoundError:
version_git = '0.0.0'
version_msg = "# Do not edit this file, pipeline versioning is governed by git
tags"
@@ -64,5 +70,5 @@
'console_scripts': ['openqa-review=openqa_review.openqa_review:main',
'tumblesle-release=openqa_review.tumblesle_release:main'],
},
- scripts=['bin/openqa-review-sles-ha', 'bin/openqa-review-daily-email'],
+ scripts=['bin/openqa-review-sles-ha', 'bin/openqa-review-daily-email',
'bin/openqa-review-functional_autoyast_concise'],
)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.8.3/tests/report25_TTT.md
new/openqa_review-1.10.0/tests/report25_TTT.md
--- old/openqa_review-1.8.3/tests/report25_TTT.md 2018-01-19
08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/tests/report25_TTT.md 2018-02-09
14:34:50.000000000 +0100
@@ -4,6 +4,13 @@
**Date:** 2016-09-19 - 14:35
**Build:** 0313 (reference 0311)
+***Changes since reference build***
+
+* failed: -11
+* passed: 0
+* soft failure: -1
+
+
**Common issues:**
* **Missing architectures**: i586
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1508%26groupid%3D25
new/openqa_review-1.10.0/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1508%26groupid%3D25
---
old/openqa_review-1.8.3/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1508%26groupid%3D25
2018-01-19 08:36:01.000000000 +0100
+++
new/openqa_review-1.10.0/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1508%26groupid%3D25
2018-02-09 14:34:50.000000000 +0100
@@ -472,6 +472,33 @@
<td>-</td>
</tr>
<tr>
+ <td class="name"><span
title="allpatterns2@i586--l2">allpatterns2@i586--vswi ...</span></td>
+
+ <td>-</td>
+
+ <td>-</td>
+
+ <td id="res_Gnome-DVD_i586_allpatterns2@i586--l2">
+
+ <span id="res-384339">
+ <a href="/tests/384339">
+ <i class="status fa fa-circle result_failed" title="Done:
failed"></i>
+ </a>
+ </span>
+
+ <span id="bug-384339">
+ <a href="https://progress.opensuse.org/issues/9773">
+ <i class="test-label label_bug fa fa-bug" title="Bug(s)
referenced: poo#9773"></i>
+ </a>
+ </span>
+
+
+</td>
+
+
+ <td>-</td>
+ </tr>
+ <tr>
<td class="name">autoupgrade_13.1</td>
<td>-</td>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/https%3A::progress.opensuse.org:issues:9773.json
new/openqa_review-1.10.0/tests/tags_labels/https%3A::progress.opensuse.org:issues:9773.json
---
old/openqa_review-1.8.3/tests/tags_labels/https%3A::progress.opensuse.org:issues:9773.json
1970-01-01 01:00:00.000000000 +0100
+++
new/openqa_review-1.10.0/tests/tags_labels/https%3A::progress.opensuse.org:issues:9773.json
2018-02-09 14:34:50.000000000 +0100
@@ -0,0 +1 @@
+{"issue": {"done_ratio": 0, "created_on": "2015-12-02T15:24:27Z",
"description": "Nothing works", "subject": "Nothing works", "tracker": {"name":
"action", "id": 4}, "start_date": "2015-12-02", "category": {"name": "New
test", "id": 164}, "author": {"name": "okurz", "id": 3}, "status": {"name":
"Resolved", "id": 5}, "id": 9772, "priority": {"name": "High", "id": 5},
"project": {"name": "openQA Tests", "id": 36}, "updated_on":
"2016-01-15T16:05:18Z"}}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_abbreviated.md
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_abbreviated.md
--- old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_abbreviated.md
1970-01-01 01:00:00.000000000 +0100
+++ new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_abbreviated.md
2018-02-09 14:34:50.000000000 +0100
@@ -0,0 +1,31 @@
+# 0
+
+
+**Date:** 2016-09-16 - 16:36
+**Build:** 1507
+
+---
+
+**Arch:** i586
+**Status: <span style="color: red;">Red</span>**
+
+**New Product bugs:**
+
+* [boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571)
+* [boo#9315715](https://bugzilla.opensuse.org/show_bug.cgi?id=9315715)
+
+
+**Existing Product bugs:**
+
+* [bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770)
+
+
+### Test issues
+
+* [poo#11948](https://progress.opensuse.org/issues/11948)
+* [poo#9772](https://progress.opensuse.org/issues/9772)
+* ssh-X@i586--l3
+
+
+
+---
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_build1508.md
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_build1508.md
--- old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_build1508.md
2018-01-19 08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_build1508.md
2018-02-09 14:34:50.000000000 +0100
@@ -11,18 +11,19 @@
**Existing Product bugs:**
-* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio: P5, assignee:
[email protected])
+* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio/severity: P5/Normal,
assignee: [email protected])
**New openQA-issues:**
* toolchain_zypper -> [poo#13694](https://progress.opensuse.org/issues/13694)
(Request to https://progress.opensuse.org/issues/13694.json was not successful,
file https%3A::progress.opensuse.org:issues:13694.json not found)
+* allpatterns2@i586--l2 ->
[poo#9773](https://progress.opensuse.org/issues/9773 "Nothing works") (Ticket
status: <span style="color: red;">Resolved</span>, prio/severity: High,
assignee: None)
**Existing openQA-issues:**
-* ssh-X@i586--l3, textmode ->
[poo#11948](https://progress.opensuse.org/issues/11948 "can not boot
pre-installed image") (Ticket status: New, prio: Normal, assignee: user1)
-* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio: High, assignee: None)
+* ssh-X@i586--l3, textmode ->
[poo#11948](https://progress.opensuse.org/issues/11948 "can not boot
pre-installed image") (Ticket status: New, prio/severity: Normal, assignee:
user1)
+* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio/severity: High, assignee: None)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues.md
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues.md
--- old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues.md
2018-01-19 08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues.md
2018-02-09 14:34:50.000000000 +0100
@@ -11,19 +11,19 @@
**New Product bugs:**
-* toolchain_zypper ->
[boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571 "no space
left on device when upgrading") (Ticket status: NEW, prio: P2, assignee:
[email protected])
+* toolchain_zypper ->
[boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571 "no space
left on device when upgrading") (Ticket status: NEW, prio/severity: P2/Major,
assignee: [email protected])
* toolchain_zypper@bar ->
[boo#9315715](https://bugzilla.opensuse.org/show_bug.cgi?id=9315715) (Ticket
not found)
**Existing Product bugs:**
-* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio: P5, assignee:
[email protected])
+* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio/severity: P5/Normal,
assignee: [email protected])
**Existing openQA-issues:**
-* textmode -> [poo#11948](https://progress.opensuse.org/issues/11948 "can not
boot pre-installed image") (Ticket status: New, prio: Normal, assignee: user1)
-* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio: High, assignee: None)
+* textmode -> [poo#11948](https://progress.opensuse.org/issues/11948 "can not
boot pre-installed image") (Ticket status: New, prio/severity: Normal,
assignee: user1)
+* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio/severity: High, assignee: None)
**TODO: review**
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues_filter_closed.md
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues_filter_closed.md
---
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues_filter_closed.md
2018-01-19 08:36:01.000000000 +0100
+++
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues_filter_closed.md
2018-02-09 14:34:50.000000000 +0100
@@ -11,7 +11,7 @@
**Existing Product bugs:**
-* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio: P5, assignee:
[email protected])
+* allpatterns, ext4@i586--l2, gnome, minimal+base ->
[bsc#822770](https://bugzilla.opensuse.org/show_bug.cgi?id=822770 "Install of
grub2-efi failed") (Ticket status: RESOLVED (FOOBAR), prio/severity: P5/Normal,
assignee: [email protected])
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues_filter_unassigned.md
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues_filter_unassigned.md
---
old/openqa_review-1.8.3/tests/tags_labels/report25_bugrefs_query_issues_filter_unassigned.md
2018-01-19 08:36:01.000000000 +0100
+++
new/openqa_review-1.10.0/tests/tags_labels/report25_bugrefs_query_issues_filter_unassigned.md
2018-02-09 14:34:50.000000000 +0100
@@ -11,12 +11,12 @@
**New Product bugs:**
-* toolchain_zypper ->
[boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571 "no space
left on device when upgrading") (Ticket status: NEW, prio: P2, assignee:
[email protected])
+* toolchain_zypper ->
[boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571 "no space
left on device when upgrading") (Ticket status: NEW, prio/severity: P2/Major,
assignee: [email protected])
**Existing openQA-issues:**
-* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio: High, assignee: None)
+* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio/severity: High, assignee: None)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.8.3/tests/test_openqa_review.py
new/openqa_review-1.10.0/tests/test_openqa_review.py
--- old/openqa_review-1.8.3/tests/test_openqa_review.py 2018-01-19
08:36:01.000000000 +0100
+++ new/openqa_review-1.10.0/tests/test_openqa_review.py 2018-02-09
14:34:50.000000000 +0100
@@ -48,6 +48,8 @@
args.show_empty = True
args.bugrefs = False
args.include_softfails = True
+ args.short_failure_str = False
+ args.abbreviate_test_issues = False
args.query_issue_status = False
args.query_issue_status_help = True
args.report_links = False
@@ -113,7 +115,7 @@
def compare_report(report, ref_report_path):
# If the reference report should be written initially or updated one can
just write the report as string to file
- # open(os.path.join(os.path.dirname(os.path.realpath(ref_port_path)),
ref_report_filename), 'w').write(str(report))
+ # open(ref_report_path, 'w').write(str(report))
ref_report = open(ref_report_path).read()
# for simpler display of the diff in case of differences it helps to have
# both reports in same encoding, i.e. casting to str
@@ -409,6 +411,15 @@
compare_report(report, os.path.join(args.load_dir, 'report25_bugrefs.md'))
+def test_bugrefs_with_abbreviated_format_can_be_used():
+ args = bugrefs_test_args_factory()
+ args.verbose_test = 1
+ args.short_failure_str = True
+ args.abbreviate_test_issues = True
+ report = str(openqa_review.generate_report(args))
+ compare_report(report, os.path.join(args.load_dir,
'report25_bugrefs_abbreviated.md'))
+
+
def test_bugrefs_with_report_links():
args = bugrefs_test_args_factory()
args.report_links = True