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.

Sign up for GitHub

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

Jump to bottom

Is the ansible.cfg needed here? #19

Closed
swapdisk opened this issue Oct 16, 2023 · 1 comment · Fixed by #20
Closed

Is the ansible.cfg needed here? #19

swapdisk opened this issue Oct 16, 2023 · 1 comment · Fixed by #20

Comments

@swapdisk
Copy link
Member

@ygalblum, we ran into an issue where the ansible.cfg in this repo supersedes the one on the AAP/Tower server. Can we remove this file from the repo?
@ygalblum ygalblum mentioned this issue Oct 17, 2023
Merged
@ygalblum
Copy link
Collaborator

The ansible.cfg file is used for running the tests without having to install the collection on the tested machine. I can see that it is not packed into the collection, so I'm not sure why you even have it when running a playbook that uses the role.

Having said that, since it is used for testing, I've moved it there and added a README describing how to run the test (#20)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move ansible.cfg to tests ygalblum/ipu-lvm-snapshot
2 participants
@ygalblum @swapdisk