cloud-init

Pylint throws lots of errors and warnings

Bug #914739 reported by Juerg Haefliger
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
cloud-init
Fix Released
Undecided
Unassigned

Bug Description

When running pylint against the cloud-init source, lots of errors and warnings are reported. Part of the full output is shown below. Some of them are bogus but some are serious and need to be addressed. I'll post a series of patches shortly that will take care of these errors/warnings.

No config file found, using default configuration
************* Module cloud-init-cfg
C0103: 1: Invalid name "cloud-init-cfg" (should match (([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$)
C0111: 1: Missing docstring
C0103: 28:Usage: Invalid name "Usage" (should match [a-z_][a-z0-9_]{2,30}$)
C0111: 28:Usage: Missing docstring
C0111: 31:main: Missing docstring
C0322: 53:main: Operator not preceded by a space
        name=sys.argv[1]
            ^
C0322: 59:main: Operator not preceded by a space
            run_args=sys.argv[3:]
                    ^
C0103: 69:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0324: 73:main: Comma not followed by a space
    cc = CC.CloudConfig(cfg_path,cloud)
                                ^^
C0103: 73:main: Invalid name "cc" (should match [a-z_][a-z0-9_]{2,30}$)
W0703: 78:main: Catch "Exception"
C0324: 76:main: Comma not followed by a space
        (outfmt, errfmt) = CC.get_output_cfg(cc.cfg,modename)
                                                   ^^
C0103: 78:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0324: 88:main: Comma not followed by a space
        module_list = CC.read_cc_modules(cc.cfg,modlist_cfg_name)
                                               ^^
C0324: 90:main: Comma not followed by a space
            err("no modules to run in cloud_config [%s]" % modename,log)
                                                                   ^^
C0324: 95:main: Comma not followed by a space
    failures = CC.run_cc_modules(cc,module_list,log)
                                   ^^
C0324: 97:main: Comma not followed by a space
        err("errors running cloud_config [%s]: %s" % (modename,failures), log)
                                                              ^^
R0912: 31:main: Too many branches (14/12)
R0915: 31:main: Too many statements (51/50)
C0324:100:err: Comma not followed by a space
def err(msg,log=None):
           ^^
C0111:100:err: Missing docstring
C0324:105:fail: Comma not followed by a space
def fail(msg,log=None):
            ^^
C0111:105:fail: Missing docstring
C0324:106:fail: Comma not followed by a space
    err(msg,log)
           ^^
W0611: 26: Unused import traceback
************* Module cloud-init
W0311: 59: Bad indentation. Found 7 spaces, expected 8
W0311: 60: Bad indentation. Found 7 spaces, expected 8
W0311: 61: Bad indentation. Found 7 spaces, expected 8
W0311: 63: Bad indentation. Found 7 spaces, expected 8
W0311: 64: Bad indentation. Found 7 spaces, expected 8
C0103: 1: Invalid name "cloud-init" (should match (([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$)
C0111: 1: Missing docstring

C0111: 33:warn: Missing docstring
C0111: 36:main: Missing docstring
R0914: 36:main: Too many local variables (27/15)
C0324: 57:main: Comma not followed by a space
    now = time.strftime("%a, %d %b %Y %H:%M:%S %z",time.gmtime())
                                                  ^^
C0322: 59:main: Operator not preceded by a space
       uptimef=open("/proc/uptime")
              ^
C0322: 60:main: Operator not preceded by a space
       uptime=uptimef.read().split(" ")[0]
             ^
C0103: 62:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
W0703: 68:main: Catch "Exception"
C0103: 68:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0103: 72:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
W0703: 79:main: Catch "Exception"
C0103: 79:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
W0703: 87:main: Catch "Exception"
C0103: 87:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0103: 99:main: Invalid name "f" (should match [a-z_][a-z0-9_]{2,30}$)
W0702:104:main: No exception type(s) specified
C0103:101:main: Invalid name "fp" (should match [a-z_][a-z0-9_]{2,30}$)

C0324:112:main: Comma not followed by a space
        manclean = util.get_cfg_option_bool(cfg, 'manual_cache_clean',False)
                                                                     ^^
C0103:119:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0321:120:main: More than one statement on a single line
C0103:132:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)

C0103:157:main: Invalid name "cc" (should match [a-z_][a-z0-9_]{2,30}$)
W0703:167:main: Catch "Exception"
C0324:165:main: Comma not followed by a space
            warn("stdout, stderr changing to (%s,%s)" % (outfmt,errfmt))
                                                               ^^
C0103:167:main: Invalid name "e" (should match [a-z_][a-z0-9_]{2,30}$)
C0324:176:main: Comma not followed by a space
        if isinstance(cc_ready,str):
                              ^^
C0324:184:main: Comma not followed by a space
        failures = CC.run_cc_modules(cc,module_list,log)
                                       ^^

R0912: 36:main: Too many branches (24/12)
R0915: 36:main: Too many statements (111/50)

Related branches

lp:~cloud-init-dev/cloud-init/trunk
Chad Smith (community): Needs Fixing
Hannes: Pending requested
Diff: 51583 lines (+11/-49380) (has conflicts)
424 files modified
.bzrignore (+0/-4)
ChangeLog (+0/-761)
HACKING.rst (+0/-48)
LICENSE (+0/-674)
MANIFEST.in (+0/-8)
Makefile (+0/-82)
README (+11/-0)
TODO.rst (+0/-43)
bin/cloud-init (+0/-673)
cloudinit/__init__.py (+0/-21)
cloudinit/cloud.py (+0/-109)
cloudinit/config/__init__.py (+0/-58)
cloudinit/config/cc_apt_configure.py (+0/-292)
cloudinit/config/cc_apt_pipelining.py (+0/-57)
cloudinit/config/cc_bootcmd.py (+0/-54)
cloudinit/config/cc_byobu.py (+0/-80)
cloudinit/config/cc_ca_certs.py (+0/-104)
cloudinit/config/cc_chef.py (+0/-342)
cloudinit/config/cc_debug.py (+0/-109)
cloudinit/config/cc_disable_ec2_metadata.py (+0/-36)
cloudinit/config/cc_emit_upstart.py (+0/-69)
cloudinit/config/cc_fan.py (+0/-101)
cloudinit/config/cc_final_message.py (+0/-73)
cloudinit/config/cc_foo.py (+0/-52)
cloudinit/config/cc_growpart.py (+0/-300)
cloudinit/config/cc_grub_dpkg.py (+0/-73)
cloudinit/config/cc_keys_to_console.py (+0/-62)
cloudinit/config/cc_landscape.py (+0/-99)
cloudinit/config/cc_locale.py (+0/-37)
cloudinit/config/cc_lxd.py (+0/-176)
cloudinit/config/cc_mcollective.py (+0/-88)
cloudinit/config/cc_migrator.py (+0/-85)
cloudinit/config/cc_mounts.py (+0/-405)
cloudinit/config/cc_package_update_upgrade_install.py (+0/-99)
cloudinit/config/cc_phone_home.py (+0/-122)
cloudinit/config/cc_power_state_change.py (+0/-223)
cloudinit/config/cc_puppet.py (+0/-118)
cloudinit/config/cc_resizefs.py (+0/-185)
cloudinit/config/cc_resolv_conf.py (+0/-116)
cloudinit/config/cc_rh_subscription.py (+0/-408)
cloudinit/config/cc_rightscale_userdata.py (+0/-102)
cloudinit/config/cc_rsyslog.py (+0/-366)
cloudinit/config/cc_runcmd.py (+0/-38)
cloudinit/config/cc_salt_minion.py (+0/-59)
cloudinit/config/cc_scripts_per_boot.py (+0/-41)
cloudinit/config/cc_scripts_per_instance.py (+0/-41)
cloudinit/config/cc_scripts_per_once.py (+0/-41)
cloudinit/config/cc_scripts_user.py (+0/-42)
cloudinit/config/cc_scripts_vendor.py (+0/-43)
cloudinit/config/cc_seed_random.py (+0/-94)
cloudinit/config/cc_set_hostname.py (+0/-37)
cloudinit/config/cc_set_passwords.py (+0/-167)
cloudinit/config/cc_snappy.py (+0/-304)
cloudinit/config/cc_ssh.py (+0/-142)
cloudinit/config/cc_ssh_authkey_fingerprints.py (+0/-105)
cloudinit/config/cc_ssh_import_id.py (+0/-99)
cloudinit/config/cc_timezone.py (+0/-39)
cloudinit/config/cc_ubuntu_init_switch.py (+0/-162)
cloudinit/config/cc_update_etc_hosts.py (+0/-60)
cloudinit/config/cc_update_hostname.py (+0/-43)
cloudinit/config/cc_users_groups.py (+0/-34)
cloudinit/config/cc_write_files.py (+0/-103)
cloudinit/config/cc_yum_add_repo.py (+0/-107)
cloudinit/cs_utils.py (+0/-105)
cloudinit/distros/__init__.py (+0/-950)
cloudinit/distros/arch.py (+0/-201)
cloudinit/distros/debian.py (+0/-227)
cloudinit/distros/fedora.py (+0/-31)
cloudinit/distros/freebsd.py (+0/-417)
cloudinit/distros/gentoo.py (+0/-160)
cloudinit/distros/net_util.py (+0/-182)
cloudinit/distros/parsers/__init__.py (+0/-28)
cloudinit/distros/parsers/hostname.py (+0/-88)
cloudinit/distros/parsers/hosts.py (+0/-92)
cloudinit/distros/parsers/resolv_conf.py (+0/-169)
cloudinit/distros/parsers/sys_conf.py (+0/-113)
cloudinit/distros/rhel.py (+0/-222)
cloudinit/distros/rhel_util.py (+0/-89)
cloudinit/distros/sles.py (+0/-179)
cloudinit/distros/ubuntu.py (+0/-31)
cloudinit/ec2_utils.py (+0/-201)
cloudinit/filters/__init__.py (+0/-21)
cloudinit/filters/launch_index.py (+0/-75)
cloudinit/handlers/__init__.py (+0/-274)
cloudinit/handlers/boot_hook.py (+0/-70)
cloudinit/handlers/cloud_config.py (+0/-163)
cloudinit/handlers/shell_script.py (+0/-55)
cloudinit/handlers/upstart_job.py (+0/-119)
cloudinit/helpers.py (+0/-458)
cloudinit/importer.py (+0/-58)
cloudinit/log.py (+0/-155)
cloudinit/mergers/__init__.py (+0/-166)
cloudinit/mergers/m_dict.py (+0/-88)
cloudinit/mergers/m_list.py (+0/-89)
cloudinit/mergers/m_str.py (+0/-46)
cloudinit/net/__init__.py (+0/-771)
cloudinit/net/network_state.py (+0/-447)
cloudinit/net/udev.py (+0/-54)
cloudinit/netinfo.py (+0/-249)
cloudinit/patcher.py (+0/-58)
cloudinit/registry.py (+0/-37)
cloudinit/reporting/__init__.py (+0/-42)
cloudinit/reporting/events.py (+0/-248)
cloudinit/reporting/handlers.py (+0/-91)
cloudinit/safeyaml.py (+0/-32)
cloudinit/settings.py (+0/-68)
cloudinit/signal_handler.py (+0/-71)
cloudinit/sources/DataSourceAltCloud.py (+0/-293)
cloudinit/sources/DataSourceAzure.py (+0/-651)
cloudinit/sources/DataSourceBigstep.py (+0/-57)
cloudinit/sources/DataSourceCloudSigma.py (+0/-141)
cloudinit/sources/DataSourceCloudStack.py (+0/-253)
cloudinit/sources/DataSourceConfigDrive.py (+0/-424)
cloudinit/sources/DataSourceDigitalOcean.py (+0/-110)
cloudinit/sources/DataSourceEc2.py (+0/-211)
cloudinit/sources/DataSourceGCE.py (+0/-167)
cloudinit/sources/DataSourceMAAS.py (+0/-353)
cloudinit/sources/DataSourceNoCloud.py (+0/-336)
cloudinit/sources/DataSourceNone.py (+0/-57)
cloudinit/sources/DataSourceOVF.py (+0/-429)
cloudinit/sources/DataSourceOpenNebula.py (+0/-455)
cloudinit/sources/DataSourceOpenStack.py (+0/-173)
cloudinit/sources/DataSourceSmartOS.py (+0/-536)
cloudinit/sources/__init__.py (+0/-337)
cloudinit/sources/helpers/__init__.py (+0/-13)
cloudinit/sources/helpers/azure.py (+0/-279)
cloudinit/sources/helpers/openstack.py (+0/-499)
cloudinit/sources/helpers/vmware/__init__.py (+0/-13)
cloudinit/sources/helpers/vmware/imc/__init__.py (+0/-13)
cloudinit/sources/helpers/vmware/imc/boot_proto.py (+0/-25)
cloudinit/sources/helpers/vmware/imc/config.py (+0/-95)
cloudinit/sources/helpers/vmware/imc/config_file.py (+0/-129)
cloudinit/sources/helpers/vmware/imc/config_namespace.py (+0/-25)
cloudinit/sources/helpers/vmware/imc/config_nic.py (+0/-247)
cloudinit/sources/helpers/vmware/imc/config_source.py (+0/-23)
cloudinit/sources/helpers/vmware/imc/guestcust_error.py (+0/-24)
cloudinit/sources/helpers/vmware/imc/guestcust_event.py (+0/-27)
cloudinit/sources/helpers/vmware/imc/guestcust_state.py (+0/-25)
cloudinit/sources/helpers/vmware/imc/guestcust_util.py (+0/-128)
cloudinit/sources/helpers/vmware/imc/ipv4_mode.py (+0/-45)
cloudinit/sources/helpers/vmware/imc/nic.py (+0/-147)
cloudinit/sources/helpers/vmware/imc/nic_base.py (+0/-154)
cloudinit/ssh_util.py (+0/-314)
cloudinit/stages.py (+0/-844)
cloudinit/templater.py (+0/-149)
cloudinit/type_utils.py (+0/-52)
cloudinit/url_helper.py (+0/-509)
cloudinit/user_data.py (+0/-352)
cloudinit/util.py (+0/-2226)
cloudinit/version.py (+0/-27)
config/cloud.cfg (+0/-115)
config/cloud.cfg-freebsd (+0/-88)
config/cloud.cfg.d/05_logging.cfg (+0/-66)
config/cloud.cfg.d/README (+0/-3)
doc/README (+0/-4)
doc/examples/cloud-config-add-apt-repos.txt (+0/-34)
doc/examples/cloud-config-archive-launch-index.txt (+0/-30)
doc/examples/cloud-config-archive.txt (+0/-16)
doc/examples/cloud-config-boot-cmds.txt (+0/-15)
doc/examples/cloud-config-ca-certs.txt (+0/-31)
doc/examples/cloud-config-chef-oneiric.txt (+0/-90)
doc/examples/cloud-config-chef.txt (+0/-95)
doc/examples/cloud-config-datasources.txt (+0/-73)
doc/examples/cloud-config-disk-setup.txt (+0/-251)
doc/examples/cloud-config-final-message.txt (+0/-7)
doc/examples/cloud-config-gluster.txt (+0/-18)
doc/examples/cloud-config-growpart.txt (+0/-31)
doc/examples/cloud-config-install-packages.txt (+0/-15)
doc/examples/cloud-config-landscape.txt (+0/-22)
doc/examples/cloud-config-launch-index.txt (+0/-23)
doc/examples/cloud-config-lxd.txt (+0/-55)
doc/examples/cloud-config-mcollective.txt (+0/-49)
doc/examples/cloud-config-mount-points.txt (+0/-46)
doc/examples/cloud-config-phone-home.txt (+0/-14)
doc/examples/cloud-config-power-state.txt (+0/-40)
doc/examples/cloud-config-puppet.txt (+0/-51)
doc/examples/cloud-config-reporting.txt (+0/-17)
doc/examples/cloud-config-resolv-conf.txt (+0/-20)
doc/examples/cloud-config-rh_subscription.txt (+0/-49)
doc/examples/cloud-config-rsyslog.txt (+0/-46)
doc/examples/cloud-config-run-cmds.txt (+0/-22)
doc/examples/cloud-config-salt-minion.txt (+0/-53)
doc/examples/cloud-config-seed-random.txt (+0/-32)
doc/examples/cloud-config-ssh-keys.txt (+0/-46)
doc/examples/cloud-config-update-apt.txt (+0/-7)
doc/examples/cloud-config-update-packages.txt (+0/-8)
doc/examples/cloud-config-user-groups.txt (+0/-109)
doc/examples/cloud-config-vendor-data.txt (+0/-16)
doc/examples/cloud-config-write-files.txt (+0/-33)
doc/examples/cloud-config-yum-repo.txt (+0/-20)
doc/examples/cloud-config.txt (+0/-645)
doc/examples/include-once.txt (+0/-7)
doc/examples/include.txt (+0/-5)
doc/examples/kernel-cmdline.txt (+0/-18)
doc/examples/part-handler-v2.txt (+0/-38)
doc/examples/part-handler.txt (+0/-23)
doc/examples/plain-ignored.txt (+0/-2)
doc/examples/seed/README (+0/-22)
doc/examples/seed/meta-data (+0/-30)
doc/examples/seed/user-data (+0/-3)
doc/examples/upstart-cloud-config.txt (+0/-12)
doc/examples/upstart-rclocal.txt (+0/-12)
doc/examples/user-script.txt (+0/-8)
doc/merging.rst (+0/-188)
doc/rtd/conf.py (+0/-77)
doc/rtd/index.rst (+0/-31)
doc/rtd/static/logo.svg (+0/-89)
doc/rtd/topics/availability.rst (+0/-20)
doc/rtd/topics/capabilities.rst (+0/-24)
doc/rtd/topics/datasources.rst (+0/-200)
doc/rtd/topics/dir_layout.rst (+0/-81)
doc/rtd/topics/examples.rst (+0/-133)
doc/rtd/topics/format.rst (+0/-159)
doc/rtd/topics/hacking.rst (+0/-1)
doc/rtd/topics/merging.rst (+0/-5)
doc/rtd/topics/modules.rst (+0/-342)
doc/rtd/topics/moreinfo.rst (+0/-12)
doc/sources/altcloud/README.rst (+0/-87)
doc/sources/azure/README.rst (+0/-134)
doc/sources/cloudsigma/README.rst (+0/-38)
doc/sources/cloudstack/README.rst (+0/-29)
doc/sources/configdrive/README.rst (+0/-123)
doc/sources/digitalocean/README.rst (+0/-21)
doc/sources/kernel-cmdline.txt (+0/-48)
doc/sources/nocloud/README.rst (+0/-71)
doc/sources/opennebula/README.rst (+0/-142)
doc/sources/openstack/README.rst (+0/-24)
doc/sources/ovf/README (+0/-83)
doc/sources/ovf/example/ovf-env.xml (+0/-46)
doc/sources/ovf/example/ubuntu-server.ovf (+0/-130)
doc/sources/ovf/make-iso (+0/-156)
doc/sources/ovf/ovf-env.xml.tmpl (+0/-28)
doc/sources/ovf/ovfdemo.pem (+0/-27)
doc/sources/ovf/user-data (+0/-7)
doc/sources/smartos/README.rst (+0/-149)
doc/status.txt (+0/-53)
doc/userdata.txt (+0/-79)
doc/var-lib-cloud.txt (+0/-63)
doc/vendordata.txt (+0/-53)
packages/bddeb (+0/-260)
packages/brpm (+0/-277)
packages/debian/changelog.in (+0/-6)
packages/debian/cloud-init.postinst (+0/-16)
packages/debian/cloud-init.preinst (+0/-20)
packages/debian/compat (+0/-1)
packages/debian/control.in (+0/-29)
packages/debian/copyright (+0/-29)
packages/debian/dirs (+0/-6)
packages/debian/rules.in (+0/-23)
packages/debian/watch (+0/-2)
packages/redhat/cloud-init.spec.in (+0/-200)
packages/suse/cloud-init.spec.in (+0/-163)
requirements.txt (+0/-36)
setup.py (+0/-215)
systemd/cloud-config.service (+0/-16)
systemd/cloud-config.target (+0/-11)
systemd/cloud-final.service (+0/-17)
systemd/cloud-init-generator (+0/-133)
systemd/cloud-init-local.service (+0/-22)
systemd/cloud-init.service (+0/-18)
systemd/cloud-init.target (+0/-6)
sysvinit/debian/cloud-config (+0/-64)
sysvinit/debian/cloud-final (+0/-66)
sysvinit/debian/cloud-init (+0/-64)
sysvinit/debian/cloud-init-local (+0/-63)
sysvinit/freebsd/cloudconfig (+0/-35)
sysvinit/freebsd/cloudfinal (+0/-35)
sysvinit/freebsd/cloudinit (+0/-35)
sysvinit/freebsd/cloudinitlocal (+0/-35)
sysvinit/gentoo/cloud-config (+0/-13)
sysvinit/gentoo/cloud-final (+0/-11)
sysvinit/gentoo/cloud-init (+0/-12)
sysvinit/gentoo/cloud-init-local (+0/-13)
sysvinit/redhat/cloud-config (+0/-121)
sysvinit/redhat/cloud-final (+0/-121)
sysvinit/redhat/cloud-init (+0/-121)
sysvinit/redhat/cloud-init-local (+0/-124)
templates/chef_client.rb.tmpl (+0/-58)
templates/hosts.debian.tmpl (+0/-26)
templates/hosts.freebsd.tmpl (+0/-24)
templates/hosts.redhat.tmpl (+0/-24)
templates/hosts.suse.tmpl (+0/-26)
templates/resolv.conf.tmpl (+0/-30)
templates/sources.list.debian.tmpl (+0/-32)
templates/sources.list.ubuntu.tmpl (+0/-58)
test-requirements.txt (+0/-17)
tests/configs/sample1.yaml (+0/-52)
tests/data/filter_cloud_multipart.yaml (+0/-30)
tests/data/filter_cloud_multipart_1.email (+0/-11)
tests/data/filter_cloud_multipart_2.email (+0/-39)
tests/data/filter_cloud_multipart_header.email (+0/-11)
tests/data/merge_sources/expected1.yaml (+0/-1)
tests/data/merge_sources/expected10.yaml (+0/-7)
tests/data/merge_sources/expected11.yaml (+0/-5)
tests/data/merge_sources/expected12.yaml (+0/-5)
tests/data/merge_sources/expected2.yaml (+0/-3)
tests/data/merge_sources/expected3.yaml (+0/-1)
tests/data/merge_sources/expected4.yaml (+0/-2)
tests/data/merge_sources/expected5.yaml (+0/-7)
tests/data/merge_sources/expected6.yaml (+0/-9)
tests/data/merge_sources/expected7.yaml (+0/-38)
tests/data/merge_sources/expected8.yaml (+0/-7)
tests/data/merge_sources/expected9.yaml (+0/-5)
tests/data/merge_sources/source1-1.yaml (+0/-3)
tests/data/merge_sources/source1-2.yaml (+0/-5)
tests/data/merge_sources/source10-1.yaml (+0/-6)
tests/data/merge_sources/source10-2.yaml (+0/-6)
tests/data/merge_sources/source11-1.yaml (+0/-5)
tests/data/merge_sources/source11-2.yaml (+0/-3)
tests/data/merge_sources/source11-3.yaml (+0/-3)
tests/data/merge_sources/source12-1.yaml (+0/-8)
tests/data/merge_sources/source12-2.yaml (+0/-5)
tests/data/merge_sources/source2-1.yaml (+0/-6)
tests/data/merge_sources/source2-2.yaml (+0/-5)
tests/data/merge_sources/source3-1.yaml (+0/-4)
tests/data/merge_sources/source3-2.yaml (+0/-4)
tests/data/merge_sources/source4-1.yaml (+0/-3)
tests/data/merge_sources/source4-2.yaml (+0/-6)
tests/data/merge_sources/source5-1.yaml (+0/-6)
tests/data/merge_sources/source5-2.yaml (+0/-8)
tests/data/merge_sources/source6-1.yaml (+0/-5)
tests/data/merge_sources/source6-2.yaml (+0/-8)
tests/data/merge_sources/source7-1.yaml (+0/-27)
tests/data/merge_sources/source7-2.yaml (+0/-17)
tests/data/merge_sources/source8-1.yaml (+0/-7)
tests/data/merge_sources/source8-2.yaml (+0/-6)
tests/data/merge_sources/source9-1.yaml (+0/-5)
tests/data/merge_sources/source9-2.yaml (+0/-6)
tests/data/mountinfo_precise_ext4.txt (+0/-24)
tests/data/mountinfo_raring_btrfs.txt (+0/-13)
tests/data/roots/simple_ubuntu/etc/networks/interfaces (+0/-3)
tests/data/user_data.1.txt (+0/-15)
tests/data/vmware/cust-dhcp-2nic.cfg (+0/-34)
tests/data/vmware/cust-static-2nic.cfg (+0/-39)
tests/unittests/helpers.py (+0/-346)
tests/unittests/test__init__.py (+0/-220)
tests/unittests/test_builtin_handlers.py (+0/-73)
tests/unittests/test_cli.py (+0/-54)
tests/unittests/test_cs_util.py (+0/-84)
tests/unittests/test_data.py (+0/-559)
tests/unittests/test_datasource/test_altcloud.py (+0/-452)
tests/unittests/test_datasource/test_azure.py (+0/-646)
tests/unittests/test_datasource/test_azure_helper.py (+0/-421)
tests/unittests/test_datasource/test_cloudsigma.py (+0/-99)
tests/unittests/test_datasource/test_cloudstack.py (+0/-87)
tests/unittests/test_datasource/test_configdrive.py (+0/-407)
tests/unittests/test_datasource/test_digitalocean.py (+0/-127)
tests/unittests/test_datasource/test_gce.py (+0/-166)
tests/unittests/test_datasource/test_maas.py (+0/-163)
tests/unittests/test_datasource/test_nocloud.py (+0/-187)
tests/unittests/test_datasource/test_opennebula.py (+0/-300)
tests/unittests/test_datasource/test_openstack.py (+0/-347)
tests/unittests/test_datasource/test_smartos.py (+0/-585)
tests/unittests/test_distros/test_generic.py (+0/-233)
tests/unittests/test_distros/test_hostname.py (+0/-38)
tests/unittests/test_distros/test_hosts.py (+0/-41)
tests/unittests/test_distros/test_netconfig.py (+0/-321)
tests/unittests/test_distros/test_resolv.py (+0/-67)
tests/unittests/test_distros/test_sysconfig.py (+0/-82)
tests/unittests/test_distros/test_user_data_normalize.py (+0/-297)
tests/unittests/test_ec2_util.py (+0/-139)
tests/unittests/test_filters/test_launch_index.py (+0/-132)
tests/unittests/test_handler/test_handler_apt_configure.py (+0/-109)
tests/unittests/test_handler/test_handler_ca_certs.py (+0/-271)
tests/unittests/test_handler/test_handler_chef.py (+0/-192)
tests/unittests/test_handler/test_handler_debug.py (+0/-81)
tests/unittests/test_handler/test_handler_disk_setup.py (+0/-30)
tests/unittests/test_handler/test_handler_growpart.py (+0/-220)
tests/unittests/test_handler/test_handler_locale.py (+0/-67)
tests/unittests/test_handler/test_handler_lxd.py (+0/-134)
tests/unittests/test_handler/test_handler_mounts.py (+0/-133)
tests/unittests/test_handler/test_handler_power_state.py (+0/-127)
tests/unittests/test_handler/test_handler_rsyslog.py (+0/-174)
tests/unittests/test_handler/test_handler_seed_random.py (+0/-227)
tests/unittests/test_handler/test_handler_set_hostname.py (+0/-72)
tests/unittests/test_handler/test_handler_snappy.py (+0/-306)
tests/unittests/test_handler/test_handler_timezone.py (+0/-76)
tests/unittests/test_handler/test_handler_write_files.py (+0/-112)
tests/unittests/test_handler/test_handler_yum_add_repo.py (+0/-68)
tests/unittests/test_helpers.py (+0/-33)
tests/unittests/test_merging.py (+0/-257)
tests/unittests/test_net.py (+0/-130)
tests/unittests/test_pathprefix2dict.py (+0/-44)
tests/unittests/test_registry.py (+0/-28)
tests/unittests/test_reporting.py (+0/-369)
tests/unittests/test_rh_subscription.py (+0/-214)
tests/unittests/test_runs/test_merge_run.py (+0/-54)
tests/unittests/test_runs/test_simple_run.py (+0/-81)
tests/unittests/test_sshutil.py (+0/-171)
tests/unittests/test_templating.py (+0/-119)
tests/unittests/test_util.py (+0/-489)
tests/unittests/test_vmware_config_file.py (+0/-103)
tools/21-cloudinit.conf (+0/-6)
tools/Z99-cloud-locale-test.sh (+0/-98)
tools/build-on-freebsd (+0/-66)
tools/ccfg-merge-debug (+0/-90)
tools/cloud-init-per (+0/-60)
tools/hacking.py (+0/-170)
tools/make-dist-tarball (+0/-21)
tools/make-mime.py (+0/-60)
tools/make-tarball (+0/-39)
tools/mock-meta.py (+0/-454)
tools/motd-hook (+0/-35)
tools/read-dependencies (+0/-29)
tools/read-version (+0/-26)
tools/run-pep8 (+0/-22)
tools/run-pyflakes (+0/-18)
tools/run-pyflakes3 (+0/-2)
tools/tox-venv (+0/-42)
tools/uncloud-init (+0/-141)
tools/validate-yaml.py (+0/-25)
tools/write-ssh-key-fingerprints (+0/-38)
tox.ini (+0/-35)
udev/66-azure-ephemeral.rules (+0/-18)
udev/79-cloud-init-net-wait.rules (+0/-10)
udev/cloud-init-wait (+0/-70)
upstart/cloud-config.conf (+0/-9)
upstart/cloud-final.conf (+0/-10)
upstart/cloud-init-blocknet.conf (+0/-83)
upstart/cloud-init-container.conf (+0/-57)
upstart/cloud-init-local.conf (+0/-16)
upstart/cloud-init-nonet.conf (+0/-66)
upstart/cloud-init.conf (+0/-9)
upstart/cloud-log-shutdown.conf (+0/-19)
JJ Asghar: Pending requested
Diff: 0 lines
Scott Moser: Pending requested
Diff: 3061 lines (+2239/-175)
39 files modified
ChangeLog (+39/-9)
cloud-init.py (+31/-1)
cloudinit/CloudConfig/__init__.py (+1/-1)
cloudinit/CloudConfig/cc_apt_pipelining.py (+53/-0)
cloudinit/CloudConfig/cc_ca_certs.py (+3/-1)
cloudinit/CloudConfig/cc_chef.py (+5/-5)
cloudinit/CloudConfig/cc_landscape.py (+5/-0)
cloudinit/CloudConfig/cc_resizefs.py (+31/-12)
cloudinit/CloudConfig/cc_salt_minion.py (+2/-1)
cloudinit/CloudConfig/cc_update_etc_hosts.py (+1/-1)
cloudinit/DataSource.py (+4/-1)
cloudinit/DataSourceCloudStack.py (+92/-0)
cloudinit/DataSourceConfigDrive.py (+231/-0)
cloudinit/DataSourceEc2.py (+2/-84)
cloudinit/DataSourceMAAS.py (+345/-0)
cloudinit/DataSourceNoCloud.py (+75/-2)
cloudinit/DataSourceOVF.py (+1/-1)
cloudinit/SshUtil.py (+2/-0)
cloudinit/UserDataHandler.py (+2/-2)
cloudinit/__init__.py (+44/-9)
cloudinit/netinfo.py (+5/-5)
cloudinit/util.py (+253/-30)
config/cloud.cfg (+3/-1)
debian.trunk/control (+1/-0)
doc/configdrive/README (+118/-0)
doc/examples/cloud-config-chef-oneiric.txt (+90/-0)
doc/examples/cloud-config-chef.txt (+48/-6)
doc/examples/cloud-config-datasources.txt (+18/-0)
doc/examples/cloud-config.txt (+11/-0)
doc/kernel-cmdline.txt (+48/-0)
doc/nocloud/README (+55/-0)
setup.py (+1/-0)
tests/unittests/test__init__.py (+242/-0)
tests/unittests/test_datasource/test_maas.py (+153/-0)
tests/unittests/test_handler/test_handler_ca_certs.py (+4/-0)
tests/unittests/test_userdata.py (+107/-0)
tests/unittests/test_util.py (+18/-2)
tools/Z99-cloud-locale-test.sh (+92/-0)
tools/run-pylint (+3/-1)
lp:ubuntu/precise/cloud-init
Revision history for this message
Juerg Haefliger (juergh) wrote :

The attached tarball contains a patch series that takes care of the pylint errors.

Revision history for this message
Juerg Haefliger (juergh) wrote :

Above patchset fixes the following errors:

************* Module cloud-init-query
E1101: 36:main: Module 'cloudinit' has no 'cloud_config' member
************* Module cloudinit
E0211:583:InternalPartHandler.__repr__: Method has no argument
E0602:584:InternalPartHandler.__repr__: Undefined variable 'self'
************* Module cloudinit.util
E0702:233:read_seeded: Raising NoneType while only classes, instances or string are allowed
************* Module cloudinit.DataSourceEc2
E0102:235:wait_for_metadata_service.status_cb: function already defined line 205
************* Module cloudinit.UserDataHandler
E1101: 70:do_include: Module 'hashlib' has no 'md5' member
************* Module cloudinit.DataSource
E1103:122:DataSource.get_hostname: Instance of 'bool' has no 'split' member (but some types could not be inferred)
************* Module cloudinit.netinfo
E1103: 7:netdev_info: Instance of 'list' has no 'splitlines' member (but some types could not be inferred)
E1103: 45:route_info: Instance of 'list' has no 'splitlines' member (but some types could not be inferred)
************* Module cloudinit.SshUtil
E0101: 20:AuthKeyEntry.__init__: Explicit return in __init__
************* Module cloudinit.CloudConfig.cc_phone_home
E1101: 36:handle: Module 'cloudinit.util' has no 'readconf' member
E0702: 98:handle: Raising NoneType while only classes, instances or string are allowed
************* Module cloudinit.CloudConfig.cc_bootcmd
E0602: 38:handle: Undefined variable 'os'
************* Module cloudinit.CloudConfig.cc_resizefs
E1103: 65:handle: Instance of 'list' has no 'rstrip' member (but some types could not be inferred)
E1103: 67:handle: Instance of 'list' has no 'startswith' member (but some types could not be inferred)
************* Module cloudinit.CloudConfig.cc_apt_update_upgrade
E1103:125:get_release: Instance of 'list' has no 'strip' member (but some types could not be inferred)

Revision history for this message
Juerg Haefliger (juergh) wrote :
Download full text (3.4 KiB)

Attached patchset fixes the following pylint warnings. The patches get rid of a total of 281 warnings.

Juerg Haefliger (13):
  Fix pylint warnings W0311 (bad indentation)
  Fix pylint warnings W0312 (found indentation with tabs instead of spaces)
  Fix pylint warnings W0301 (unnecessary semicolon)
  Fix pylint warnings W0611 (unused import)
  Fix pylint warnings W0613 (unused argument)
  Fix pylint warnings W0612 (unused variable)
  Fix pylint warnings W0622 (redefining built-in 'xyz')
  Fix pylint warnings W0104 (statement seems to have no effect)
  Fix pylint warnings W0107 (unnecessary pass statement)
  Fix pylint warnings W0102 (dangerous default value as argument)
  Fix pylint warnings W0141 (used builtin function xyz)
  Fix pylint warnings W0201 (attribute xyz defined outside __init__)
  Disable bogus pylint warnings

 cloud-init-cfg.py | 1 -
 cloud-init-query.py | 4 -
 cloud-init.py | 10 +-
 cloudinit/CloudConfig/__init__.py | 6 +-
 cloudinit/CloudConfig/cc_apt_update_upgrade.py | 26 +++--
 cloudinit/CloudConfig/cc_bootcmd.py | 2 +-
 cloudinit/CloudConfig/cc_byobu.py | 2 +-
 cloudinit/CloudConfig/cc_chef.py | 12 +--
 cloudinit/CloudConfig/cc_disable_ec2_metadata.py | 2 +-
 cloudinit/CloudConfig/cc_final_message.py | 2 +-
 cloudinit/CloudConfig/cc_foo.py | 10 +-
 cloudinit/CloudConfig/cc_grub_dpkg.py | 4 +-
 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 | 6 +-
 cloudinit/CloudConfig/cc_mounts.py | 4 +-
 cloudinit/CloudConfig/cc_phone_home.py | 3 +-
 cloudinit/CloudConfig/cc_puppet.py | 2 +-
 cloudinit/CloudConfig/cc_resizefs.py | 14 +-
 cloudinit/CloudConfig/cc_rightscale_userdata.py | 8 +-
 cloudinit/CloudConfig/cc_rsyslog.py | 8 +-
 cloudinit/CloudConfig/cc_runcmd.py | 4 +-
 cloudinit/CloudConfig/cc_scripts_per_boot.py | 7 +-
 cloudinit/CloudConfig/cc_scripts_per_instance.py | 7 +-
 cloudinit/CloudConfig/cc_scripts_per_once.py | 7 +-
 cloudinit/CloudConfig/cc_scripts_user.py | 7 +-
 cloudinit/CloudConfig/cc_set_hostname.py | 8 +-
 cloudinit/CloudConfig/cc_set_passwords.py | 6 +-
 cloudinit/CloudConfig/cc_ssh.py | 2 +-
 cloudinit/CloudConfig/cc_ssh_import_id.py | 2 +-
 cloudinit/CloudConfig/cc_timezone.py | 4 +-
 cloudinit/CloudConfig/cc_update_etc_hosts.py | 64 +++++-----
 cloudinit/CloudConfig/cc_update_hostname.py | 6 +-
 cloudinit/DataSource.py | 12 +-
 cloudinit/DataSourceEc2.py | 10 +-
 cloudinit/DataSourceNoCloud.py | 8 +-
 cloudinit/DataSourceOVF.py | 10 +-
 cloudinit/SshUtil.py | 8 +-
 cloudinit/U...

Read more...

Scott Moser (smoser)
Changed in cloud-init:
status: New → Fix Committed
Revision history for this message
Juerg Haefliger (juergh) wrote :

Attached patchset fixed the following pylint convention violations. The patches fix a total of 450 violations.

Juerg Haefliger (4):
  Fix pylint conventions C0321 (more than one statement on a single line)
  Fix pylint conventions C0322 (operator not preceded by a space)
  Fix pylint conventions C0324 (comma not followed by a space)
  Fix pylint conventions C0301 (line too long)

 cloud-init-cfg.py | 22 ++--
 cloud-init-query.py | 2 +-
 cloud-init.py | 27 ++--
 cloudinit/CloudConfig/__init__.py | 89 +++++++-----
 cloudinit/CloudConfig/cc_apt_update_upgrade.py | 55 ++++----
 cloudinit/CloudConfig/cc_bootcmd.py | 6 +-
 cloudinit/CloudConfig/cc_byobu.py | 9 +-
 cloudinit/CloudConfig/cc_chef.py | 39 +++---
 cloudinit/CloudConfig/cc_disable_ec2_metadata.py | 4 +-
 cloudinit/CloudConfig/cc_final_message.py | 10 +-
 cloudinit/CloudConfig/cc_foo.py | 2 +-
 cloudinit/CloudConfig/cc_grub_dpkg.py | 25 ++--
 cloudinit/CloudConfig/cc_keys_to_console.py | 10 +-
 cloudinit/CloudConfig/cc_landscape.py | 2 +-
 cloudinit/CloudConfig/cc_locale.py | 7 +-
 cloudinit/CloudConfig/cc_mcollective.py | 32 +++--
 cloudinit/CloudConfig/cc_mounts.py | 52 +++++---
 cloudinit/CloudConfig/cc_phone_home.py | 12 +-
 cloudinit/CloudConfig/cc_puppet.py | 18 ++-
 cloudinit/CloudConfig/cc_resizefs.py | 16 ++-
 cloudinit/CloudConfig/cc_rightscale_userdata.py | 10 +-
 cloudinit/CloudConfig/cc_rsyslog.py | 9 +-
 cloudinit/CloudConfig/cc_runcmd.py | 6 +-
 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 | 6 +-
 cloudinit/CloudConfig/cc_set_passwords.py | 34 +++---
 cloudinit/CloudConfig/cc_ssh.py | 30 +++--
 cloudinit/CloudConfig/cc_ssh_import_id.py | 9 +-
 cloudinit/CloudConfig/cc_timezone.py | 9 +-
 cloudinit/CloudConfig/cc_update_etc_hosts.py | 6 +-
 cloudinit/CloudConfig/cc_update_hostname.py | 17 ++-
 cloudinit/DataSource.py | 16 ++-
 cloudinit/DataSourceEc2.py | 34 +++--
 cloudinit/DataSourceNoCloud.py | 40 +++---
 cloudinit/DataSourceOVF.py | 68 +++++----
 cloudinit/SshUtil.py | 23 ++--
 cloudinit/UserDataHandler.py | 23 ++--
 cloudinit/__init__.py | 161 ++++++++++++----------
 cloudinit/netinfo.py | 6 +-
 cloudinit/util.py | 151 +++++++++++---------
 43 files changed, 619 insertions(+), 486 deletions(-)

Revision history for this message
Juerg Haefliger (juergh) wrote :

Attached patch fixes the remaining PEP8 violations (http://www.python.org/dev/peps/pep-0008/').

Juerg Haefliger (1):
  PEP8 coding style fixes.

 cloud-init-cfg.py | 18 ++-
 cloud-init-query.py | 12 +-
 cloud-init.py | 16 ++-
 cloudinit/CloudConfig/__init__.py | 34 ++++--
 cloudinit/CloudConfig/cc_apt_update_upgrade.py | 38 ++++---
 cloudinit/CloudConfig/cc_bootcmd.py | 5 +-
 cloudinit/CloudConfig/cc_byobu.py | 11 +-
 cloudinit/CloudConfig/cc_chef.py | 18 ++-
 cloudinit/CloudConfig/cc_disable_ec2_metadata.py | 1 +
 cloudinit/CloudConfig/cc_final_message.py | 4 +-
 cloudinit/CloudConfig/cc_foo.py | 1 +
 cloudinit/CloudConfig/cc_grub_dpkg.py | 10 +-
 cloudinit/CloudConfig/cc_keys_to_console.py | 3 +-
 cloudinit/CloudConfig/cc_landscape.py | 6 +-
 cloudinit/CloudConfig/cc_locale.py | 4 +-
 cloudinit/CloudConfig/cc_mcollective.py | 8 +-
 cloudinit/CloudConfig/cc_mounts.py | 28 +++---
 cloudinit/CloudConfig/cc_phone_home.py | 15 ++-
 cloudinit/CloudConfig/cc_puppet.py | 6 +-
 cloudinit/CloudConfig/cc_resizefs.py | 13 +-
 cloudinit/CloudConfig/cc_rightscale_userdata.py | 9 +-
 cloudinit/CloudConfig/cc_rsyslog.py | 9 +-
 cloudinit/CloudConfig/cc_runcmd.py | 3 +-
 cloudinit/CloudConfig/cc_scripts_per_boot.py | 1 +
 cloudinit/CloudConfig/cc_scripts_per_instance.py | 1 +
 cloudinit/CloudConfig/cc_scripts_per_once.py | 1 +
 cloudinit/CloudConfig/cc_scripts_user.py | 1 +
 cloudinit/CloudConfig/cc_set_hostname.py | 4 +-
 cloudinit/CloudConfig/cc_set_passwords.py | 25 +++--
 cloudinit/CloudConfig/cc_ssh.py | 30 +++---
 cloudinit/CloudConfig/cc_ssh_import_id.py | 9 +-
 cloudinit/CloudConfig/cc_timezone.py | 5 +-
 cloudinit/CloudConfig/cc_update_etc_hosts.py | 8 +-
 cloudinit/CloudConfig/cc_update_hostname.py | 12 +-
 cloudinit/DataSource.py | 31 +++---
 cloudinit/DataSourceEc2.py | 39 ++++---
 cloudinit/DataSourceNoCloud.py | 31 +++--
 cloudinit/DataSourceOVF.py | 66 ++++++-----
 cloudinit/SshUtil.py | 30 +++--
 cloudinit/UserDataHandler.py | 44 +++++---
 cloudinit/__init__.py | 131 ++++++++++++---------
 cloudinit/netinfo.py | 23 +++--
 cloudinit/util.py | 96 +++++++++++-----
 43 files changed, 501 insertions(+), 359 deletions(-)

Revision history for this message
Juerg Haefliger (juergh) wrote :

Sorry, link to PEP8 in the previous comment is broken. Correct link is: http://www.python.org/dev/peps/pep-0008/

Revision history for this message
Juerg Haefliger (juergh) wrote :

With the attached patches, the code is now clean when checked as follows:

#!/bin/bash

files='cloud*.py cloudinit/*.py cloudinit/CloudConfig/*.py'

cmd=(
    pylint
    --reports=n
    --include-ids=y
    --max-line-length=79

    --disable=R
    --disable=I

    --disable=W0142 # Used * or ** magic
    --disable=W0402 # Uses of a deprecated module
    --disable=W0404 # Reimport xyz
    --disable=W0511 # TODO/FIXME note
    --disable=W0603 # Using the global statement
    --disable=W0621 # Redefining name xyz from outer scope
    --disable=W0702 # No exception type(s) specified
    --disable=W0703 # Catch "Exception"

    --disable=C0103 # Invalid name
    --disable=C0111 # Missing docstring

    $files
)

echo -e "\nRunning pylint:"
echo ${cmd[*]}
eval ${cmd[*]}

cmd=(
    pep8

    --ignore=E501 # Line too long (these are caught by pylint above)

    $files
)

echo -e "\nRunning pep8:"
echo ${cmd[*]}
eval ${cmd[*]}

Revision history for this message
Juerg Haefliger (juergh) wrote :

Final patch that adds HP to the copyright notice to please the legal department.

Juerg Haefliger (1):
  Fix license to please HP legal

 cloud-init-cfg.py | 2 ++
 cloud-init-query.py | 2 ++
 cloud-init.py | 2 ++
 cloudinit/CloudConfig/__init__.py | 3 +++
 cloudinit/CloudConfig/cc_apt_update_upgrade.py | 3 +++
 cloudinit/CloudConfig/cc_bootcmd.py | 2 ++
 cloudinit/CloudConfig/cc_byobu.py | 3 +++
 cloudinit/CloudConfig/cc_chef.py | 3 +++
 cloudinit/CloudConfig/cc_disable_ec2_metadata.py | 2 ++
 cloudinit/CloudConfig/cc_final_message.py | 3 +++
 cloudinit/CloudConfig/cc_foo.py | 2 ++
 cloudinit/CloudConfig/cc_grub_dpkg.py | 2 ++
 cloudinit/CloudConfig/cc_keys_to_console.py | 3 +++
 cloudinit/CloudConfig/cc_landscape.py | 3 +++
 cloudinit/CloudConfig/cc_locale.py | 3 +++
 cloudinit/CloudConfig/cc_mcollective.py | 2 ++
 cloudinit/CloudConfig/cc_mounts.py | 3 +++
 cloudinit/CloudConfig/cc_phone_home.py | 2 ++
 cloudinit/CloudConfig/cc_puppet.py | 3 +++
 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 | 3 +++
 cloudinit/CloudConfig/cc_ssh_import_id.py | 3 +++
 cloudinit/CloudConfig/cc_timezone.py | 2 ++
 cloudinit/CloudConfig/cc_update_etc_hosts.py | 3 +++
 cloudinit/CloudConfig/cc_update_hostname.py | 3 +++
 cloudinit/DataSource.py | 2 ++
 cloudinit/DataSourceEc2.py | 2 ++
 cloudinit/DataSourceNoCloud.py | 2 ++
 cloudinit/DataSourceOVF.py | 2 ++
 cloudinit/SshUtil.py | 19 +++++++++++++++++++
 cloudinit/UserDataHandler.py | 3 +++
 cloudinit/__init__.py | 2 ++
 cloudinit/netinfo.py | 21 +++++++++++++++++++++
 cloudinit/util.py | 3 +++
 43 files changed, 138 insertions(+), 0 deletions(-)

Scott Moser (smoser)
Changed in cloud-init:
status: Fix Committed → Fix Released
Revision history for this message
James Falcon (falcojr) wrote :

Tracked in Github Issues as https://github.com/canonical/cloud-init/issues/2234

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.