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

[007]: Add an nsinit demo launching a container from IPFS #22

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

wking
Copy link

@wking wking commented Apr 16, 2015

Without bothering with Docker's image format, layer tarballs, etc.
Details in the demo's README.md. I'm happy to rebase around #19
(which is currently also using the 005 number) if that PR lands first.

@jbenet jbenet added the backlog label Apr 16, 2015
wking added a commit to wking/ipfs-infrastructure that referenced this pull request Apr 16, 2015
This is the example filesystem used for my nsinit container demo [1].

[1]: ipfs-shipyard/container-demos#22
@jbenet
Copy link
Contributor

jbenet commented Apr 17, 2015

@wking may be easier to turn this into [006]. its ok if the demos on the repo are weird numbers. we can reorder them later.

file permissions][no-exec], so you can't launch `nsinit` directly from
your IPNS directory. As a workaround, the demo `Makefile` currently
extracts the IPFS directory to your local filesystem, and then tweaks
the file mode locally.
Copy link
Contributor

Choose a reason for hiding this comment

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

@whyrusleeping maybe we can add perms to unixfs soon? we should consider it for sure in next week's sprint. ipfs/team-mgmt#5

@jbenet
Copy link
Contributor

jbenet commented Apr 17, 2015

@wking this looks great. i havent tried it yet, but maybe i can try it tomorrow.

@wking
Copy link
Author

wking commented Apr 17, 2015

On Thu, Apr 16, 2015 at 05:59:26PM -0700, Juan Batiz-Benet wrote:

@wking may be easier to turn this into [006]. its ok if the demos
on the repo are weird numbers. we can reorder them later.

Maybe I should make it [101], since it's the first demo in a
nsinit-based track, which is pretty independent of @hosh's existing
virtual machine and Docker-based demos. Or maybe [0xx] for VMs, [1xx]
for Docker, and [2xx] for nsinit, where the hundreds digit is just the
order in which they arrived. I'm happy to use a different hundreds
ordering if you'd prefer something else.

@wking wking force-pushed the nsinit-container-from-ipfs-filesystem branch from 4c32056 to 075aaa4 Compare April 17, 2015 22:02
wking added 3 commits April 17, 2015 15:05
Without bothering with Docker's image format, layer tarballs, etc.
Without this running you can still 'ipfs get ...' local objects, but
remote objects will give you:

  $ ipfs ls QmYRpzwXkSnCPPNL4b2gn5AFnyyMHFxhS9Nrbau5ycw92H
  Error: blockstore: block not found
@wking wking force-pushed the nsinit-container-from-ipfs-filesystem branch from 075aaa4 to 94a4f54 Compare April 17, 2015 22:10
@wking wking changed the title [005]: Add an nsinit demo launching a container from IPFS [007]: Add an nsinit demo launching a container from IPFS Apr 17, 2015
@wking wking mentioned this pull request Apr 17, 2015
@wking
Copy link
Author

wking commented Apr 18, 2015 via email

@wking
Copy link
Author

wking commented Apr 20, 2015

On Mon, Apr 20, 2015 at 01:13:03PM -0700, Juan Batiz-Benet wrote:

Assigned #22 to @wking.

I'm not sure what I'm getting assigned to do here ;). I'll work on
getting access mode metadata in place, and that will remove one of the
current pain points here.

wking added 2 commits June 1, 2015 14:59
You can still build it from source if you want, but this lets you skip
that step if you'd rather.  I've also included some of the basic
libraries from the Gentoo stage3 so we can handle a bit of dynamic
linking in Go binaries.
This is one less step folks need to run to launch from the directory,
although it does mean we have something else to update if the
libcontainer folks change their default config format, or tweak the
default values too far.

Conflicts:
	007-nsinit-container-from-ipfs-filesystem/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants