summaryrefslogtreecommitdiffstats
path: root/ipaserver/install
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 /ipaserver/install
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 'ipaserver/install')
-rw-r--r--ipaserver/install/dns.py3
-rw-r--r--ipaserver/install/sysupgrade.py3
2 files changed, 0 insertions, 6 deletions
diff --git a/ipaserver/install/dns.py b/ipaserver/install/dns.py
index dedafec28..912b94fb7 100644
--- a/ipaserver/install/dns.py
+++ b/ipaserver/install/dns.py
@@ -314,9 +314,6 @@ def install_check(standalone, api, replica, options, hostname):
def install(standalone, replica, options, api=api):
- global ip_addresses
- global reverse_zones
-
local_dnskeysyncd_dn = DN(('cn', 'DNSKeySync'), ('cn', api.env.host),
('cn', 'masters'), ('cn', 'ipa'), ('cn', 'etc'),
api.env.basedn)
diff --git a/ipaserver/install/sysupgrade.py b/ipaserver/install/sysupgrade.py
index 1eba38cd0..e9192ac2f 100644
--- a/ipaserver/install/sysupgrade.py
+++ b/ipaserver/install/sysupgrade.py
@@ -35,17 +35,14 @@ def _load_sstore():
def get_upgrade_state(module, state):
_load_sstore()
- global _sstore
return _sstore.get_state(module, state)
def set_upgrade_state(module, state, value):
_load_sstore()
- global _sstore
_sstore.backup_state(module, state, value)
def remove_upgrade_state(module, state):
_load_sstore()
- global _sstore
_sstore.delete_state(module, state)
def remove_upgrade_file():