Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Ansible default plugin paths in config files #958

Merged
merged 1 commit into from
Mar 8, 2018
Merged

Conversation

fullyint
Copy link
Contributor

@fullyint fullyint commented Mar 7, 2018

ansible/ansible#12378 changed Ansible's default plugin paths. At the time, Ansible set these defaults in lib/ansible/constants.py but now in lib/ansible/config/base.yml (i.e., ansible config;
location change in ansible/ansible#27448).

This PR updates these default paths in Trellis ansible.cfg and lib/trellis/vagrant.rb.

The CHANGELOG entry marks this as [BREAKING] for the uncommon case that users have plugins in any of the out-of-date locations.

@fullyint fullyint merged commit f7cc5cc into master Mar 8, 2018
@fullyint fullyint deleted the plugin-paths branch March 8, 2018 01:52
@partounian
Copy link
Contributor

To implement the past few commits, do we just need to run deploy or do we need reconfigure the server with the ansible-playbook command?

@swalkinshaw
Copy link
Member

Just the commits in this PR? Don't need to do anything other than merge.

primozcigler added a commit to proteusthemes/pt-ops that referenced this pull request Mar 13, 2018
* trellis/master:
  Ensure Diffie-Hellman group is generated for Let's Encrypt (roots#964)
  Fix raw_vars feature to properly handle int values (roots#959)
  Update Ansible default plugin paths in config files (roots#958)
  Add Nginx ssl.no-default.conf to drop requests for unknown hosts (roots#888)
  Disable memcached UDP support by default
  Git: Ignore `vagrant.local.yml`
  Add sponsors [ci skip]
  Set Vagrant's ansible compatibility_mode
  Check and stop php7.1-fpm service
  Update to PHP 7.2
  Fix failed_when in template_root check with wp-cli 1.5.0 (roots#948)
  Bump Ansible version_tested_max to 2.4.3.0 (roots#945)
  Update wp-cli to 1.5.0 (roots#944)
  Update `vagrant_box_version` to `>= 201801.02.0`
  Update `vagrant_box_version` to `<= 201801.02.0`
  Update Vagrant requirement to v2.0.1.
  Support for IPv6 in Letsencrypt challenges
  Allow Trellis VMs to be created/provisioned in Windows Subsystem for Linux.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants