Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2017-03-16 09:43:30
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Thu Mar 16 09:43:30 2017 rev:11 rq:479748 version:1.6.4
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2017-03-12 20:04:02.289433026 +0100
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2017-03-16 09:51:58.802010660 +0100
@@ -1,0 +2,14 @@
+Wed Mar 15 16:21:22 UTC 2017 - ok...@suse.com
+
+- Update to version 1.6.4:
+ * Remove unused webtests (#58)
+ * dashboard: Prevent duplicate reminder comments
+ * Extend review-daily-email for use with opensuse.org
+ * Install 'openqa-review-daily-email' script
+
+-------------------------------------------------------------------
+Wed Mar 15 16:21:09 UTC 2017 - ok...@suse.com
+
+- Include new binary 'openqa-review-daily-email'
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.6.2.tar.gz
New:
----
python-openqa_review-1.6.4.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.sgBOo0/_old 2017-03-16 09:51:59.289941560 +0100
+++ /var/tmp/diff_new_pack.sgBOo0/_new 2017-03-16 09:51:59.293940993 +0100
@@ -18,7 +18,7 @@
%define short_name openqa_review
Name: python-%{short_name}
-Version: 1.6.2
+Version: 1.6.4
Release: 0
Summary: A review helper script for openQA
License: MIT
@@ -61,6 +61,7 @@
%{python_sitelib}
%attr(755, -, -) %{_bindir}/openqa-review
%attr(755, -, -) %{_bindir}/openqa-review-sles-ha
+%attr(755, -, -) %{_bindir}/openqa-review-daily-email
%attr(755, -, -) %{_bindir}/tumblesle-release
%changelog
++++++ python-openqa_review-1.6.2.tar.gz -> python-openqa_review-1.6.4.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/bin/openqa-review-daily-email
new/openqa_review-1.6.4/bin/openqa-review-daily-email
--- old/openqa_review-1.6.2/bin/openqa-review-daily-email 1970-01-01
01:00:00.000000000 +0100
+++ new/openqa_review-1.6.4/bin/openqa-review-daily-email 2017-03-15
16:36:12.000000000 +0100
@@ -0,0 +1,50 @@
+#!/bin/sh -e
+
+. $(dirname $0)/_common
+setup_tmpdir
+
+openqa_host="${openqa_host:-"https://openqa.suse.de"}";
+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) $@"}"
+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"}"
+# this is also putting reminder comments on issues. We can not do this in an
+# explicit later steps as we need all requests to be done when saving the data
+# and if we call it here and also in a later step we would end up with
+# duplicate reminder comments
+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)"
+(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
ok...@suse.de\n\nRegards,\nYour openqa_review") | mutt -s "Daily status from
${openqa_host}" -e 'my_hdr From: openqa-review <ok...@suse.de>' $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)"
+
+TPL_OPENQA_HOST=${openqa_host}
+
+TPL_OPENQA_HEADER_LINKS=$(cat <<EOF
+$(header_line "complete_report_box" "Complete report")
+$(header_line "closed_box" "Closed bugs")
+$(header_line "unassigned_box" "Unassigned bugs")
+EOF
+)
+
+TPL_OPENQA_CONTENT=$(cat <<EOF
+$(entry_block "complete_report_box" "Complete report" "${html_report}")
+$(entry_block "closed_box" "Closed bugs" "${html_report_closed}")
+$(entry_block "unassigned_box" "Unassigned bugs" "${html_report_unassigned}")
+EOF
+)
+
+eval "cat <<EOF
+$(<$TPL)
+EOF
+" > ${html_target}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.6.2/bin/openqa_review_osd_daily_email
new/openqa_review-1.6.4/bin/openqa_review_osd_daily_email
--- old/openqa_review-1.6.2/bin/openqa_review_osd_daily_email 2017-03-08
15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/bin/openqa_review_osd_daily_email 1970-01-01
01:00:00.000000000 +0100
@@ -1,53 +0,0 @@
-#!/bin/sh -e
-
-. $(dirname $0)/_common
-setup_tmpdir
-
-recv="${recv:-"openqa-suse-sta...@suse.de"}"
-html_target="${html_target:-"/suse/okurz/Export/openqa_suse_de_status.html"}"
-openqa_review_args="${openqa_review_args:-"--host http://openqa.suse.de -n -r
-T --query-issue-status --no-empty-sections --include-softfails
--running-threshold=2 --exclude-job-groups ^(Released|Development) $@"}"
-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"}"
-openqa_review_save_args="${openqa_review_save_args:-"--report-links
--reminder-comment-on-issues --save --save-dir ${tmp}"}"
-openqa_review_reminder_args="${openqa_review_reminder_args:-"${load_args}
--reminder-comment-on-issues"}"
-openqa_review="${openqa_review:-"$(which openqa-review)"}"
-TPL="${TPL:-"dashboard_files/dashboard.html.in"}"
-# the awk call is to ignore the "InsecureWarning" and the following blank line
-# that is generated by accessing with requests on an https server for which we
-# don't know the certificate, see http://unix.stackexchange.com/a/279620
-# this is actually fixed in a more recent python3-requests but we don't have
-# that yet. Anyway, we don't need certificates as we trust the internal server
-# and as long as we don't want to authenticate manually against the UI it's ok
-# to use the http:// URL
-#save_report="$(${openqa_review} $openqa_review_args $openqa_review_save_args
2>&1| awk -v nlines=2 '/Insecure.*Warning/ {for (i=0; i<nlines; i++) {getline};
next} 1')"
-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/openqa_suse_de_status.html. Status of tests and
builds on https://openqa.suse.de 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
ok...@suse.de\n\nRegards,\nYour openqa_review") | mutt -s 'Daily status from
https://openqa.suse.de' -e 'my_hdr From: openqa-review <ok...@suse.de>' $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)"
-
-TPL_OPENQA_HEADER_LINKS=$(cat <<EOF
-$(header_line "complete_report_box" "Complete report")
-$(header_line "closed_box" "Closed bugs")
-$(header_line "unassigned_box" "Unassigned bugs")
-EOF
-)
-
-TPL_OPENQA_CONTENT=$(cat <<EOF
-$(entry_block "complete_report_box" "Complete report" "${html_report}")
-$(entry_block "closed_box" "Closed bugs" "${html_report_closed}")
-$(entry_block "unassigned_box" "Unassigned bugs" "${html_report_unassigned}")
-EOF
-)
-
-eval "cat <<EOF
-$(<$TPL)
-EOF
-" > ${html_target}
-
-${openqa_review} $openqa_review_args $openqa_review_reminder_args > /dev/null
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/call_tests
new/openqa_review-1.6.4/call_tests
--- old/openqa_review-1.6.2/call_tests 1970-01-01 01:00:00.000000000 +0100
+++ new/openqa_review-1.6.4/call_tests 2017-03-15 16:36:12.000000000 +0100
@@ -0,0 +1,2 @@
+#!/bin/sh -e
+PYTHONPATH=.:$PYTHONPATH py.test "$@"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.6.2/dashboard_files/dashboard.html.in
new/openqa_review-1.6.4/dashboard_files/dashboard.html.in
--- old/openqa_review-1.6.2/dashboard_files/dashboard.html.in 2017-03-08
15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/dashboard_files/dashboard.html.in 2017-03-15
16:36:12.000000000 +0100
@@ -13,7 +13,7 @@
<!-- Bootstrap core CSS -->
<link href="dashboard_files/bootstrap.css" rel="stylesheet">
- <link rel="icon" type="image/png" sizes="16x16"
href="https://openqa.suse.de/asset/5caa45a06e/logo-16.png";>
+ <link rel="icon" type="image/png" sizes="16x16"
href="${TPL_OPENQA_HOST}/asset/5caa45a06e/logo-16.png">
<link rel="icon" href="dashboard_files/logo.svg" sizes="any"
type="image/svg+xml">
<style type="text/css">
@@ -36,20 +36,20 @@
<span class="icon-bar"></span>
<span class="icon-bar"></span>
</button>
- <a class="navbar-brand" href="https://openqa.suse.de/";><img
src="dashboard_files/logo.svg" alt="openQA"></a>
+ <a class="navbar-brand" href="${TPL_OPENQA_HOST}/"><img
src="dashboard_files/logo.svg" alt="openQA"></a>
</div>
<!-- Collect the nav links, forms, and other content for
toggling -->
<div class="collapse navbar-collapse" id="navbar-collapse-1">
<ul class="nav navbar-nav">
<li>
- <a href="https://openqa.suse.de/tests";>All Tests</a>
+ <a href="${TPL_OPENQA_HOST}/tests">All Tests</a>
</li>
</ul>
<ul class="nav navbar-nav navbar-right">
<li id="user-action">
- <a href="https://openqa.suse.de/login";>Login</a>
+ <a href="${TPL_OPENQA_HOST}/login">Login</a>
</li>
</ul>
</div><!-- /.navbar-collapse -->
@@ -60,7 +60,7 @@
<div class="jumbotron">
<div class="container">
<div class="col-md-10">
- <h2>openQA review report for openqa.suse.de</h2>
+ <h2>openQA review report for <a
href="${TPL_OPENQA_HOST}">${TPL_OPENQA_HOST}</a></h2>
${TPL_OPENQA_HEADER_LINKS}
@@ -89,7 +89,4 @@
</div>
</div>
</footer>
-
-
-
</body></html>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/openqa_review/openqa_review.py
new/openqa_review-1.6.4/openqa_review/openqa_review.py
--- old/openqa_review-1.6.2/openqa_review/openqa_review.py 2017-03-08
15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/openqa_review/openqa_review.py 2017-03-15
16:36:12.000000000 +0100
@@ -55,9 +55,8 @@
- Human friendly progress notification and wait spinner
- Accept multiple entries for '--job-group(-urls)'
- Ensure report entries are in same alphabetical order with OrderedDict
- - tox.ini: Local tests, webtests, doctests, check with flake8
+ - tox.ini: Local tests, doctests, check with flake8
- Generate version based on git describe
- - tests: Make slow webtests ignorable by marker
- Add support to parse all job groups
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/pytest_no_webtest
new/openqa_review-1.6.4/pytest_no_webtest
--- old/openqa_review-1.6.2/pytest_no_webtest 2017-03-08 15:20:04.000000000
+0100
+++ new/openqa_review-1.6.4/pytest_no_webtest 1970-01-01 01:00:00.000000000
+0100
@@ -1,2 +0,0 @@
-#!/bin/sh -e
-PYTHONPATH=.:$PYTHONPATH py.test -m "not webtest" "$@"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/setup.py
new/openqa_review-1.6.4/setup.py
--- old/openqa_review-1.6.2/setup.py 2017-03-08 15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/setup.py 2017-03-15 16:36:12.000000000 +0100
@@ -58,5 +58,5 @@
'console_scripts': ['openqa-review=openqa_review.openqa_review:main',
'tumblesle-release=openqa_review.tumblesle_release:main'],
},
- scripts=['bin/openqa-review-sles-ha'],
+ scripts=['bin/openqa-review-sles-ha', 'bin/openqa-review-daily-email'],
)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/tests/test_openqa_review.py
new/openqa_review-1.6.4/tests/test_openqa_review.py
--- old/openqa_review-1.6.2/tests/test_openqa_review.py 2017-03-08
15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/tests/test_openqa_review.py 2017-03-15
16:36:12.000000000 +0100
@@ -458,29 +458,3 @@
report = str(openqa_review.generate_report(args))
assert 'ppc64le' in report
-
-
-@pytest.mark.webtest
-def test_default_returns_valid_markdown_document():
- args = args_factory()
- report = openqa_review.generate_report(args)
- assert '**Common issues:**' in report
-
-
-@pytest.mark.webtest
-def
test_single_job_group_with_extended_test_output_returns_valid_markdown_document():
- args = args_factory()
- args.job_group_urls = args.host + '/group_overview/25'
- report = openqa_review.generate_report(args)
- assert '**Common issues:**' in report
-
-
-@pytest.mark.webtest
-def test_single_job_group_pages_can_be_cached_from_web():
- args = args_factory()
- args.job_group_urls = args.host + '/group_overview/25'
-
- with TemporaryDirectory() as tmp_dir:
- args.save_dir = tmp_dir
- report = openqa_review.generate_report(args)
- assert '**Common issues:**' in report
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.6.2/tox.ini
new/openqa_review-1.6.4/tox.ini
--- old/openqa_review-1.6.2/tox.ini 2017-03-08 15:20:04.000000000 +0100
+++ new/openqa_review-1.6.4/tox.ini 2017-03-15 16:36:12.000000000 +0100
@@ -47,7 +47,7 @@
# work. 'py.test --doctest-modules' fails when combined with '--cov' and its
# individual call with '--cov-append' also did not show success
commands =
- py.test --cov=openqa_review -m "not webtest"
+ py.test --cov=openqa_review
deps = -rrequirements.txt
pytest
pytest-cov
@@ -55,19 +55,7 @@
[testenv]
commands =
- py.test {posargs:-m 'not webtest'}
-deps = -rrequirements.txt
- pytest
- pytest-mock
-
-[testenv:webtests]
-# This testenv could be called explicitly, e.g. by 'tox -e webtests'.
-# It is not called by default, which is what we want
-commands =
py.test
deps = -rrequirements.txt
pytest
-
-[pytest]
-markers =
- webtest: relying on network access, slow
+ pytest-mock
![https://openqa.suse.de/asset/5caa45a06e/logo-16.png"](https://openqa.suse.de/asset/5caa45a06e/logo-16.png"){kind=link}