{"action":"created","issue":{"url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/688","labels_url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/688/labels{/name}","comments_url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/688/comments","events_url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/688/events","html_url":"https://github.com/GoogleCloudPlatform/gcloud-python/pull/688","id":59874731,"number":688,"title":"Removing HTTP request in _PropertyMixin.properties.","user":{"login":"dhermes","id":520669,"avatar_url":"https://avatars.githubusercontent.com/u/520669?v=3","gravatar_id":"","url":"https://api.github.com/users/dhermes","html_url":"https://github.com/dhermes","followers_url":"https://api.github.com/users/dhermes/followers","following_url":"https://api.github.com/users/dhermes/following{/other_user}","gists_url":"https://api.github.com/users/dhermes/gists{/gist_id}","starred_url":"https://api.github.com/users/dhermes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dhermes/subscriptions","organizations_url":"https://api.github.com/users/dhermes/orgs","repos_url":"https://api.github.com/users/dhermes/repos","events_url":"https://api.github.com/users/dhermes/events{/privacy}","received_events_url":"https://api.github.com/users/dhermes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/labels/cla:%20yes","name":"cla: yes","color":"009800"},{"url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/labels/storage","name":"storage","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-03-04T22:41:56Z","updated_at":"2015-03-09T18:47:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/pulls/688","html_url":"https://github.com/GoogleCloudPlatform/gcloud-python/pull/688","diff_url":"https://github.com/GoogleCloudPlatform/gcloud-python/pull/688.diff","patch_url":"https://github.com/GoogleCloudPlatform/gcloud-python/pull/688.patch"},"body":"~~**NOTE 1**: Has #683 as diffbase.~~\r\n\r\n**NOTE 2**: There are many places here where the tests just give up and call `_reload_properties()`. This should not be. We should be clear on which methods should be just returning local data and which should make an HTTP request (e.g. update to `Bucket.get_logging()` here). (I also partially point out this type of problem in #683)\r\n\r\nAlso removing dead code:\r\n\r\n- `_PropertyMixin.CUSTOM_PROPERTY_ACCESSORS` (and subclasses use of this)\r\n- `_PropertyMixin._get_property` (only consumer of `CUSTOM_PROPERTY_ACCESSORS`)"},"comment":{"url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/comments/77916968","html_url":"https://github.com/GoogleCloudPlatform/gcloud-python/pull/688#issuecomment-77916968","issue_url":"https://api.github.com/repos/GoogleCloudPlatform/gcloud-python/issues/688","id":77916968,"user":{"login":"dhermes","id":520669,"avatar_url":"https://avatars.githubusercontent.com/u/520669?v=3","gravatar_id":"","url":"https://api.github.com/users/dhermes","html_url":"https://github.com/dhermes","followers_url":"https://api.github.com/users/dhermes/followers","following_url":"https://api.github.com/users/dhermes/following{/other_user}","gists_url":"https://api.github.com/users/dhermes/gists{/gist_id}","starred_url":"https://api.github.com/users/dhermes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dhermes/subscriptions","organizations_url":"https://api.github.com/users/dhermes/orgs","repos_url":"https://api.github.com/users/dhermes/repos","events_url":"https://api.github.com/users/dhermes/events{/privacy}","received_events_url":"https://api.github.com/users/dhermes/received_events","type":"User","site_admin":false},"created_at":"2015-03-09T18:47:11Z","updated_at":"2015-03-09T18:47:11Z","body":"@tseaver After #683 the implementation is kind of in flux and moving towards a new equilibrium.\r\n\r\nI am more worried about being consistent in methods like `get_logging()` that are essentially just getters."}}