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

Reprounzip unpacker containerexec #276

Open
wants to merge 7 commits into
base: 1.x
Choose a base branch
from

Conversation

PhilippWendler
Copy link

This is a new unpacker for reprounzip. It only works on Linux and creates a container similar to a Docker container. However, it is based on our tool BenchExec.

This unpacker provides the following advantages compared to the existing unpackers:

  • no special setup or manual installation of further software necessary (BenchExec is a pure-Python package and installed automatically as a dependency)
  • works as regular user, does not require root access
  • provides strong isolation like a container

The system requirements are a Linux kernel with namespaces (present since Linux 3.8).

Maybe because of these advantages you would even like to bundle it as one of the default unpackers together with ReproZip, at least on Linux?

This is our first contribution to ReproZip, so there are maybe some things that could be done in a better way, but we would be happy to improve them if required.

@remram44 remram44 added A-pending-review Attention: This is ready for review and can be merged afterwards C-unpacker Component: New unpacker or common unpacker functionalities T-enhancement Type: En enhancement to existing code, or a new feature labels Oct 2, 2017
@remram44 remram44 added this to the 1.0.11 milestone Oct 2, 2017
@remram44 remram44 self-requested a review October 2, 2017 13:20
@remram44 remram44 modified the milestones: 1.0.11, 1.0.12 Nov 2, 2017
@PhilippWendler
Copy link
Author

Any news on this? Is there anything we need to do?

@remram44
Copy link
Member

Hi @PhilippWendler! I just need to find time to go over this and merge it. We are very interested in integrating ReproZip with more tools, I just didn't have the cycles. There are currently a few PRs that have accumulated on ReproZip and that I need to go over.

This has really been lingering for a while, I apologize for not getting back to you sooner. Hopefully I can take a good look this week.

Sorry about that!

@PhilippWendler
Copy link
Author

No worries, just wanted to know whether there is something we need to do.

@PhilippWendler
Copy link
Author

Is there still a chance that this will be merged at some point?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-pending-review Attention: This is ready for review and can be merged afterwards C-unpacker Component: New unpacker or common unpacker functionalities T-enhancement Type: En enhancement to existing code, or a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants