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

minor: support debian 12 #201

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

Conversation

gardar
Copy link
Member

@gardar gardar commented Aug 22, 2023

No description provided.

@github-actions github-actions bot added minor and removed minor labels Aug 22, 2023
@github-actions github-actions bot added minor and removed minor labels Aug 22, 2023
@github-actions github-actions bot added minor and removed minor labels Aug 22, 2023
@github-actions github-actions bot added minor and removed minor labels Oct 27, 2023
@SuperQ
Copy link
Contributor

SuperQ commented Oct 27, 2023

Failed to find required executable \"rsync\" in paths

@gardar
Copy link
Member Author

gardar commented Nov 3, 2023

Failed to find required executable \"rsync\" in paths

Yep, the debian 12 container doesn't come with python so I configured molecule to prepare the image pre_build_image: false which should install python, etc.

But to do that the molecule docker driver uses the synchronize ansible module which requires rsync, where IMHO the copy module would be sufficient.

It's a catch 22 situation and I've actually thought about doing a PR to address this in the docker driver.

@gardar
Copy link
Member Author

gardar commented Dec 7, 2023

Failed to find required executable \"rsync\" in paths

Yep, the debian 12 container doesn't come with python so I configured molecule to prepare the image pre_build_image: false which should install python, etc.

But to do that the molecule docker driver uses the synchronize ansible module which requires rsync, where IMHO the copy module would be sufficient.

It's a catch 22 situation and I've actually thought about doing a PR to address this in the docker driver.

Hopefully test-kitchen/dokken-images#117 will get approved which should resolve this issue.

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.

2 participants