-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
This role overrides php_packages, which is also used in geerlingguy.php #46
Comments
This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
Today I have found this issue as affecting my playbook. I also have examined and tested the proposed solution - PR #39. It works with both - playbook vars and host vars. @mloveday , as a temporary solution you can create host vars to avoid overwriting variables and also avoid endless waiting for PR #39 to be merged. I.e. create file |
This issue is no longer marked for closure. |
@mloveday, another option is to use my fork webarchitect609.php_versions, where it has been fixed. |
It works for me. |
Same for me. Thanks for this workaround! |
This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
This issue has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details. |
@geerlingguy, I would like you to check this issue again. I believe this is a bug, formulated as: impossible to set The fix is simple and I re-make PR #39 as a fresh PR #55 . Thank you! :) |
This issue is no longer marked for closure. |
Fixed in #55. |
Our playbook has the following config defined:
When we run the playbook, the php packages we were specifying were not being installed. We found that
geerlingguy.php_versions
was settingphp_packages
to that role's default value, andgeerlingguy.php
was then using this default over our user-specified value. We tried specifyingphp_packages
in vars as above, in a vars file, and inline with the role itself (- { role: geerlingguy.php, vars: php_packages: ... }
), but nothing changed.The only way we could work around this was by defining the
php_packages
variable in a new role after php-versions had run:... with the php-packages role containing the following ...
Firstly, is there something I'm doing wrong in specifying the
php_packages
variable? If it's something I've done wrong, what should I change and why is this particular use case different to others?Is it just a case of renaming this role's (internal only?) variable to something more unique? If so, I'm happy to provide a PR.
Thanks!
The text was updated successfully, but these errors were encountered: