From 32f1967029e0b7d5f861296e4c2a05afb85745b3 Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:50:20 +0100 Subject: [PATCH 03/13] Fix pylint warnings W0301 (unnecessary semicolon) Juerg Haefliger --- cloudinit/DataSourceOVF.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'cloudinit/DataSourceOVF.py') diff --git a/cloudinit/DataSourceOVF.py b/cloudinit/DataSourceOVF.py index a9089d70..76c5dbd8 100644 --- a/cloudinit/DataSourceOVF.py +++ b/cloudinit/DataSourceOVF.py @@ -99,7 +99,7 @@ class DataSourceOVF(DataSource.DataSource): md = util.mergedict(md,defaults) self.seed = ",".join(found) - self.metadata = md; + self.metadata = md self.userdata_raw = ud self.cfg = cfg return True -- 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/DataSourceOVF.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 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/DataSourceOVF.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 From 7c38d4b469d3863443ec3322f0def25672c545db Mon Sep 17 00:00:00 2001 From: Scott Moser Date: Thu, 12 Jan 2012 16:56:05 +0100 Subject: [PATCH 13/13] Disable bogus pylint warnings From: Juerg Haefliger --- cloudinit/DataSourceOVF.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'cloudinit/DataSourceOVF.py') diff --git a/cloudinit/DataSourceOVF.py b/cloudinit/DataSourceOVF.py index e4d28aba..1c510688 100644 --- a/cloudinit/DataSourceOVF.py +++ b/cloudinit/DataSourceOVF.py @@ -70,7 +70,7 @@ class DataSourceOVF(DataSource.DataSource): if contents: (md, ud, cfg) = read_ovf_environment(contents) self.environment = contents - found.append(name) + found.append(name) # pylint: disable=W0631 # There was no OVF transports found if len(found) == 0: -- cgit v1.2.3