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

Add simple documentation how to use c/image with podman's rootless mode #1405

Merged
merged 1 commit into from Nov 12, 2021

Conversation

dcermak
Copy link
Contributor

@dcermak dcermak commented Nov 10, 2021

this fixes #1400

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
@mtrmac @giuseppe @lsm5 PTAL

@vrothberg
Copy link
Member

vrothberg commented Nov 10, 2021

Thanks, @dcermak. In order to pass CI you need to sign your commit (e.g., via git commit --amend -s) and repush.

@dcermak
Copy link
Contributor Author

dcermak commented Nov 10, 2021 via email

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

doc.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sure, this place works well enough.

doc.go Show resolved Hide resolved
@TomSweeneyRedHat
Copy link
Member

@dcermak thanks for the PR. It needs an update and LGTM once @mtrmac 's comments are addressed.

this fixes containers#1400

Signed-off-by: Dan Čermák <dcermak@suse.com>
@TomSweeneyRedHat
Copy link
Member

LGTM

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vrothberg vrothberg merged commit 1895e31 into containers:main Nov 12, 2021
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.

Document how to access rootless containers-storage:
5 participants