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

Using copy instead of unarchive to deploy incexc files #96

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ohmer1
Copy link

@ohmer1 ohmer1 commented Jul 12, 2023

I tried to edit incexc configuration files and wondered why the changes didn't apply to my server when running my playbook. I found two issues.

First issue, I found I had to edit the .tar.gz and not the incexc files directly. It's confusing and error prone I think (At first, I edited the individual file without reading the task file). I proceed to edit the .tar.gz anyway. I ran the playbook again and still no apply. The second issue I found is caused by the created parameter used in the task, Ansible never updates the directory again if incexc/audio_exclusions exists at the remote server.

I propose to use the copy module instead of the unarchive module to update theses file. The directory doesn't content a lot of files, so there is not much performance penalty.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant