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

Fix check mode #163

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

x4e-jonas
Copy link

@x4e-jonas x4e-jonas commented Jul 31, 2024

The role fails when Ansible is run in check mode because several directories do not exist.

msg: dest '/tmp/rclone_setup' must be an existing dir

I think its safe to create files under /tmp but the rclone.fact needs an exclusion.

@stefangweichinger
Copy link
Owner

thanks. I consider modifying the create task anyway, so pls let's see if that maybe also fixes this issue.

@x4e-jonas
Copy link
Author

You mean the change in bfdd10a? I'm still getting the error above in check mode with the newly released 0.2.0.

@stefangweichinger
Copy link
Owner

You mean the change in bfdd10a? I'm still getting the error above in check mode with the newly released 0.2.0.

Do we have to add a molecule test for the check mode also?

@x4e-jonas
Copy link
Author

I think you can just run molecule check: e17f08c

@stefangweichinger
Copy link
Owner

I think that needs more thought and preparation. I am not sure yet about the setup and how to properly use the various sequences in molecule.

@x4e-jonas
Copy link
Author

Sure, let me know if I can help.

@stefangweichinger
Copy link
Owner

Sure, let me know if I can help.

thanks. pls allow some time ... summer time and vacation etc

@stefangweichinger
Copy link
Owner

sorry for being so late here. busy times

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