Skip to content
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

Merged
merged 1 commit into from Mar 23, 2021
Merged

Make molecule reuse prerun from ansible-lint #3077

merged 1 commit into from Mar 23, 2021

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Mar 19, 2021

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.

@ssbarnea ssbarnea force-pushed the os/prerun branch 5 times, most recently from 48ff10a to 3fbc320 Compare March 19, 2021 16:18
@ssbarnea ssbarnea changed the title Make molecule reuse prerun from ansible-lint WIP: Make molecule reuse prerun from ansible-lint Mar 19, 2021
@ssbarnea ssbarnea force-pushed the os/prerun branch 8 times, most recently from f3ad3b7 to a763021 Compare March 22, 2021 10:49
@ssbarnea ssbarnea force-pushed the os/prerun branch 2 times, most recently from 2014129 to aea296c Compare March 22, 2021 17:16
@ssbarnea ssbarnea changed the title WIP: Make molecule reuse prerun from ansible-lint Make molecule reuse prerun from ansible-lint Mar 22, 2021
@ssbarnea ssbarnea added this to the 3.3.0 milestone Mar 22, 2021
Copy link
Contributor

@tadeboro tadeboro left a 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.

docs/configuration.rst Outdated Show resolved Hide resolved
docs/configuration.rst Outdated Show resolved Hide resolved
@apatard
Copy link
Contributor

apatard commented Mar 23, 2021

I've done a quick test. While it doesnt solve my bug, this change make molecule behaves in the same way as ansible-lint,
which is nice. The prerun option is working as expected too.
I found some typos in the doc. Except for the earliers comments, I'm happy with this

@ssbarnea ssbarnea force-pushed the os/prerun branch 4 times, most recently from 6dab445 to 4ebcf18 Compare March 23, 2021 10:07
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants