{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/16614","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/16614/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/16614/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/16614/events","html_url":"https://github.com/kubernetes/kubernetes/pull/16614","id":114350010,"number":16614,"title":"Refactor volume.Builder.IsReadOnly to GetReadWriteMode","user":{"login":"pmorie","id":366488,"avatar_url":"https://avatars.githubusercontent.com/u/366488?v=3","gravatar_id":"","url":"https://api.github.com/users/pmorie","html_url":"https://github.com/pmorie","followers_url":"https://api.github.com/users/pmorie/followers","following_url":"https://api.github.com/users/pmorie/following{/other_user}","gists_url":"https://api.github.com/users/pmorie/gists{/gist_id}","starred_url":"https://api.github.com/users/pmorie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pmorie/subscriptions","organizations_url":"https://api.github.com/users/pmorie/orgs","repos_url":"https://api.github.com/users/pmorie/repos","events_url":"https://api.github.com/users/pmorie/events{/privacy}","received_events_url":"https://api.github.com/users/pmorie/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cla:%20yes","name":"cla: yes","color":"bfe5bf"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/L","name":"size/L","color":"ee9900"}],"state":"open","locked":false,"assignee":{"login":"vishh","id":7063592,"avatar_url":"https://avatars.githubusercontent.com/u/7063592?v=3","gravatar_id":"","url":"https://api.github.com/users/vishh","html_url":"https://github.com/vishh","followers_url":"https://api.github.com/users/vishh/followers","following_url":"https://api.github.com/users/vishh/following{/other_user}","gists_url":"https://api.github.com/users/vishh/gists{/gist_id}","starred_url":"https://api.github.com/users/vishh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vishh/subscriptions","organizations_url":"https://api.github.com/users/vishh/orgs","repos_url":"https://api.github.com/users/vishh/repos","events_url":"https://api.github.com/users/vishh/events{/privacy}","received_events_url":"https://api.github.com/users/vishh/received_events","type":"User","site_admin":false},"milestone":null,"comments":12,"created_at":"2015-10-30T20:26:55Z","updated_at":"2015-11-06T04:37:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/16614","html_url":"https://github.com/kubernetes/kubernetes/pull/16614","diff_url":"https://github.com/kubernetes/kubernetes/pull/16614.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/16614.patch"},"body":"Resolves #16514."},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/154283633","html_url":"https://github.com/kubernetes/kubernetes/pull/16614#issuecomment-154283633","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/16614","id":154283633,"user":{"login":"pmorie","id":366488,"avatar_url":"https://avatars.githubusercontent.com/u/366488?v=3","gravatar_id":"","url":"https://api.github.com/users/pmorie","html_url":"https://github.com/pmorie","followers_url":"https://api.github.com/users/pmorie/followers","following_url":"https://api.github.com/users/pmorie/following{/other_user}","gists_url":"https://api.github.com/users/pmorie/gists{/gist_id}","starred_url":"https://api.github.com/users/pmorie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pmorie/subscriptions","organizations_url":"https://api.github.com/users/pmorie/orgs","repos_url":"https://api.github.com/users/pmorie/repos","events_url":"https://api.github.com/users/pmorie/events{/privacy}","received_events_url":"https://api.github.com/users/pmorie/received_events","type":"User","site_admin":false},"created_at":"2015-11-06T04:37:47Z","updated_at":"2015-11-06T04:37:47Z","body":"@vishh \r\n\r\n> The existing string type is mildly confusing already. I'm concerned that we will overload this type with those attributes in the future. I'd be less concerned if we expose an interface and use whatever representation underneath\r\n\r\nAccepted.  How about:\r\n\r\n```go\r\ntype Attributes struct {\r\n  ReadOnly bool\r\n  Managed  bool\r\n}\r\n\r\ntype Builder interface {\r\n  GetAttributes() Attributes\r\n}\r\n```\r\n\r\nUnless we want to attach behaviors to this"}}