-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
module_utils fixes in collections #55118
module_utils fixes in collections #55118
Conversation
* fixed Windows module_utils in collections * fixed more Python module_utils cases (from X import module) * "medium style" Ansiballz modules now work properly with collections (ie, non-replacer but also not using basic.py) * added more tests * split Windows/POSIX exec
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.
Windows side looks good to me, just had an unrelated question around the shebang code smell ignore that I was mostly curious about
@@ -30,6 +30,10 @@ def main(): | |||
} | |||
|
|||
skip = set([ | |||
'test/integration/targets/collections/collection_root_user/ansible_collections/testns/testcoll/plugins/modules/win_csbasic_only.ps1', |
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.
Are these getting picked up because they are not in a library
dir?
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.
Yep; collections calls that dir modules
. I originally added the new path to the is_module
check, but that broke a bunch of other things, so for the moment we'll need the ignore...
SUMMARY
ISSUE TYPE
COMPONENT NAME
module_common.py
ADDITIONAL INFORMATION