{"action":"created","issue":{"url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/513","repository_url":"https://api.github.com/repos/opencontainers/runtime-spec","labels_url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/513/labels{/name}","comments_url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/513/comments","events_url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/513/events","html_url":"https://github.com/opencontainers/runtime-spec/pull/513","id":164148308,"number":513,"title":"Merge branch 'runtime-spec' of git://github.com/wking/oci-command-line-api","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,"assignees":[],"milestone":null,"comments":30,"created_at":"2016-07-06T18:54:35Z","updated_at":"2016-07-11T22:18:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/opencontainers/runtime-spec/pulls/513","html_url":"https://github.com/opencontainers/runtime-spec/pull/513","diff_url":"https://github.com/opencontainers/runtime-spec/pull/513.diff","patch_url":"https://github.com/opencontainers/runtime-spec/pull/513.patch"},"body":"In today's meeting [I said I'd PR this][1], so here it is.  I've kept the history to preserve previous review and contribution information.  I've tried to summarize any discussion that happened elsewhere in the commit messages, but there may be additional background in [the old repo's PRs][2].\n\n[1]: http://ircbot.wl.linuxfoundation.org/meetings/opencontainers/2016/opencontainers.2016-07-06-17.00.log.html#l-24\n[2]: https://github.com/wking/oci-command-line-api/pulls?q=is%3Apr+is%3Aclosed"},"comment":{"url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/comments/231882879","html_url":"https://github.com/opencontainers/runtime-spec/pull/513#issuecomment-231882879","issue_url":"https://api.github.com/repos/opencontainers/runtime-spec/issues/513","id":231882879,"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},"created_at":"2016-07-11T22:18:28Z","updated_at":"2016-07-11T22:18:28Z","body":"On Mon, Jul 11, 2016 at 03:04:51PM -0700, Doug Davis wrote:\n> The volume of traffic on this makes it nearly impossible to follow.\n\nI'm happy to punt back to per-issue PRs in wking/oci-command-line-api.\nMy initial feeling was that the current content still needed polishing\n[1], but folks wanted a PR here for discussion ;).  I'll attempt to\nsummarize:\n\n* There are some Travis errors due to whitespace, but I think we\n  should ignore them [2].\n* There are several components that belong in this PR, but are blocked\n  on and up/down decision on #508 [3].  I'm implementing #508 in ccon\n  to help push that PR forward [4].\n* There is disagreement about whether third-party shims that provide\n  compatibility with this API are acceptable for compliance badging\n  [5].\n\nI'm not aware of other open issues from this discussion, but I may\nhave missed something above.\n\n> I would strongly suggest that this group adopt a minimal PR that\n> just documents what the current runc supports today and only those\n> things defined by the runtime spec itself - like my PR #511 tries to\n> do.\n\nI don't think #511 is useful enough for compliance testing without\nwording about when a given command will exit and a cross-platform way\nto recover the container process's exit code [6].\n\n> Worrying about things that are not part of the runtime spec (like\n> \"version\") but be something we can work on at some point, but right\n> now I really don't think that's critical for interop. What's most\n> important are the runtime actions.\n\nI think we've settled down on that particular issue (at least, initial\nreaction to opencontainers/runc#940 seems positive).\n\n[1]: https://groups.google.com/a/opencontainers.org/d/msg/dev/pg0zqms52zk/tbNhIKGgBQAJ\n     Subject: Re: tomorrow's call\n     Date: Tue, 28 Jun 2016 21:36:22 -0700\n     Message-ID: <20160629043622.GE21514@odin.tremily.us>\n[2]: https://github.com/opencontainers/runtime-spec/pull/513#issuecomment-230875797\n[3]: https://github.com/opencontainers/runtime-spec/pull/513#issuecomment-231844933\n[4]: https://github.com/opencontainers/runtime-spec/pull/513#discussion_r70188116\n[5]: https://github.com/opencontainers/runtime-spec/pull/513#issuecomment-231874498\n[6]: https://github.com/opencontainers/runtime-spec/pull/513#discussion_r70189938\n"}}
{"actor":{"display_login":"wking"}}