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

Buildah connection rootless #28

Merged
merged 1 commit into from May 5, 2020
Merged

Buildah connection rootless #28

merged 1 commit into from May 5, 2020

Conversation

sshnaidm
Copy link
Member

@sshnaidm sshnaidm commented May 2, 2020

Support rootless containers with buildah connection
Fixes ansible/ansible#63316

@sshnaidm
Copy link
Member Author

@sshnaidm sshnaidm commented May 4, 2020

@TomasTomecek can you please take a look? I'd like to have it as a temporary solution for buildah rootless, until I make it work with unshare, if it's possible. Currently unshare mount doesn't work for me well.

Copy link

@TomasTomecek TomasTomecek left a comment

Looks good, worked for me, even with mount:

TASK [Run a sample command] ************************************************************
task path: /home/tt/g/TomasTomecek/ansible-bender/simple-playbook.yaml:21             
Using buildah connection from collection                                              
<fedora-working-container> RUN [b'buildah', b'mount', b'--', b'fedora-working-container']                                                                                    
STDOUT b'/home/tt/.local/share/containers/storage/overlay/06b201fda221beaadcb651869c1f8b191e6ada2dd00bcbc53d7f3809360dffe1/merged\n'
STDERR b''                                                                            
RC CODE 0                                  

Decided to blog about this :D https://blog.tomecek.net/post/ansible-collections/

@sshnaidm sshnaidm merged commit 3d1c38c into containers:master May 5, 2020
10 checks passed
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