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
support collection-requirements.yml #149
Conversation
There is no way for a role to automatically require collections for module/plugin dependencies. Use the non-standard file `meta/collection-requirements.yml` for this. Note that `meta/requirements.yml` is supposed to be a list of roles only, in the format described at https://docs.ansible.com/ansible/latest/galaxy/user_guide.html#installing-multiple-roles-from-multiple-files If we find a `meta/requirements.yml` file, and it is not in `list` format, assume it is an old one that has not yet been renamed to `meta/collection-requirements.yml`, use it, and warn the user.
This is related to linux-system-roles/tox-lsr#78 |
continue # legacy role format | ||
if rqf == legacy_rqf: | ||
logging.warning( | ||
"The role %s is still using %s - please convert to %s instead", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering to convert requirements.yml
to collection-requirements.yml
, are there anything else to do other than renaming the filename?
continue # legacy role format | ||
if rqf == legacy_rqf: | ||
logging.warning( | ||
"The role %s is still using %s - please convert to %s instead", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering to convert requirements.yml
to collection-requirements.yml
, are there anything else to do other than renaming the filename?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There is no way for a role to automatically require collections for
module/plugin dependencies. Use the non-standard file
meta/collection-requirements.yml
for this. Note thatmeta/requirements.yml
is supposed to be a list of roles only, inthe format described at https://docs.ansible.com/ansible/latest/galaxy/user_guide.html#installing-multiple-roles-from-multiple-files
If we find a
meta/requirements.yml
file, and it is not inlist
format, assume it is an old one that has not yet been renamed to
meta/collection-requirements.yml
, use it, and warn the user.