{"action":"created","issue":{"url":"https://api.github.com/repos/contiv/volplugin/issues/272","repository_url":"https://api.github.com/repos/contiv/volplugin","labels_url":"https://api.github.com/repos/contiv/volplugin/issues/272/labels{/name}","comments_url":"https://api.github.com/repos/contiv/volplugin/issues/272/comments","events_url":"https://api.github.com/repos/contiv/volplugin/issues/272/events","html_url":"https://github.com/contiv/volplugin/pull/272","id":156069594,"number":272,"title":"Lock free operation","user":{"login":"erikh","id":18923,"avatar_url":"https://avatars.githubusercontent.com/u/18923?v=3","gravatar_id":"","url":"https://api.github.com/users/erikh","html_url":"https://github.com/erikh","followers_url":"https://api.github.com/users/erikh/followers","following_url":"https://api.github.com/users/erikh/following{/other_user}","gists_url":"https://api.github.com/users/erikh/gists{/gist_id}","starred_url":"https://api.github.com/users/erikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erikh/subscriptions","organizations_url":"https://api.github.com/users/erikh/orgs","repos_url":"https://api.github.com/users/erikh/repos","events_url":"https://api.github.com/users/erikh/events{/privacy}","received_events_url":"https://api.github.com/users/erikh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-05-20T23:37:51Z","updated_at":"2016-05-21T07:58:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiv/volplugin/pulls/272","html_url":"https://github.com/contiv/volplugin/pull/272","diff_url":"https://github.com/contiv/volplugin/pull/272.diff","patch_url":"https://github.com/contiv/volplugin/pull/272.patch"},"body":"This patch implements lock free operation as a toggle. What it does internally is replace the hostname for the exclusive lock on mount to a static string; this way any host can \"take\" the lock, but it still prevents mounted volumes from existing after a Remove call is issued (it will fail if there are any mounted).\r\n\r\nIt's not quite ready; there are still some tests to write and some cleanups to do. I was hoping for a preliminary review + CI pass before I move further, as the necessary code is here already.\r\n\r\n@mapuri @unclejack @dseevr "},"comment":{"url":"https://api.github.com/repos/contiv/volplugin/issues/comments/220764854","html_url":"https://github.com/contiv/volplugin/pull/272#issuecomment-220764854","issue_url":"https://api.github.com/repos/contiv/volplugin/issues/272","id":220764854,"user":{"login":"erikh","id":18923,"avatar_url":"https://avatars.githubusercontent.com/u/18923?v=3","gravatar_id":"","url":"https://api.github.com/users/erikh","html_url":"https://github.com/erikh","followers_url":"https://api.github.com/users/erikh/followers","following_url":"https://api.github.com/users/erikh/following{/other_user}","gists_url":"https://api.github.com/users/erikh/gists{/gist_id}","starred_url":"https://api.github.com/users/erikh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erikh/subscriptions","organizations_url":"https://api.github.com/users/erikh/orgs","repos_url":"https://api.github.com/users/erikh/repos","events_url":"https://api.github.com/users/erikh/events{/privacy}","received_events_url":"https://api.github.com/users/erikh/received_events","type":"User","site_admin":false},"created_at":"2016-05-21T07:58:57Z","updated_at":"2016-05-21T07:58:57Z","body":"The NFS situation is different, many containers can take a specific \nvolume without consequence, where in the ceph case the RBD devices must \nbe locked.\n\nI still need to patch in driver hints, too.\n\n-Erik\n\nOn 20 May 2016, at 17:53, Madhav Puri wrote:\n\n> @erikh changes look good but I seem to be missing on the use case. Is \n> this only needed for the nfs backend or it this going to be a more \n> general purpose config driven by user?\n>\n> ---\n> You are receiving this because you were mentioned.\n> Reply to this email directly or view it on GitHub:\n> https://github.com/contiv/volplugin/pull/272#issuecomment-220749126\n"}}
{"actor":{"display_login":"erikh"}}