{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/21263","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/21263/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/21263/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/21263/events","html_url":"https://github.com/kubernetes/kubernetes/pull/21263","id":133724924,"number":21263,"title":"Add API Conditions for PV&PVC ","user":{"login":"markturansky","id":8001968,"avatar_url":"https://avatars.githubusercontent.com/u/8001968?v=3","gravatar_id":"","url":"https://api.github.com/users/markturansky","html_url":"https://github.com/markturansky","followers_url":"https://api.github.com/users/markturansky/followers","following_url":"https://api.github.com/users/markturansky/following{/other_user}","gists_url":"https://api.github.com/users/markturansky/gists{/gist_id}","starred_url":"https://api.github.com/users/markturansky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markturansky/subscriptions","organizations_url":"https://api.github.com/users/markturansky/orgs","repos_url":"https://api.github.com/users/markturansky/repos","events_url":"https://api.github.com/users/markturansky/events{/privacy}","received_events_url":"https://api.github.com/users/markturansky/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/area/api","name":"area/api","color":"e1ed21"},{"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/kind/api-change","name":"kind/api-change","color":"c7def8"},{"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/XXL","name":"size/XXL","color":"ee0000"}],"state":"open","locked":false,"assignee":{"login":"thockin","id":5595220,"avatar_url":"https://avatars.githubusercontent.com/u/5595220?v=3","gravatar_id":"","url":"https://api.github.com/users/thockin","html_url":"https://github.com/thockin","followers_url":"https://api.github.com/users/thockin/followers","following_url":"https://api.github.com/users/thockin/following{/other_user}","gists_url":"https://api.github.com/users/thockin/gists{/gist_id}","starred_url":"https://api.github.com/users/thockin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thockin/subscriptions","organizations_url":"https://api.github.com/users/thockin/orgs","repos_url":"https://api.github.com/users/thockin/repos","events_url":"https://api.github.com/users/thockin/events{/privacy}","received_events_url":"https://api.github.com/users/thockin/received_events","type":"User","site_admin":false},"milestone":null,"comments":22,"created_at":"2016-02-15T14:24:16Z","updated_at":"2016-02-29T12:56:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/21263","html_url":"https://github.com/kubernetes/kubernetes/pull/21263","diff_url":"https://github.com/kubernetes/kubernetes/pull/21263.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/21263.patch"},"body":"Adds `PersistentVolumeCondition` and `PersistentVolumeClaimCondition`.\r\n\r\nVolume and Claim both have a Bound condition that wants to be satisfied (true).  Volume also has Provisioned and Recycled.\r\n\r\nBound - always wants to be bound, either to a volume or claim\r\nProvisioned - is the state of the volume's provisioning. Status=False for volumes that require provisioning.\r\nRecycled - is the state of the volume's Reclaim policy after the volume is released of its claim.  Volumes can have a Reclaim policy of Recycle/Delete/Retain.  \r\n\r\nI have another PR that uses Conditions to drive a newly rewritten and clean PVController.\r\n\r\n@saad-ali @thockin @bgrant0607 @kubernetes/rh-storage "},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/190198627","html_url":"https://github.com/kubernetes/kubernetes/pull/21263#issuecomment-190198627","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/21263","id":190198627,"user":{"login":"jsafrane","id":1745006,"avatar_url":"https://avatars.githubusercontent.com/u/1745006?v=3","gravatar_id":"","url":"https://api.github.com/users/jsafrane","html_url":"https://github.com/jsafrane","followers_url":"https://api.github.com/users/jsafrane/followers","following_url":"https://api.github.com/users/jsafrane/following{/other_user}","gists_url":"https://api.github.com/users/jsafrane/gists{/gist_id}","starred_url":"https://api.github.com/users/jsafrane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsafrane/subscriptions","organizations_url":"https://api.github.com/users/jsafrane/orgs","repos_url":"https://api.github.com/users/jsafrane/repos","events_url":"https://api.github.com/users/jsafrane/events{/privacy}","received_events_url":"https://api.github.com/users/jsafrane/received_events","type":"User","site_admin":false},"created_at":"2016-02-29T12:56:41Z","updated_at":"2016-02-29T12:56:41Z","body":"I guess that marking Volume/Claim Status.Phase as deprecated is for subsequent PRs. when the conditions are actually implemented. LGTM if so."}}