-
Notifications
You must be signed in to change notification settings - Fork 667
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
Make molecule reuse prerun from ansible-lint #3077
Conversation
48ff10a
to
3fbc320
Compare
f3ad3b7
to
a763021
Compare
2014129
to
aea296c
Compare
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 tested changes on our collections and it works just fine. Having to install ansible-lint just for setting up the test run is a bit of a bummer, but you cannot always win I guess.
I've done a quick test. While it doesnt solve my bug, this change make molecule behaves in the same way as ansible-lint, |
6dab445
to
4ebcf18
Compare
As ansible-lint is able to detect repository layout and run prepare ansible from running it, we rely on its logic in order to assure ansible is able to run properly. This means that dependencies will be installed into .cache folder and that the role import path will be updated to include it.
As ansible-lint is able to detect repository layout and run prepare ansible from running it, we rely on its logic in order to assure ansible is able to run properly.
This means that dependencies will be installed into .cache folder and that the role import path will be updated to include it.