Skip to content

Conversation

@rhatdan
Copy link
Member

@rhatdan rhatdan commented Mar 21, 2019

Update containers/storage and containers/image to define location of local storage.

Signed-off-by: Daniel J Walsh [email protected]

@rhatdan
Copy link
Member Author

rhatdan commented Mar 21, 2019

@giuseppe PTAL, I am trying to get skopeo to work when running as non root to pull content out of the users store.

It looks like skopeo needs to enter the user namespace to be allowed to do the copy.

./skopeo copy containers-storage:alpine dir:/tmp/alpine
FATA[0000] Invalid source name containers-storage:alpine: chown /home/dwalsh/.local/share/containers/storage/overlay/l: operation not permitted

@rhatdan
Copy link
Member Author

rhatdan commented Mar 21, 2019

@vrothberg @mtrmac PTAL

1 similar comment
@rhatdan
Copy link
Member Author

rhatdan commented Mar 21, 2019

@vrothberg @mtrmac PTAL

@mtrmac
Copy link
Contributor

mtrmac commented Mar 21, 2019

@rhatdan ACK. Please rebase on top of c/image master, until we merge this all other c/image PRs will fail tests.

@rhatdan rhatdan changed the title [wip] Vendor in latest containers/storage and containers/image Vendor in latest containers/storage and containers/image Mar 22, 2019
@vrothberg
Copy link
Member

@rhatdan, can you update the vendor.conf and let it point to the regular projects? Once that's merged, the CI over at containers/image will pass again.

Update containers/storage and containers/image to define location of local storage.

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan
Copy link
Member Author

rhatdan commented Mar 25, 2019

@vrothberg @mtrmac Updated. Ready to merge?

@vrothberg
Copy link
Member

LGTM, let's do this :)

@vrothberg vrothberg merged commit 0975497 into containers:master Mar 25, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants