summaryrefslogtreecommitdiffstats
path: root/daemons
diff options
context:
space:
mode:
authorMartin Basti <mbasti@redhat.com>2016-09-22 14:39:16 +0200
committerMartin Basti <mbasti@redhat.com>2016-09-23 09:23:41 +0200
commit9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696 (patch)
tree05b54fce4f950dc16051e2bf081f33698cd2a203 /daemons
parent4d7d53c9664c9e68d7c6cda1a65cae7551423df7 (diff)
downloadfreeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.gz
freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.xz
freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.zip
Pylint: enable global-variable-not-assigned check
the global keyword should be used only when variable from outside is assigned inside, otherwise it has no effect and just confuses developers Reviewed-By: Tomas Krizek <tkrizek@redhat.com>
Diffstat (limited to 'daemons')
-rwxr-xr-xdaemons/dnssec/ipa-dnskeysyncd4
1 files changed, 3 insertions, 1 deletions
diff --git a/daemons/dnssec/ipa-dnskeysyncd b/daemons/dnssec/ipa-dnskeysyncd
index a381c29b5..dfe4cb4be 100755
--- a/daemons/dnssec/ipa-dnskeysyncd
+++ b/daemons/dnssec/ipa-dnskeysyncd
@@ -38,7 +38,9 @@ KEYTAB_FB = paths.IPA_DNSKEYSYNCD_KEYTAB
# Shutdown handler
def commenceShutdown(signum, stack):
# Declare the needed global variables
- global watcher_running, ldap_connection, log
+ global watcher_running
+ global ldap_connection # pylint: disable=global-variable-not-assigned
+
log.info('Signal %s received: Shutting down!', signum)
# We are no longer running