Update ansible to 2.5.1
ClosedPublic
Actions

Authored by mcritchlow on Apr 24 2018, 4:17 PM.

Details

Reviewers

joebonrichie
• sunnyflunk

Group Reviewers

Triage Team

Commits

R360:d1c4265b4238: Update ansible to 2.5.1

Summary

New simpler and more intuitive 'loop' keyword for task loops. The with_<lookup> loops will likely be deprecated in the near future and eventually removed.
Added fact namespacing; from now on facts will be available under ansible_facts namespace (for example: ansible_facts.os_distribution) without the ansible_ prefix. They will continue to be added into the main namespace directly, but now with a configuration toggle to enable this. This is currently on by default, but in the future it will default to off.
Ansible Network improvements
Changelog
Ansible 2.5 Porting Guide

Test Plan

Installed locally, confirmed version
Ran a few local playbooks
Confirmed new YAML callback plugin introduced in 2.5 works

Diff Detail

Repository

R360 ansible

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

mcritchlow created this revision.Apr 24 2018, 4:17 PM

Herald added a reviewer: Triage Team. · View Herald TranscriptApr 24 2018, 4:17 PM

mcritchlow requested review of this revision.Apr 24 2018, 4:17 PM

mcritchlow mentioned this in D2801: Update ansible to 2.5.1.Apr 26 2018, 3:28 AM

UnlawfulMonad added a subscriber: UnlawfulMonad.Apr 26 2018, 5:29 AM

Just a small change needed

package.yml
6	Should be `GPL-3.0-or-later` to follow updated SPDX licence identifiers.

This revision now requires changes to proceed.Apr 26 2018, 9:40 AM

Appease the future (update license as requested)

In D2789#44066, @joebonrichie wrote:

Just a small change needed

Thanks, fixed. Since this has now happened a few times, I've written up a little script to test this when updating future packages, because it's just too damn easy to forget to look at..

https://github.com/mcritchlow/dotfiles/commit/6564d6b13786279c2fa935a770a5e1e1d6733945

@mcritchlow fyi, I don't think your alias will work as intended

updatePackage() {
    /usr/share/ypkg/yupdate.py $1 $2
    validate-license
}

This revision was not accepted when it landed; it landed in state Needs Review.Apr 27 2018, 12:19 AM

Closed by commit R360:d1c4265b4238: Update ansible to 2.5.1 (authored by Matt Critchlow <matt.critchlow@gmail.com>, committed by • sunnyflunk). · Explain Why

This revision was automatically updated to reflect the committed changes.

In D2789#44143, @sunnyflunk wrote:
@mcritchlow fyi, I don't think your alias will work as intended
updatePackage() {
    /usr/share/ypkg/yupdate.py $1 $2
    validate-license
}

Derp.. yeah arguments are a thing. thanks :)