From 029ac0b55f686180167fa3c929fb41ad3574b656 Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:49:17 +0100 Subject: [PATCH 01/13] Fix pylint warnings W0311 (bad indentation) From: Juerg Haefliger --- cloudinit/CloudConfig/cc_chef.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'cloudinit/CloudConfig/cc_chef.py') diff --git a/cloudinit/CloudConfig/cc_chef.py b/cloudinit/CloudConfig/cc_chef.py index 14960a3c..f093d0b2 100644 --- a/cloudinit/CloudConfig/cc_chef.py +++ b/cloudinit/CloudConfig/cc_chef.py @@ -90,9 +90,9 @@ def get_ruby_packages(version): def install_chef_from_gems(ruby_version, chef_version = None): cc.install_packages(get_ruby_packages(ruby_version)) if not os.path.exists('/usr/bin/gem'): - os.symlink('/usr/bin/gem%s' % ruby_version, '/usr/bin/gem') + os.symlink('/usr/bin/gem%s' % ruby_version, '/usr/bin/gem') if not os.path.exists('/usr/bin/ruby'): - os.symlink('/usr/bin/ruby%s' % ruby_version, '/usr/bin/ruby') + os.symlink('/usr/bin/ruby%s' % ruby_version, '/usr/bin/ruby') if chef_version: subprocess.check_call(['/usr/bin/gem','install','chef', '-v %s' % chef_version, '--no-ri', -- cgit v1.2.3 From df5d51c25523ea04d45f5f077e94885d1847e526 Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:50:56 +0100 Subject: [PATCH 04/13] Fix pylint warnings W0611 (unused import) From: Juerg Haefliger --- cloud-init-cfg.py | 1 - cloud-init-query.py | 3 --- cloudinit/CloudConfig/cc_chef.py | 5 +---- cloudinit/CloudConfig/cc_foo.py | 6 ++++-- cloudinit/CloudConfig/cc_grub_dpkg.py | 2 +- cloudinit/CloudConfig/cc_mcollective.py | 4 +--- cloudinit/CloudConfig/cc_resizefs.py | 2 +- cloudinit/CloudConfig/cc_rightscale_userdata.py | 3 ++- cloudinit/CloudConfig/cc_rsyslog.py | 2 +- cloudinit/CloudConfig/cc_runcmd.py | 2 +- cloudinit/CloudConfig/cc_scripts_per_boot.py | 5 +++-- cloudinit/CloudConfig/cc_scripts_per_instance.py | 5 +++-- cloudinit/CloudConfig/cc_scripts_per_once.py | 5 +++-- cloudinit/CloudConfig/cc_scripts_user.py | 5 +++-- cloudinit/CloudConfig/cc_set_hostname.py | 2 +- cloudinit/CloudConfig/cc_set_passwords.py | 2 +- cloudinit/CloudConfig/cc_timezone.py | 2 +- cloudinit/DataSourceEc2.py | 5 +---- cloudinit/DataSourceNoCloud.py | 6 +----- cloudinit/DataSourceOVF.py | 2 -- cloudinit/UserDataHandler.py | 2 -- 21 files changed, 29 insertions(+), 42 deletions(-) (limited to 'cloudinit/CloudConfig/cc_chef.py') diff --git a/cloud-init-cfg.py b/cloud-init-cfg.py index 2930cd88..de64ef9c 100755 --- a/cloud-init-cfg.py +++ b/cloud-init-cfg.py @@ -23,7 +23,6 @@ import cloudinit.util as util import cloudinit.CloudConfig as CC import logging import os -import traceback def Usage(out = sys.stdout): out.write("Usage: %s name\n" % sys.argv[0]) diff --git a/cloud-init-query.py b/cloud-init-query.py index 3e8c24ab..d4b61a2b 100755 --- a/cloud-init-query.py +++ b/cloud-init-query.py @@ -20,9 +20,6 @@ import sys import cloudinit import cloudinit.CloudConfig -import logging -import os -import traceback def Usage(out = sys.stdout): out.write("Usage: %s name\n" % sys.argv[0]) diff --git a/cloudinit/CloudConfig/cc_chef.py b/cloudinit/CloudConfig/cc_chef.py index f093d0b2..08a9f40e 100644 --- a/cloudinit/CloudConfig/cc_chef.py +++ b/cloudinit/CloudConfig/cc_chef.py @@ -14,13 +14,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import os -import pwd -import socket import subprocess import json -import StringIO -import ConfigParser import cloudinit.CloudConfig as cc import cloudinit.util as util diff --git a/cloudinit/CloudConfig/cc_foo.py b/cloudinit/CloudConfig/cc_foo.py index fb14707e..c15e8d75 100644 --- a/cloudinit/CloudConfig/cc_foo.py +++ b/cloudinit/CloudConfig/cc_foo.py @@ -15,10 +15,12 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . -import cloudinit -import cloudinit.util as util + +#import cloudinit +#import cloudinit.util as util from cloudinit.CloudConfig import per_instance frequency = per_instance + def handle(name,cfg,cloud,log,args): print "hi" diff --git a/cloudinit/CloudConfig/cc_grub_dpkg.py b/cloudinit/CloudConfig/cc_grub_dpkg.py index b26e90e8..588f1403 100644 --- a/cloudinit/CloudConfig/cc_grub_dpkg.py +++ b/cloudinit/CloudConfig/cc_grub_dpkg.py @@ -15,8 +15,8 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -import subprocess import traceback import os diff --git a/cloudinit/CloudConfig/cc_mcollective.py b/cloudinit/CloudConfig/cc_mcollective.py index c7912aa4..656dd406 100644 --- a/cloudinit/CloudConfig/cc_mcollective.py +++ b/cloudinit/CloudConfig/cc_mcollective.py @@ -16,11 +16,9 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import os -import pwd -import socket import subprocess -import fileinput import StringIO import ConfigParser import cloudinit.CloudConfig as cc diff --git a/cloudinit/CloudConfig/cc_resizefs.py b/cloudinit/CloudConfig/cc_resizefs.py index d3a22fd6..83062642 100644 --- a/cloudinit/CloudConfig/cc_resizefs.py +++ b/cloudinit/CloudConfig/cc_resizefs.py @@ -15,9 +15,9 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util import subprocess -import traceback import os import stat import tempfile diff --git a/cloudinit/CloudConfig/cc_rightscale_userdata.py b/cloudinit/CloudConfig/cc_rightscale_userdata.py index 4875acbe..ce0c2fe9 100644 --- a/cloudinit/CloudConfig/cc_rightscale_userdata.py +++ b/cloudinit/CloudConfig/cc_rightscale_userdata.py @@ -32,8 +32,9 @@ ## Therefore, this must run before that. ## ## + import cloudinit.util as util -from cloudinit.CloudConfig import per_once, per_always, per_instance +from cloudinit.CloudConfig import per_instance from cloudinit import get_ipath_cur from urlparse import parse_qs diff --git a/cloudinit/CloudConfig/cc_rsyslog.py b/cloudinit/CloudConfig/cc_rsyslog.py index a8ee8476..dfd72c33 100644 --- a/cloudinit/CloudConfig/cc_rsyslog.py +++ b/cloudinit/CloudConfig/cc_rsyslog.py @@ -15,10 +15,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit import logging import cloudinit.util as util -import subprocess import traceback DEF_FILENAME = "20-cloud-config.conf" diff --git a/cloudinit/CloudConfig/cc_runcmd.py b/cloudinit/CloudConfig/cc_runcmd.py index f030980a..633f411b 100644 --- a/cloudinit/CloudConfig/cc_runcmd.py +++ b/cloudinit/CloudConfig/cc_runcmd.py @@ -15,7 +15,7 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . -import cloudinit + import cloudinit.util as util def handle(name,cfg,cloud,log,args): diff --git a/cloudinit/CloudConfig/cc_scripts_per_boot.py b/cloudinit/CloudConfig/cc_scripts_per_boot.py index 4e407fb7..2af060af 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_boot.py +++ b/cloudinit/CloudConfig/cc_scripts_per_boot.py @@ -15,9 +15,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -from cloudinit.CloudConfig import per_once, per_always, per_instance -from cloudinit import get_cpath, get_ipath_cur +from cloudinit.CloudConfig import per_always +from cloudinit import get_cpath frequency = per_always runparts_path = "%s/%s" % (get_cpath(), "scripts/per-boot") diff --git a/cloudinit/CloudConfig/cc_scripts_per_instance.py b/cloudinit/CloudConfig/cc_scripts_per_instance.py index 22b41185..e00f5e7b 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_instance.py +++ b/cloudinit/CloudConfig/cc_scripts_per_instance.py @@ -15,9 +15,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -from cloudinit.CloudConfig import per_once, per_always, per_instance -from cloudinit import get_cpath, get_ipath_cur +from cloudinit.CloudConfig import per_instance +from cloudinit import get_cpath frequency = per_instance runparts_path = "%s/%s" % (get_cpath(), "scripts/per-instance") diff --git a/cloudinit/CloudConfig/cc_scripts_per_once.py b/cloudinit/CloudConfig/cc_scripts_per_once.py index 9d752325..3c1a151b 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_once.py +++ b/cloudinit/CloudConfig/cc_scripts_per_once.py @@ -15,9 +15,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -from cloudinit.CloudConfig import per_once, per_always, per_instance -from cloudinit import get_cpath, get_ipath_cur +from cloudinit.CloudConfig import per_once +from cloudinit import get_cpath frequency = per_once runparts_path = "%s/%s" % (get_cpath(), "scripts/per-once") diff --git a/cloudinit/CloudConfig/cc_scripts_user.py b/cloudinit/CloudConfig/cc_scripts_user.py index bafecd23..2f44b66b 100644 --- a/cloudinit/CloudConfig/cc_scripts_user.py +++ b/cloudinit/CloudConfig/cc_scripts_user.py @@ -15,9 +15,10 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -from cloudinit.CloudConfig import per_once, per_always, per_instance -from cloudinit import get_cpath, get_ipath_cur +from cloudinit.CloudConfig import per_instance +from cloudinit import get_ipath_cur frequency = per_instance runparts_path = "%s/%s" % (get_ipath_cur(), "scripts") diff --git a/cloudinit/CloudConfig/cc_set_hostname.py b/cloudinit/CloudConfig/cc_set_hostname.py index bc190049..1a70d9e0 100644 --- a/cloudinit/CloudConfig/cc_set_hostname.py +++ b/cloudinit/CloudConfig/cc_set_hostname.py @@ -15,8 +15,8 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -import subprocess def handle(name,cfg,cloud,log,args): if util.get_cfg_option_bool(cfg,"preserve_hostname",False): diff --git a/cloudinit/CloudConfig/cc_set_passwords.py b/cloudinit/CloudConfig/cc_set_passwords.py index edfbaeeb..c7fa4d64 100644 --- a/cloudinit/CloudConfig/cc_set_passwords.py +++ b/cloudinit/CloudConfig/cc_set_passwords.py @@ -15,8 +15,8 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + import cloudinit.util as util -import subprocess import sys import random import string diff --git a/cloudinit/CloudConfig/cc_timezone.py b/cloudinit/CloudConfig/cc_timezone.py index e8a453f9..9a6e50d1 100644 --- a/cloudinit/CloudConfig/cc_timezone.py +++ b/cloudinit/CloudConfig/cc_timezone.py @@ -15,9 +15,9 @@ # # You should have received a copy of the GNU General Public License # along with this program. If not, see . + from cloudinit.CloudConfig import per_instance from cloudinit import util -import subprocess import os.path import shutil diff --git a/cloudinit/DataSourceEc2.py b/cloudinit/DataSourceEc2.py index aee10ffa..eed310c9 100644 --- a/cloudinit/DataSourceEc2.py +++ b/cloudinit/DataSourceEc2.py @@ -18,16 +18,13 @@ import DataSource -from cloudinit import seeddir, log +from cloudinit import seeddir, log # pylint: disable=W0611 import cloudinit.util as util import socket import urllib2 import time -import sys import boto.utils as boto_utils import os.path -import errno -import urlparse class DataSourceEc2(DataSource.DataSource): api_ver = '2009-04-04' diff --git a/cloudinit/DataSourceNoCloud.py b/cloudinit/DataSourceNoCloud.py index af99fc2b..2f4bd604 100644 --- a/cloudinit/DataSourceNoCloud.py +++ b/cloudinit/DataSourceNoCloud.py @@ -18,12 +18,8 @@ import DataSource -from cloudinit import seeddir, log +from cloudinit import seeddir, log # pylint: disable=W0611 import cloudinit.util as util -import sys -import os.path -import os -import errno class DataSourceNoCloud(DataSource.DataSource): metadata = None diff --git a/cloudinit/DataSourceOVF.py b/cloudinit/DataSourceOVF.py index 76c5dbd8..4f4187fb 100644 --- a/cloudinit/DataSourceOVF.py +++ b/cloudinit/DataSourceOVF.py @@ -23,9 +23,7 @@ import cloudinit.util as util import sys import os.path import os -import errno from xml.dom import minidom -from xml.dom import Node import base64 import re import tempfile diff --git a/cloudinit/UserDataHandler.py b/cloudinit/UserDataHandler.py index 541ee87a..ac2cdda9 100644 --- a/cloudinit/UserDataHandler.py +++ b/cloudinit/UserDataHandler.py @@ -20,12 +20,10 @@ import email from email.mime.multipart import MIMEMultipart from email.mime.text import MIMEText from email.mime.base import MIMEBase -from email import encoders import yaml import cloudinit import cloudinit.util as util import hashlib -import os import urllib starts_with_mappings={ -- cgit v1.2.3 From 234cefd314a7e1cfb51fad3bdf344d801a8b238d Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:51:47 +0100 Subject: [PATCH 05/13] Fix pylint warnings W0613 (unused argument) From: Juerg Haefliger --- cloudinit/CloudConfig/cc_apt_update_upgrade.py | 2 +- cloudinit/CloudConfig/cc_bootcmd.py | 2 +- cloudinit/CloudConfig/cc_byobu.py | 2 +- cloudinit/CloudConfig/cc_chef.py | 2 +- cloudinit/CloudConfig/cc_disable_ec2_metadata.py | 2 +- cloudinit/CloudConfig/cc_final_message.py | 2 +- cloudinit/CloudConfig/cc_foo.py | 2 +- cloudinit/CloudConfig/cc_grub_dpkg.py | 2 +- cloudinit/CloudConfig/cc_keys_to_console.py | 2 +- cloudinit/CloudConfig/cc_landscape.py | 2 +- cloudinit/CloudConfig/cc_locale.py | 2 +- cloudinit/CloudConfig/cc_mcollective.py | 2 +- cloudinit/CloudConfig/cc_mounts.py | 2 +- cloudinit/CloudConfig/cc_phone_home.py | 2 +- cloudinit/CloudConfig/cc_puppet.py | 2 +- cloudinit/CloudConfig/cc_resizefs.py | 2 +- cloudinit/CloudConfig/cc_rightscale_userdata.py | 2 +- cloudinit/CloudConfig/cc_rsyslog.py | 2 +- cloudinit/CloudConfig/cc_runcmd.py | 2 +- cloudinit/CloudConfig/cc_scripts_per_boot.py | 2 +- cloudinit/CloudConfig/cc_scripts_per_instance.py | 2 +- cloudinit/CloudConfig/cc_scripts_per_once.py | 2 +- cloudinit/CloudConfig/cc_scripts_user.py | 2 +- cloudinit/CloudConfig/cc_set_hostname.py | 2 +- cloudinit/CloudConfig/cc_set_passwords.py | 2 +- cloudinit/CloudConfig/cc_ssh.py | 2 +- cloudinit/CloudConfig/cc_ssh_import_id.py | 2 +- cloudinit/CloudConfig/cc_timezone.py | 2 +- cloudinit/CloudConfig/cc_update_etc_hosts.py | 4 ++-- cloudinit/CloudConfig/cc_update_hostname.py | 2 +- cloudinit/DataSource.py | 2 +- cloudinit/__init__.py | 10 +++++----- 32 files changed, 37 insertions(+), 37 deletions(-) (limited to 'cloudinit/CloudConfig/cc_chef.py') diff --git a/cloudinit/CloudConfig/cc_apt_update_upgrade.py b/cloudinit/CloudConfig/cc_apt_update_upgrade.py index 0677ceb5..e9fd2c07 100644 --- a/cloudinit/CloudConfig/cc_apt_update_upgrade.py +++ b/cloudinit/CloudConfig/cc_apt_update_upgrade.py @@ -22,7 +22,7 @@ import os import glob import cloudinit.CloudConfig as cc -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): update = util.get_cfg_option_bool(cfg, 'apt_update', False) upgrade = util.get_cfg_option_bool(cfg, 'apt_upgrade', False) diff --git a/cloudinit/CloudConfig/cc_bootcmd.py b/cloudinit/CloudConfig/cc_bootcmd.py index 5a9e4356..fc925447 100644 --- a/cloudinit/CloudConfig/cc_bootcmd.py +++ b/cloudinit/CloudConfig/cc_bootcmd.py @@ -21,7 +21,7 @@ import tempfile from cloudinit.CloudConfig import per_always frequency = per_always -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): if not cfg.has_key("bootcmd"): return diff --git a/cloudinit/CloudConfig/cc_byobu.py b/cloudinit/CloudConfig/cc_byobu.py index 406a1f67..dd510dda 100644 --- a/cloudinit/CloudConfig/cc_byobu.py +++ b/cloudinit/CloudConfig/cc_byobu.py @@ -19,7 +19,7 @@ import cloudinit.util as util import subprocess import traceback -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: value = args[0] else: diff --git a/cloudinit/CloudConfig/cc_chef.py b/cloudinit/CloudConfig/cc_chef.py index 08a9f40e..6b8e3825 100644 --- a/cloudinit/CloudConfig/cc_chef.py +++ b/cloudinit/CloudConfig/cc_chef.py @@ -23,7 +23,7 @@ import cloudinit.util as util ruby_version_default = "1.8" -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): # If there isn't a chef key in the configuration don't do anything if not cfg.has_key('chef'): return chef_cfg = cfg['chef'] diff --git a/cloudinit/CloudConfig/cc_disable_ec2_metadata.py b/cloudinit/CloudConfig/cc_disable_ec2_metadata.py index 5cd1d1c1..f06d4dfc 100644 --- a/cloudinit/CloudConfig/cc_disable_ec2_metadata.py +++ b/cloudinit/CloudConfig/cc_disable_ec2_metadata.py @@ -21,7 +21,7 @@ from cloudinit.CloudConfig import per_always frequency = per_always -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,_log,_args): if util.get_cfg_option_bool(cfg, "disable_ec2_metadata", False): fwall="route add -host 169.254.169.254 reject" subprocess.call(fwall.split(' ')) diff --git a/cloudinit/CloudConfig/cc_final_message.py b/cloudinit/CloudConfig/cc_final_message.py index fbdb5c94..c8631d01 100644 --- a/cloudinit/CloudConfig/cc_final_message.py +++ b/cloudinit/CloudConfig/cc_final_message.py @@ -24,7 +24,7 @@ frequency = per_always final_message = "cloud-init boot finished at $TIMESTAMP. Up $UPTIME seconds" -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: msg_in = args[0] else: diff --git a/cloudinit/CloudConfig/cc_foo.py b/cloudinit/CloudConfig/cc_foo.py index c15e8d75..48d20e5b 100644 --- a/cloudinit/CloudConfig/cc_foo.py +++ b/cloudinit/CloudConfig/cc_foo.py @@ -22,5 +22,5 @@ from cloudinit.CloudConfig import per_instance frequency = per_instance -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,_cloud,_log,_args): print "hi" diff --git a/cloudinit/CloudConfig/cc_grub_dpkg.py b/cloudinit/CloudConfig/cc_grub_dpkg.py index 588f1403..97d79bdb 100644 --- a/cloudinit/CloudConfig/cc_grub_dpkg.py +++ b/cloudinit/CloudConfig/cc_grub_dpkg.py @@ -20,7 +20,7 @@ import cloudinit.util as util import traceback import os -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,_args): idevs=None idevs_empty=None diff --git a/cloudinit/CloudConfig/cc_keys_to_console.py b/cloudinit/CloudConfig/cc_keys_to_console.py index 79dd2ea9..08e8f085 100644 --- a/cloudinit/CloudConfig/cc_keys_to_console.py +++ b/cloudinit/CloudConfig/cc_keys_to_console.py @@ -21,7 +21,7 @@ import subprocess frequency = per_instance -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,_args): cmd = [ '/usr/lib/cloud-init/write-ssh-key-fingerprints' ] fp_blacklist = util.get_cfg_option_list_or_str(cfg, "ssh_fp_console_blacklist", []) key_blacklist = util.get_cfg_option_list_or_str(cfg, "ssh_key_console_blacklist", ["ssh-dss"]) diff --git a/cloudinit/CloudConfig/cc_landscape.py b/cloudinit/CloudConfig/cc_landscape.py index e3e4f7ba..22a90665 100644 --- a/cloudinit/CloudConfig/cc_landscape.py +++ b/cloudinit/CloudConfig/cc_landscape.py @@ -32,7 +32,7 @@ lsc_builtincfg = { } } -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,_args): """ Basically turn a top level 'landscape' entry with a 'client' dict and render it to ConfigObj format under '[client]' section in diff --git a/cloudinit/CloudConfig/cc_locale.py b/cloudinit/CloudConfig/cc_locale.py index 8542f641..8e91d3bf 100644 --- a/cloudinit/CloudConfig/cc_locale.py +++ b/cloudinit/CloudConfig/cc_locale.py @@ -28,7 +28,7 @@ def apply_locale(locale, cfgfile): util.render_to_file('default-locale', cfgfile, { 'locale' : locale }) -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,args): if len(args) != 0: locale = args[0] else: diff --git a/cloudinit/CloudConfig/cc_mcollective.py b/cloudinit/CloudConfig/cc_mcollective.py index 656dd406..38fe4a3c 100644 --- a/cloudinit/CloudConfig/cc_mcollective.py +++ b/cloudinit/CloudConfig/cc_mcollective.py @@ -38,7 +38,7 @@ class FakeSecHead(object): finally: self.sechead = None else: return self.fp.readline() -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,_log,_args): # If there isn't a mcollective key in the configuration don't do anything if not cfg.has_key('mcollective'): return mcollective_cfg = cfg['mcollective'] diff --git a/cloudinit/CloudConfig/cc_mounts.py b/cloudinit/CloudConfig/cc_mounts.py index db382f04..327b480f 100644 --- a/cloudinit/CloudConfig/cc_mounts.py +++ b/cloudinit/CloudConfig/cc_mounts.py @@ -31,7 +31,7 @@ def is_mdname(name): return True return False -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): # fs_spec, fs_file, fs_vfstype, fs_mntops, fs-freq, fs_passno defvals = [ None, None, "auto", "defaults,nobootwait", "0", "2" ] defvals = cfg.get("mount_default_fields", defvals) diff --git a/cloudinit/CloudConfig/cc_phone_home.py b/cloudinit/CloudConfig/cc_phone_home.py index f291e1d4..3dcec17f 100644 --- a/cloudinit/CloudConfig/cc_phone_home.py +++ b/cloudinit/CloudConfig/cc_phone_home.py @@ -31,7 +31,7 @@ post_list_all = [ 'pub_key_dsa', 'pub_key_rsa', 'pub_key_ecdsa', 'instance_id', # url: http://my.foo.bar/$INSTANCE_ID/ # post: [ pub_key_dsa, pub_key_rsa, pub_key_ecdsa, instance_id # -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,args): if len(args) != 0: ph_cfg = util.readconf(args[0]) else: diff --git a/cloudinit/CloudConfig/cc_puppet.py b/cloudinit/CloudConfig/cc_puppet.py index c635d1f5..3748559a 100644 --- a/cloudinit/CloudConfig/cc_puppet.py +++ b/cloudinit/CloudConfig/cc_puppet.py @@ -25,7 +25,7 @@ import ConfigParser import cloudinit.CloudConfig as cc import cloudinit.util as util -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): # If there isn't a puppet key in the configuration don't do anything if not cfg.has_key('puppet'): return puppet_cfg = cfg['puppet'] diff --git a/cloudinit/CloudConfig/cc_resizefs.py b/cloudinit/CloudConfig/cc_resizefs.py index 83062642..d4b260b8 100644 --- a/cloudinit/CloudConfig/cc_resizefs.py +++ b/cloudinit/CloudConfig/cc_resizefs.py @@ -25,7 +25,7 @@ from cloudinit.CloudConfig import per_always frequency = per_always -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: resize_root = False if str(args[0]).lower() in [ 'true', '1', 'on', 'yes']: diff --git a/cloudinit/CloudConfig/cc_rightscale_userdata.py b/cloudinit/CloudConfig/cc_rightscale_userdata.py index ce0c2fe9..050d89aa 100644 --- a/cloudinit/CloudConfig/cc_rightscale_userdata.py +++ b/cloudinit/CloudConfig/cc_rightscale_userdata.py @@ -42,7 +42,7 @@ frequency = per_instance my_name = "cc_rightscale_userdata" my_hookname = 'CLOUD_INIT_REMOTE_HOOK' -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,cloud,log,_args): try: ud = cloud.get_userdata_raw() except: diff --git a/cloudinit/CloudConfig/cc_rsyslog.py b/cloudinit/CloudConfig/cc_rsyslog.py index dfd72c33..4e427ae0 100644 --- a/cloudinit/CloudConfig/cc_rsyslog.py +++ b/cloudinit/CloudConfig/cc_rsyslog.py @@ -24,7 +24,7 @@ import traceback DEF_FILENAME = "20-cloud-config.conf" DEF_DIR = "/etc/rsyslog.d" -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,_args): # rsyslog: # - "*.* @@192.158.1.1" # - content: "*.* @@192.0.2.1:10514" diff --git a/cloudinit/CloudConfig/cc_runcmd.py b/cloudinit/CloudConfig/cc_runcmd.py index 633f411b..d255223b 100644 --- a/cloudinit/CloudConfig/cc_runcmd.py +++ b/cloudinit/CloudConfig/cc_runcmd.py @@ -18,7 +18,7 @@ import cloudinit.util as util -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): if not cfg.has_key("runcmd"): return outfile="%s/runcmd" % cloud.get_ipath('scripts') diff --git a/cloudinit/CloudConfig/cc_scripts_per_boot.py b/cloudinit/CloudConfig/cc_scripts_per_boot.py index 2af060af..fb643c6d 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_boot.py +++ b/cloudinit/CloudConfig/cc_scripts_per_boot.py @@ -23,7 +23,7 @@ from cloudinit import get_cpath frequency = per_always runparts_path = "%s/%s" % (get_cpath(), "scripts/per-boot") -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,_cloud,log,_args): try: util.runparts(runparts_path) except: diff --git a/cloudinit/CloudConfig/cc_scripts_per_instance.py b/cloudinit/CloudConfig/cc_scripts_per_instance.py index e00f5e7b..b0f0601a 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_instance.py +++ b/cloudinit/CloudConfig/cc_scripts_per_instance.py @@ -23,7 +23,7 @@ from cloudinit import get_cpath frequency = per_instance runparts_path = "%s/%s" % (get_cpath(), "scripts/per-instance") -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,_cloud,log,_args): try: util.runparts(runparts_path) except: diff --git a/cloudinit/CloudConfig/cc_scripts_per_once.py b/cloudinit/CloudConfig/cc_scripts_per_once.py index 3c1a151b..2ab81840 100644 --- a/cloudinit/CloudConfig/cc_scripts_per_once.py +++ b/cloudinit/CloudConfig/cc_scripts_per_once.py @@ -23,7 +23,7 @@ from cloudinit import get_cpath frequency = per_once runparts_path = "%s/%s" % (get_cpath(), "scripts/per-once") -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,_cloud,log,_args): try: util.runparts(runparts_path) except: diff --git a/cloudinit/CloudConfig/cc_scripts_user.py b/cloudinit/CloudConfig/cc_scripts_user.py index 2f44b66b..9c7f2322 100644 --- a/cloudinit/CloudConfig/cc_scripts_user.py +++ b/cloudinit/CloudConfig/cc_scripts_user.py @@ -23,7 +23,7 @@ from cloudinit import get_ipath_cur frequency = per_instance runparts_path = "%s/%s" % (get_ipath_cur(), "scripts") -def handle(name,cfg,cloud,log,args): +def handle(_name,_cfg,_cloud,log,_args): try: util.runparts(runparts_path) except: diff --git a/cloudinit/CloudConfig/cc_set_hostname.py b/cloudinit/CloudConfig/cc_set_hostname.py index 1a70d9e0..129e9165 100644 --- a/cloudinit/CloudConfig/cc_set_hostname.py +++ b/cloudinit/CloudConfig/cc_set_hostname.py @@ -18,7 +18,7 @@ import cloudinit.util as util -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): if util.get_cfg_option_bool(cfg,"preserve_hostname",False): log.debug("preserve_hostname is set. not setting hostname") return(True) diff --git a/cloudinit/CloudConfig/cc_set_passwords.py b/cloudinit/CloudConfig/cc_set_passwords.py index c7fa4d64..f44d450c 100644 --- a/cloudinit/CloudConfig/cc_set_passwords.py +++ b/cloudinit/CloudConfig/cc_set_passwords.py @@ -21,7 +21,7 @@ import sys import random import string -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: # if run from command line, and give args, wipe the chpasswd['list'] password = args[0] diff --git a/cloudinit/CloudConfig/cc_ssh.py b/cloudinit/CloudConfig/cc_ssh.py index 9f5dc567..0aad2187 100644 --- a/cloudinit/CloudConfig/cc_ssh.py +++ b/cloudinit/CloudConfig/cc_ssh.py @@ -26,7 +26,7 @@ DISABLE_ROOT_OPTS="no-port-forwarding,no-agent-forwarding,no-X11-forwarding,comm global_log = None -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): global global_log global_log = log diff --git a/cloudinit/CloudConfig/cc_ssh_import_id.py b/cloudinit/CloudConfig/cc_ssh_import_id.py index bf1314be..7e7a54a1 100644 --- a/cloudinit/CloudConfig/cc_ssh_import_id.py +++ b/cloudinit/CloudConfig/cc_ssh_import_id.py @@ -19,7 +19,7 @@ import cloudinit.util as util import subprocess import traceback -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: user = args[0] ids = [ ] diff --git a/cloudinit/CloudConfig/cc_timezone.py b/cloudinit/CloudConfig/cc_timezone.py index 9a6e50d1..26b2796d 100644 --- a/cloudinit/CloudConfig/cc_timezone.py +++ b/cloudinit/CloudConfig/cc_timezone.py @@ -24,7 +24,7 @@ import shutil frequency = per_instance tz_base = "/usr/share/zoneinfo" -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,_cloud,log,args): if len(args) != 0: timezone = args[0] else: diff --git a/cloudinit/CloudConfig/cc_update_etc_hosts.py b/cloudinit/CloudConfig/cc_update_etc_hosts.py index cf2ad046..9e3a1dbb 100644 --- a/cloudinit/CloudConfig/cc_update_etc_hosts.py +++ b/cloudinit/CloudConfig/cc_update_etc_hosts.py @@ -21,7 +21,7 @@ import StringIO frequency = per_always -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): ( hostname, fqdn ) = util.get_hostname_fqdn(cfg, cloud) manage_hosts = util.get_cfg_option_bool(cfg,"manage_etc_hosts", False) @@ -48,7 +48,7 @@ def handle(name,cfg,cloud,log,args): log.debug("not managing /etc/hosts") -def update_etc_hosts(hostname, fqdn, log): +def update_etc_hosts(hostname, fqdn, _log): with open('/etc/hosts', 'r') as etchosts: header = "# Added by cloud-init\n" hosts_line = "127.0.1.1\t%s %s\n" % (fqdn, hostname) diff --git a/cloudinit/CloudConfig/cc_update_hostname.py b/cloudinit/CloudConfig/cc_update_hostname.py index 3f55c73b..871e40bf 100644 --- a/cloudinit/CloudConfig/cc_update_hostname.py +++ b/cloudinit/CloudConfig/cc_update_hostname.py @@ -22,7 +22,7 @@ from cloudinit.CloudConfig import per_always frequency = per_always -def handle(name,cfg,cloud,log,args): +def handle(_name,cfg,cloud,log,_args): if util.get_cfg_option_bool(cfg,"preserve_hostname",False): log.debug("preserve_hostname is set. not updating hostname") return diff --git a/cloudinit/DataSource.py b/cloudinit/DataSource.py index 06061a2e..66763181 100644 --- a/cloudinit/DataSource.py +++ b/cloudinit/DataSource.py @@ -82,7 +82,7 @@ class DataSource: return(keys) - def device_name_to_device(self, name): + def device_name_to_device(self, _name): # translate a 'name' to a device # the primary function at this point is on ec2 # to consult metadata service, that has diff --git a/cloudinit/__init__.py b/cloudinit/__init__.py index d01d443d..5460d6e1 100644 --- a/cloudinit/__init__.py +++ b/cloudinit/__init__.py @@ -365,7 +365,7 @@ class CloudInit: continue handler_call_end(mod, data, frequency) - def handle_user_script(self,data,ctype,filename,payload, frequency): + def handle_user_script(self,_data,ctype,filename,payload, _frequency): if ctype == "__end__": return if ctype == "__begin__": # maybe delete existing things here @@ -376,7 +376,7 @@ class CloudInit: util.write_file("%s/%s" % (scriptsdir,filename), util.dos2unix(payload), 0700) - def handle_upstart_job(self,data,ctype,filename,payload, frequency): + def handle_upstart_job(self,_data,ctype,filename,payload, frequency): # upstart jobs are only written on the first boot if frequency != per_instance: return @@ -388,7 +388,7 @@ class CloudInit: util.write_file("%s/%s" % ("/etc/init",filename), util.dos2unix(payload), 0644) - def handle_cloud_config(self,data,ctype,filename,payload, frequency): + def handle_cloud_config(self,_data,ctype,filename,payload, _frequency): if ctype == "__begin__": self.cloud_config_str="" return @@ -408,7 +408,7 @@ class CloudInit: self.cloud_config_str+="\n#%s\n%s" % (filename,payload) - def handle_cloud_boothook(self,data,ctype,filename,payload, frequency): + def handle_cloud_boothook(self,_data,ctype,filename,payload, _frequency): if ctype == "__end__": return if ctype == "__begin__": return @@ -538,7 +538,7 @@ def handler_handle_part(mod, data, ctype, filename, payload, frequency): else: mod.handle_part(data, ctype, filename, payload, frequency) -def partwalker_handle_handler(pdata, ctype, filename, payload): +def partwalker_handle_handler(pdata, _ctype, _filename, payload): curcount = pdata['handlercount'] modname = 'part-handler-%03d' % curcount -- cgit v1.2.3 From 6a2031c4dc005d9a0f42b10200b828eee5cf166f Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:54:54 +0100 Subject: [PATCH 06/13] Fix pylint warnings W0612 (unused variable) From: Juerg Haefliger --- cloud-init-query.py | 1 - cloudinit/CloudConfig/cc_apt_update_upgrade.py | 4 ++-- cloudinit/CloudConfig/cc_chef.py | 1 - cloudinit/CloudConfig/cc_resizefs.py | 4 ++-- cloudinit/CloudConfig/cc_rightscale_userdata.py | 3 +-- cloudinit/CloudConfig/cc_rsyslog.py | 4 +--- cloudinit/CloudConfig/cc_set_hostname.py | 4 ++-- cloudinit/CloudConfig/cc_set_passwords.py | 2 +- cloudinit/CloudConfig/cc_update_etc_hosts.py | 2 +- cloudinit/CloudConfig/cc_update_hostname.py | 4 ++-- cloudinit/DataSource.py | 4 ++-- cloudinit/DataSourceEc2.py | 5 ++--- cloudinit/DataSourceOVF.py | 4 ++-- cloudinit/SshUtil.py | 8 ++++---- cloudinit/UserDataHandler.py | 4 +--- cloudinit/__init__.py | 8 ++++---- cloudinit/netinfo.py | 8 ++++---- cloudinit/util.py | 7 +++---- 18 files changed, 34 insertions(+), 43 deletions(-) (limited to 'cloudinit/CloudConfig/cc_chef.py') diff --git a/cloud-init-query.py b/cloud-init-query.py index d4b61a2b..cfe9b429 100755 --- a/cloud-init-query.py +++ b/cloud-init-query.py @@ -31,7 +31,6 @@ def main(): sys.exit(1) cc = cloudinit.CloudConfig.CloudConfig(cloudinit.cloud_config) - cloud_config = cc.cfg data = { 'user_data' : cc.cloud.get_userdata(), 'user_data_raw' : cc.cloud.get_userdata_raw(), diff --git a/cloudinit/CloudConfig/cc_apt_update_upgrade.py b/cloudinit/CloudConfig/cc_apt_update_upgrade.py index e9fd2c07..22651067 100644 --- a/cloudinit/CloudConfig/cc_apt_update_upgrade.py +++ b/cloudinit/CloudConfig/cc_apt_update_upgrade.py @@ -121,7 +121,7 @@ def rename_apt_lists(omirror,new_mirror,lists_d="/var/lib/apt/lists"): os.rename(file,"%s%s" % (nprefix, file[olen:])) def get_release(): - stdout, stderr = subprocess.Popen(['lsb_release', '-cs'], stdout=subprocess.PIPE).communicate() + stdout, _stderr = subprocess.Popen(['lsb_release', '-cs'], stdout=subprocess.PIPE).communicate() return(stdout.strip()) def generate_sources_list(codename, mirror): @@ -202,7 +202,7 @@ def find_apt_mirror(cloud, cfg): if not mirror and cloud: # if we have a fqdn, then search its domain portion first - ( hostname, fqdn ) = util.get_hostname_fqdn(cfg, cloud) + ( _hostname, fqdn ) = util.get_hostname_fqdn(cfg, cloud) mydom = ".".join(fqdn.split(".")[1:]) if mydom: doms.append(".%s" % mydom) diff --git a/cloudinit/CloudConfig/cc_chef.py b/cloudinit/CloudConfig/cc_chef.py index 6b8e3825..977fe80f 100644 --- a/cloudinit/CloudConfig/cc_chef.py +++ b/cloudinit/CloudConfig/cc_chef.py @@ -41,7 +41,6 @@ def handle(_name,cfg,cloud,log,_args): with open('/etc/chef/validation.pem', 'w') as validation_key_fh: validation_key_fh.write(validation_key) - validation_name = chef_cfg.get('validation_name','chef-validator') # create the chef config from template util.render_to_file('chef_client.rb', '/etc/chef/client.rb', {'server_url': chef_cfg['server_url'], diff --git a/cloudinit/CloudConfig/cc_resizefs.py b/cloudinit/CloudConfig/cc_resizefs.py index d4b260b8..adec70be 100644 --- a/cloudinit/CloudConfig/cc_resizefs.py +++ b/cloudinit/CloudConfig/cc_resizefs.py @@ -53,7 +53,7 @@ def handle(_name,cfg,_cloud,log,args): cmd = [ 'blkid', '-c', '/dev/null', '-sTYPE', '-ovalue', devpth ] try: - (fstype,err) = util.subp(cmd) + (fstype,_err) = util.subp(cmd) except subprocess.CalledProcessError as e: log.warn("Failed to get filesystem type of maj=%s, min=%s via: %s" % (os.major(st_dev), os.minor(st_dev), cmd)) @@ -74,7 +74,7 @@ def handle(_name,cfg,_cloud,log,args): return try: - (out,err) = util.subp(resize_cmd) + util.subp(resize_cmd) except subprocess.CalledProcessError as e: log.warn("Failed to resize filesystem (%s)" % resize_cmd) log.warn("output=%s\nerror=%s\n", e.output[0], e.output[1]) diff --git a/cloudinit/CloudConfig/cc_rightscale_userdata.py b/cloudinit/CloudConfig/cc_rightscale_userdata.py index 050d89aa..2b43023c 100644 --- a/cloudinit/CloudConfig/cc_rightscale_userdata.py +++ b/cloudinit/CloudConfig/cc_rightscale_userdata.py @@ -50,7 +50,7 @@ def handle(_name,_cfg,cloud,log,_args): return try: - mdict = parse_qs(cloud.get_userdata_raw()) + mdict = parse_qs(ud) if not my_hookname in mdict: return except: log.warn("failed to urlparse.parse_qa(userdata_raw())") @@ -58,7 +58,6 @@ def handle(_name,_cfg,cloud,log,_args): scripts_d = get_ipath_cur('scripts') i = 0 - errors = [ ] first_e = None for url in mdict[my_hookname]: fname = "%s/rightscale-%02i" % (scripts_d,i) diff --git a/cloudinit/CloudConfig/cc_rsyslog.py b/cloudinit/CloudConfig/cc_rsyslog.py index 4e427ae0..ab85a6d8 100644 --- a/cloudinit/CloudConfig/cc_rsyslog.py +++ b/cloudinit/CloudConfig/cc_rsyslog.py @@ -38,8 +38,6 @@ def handle(_name,cfg,_cloud,log,_args): def_dir = cfg.get('rsyslog_dir', DEF_DIR) def_fname = cfg.get('rsyslog_filename', DEF_FILENAME) - entries = cfg['rsyslog'] - files = [ ] elst = [ ] for ent in cfg['rsyslog']: @@ -80,7 +78,7 @@ def handle(_name,cfg,_cloud,log,_args): # it will also return failure on the attempt, so 'restarted' # won't get set log.debug("restarting rsyslog") - p = util.subp(['service', 'rsyslog', 'restart']) + util.subp(['service', 'rsyslog', 'restart']) restarted = True except Exception as e: diff --git a/cloudinit/CloudConfig/cc_set_hostname.py b/cloudinit/CloudConfig/cc_set_hostname.py index 129e9165..4f19b0c8 100644 --- a/cloudinit/CloudConfig/cc_set_hostname.py +++ b/cloudinit/CloudConfig/cc_set_hostname.py @@ -23,10 +23,10 @@ def handle(_name,cfg,cloud,log,_args): log.debug("preserve_hostname is set. not setting hostname") return(True) - ( hostname, fqdn ) = util.get_hostname_fqdn(cfg, cloud) + ( hostname, _fqdn ) = util.get_hostname_fqdn(cfg, cloud) try: set_hostname(hostname, log) - except Exception as e: + except Exception: util.logexc(log) log.warn("failed to set hostname to %s\n", hostname) diff --git a/cloudinit/CloudConfig/cc_set_passwords.py b/cloudinit/CloudConfig/cc_set_passwords.py index f44d450c..07e3ca1b 100644 --- a/cloudinit/CloudConfig/cc_set_passwords.py +++ b/cloudinit/CloudConfig/cc_set_passwords.py @@ -115,7 +115,7 @@ def handle(_name,cfg,_cloud,log,args): return def rand_str(strlen=32, select_from=string.letters+string.digits): - return("".join([random.choice(select_from) for x in range(0, strlen)])) + return("".join([random.choice(select_from) for _x in range(0, strlen)])) def rand_user_password(pwlen=9): selfrom=(string.letters.translate(None,'loLOI') + diff --git a/cloudinit/CloudConfig/cc_update_etc_hosts.py b/cloudinit/CloudConfig/cc_update_etc_hosts.py index 9e3a1dbb..1c245e67 100644 --- a/cloudinit/CloudConfig/cc_update_etc_hosts.py +++ b/cloudinit/CloudConfig/cc_update_etc_hosts.py @@ -34,7 +34,7 @@ def handle(_name,cfg,cloud,log,_args): util.render_to_file('hosts', '/etc/hosts', \ { 'hostname' : hostname, 'fqdn' : fqdn }) - except Exception as e: + except Exception: log.warn("failed to update /etc/hosts") raise elif manage_hosts == "localhost": diff --git a/cloudinit/CloudConfig/cc_update_hostname.py b/cloudinit/CloudConfig/cc_update_hostname.py index 871e40bf..4bc1cb2b 100644 --- a/cloudinit/CloudConfig/cc_update_hostname.py +++ b/cloudinit/CloudConfig/cc_update_hostname.py @@ -27,11 +27,11 @@ def handle(_name,cfg,cloud,log,_args): log.debug("preserve_hostname is set. not updating hostname") return - ( hostname, fqdn ) = util.get_hostname_fqdn(cfg, cloud) + ( hostname, _fqdn ) = util.get_hostname_fqdn(cfg, cloud) try: prev ="%s/%s" % (cloud.get_cpath('data'),"previous-hostname") update_hostname(hostname, prev, log) - except Exception as e: + except Exception: log.warn("failed to set hostname\n") raise diff --git a/cloudinit/DataSource.py b/cloudinit/DataSource.py index 66763181..bdcac4ef 100644 --- a/cloudinit/DataSource.py +++ b/cloudinit/DataSource.py @@ -69,7 +69,7 @@ class DataSource: if isinstance(self.metadata['public-keys'], str): return([self.metadata['public-keys'],]) - for keyname, klist in self.metadata['public-keys'].items(): + for _keyname, klist in self.metadata['public-keys'].items(): # lp:506332 uec metadata service responds with # data that makes boto populate a string for 'klist' rather # than a list. @@ -197,7 +197,7 @@ def is_ipv4(instr): return False try: - r = filter(lambda x: int(x) < 256 and x > 0, toks) + filter(lambda x: int(x) < 256 and x > 0, toks) except: return False diff --git a/cloudinit/DataSourceEc2.py b/cloudinit/DataSourceEc2.py index eed310c9..9191e647 100644 --- a/cloudinit/DataSourceEc2.py +++ b/cloudinit/DataSourceEc2.py @@ -89,7 +89,7 @@ class DataSourceEc2(DataSource.DataSource): max_wait = 120 try: max_wait = int(mcfg.get("max_wait",max_wait)) - except Exception as e: + except Exception: util.logexc(log) log.warn("Failed to get max wait. using %s" % max_wait) @@ -99,7 +99,7 @@ class DataSourceEc2(DataSource.DataSource): timeout = 50 try: timeout = int(mcfg.get("timeout",timeout)) - except Exception as e: + except Exception: util.logexc(log) log.warn("Failed to get timeout, using %s" % timeout) @@ -226,7 +226,6 @@ def wait_for_metadata_service(urls, max_wait=None, timeout=None, status_cb=None) starttime = time.time() sleeptime = 1 - timeout_orig = timeout if status_cb == None: def status_cb(msg): return diff --git a/cloudinit/DataSourceOVF.py b/cloudinit/DataSourceOVF.py index 4f4187fb..e4d28aba 100644 --- a/cloudinit/DataSourceOVF.py +++ b/cloudinit/DataSourceOVF.py @@ -64,7 +64,7 @@ class DataSourceOVF(DataSource.DataSource): np = { 'iso' : transport_iso9660, 'vmware-guestd' : transport_vmware_guestd, } for name, transfunc in np.iteritems(): - (contents, dev, fname) = transfunc() + (contents, _dev, _fname) = transfunc() if contents: break if contents: @@ -171,7 +171,7 @@ def transport_iso9660(require_iso=False): mounted = { } for mpline in mounts: - (dev,mp,fstype,opts,freq,passno) = mpline.split() + (dev,mp,fstype,_opts,_freq,_passno) = mpline.split() mounted[dev]=(dev,fstype,mp,False) mp = mp.replace("\\040"," ") if fstype != "iso9660" and require_iso: continue diff --git a/cloudinit/SshUtil.py b/cloudinit/SshUtil.py index fdd3bb27..125ca618 100644 --- a/cloudinit/SshUtil.py +++ b/cloudinit/SshUtil.py @@ -46,7 +46,7 @@ class AuthKeyEntry(): elif curc == '"': quoted = not quoted i = i + 1 - except IndexError as e: + except IndexError: self.is_comment = True return() @@ -54,7 +54,7 @@ class AuthKeyEntry(): self.options = ent[0:i] (self.keytype, self.base64, self.comment) = \ ent[i+1:].split(None,3) - except ValueError as e: + except ValueError: # we did not understand this line self.is_comment = True @@ -84,7 +84,7 @@ def update_authorized_keys(fname, keys): fp = open(fname, "r") lines = fp.readlines() # lines have carriage return fp.close() - except IOError as e: + except IOError: lines = [ ] ka_stats = { } # keys_added status @@ -133,7 +133,7 @@ def setup_user_keys(keys, user, key_prefix, log=None): akeys = akeys.replace("%h", pwent.pw_dir) akeys = akeys.replace("%u", user) authorized_keys = akeys - except Exception as e: + except Exception: authorized_keys = '%s/.ssh/authorized_keys' % pwent.pw_dir if log: util.logexc(log) diff --git a/cloudinit/UserDataHandler.py b/cloudinit/UserDataHandler.py index ac2cdda9..0afe5a18 100644 --- a/cloudinit/UserDataHandler.py +++ b/cloudinit/UserDataHandler.py @@ -77,7 +77,7 @@ def do_include(content, appendmsg): content = urllib.urlopen(line).read() if includeonce: util.write_file(includeonce_filename, content, mode=0600) - except Exception as e: + except Exception: raise process_includes(message_from_string(decomp_str(content)), appendmsg) @@ -190,7 +190,6 @@ def process_includes(msg, appendmsg=None): def message_from_string(data, headers={}): if "mime-version:" in data[0:4096].lower(): - was_mime = True msg = email.message_from_string(data) for (key,val) in headers.items(): if key in msg: @@ -198,7 +197,6 @@ def message_from_string(data, headers={}): else: msg[key] = val else: - was_mime = False mtype = headers.get("Content-Type","text/plain") maintype, subtype = mtype.split("/", 1) msg = MIMEBase(maintype, subtype, *headers) diff --git a/cloudinit/__init__.py b/cloudinit/__init__.py index 5460d6e1..537e3363 100644 --- a/cloudinit/__init__.py +++ b/cloudinit/__init__.py @@ -137,7 +137,7 @@ class CloudInit: try: conf = util.get_base_cfg(self.sysconfig,cfg_builtin, parsed_cfgs) - except Exception as e: + except Exception: conf = get_builtin_cfg() # support reading the old ConfigObj format file and merging @@ -176,7 +176,7 @@ class CloudInit: try: f=open(cache, "wb") - data = cPickle.dump(self.datasource,f) + cPickle.dump(self.datasource,f) f.close() os.chmod(cache,0400) except: @@ -360,7 +360,7 @@ class CloudInit: # give callbacks opportunity to finalize called = [ ] - for (mtype, mod) in part_handlers.iteritems(): + for (_mtype, mod) in part_handlers.iteritems(): if mod in called: continue handler_call_end(mod, data, frequency) @@ -425,7 +425,7 @@ class CloudInit: try: env=os.environ.copy() env['INSTANCE_ID']= self.datasource.get_instance_id() - ret = subprocess.check_call([filepath], env=env) + subprocess.check_call([filepath], env=env) except subprocess.CalledProcessError as e: log.error("boothooks script %s returned %i" % (filepath,e.returncode)) diff --git a/cloudinit/netinfo.py b/cloudinit/netinfo.py index a6b32c4f..9ca72b77 100644 --- a/cloudinit/netinfo.py +++ b/cloudinit/netinfo.py @@ -24,7 +24,7 @@ def netdev_info(): if toks[i] == "hwaddr": try: devs[curdev]["hwaddr"] = toks[i+1] - except IndexError as e: + except IndexError: pass for field in ("addr", "bcast", "mask"): target = "%s%s" % (field, fieldpost) @@ -33,7 +33,7 @@ def netdev_info(): if toks[i] == "%s:" % field: try: devs[curdev][target] = toks[i+1] - except IndexError as e: + except IndexError: pass elif toks[i].startswith("%s:" % field): devs[curdev][target] = toks[i][len(field)+1:] @@ -61,7 +61,7 @@ def debug_info(pre="ci-info: "): lines = [ ] try: netdev = netdev_info() - except Exception as e: + except Exception: lines.append("netdev_info failed!") netdev = {} for (dev, d) in netdev.iteritems(): @@ -69,7 +69,7 @@ def debug_info(pre="ci-info: "): (pre, dev, d["up"], d["addr"],d["mask"], d["hwaddr"])) try: routes = route_info() - except Exception as e: + except Exception: lines.append("route_info failed") routes = [] n = 0 diff --git a/cloudinit/util.py b/cloudinit/util.py index b45bb19e..90998297 100644 --- a/cloudinit/util.py +++ b/cloudinit/util.py @@ -264,7 +264,6 @@ def read_file_with_includes(fname, rel = ".", stack=[], patt = None): stack.append(fname) cur = 0 - clen = len(contents) while True: match = patt.search(contents[cur:]) if not match: break @@ -443,7 +442,7 @@ def islxc(): # we're inside a container. otherwise, no sp = subprocess.Popen(['lxc-is-container'], stdout=subprocess.PIPE, stderr=subprocess.PIPE) - out,err = sp.communicate(None) + sp.communicate(None) return(sp.returncode == 0) except OSError as e: if e.errno != errno.ENOENT: @@ -507,7 +506,7 @@ def is_resolvable(name): try: socket.getaddrinfo(name, None) return True - except socket.gaierror as e: + except socket.gaierror: return False def is_resolvable_url(url): @@ -520,7 +519,7 @@ def search_for_mirror(candidates): try: if is_resolvable_url(cand): return cand - except Exception as e: + except Exception: raise return None -- cgit v1.2.3