-
Notifications
You must be signed in to change notification settings - Fork 4
fix: make role compatible with podman #62
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
Conversation
… to fix podman without this, following error appears: Error: executable file not found in /home/jonas/Dokumente/Programmieren/public/ansible-roles/all-repos/ansible-role-bootstrap/venv/bin:/home/jonas/.nvm/versions/node/v16.15.0/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/jonas/.local/share/JetBrains/Toolbox/scripts:/home/jonas/.local/share/JetBrains/Toolbox/scripts:/usr/local/go/bin: No such file or directory: OCI runtime attempted to invoke a command that was not found
226aef8 to
63d5a36
Compare
|
A prove of this bug can be found in a CI run that was run before 63d5a36 ( |
|
lgtm when CI finishes! |
|
This PR also addresses the problem as reported by robertdebock/ansible-role-bootstrap#55. Although this issue has no further comments or reactions I link to it here anyway so people stumbling upon it find this. |
|
Currently errors on all variations of CI as tested in https://github.com/JonasPammer/ansible-role-bootstrap/actions/runs/2966687600, dont have time to investigate today |
|
My current analysis is that the problem boils down to molecule assuming the following in CI, which is not true. There is no difference in pip package versions between CI and local testing i did. only difference currently can think of is that i need to run tox with sudo because podman requires root and i can't setup rootless with my network user setup i currently have |
|
it seems to sportically work? |
in an attempt to understand or fix random CI failures
|
ok. as i am not using podman myself imma remove it from test suite. its molecule collection and extension also says its early stage :/. but i'm going to leave this PR and the actual task adjustment because as seen it does fix podman. |
molecule podman ci tests are scuffed. podman installation is scuffed too. gonna stay away from podman for a while after this encounter lol
molecule podman ci tests are scuffed. podman installation is scuffed too. gonna stay away from podman for a while after this encounter lol as per e182d43
Fixes robertdebock/ansible-role-bootstrap#66 as described in said PR
Checked this myself.
Made this to give myself an reason to try out to install and use podman.
As I couldn't get rootless podman to work with my network linux user (official not supported note can be found here) I'll still stay away with it though
Without this, following error would occur:
... even with
bootstrap_become: false(default).Note: As per https://github.com/actions/runner-images/blob/ubuntu20/20220821.1/images/linux/Ubuntu2004-Readme.md, the current Github Ubuntu VM image ships with podman preinstalled, so there was or should not be a need to change the workflow.
Required Checklist
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Optional Checklist
pre-commit was installed in local development environment (if not, a GitHub workflow will run pre-commit once you create the request)
my commits are small, single-purpose, detailed and maybe even follow the conventional commit specification for extra convenience of the reviewer