Page MenuHomeSolus

Update ansible to 2.5.1
ClosedPublic

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

Details

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

mcritchlow created this revision.Apr 24 2018, 4:17 PM
mcritchlow requested review of this revision.Apr 24 2018, 4:17 PM
joebonrichie requested changes to this revision.Apr 26 2018, 9:40 AM
joebonrichie added a subscriber: joebonrichie.

Just a small change needed

package.yml
5–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
mcritchlow updated this revision to Diff 6901.Apr 26 2018, 3:10 PM

Appease the future (update license as requested)

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

sunnyflunk accepted this revision.Apr 27 2018, 12:17 AM
sunnyflunk added a subscriber: sunnyflunk.

@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
This revision was automatically updated to reflect the committed changes.

@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 :)