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

Umoci unpack #3921

Closed
wants to merge 9 commits into from
Closed

Umoci unpack #3921

wants to merge 9 commits into from

Conversation

AdamSimpson
Copy link
Contributor

Description of the Pull Request (PR):

This is a first stab at moving from image-tools to umoci for OCI unpacking.

This fixes or addresses the following GitHub issues:

Before submitting a PR, make sure you have done the following:

Attn: @singularity-maintainers

@gvallee gvallee requested review from cclerget and ikaneshiro July 12, 2019 13:09
@AdamSimpson
Copy link
Contributor Author

AdamSimpson commented Jul 12, 2019

It looks like umoci is not going to build on darwin, as there are linux only files. Perhaps unpackTmpfs() could be pulled into separate _linux/_darwin implementations, with linux using umoci and darwin using the current image-tools method?

@GodloveD GodloveD added this to the Financial Organization milestone Jul 18, 2019
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 8, 2019
This change creates a small package to split oci layer unpacking into a
new function that allows for platform dependent builds.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 9, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 9, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 9, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
@ikaneshiro ikaneshiro mentioned this pull request Aug 9, 2019
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 14, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 14, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
ikaneshiro added a commit to ikaneshiro/singularity that referenced this pull request Aug 14, 2019
This change creates two small files to split oci layer unpacking into a
new function that allows for different behavior based on platform.

NOTE: umoci v0.4.2 is used to prevent an a dependency issue with
github.com/opencontainers/runtime-spec v1.0.0 not being backwards
compatable

Update apptainer#3921 to most recent master
@ikaneshiro
Copy link
Contributor

Closing as #4168 is merged

@ikaneshiro ikaneshiro closed this Aug 16, 2019
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.

Importing docker image results in incorrect file permissions
3 participants