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

Fixes for new molecule #94

Closed
wants to merge 2 commits into from
Closed

Fixes for new molecule #94

wants to merge 2 commits into from

Conversation

AaronJackson
Copy link
Member

No description provided.

* Correct permissions of /var/lib/clamav to allow all users to list
  the directory

* Replace use of `sigtool` with a hardcoded signature for EICAR, as
  this triggers Microsoft Defender during the `molecule converge` (a
  temp file is created locally during the `ansible.builtin.copy`).
@AaronJackson AaronJackson requested a review from manics April 23, 2024 12:54
@manics
Copy link
Member

manics commented Apr 23, 2024

Can you try upgrading ansible-lint in requirements-dev.txt? ansible/ansible-lint#3408 (comment)
Maybe ansible-compat too if necessary?

@AaronJackson
Copy link
Member Author

😱

@AaronJackson AaronJackson deleted the fixes-for-new-molecule branch April 23, 2024 13:05
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.

2 participants