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
Limiting plugin types 2 #150
Conversation
4356357
to
0390f4e
Compare
To be added to the agenda once the purpose of the other alternate plugin directories are added. |
Co-authored-by: Felix Fontein <felix@fontein.de>
* The following collections have a temporary exception to use the ``plugin_utils``, ``cli_parsers``, ``fact_diff``, and ``validate`` directories for additional plugins during the 2.10 and 3 release cycles. We will figure out a final policy which these collections will need to comply with before ansible-4: | ||
:Those recognized by ansible-core: ``doc_fragments``, ``modules``, ``module_utils``, ``terminal``, and those listed on https://docs.ansible.com/ansible/devel/plugins/plugins.html This list can be verified by looking at the last element of the package argument of each ``*_loader`` in https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/loader.py#L1126 | ||
:plugin_utils: For shared code which is only used controller-side, not in modules. | ||
:sub_plugins: For other plugins which are managed by plugins inside of collections instead of ansible-core. We use a subfolder so there aren't conflicts when ansible-core adds new plugin types. |
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.
If we think of modules as a kind of plugin, this wording suggested by @mattclay is better:
:sub_plugins: For other plugins which are managed by plugins inside of collections instead of ansible-core. We use a subfolder so there aren't conflicts when ansible-core adds new plugin types. | |
:sub_plugins: For other plugins which are managed by non-module plugins inside of collections instead of ansible-core. We use a subfolder so there aren't conflicts when ansible-core adds new plugin types. |
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.
We'll do this as a separate PR.
This was approved at today's meeting. |
SUMMARY
Polished policy for limiting plugin types.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION