Differential D1134

logger: set koji log level to INFO
ClosedPublic
Actions

Authored by kparal on Feb 17 2017, 9:40 AM.

Details

Reviewers
mkrizek
Group Reviewers
libtaskotron
Maniphest Tasks
T910: Reference link not found
Commits
rLTRNcc074c62e4ee: logger: set koji log level to INFO
Summary

The bug we depended on has been fixed in koji 1.10.0 (commit
554a2448761e7).

Test Plan

test suite works, rpmlint local execution works (and koji debug message is not shown)

Diff Detail

Repository
rLTRN libtaskotron
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kparal retitled this revision from to logger: set koji log level to INFO.Feb 17 2017, 9:40 AM
kparal updated this object.
kparal edited the test plan for this revision. (Show Details)
kparal added a reviewer: libtaskotron.
kparal added a task: T910: Reference link not found.
mkrizek accepted this revision.Feb 17 2017, 9:50 AM
mkrizek added a reviewer: mkrizek.
This revision is now accepted and ready to land.Feb 17 2017, 9:50 AM
Closed by commit rLTRNcc074c62e4ee: logger: set koji log level to INFO (authored by Kamil Páral <kparal@redhat.com>). · Explain WhyFeb 17 2017, 11:29 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathPackages
Mlibtaskotron.spec (9 lines)
Mlibtaskotron/logger.py (4 lines)

Diff 2902

View Options

libtaskotron.spec

1Name: libtaskotron 1​Name: libtaskotron
2# NOTE: if you update version, *make sure* to also update `libtaskotron/__init__.py` 2​# NOTE: if you update version, *make sure* to also update `libtaskotron/__init__.py`
3Version: 0.4.18 3​Version: 0.4.18
4Release: 3%{?dist} 4​Release: 4%{?dist}
5Summary: Taskotron Support Library 5​Summary: Taskotron Support Library
6 6​
7License: GPLv3 7​License: GPLv3
8URL: https://pagure.io/taskotron/libtaskotron 8​URL: https://pagure.io/taskotron/libtaskotron
9Source0: https://qa.fedoraproject.org/releases/%{name}/%{name}-%{version}.tar.gz 9​Source0: https://qa.fedoraproject.org/releases/%{name}/%{name}-%{version}.tar.gz
10 10​
11BuildArch: noarch 11​BuildArch: noarch
12 12​
▲ Show 20 Lines • Show All 44 Lines • ▼ Show 20 Line(s)
57libtaskotron-config contains all of the configuration files needed for using 57​libtaskotron-config contains all of the configuration files needed for using
58libtaskotron. 58​libtaskotron.
59 59​
60%package -n libtaskotron-fedora 60​%package -n libtaskotron-fedora
61Summary: Fedora specific module for libtaskotron 61​Summary: Fedora specific module for libtaskotron
62 62​
63Requires: createrepo 63​Requires: createrepo
64Requires: dnf >= 0.6.4 64​Requires: dnf >= 0.6.4
65Requires: koji 65​Requires: koji >= 1.10.0
66Requires: libtaskotron-core = %{version}-%{release} 66​Requires: libtaskotron-core = %{version}-%{release}
67Requires: mash 67​Requires: mash
68Requires: python-fedora >= 0.8.0 68​Requires: python-fedora >= 0.8.0
69Requires: python-hawkey >= 0.4.13-1 69​Requires: python-hawkey >= 0.4.13-1
70Requires: python-munch >= 2.0.2 70​Requires: python-munch >= 2.0.2
71Requires: rpm-python 71​Requires: rpm-python
72BuildRequires: koji 72​BuildRequires: koji >= 1.10.0
73BuildRequires: mash 73​BuildRequires: mash
74BuildRequires: python-fedora >= 0.8.0 74​BuildRequires: python-fedora >= 0.8.0
75BuildRequires: python-hawkey >= 0.4.13-1 75​BuildRequires: python-hawkey >= 0.4.13-1
76BuildRequires: python-rpmfluff >= 0.5.1 76​BuildRequires: python-rpmfluff >= 0.5.1
77BuildRequires: python-munch >= 2.0.2 77​BuildRequires: python-munch >= 2.0.2
78 78​
79%description -n libtaskotron-fedora 79​%description -n libtaskotron-fedora
80Module for libtaskotron which contains all functionality which is fedora-only 80​Module for libtaskotron which contains all functionality which is fedora-only
▲ Show 20 Lines • Show All 84 Lines • ▼ Show 20 Line(s)
165%dir %{python2_sitelib}/libtaskotron/ext/fedora 165​%dir %{python2_sitelib}/libtaskotron/ext/fedora
166%{python2_sitelib}/libtaskotron/ext/fedora/* 166​%{python2_sitelib}/libtaskotron/ext/fedora/*
167 167​
168%files -n libtaskotron-disposable 168​%files -n libtaskotron-disposable
169%dir %{python2_sitelib}/libtaskotron/ext/disposable 169​%dir %{python2_sitelib}/libtaskotron/ext/disposable
170%{python2_sitelib}/libtaskotron/ext/disposable/* 170​%{python2_sitelib}/libtaskotron/ext/disposable/*
171 171​
172%changelog 172​%changelog
173​* Fri Feb 17 2017 Kamil Páral <kparal@redhat.com> - 0.4.18-4
174​- require koji >= 1.10.0 because of T910
175​
173* Fri Feb 10 2017 Kamil Páral <kparal@redhat.com> - 0.4.18-3 176​* Fri Feb 10 2017 Kamil Páral <kparal@redhat.com> - 0.4.18-3
174- add python-pytest-cov builddep because the test suite now needs it, 177​- add python-pytest-cov builddep because the test suite now needs it,
175 and python-rpmfluff because we're missing it 178​ and python-rpmfluff because we're missing it
176 179​
177* Thu Feb 09 2017 Kamil Páral <kparal@redhat.com> - 0.4.18-2 180​* Thu Feb 09 2017 Kamil Páral <kparal@redhat.com> - 0.4.18-2
178- require python-resultsdb_api >= 2.0.0 181​- require python-resultsdb_api >= 2.0.0
179 182​
180* Wed Jan 11 2017 Martin Krizek <mkrizek@fedoraproject.org> - 0.4.18-1 183​* Wed Jan 11 2017 Martin Krizek <mkrizek@fedoraproject.org> - 0.4.18-1
▲ Show 20 Lines • Show All 244 Lines • Show Last 20 Lines
View Options

libtaskotron/logger.py

Show First 20 Lines • Show All 88 Lines • ▼ Show 20 Line(s)
89 89​
90 90​
91def _set_verbosity_levels(): 91​def _set_verbosity_levels():
92 '''Configure libtaskotron and other important libraries for maximum desired 92​ '''Configure libtaskotron and other important libraries for maximum desired
93 verbosity. The actual verbosity is then adjusted in our handlers. This 93​ verbosity. The actual verbosity is then adjusted in our handlers. This
94 should be only called when running as the main script.''' 94​ should be only called when running as the main script.'''
95 log.setLevel(logging.DEBUG) 95​ log.setLevel(logging.DEBUG)
96 logging.getLogger('resultsdb_api').setLevel(logging.DEBUG) 96​ logging.getLogger('resultsdb_api').setLevel(logging.DEBUG)
97 # TODO: set this to INFO once this is resolved: 97​ logging.getLogger('koji').setLevel(logging.INFO)
98 # https://fedorahosted.org/koji/ticket/317 ​
99 logging.getLogger('koji').setLevel(logging.DEBUG) ​
100 logging.getLogger('testcloud').setLevel(logging.DEBUG) 98​ logging.getLogger('testcloud').setLevel(logging.DEBUG)
101 99​
102 100​
103def init_prior_config(level_stream=None): 101​def init_prior_config(level_stream=None):
104 '''Initialize Taskotron logging with default values which do not rely on 102​ '''Initialize Taskotron logging with default values which do not rely on
105 a config file. Only stream logging is enabled here. This is used before the 103​ a config file. Only stream logging is enabled here. This is used before the
106 config file is loaded. After that a proper initialization should take place 104​ config file is loaded. After that a proper initialization should take place
107 through the :func:`init` method. 105​ through the :func:`init` method.
▲ Show 20 Lines • Show All 156 Lines • Show Last 20 Lines