{"action":"created","issue":{"url":"https://api.github.com/repos/opencontainers/specs/issues/142","labels_url":"https://api.github.com/repos/opencontainers/specs/issues/142/labels{/name}","comments_url":"https://api.github.com/repos/opencontainers/specs/issues/142/comments","events_url":"https://api.github.com/repos/opencontainers/specs/issues/142/events","html_url":"https://github.com/opencontainers/specs/pull/142","id":104740187,"number":142,"title":"config: Require the runtime to mount Spec.Mounts in order","user":{"login":"wking","id":209920,"avatar_url":"https://avatars.githubusercontent.com/u/209920?v=3","gravatar_id":"","url":"https://api.github.com/users/wking","html_url":"https://github.com/wking","followers_url":"https://api.github.com/users/wking/followers","following_url":"https://api.github.com/users/wking/following{/other_user}","gists_url":"https://api.github.com/users/wking/gists{/gist_id}","starred_url":"https://api.github.com/users/wking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wking/subscriptions","organizations_url":"https://api.github.com/users/wking/orgs","repos_url":"https://api.github.com/users/wking/repos","events_url":"https://api.github.com/users/wking/events{/privacy}","received_events_url":"https://api.github.com/users/wking/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-09-03T17:04:50Z","updated_at":"2015-09-11T00:31:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/opencontainers/specs/pulls/142","html_url":"https://github.com/opencontainers/specs/pull/142","diff_url":"https://github.com/opencontainers/specs/pull/142.diff","patch_url":"https://github.com/opencontainers/specs/pull/142.patch"},"body":"If we don't specify this, some bundle-authors or runtime-implementers\r\nmight expect the runtime to intelligently order mounts to get the\r\n\"right\" order.  But that's not possible because:\r\n\r\n    $ mkdir -p a/b/c d/e/f h\r\n    # mount --bind a/b h\r\n    # mount --bind d a/b\r\n    $ tree --charset=ascii h\r\n    h\r\n    `-- c\r\n\r\nBut in the other order:\r\n\r\n    # umount a/b\r\n    # umount h\r\n    # mount --bind d a/b\r\n    # mount --bind a/b h\r\n    $ tree --charset=ascii h\r\n    h\r\n    `-- e\r\n        `-- f\r\n\r\nSo there's no \"right\" order.  Allowing the bundle-author to specify\r\ntheir intended order is both easy to implement and unambiguous.\r\n\r\nSpun off from #136."},"comment":{"url":"https://api.github.com/repos/opencontainers/specs/issues/comments/139418309","html_url":"https://github.com/opencontainers/specs/pull/142#issuecomment-139418309","issue_url":"https://api.github.com/repos/opencontainers/specs/issues/142","id":139418309,"user":{"login":"laijs","id":125671,"avatar_url":"https://avatars.githubusercontent.com/u/125671?v=3","gravatar_id":"","url":"https://api.github.com/users/laijs","html_url":"https://github.com/laijs","followers_url":"https://api.github.com/users/laijs/followers","following_url":"https://api.github.com/users/laijs/following{/other_user}","gists_url":"https://api.github.com/users/laijs/gists{/gist_id}","starred_url":"https://api.github.com/users/laijs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/laijs/subscriptions","organizations_url":"https://api.github.com/users/laijs/orgs","repos_url":"https://api.github.com/users/laijs/repos","events_url":"https://api.github.com/users/laijs/events{/privacy}","received_events_url":"https://api.github.com/users/laijs/received_events","type":"User","site_admin":false},"created_at":"2015-09-11T00:31:10Z","updated_at":"2015-09-11T00:31:10Z","body":"LGTM\r\n\r\nI was wondering if you could add \"Suggested-by: Lai Jiangshan <jiangshanlai@gmail.com>\""}}