Merge ~d-info-e/cloud-init:fix-log-warn into cloud-init:master

Proposed by do3meli
Status: Merged
Approved by: Dan Watkins
Approved revision: d15d8b70187fbfe0cf04b1176e2ff2a2cfdcd8f8
Merge reported by: Server Team CI bot
Merged at revision: not available
Proposed branch: ~d-info-e/cloud-init:fix-log-warn
Merge into: cloud-init:master
Diff against target: 673 lines (+90/-92)
27 files modified
cloudinit/config/cc_apt_pipelining.py (+1/-1)
cloudinit/config/cc_byobu.py (+3/-3)
cloudinit/config/cc_chef.py (+10/-10)
cloudinit/config/cc_emit_upstart.py (+1/-1)
cloudinit/config/cc_final_message.py (+1/-1)
cloudinit/config/cc_growpart.py (+1/-1)
cloudinit/config/cc_keys_to_console.py (+3/-3)
cloudinit/config/cc_lxd.py (+7/-8)
cloudinit/config/cc_mounts.py (+7/-7)
cloudinit/config/cc_package_update_upgrade_install.py (+4/-3)
cloudinit/config/cc_phone_home.py (+6/-6)
cloudinit/config/cc_power_state_change.py (+6/-7)
cloudinit/config/cc_puppet.py (+5/-5)
cloudinit/config/cc_resizefs.py (+8/-9)
cloudinit/config/cc_resolv_conf.py (+2/-2)
cloudinit/config/cc_rightscale_userdata.py (+2/-2)
cloudinit/config/cc_rsyslog.py (+1/-1)
cloudinit/config/cc_scripts_per_boot.py (+2/-2)
cloudinit/config/cc_scripts_per_instance.py (+2/-2)
cloudinit/config/cc_scripts_per_once.py (+2/-2)
cloudinit/config/cc_scripts_user.py (+2/-2)
cloudinit/config/cc_scripts_vendor.py (+2/-2)
cloudinit/config/cc_seed_random.py (+1/-1)
cloudinit/config/cc_set_passwords.py (+1/-1)
cloudinit/config/cc_update_etc_hosts.py (+4/-4)
cloudinit/config/cc_yum_add_repo.py (+5/-5)
tests/unittests/test_handler/test_handler_power_state.py (+1/-1)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
Review via email: mp+374215@code.qastaging.launchpad.net

Commit message

replace any deprecated log.warn with log.warning

Commit 6797e822959b84c98cf73e02b2a6e3d6ab3fd4fe replaced
the LOG.warn calls that linters were warning about; this
also replaces calls that linters would not have recognised
(as `log` is generally a parameter in these scenarios).

LP: #1508442

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:618136a3309ad961c94b95aacb4c2a590be67cb1
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1210/
Executed test runs:
    FAILED: Checkout

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1210//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:618136a3309ad961c94b95aacb4c2a590be67cb1
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1211/
Executed test runs:
    SUCCESS: Checkout
    FAILED: Unit & Style Tests

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1211//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:618136a3309ad961c94b95aacb4c2a590be67cb1
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1212/
Executed test runs:
    SUCCESS: Checkout
    FAILED: Unit & Style Tests

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1212//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Dan Watkins (oddbloke) wrote :

The lint failures from the previous run were not addressed; that's what's failing here, too.

Revision history for this message
do3meli (d-info-e) wrote :

hi dan,
i have fixed the outstanding pycodestyle issues i found in my local env.
please kick in another CI run.
cheers, dom

Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:618136a3309ad961c94b95aacb4c2a590be67cb1
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1213/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/1213//rebuild

review: Approve (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) :
review: Approve (continuous-integration)

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Update scan failed

At least one of the branches involved have failed to scan. You can manually schedule a rescan if required.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches