{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/ceph/issues/8983","repository_url":"https://api.github.com/repos/ceph/ceph","labels_url":"https://api.github.com/repos/ceph/ceph/issues/8983/labels{/name}","comments_url":"https://api.github.com/repos/ceph/ceph/issues/8983/comments","events_url":"https://api.github.com/repos/ceph/ceph/issues/8983/events","html_url":"https://github.com/ceph/ceph/pull/8983","id":153571977,"number":8983,"title":"DNM: Improve list-inconsistent json format","user":{"login":"dzafman","id":2467610,"avatar_url":"https://avatars.githubusercontent.com/u/2467610?v=3","gravatar_id":"","url":"https://api.github.com/users/dzafman","html_url":"https://github.com/dzafman","followers_url":"https://api.github.com/users/dzafman/followers","following_url":"https://api.github.com/users/dzafman/following{/other_user}","gists_url":"https://api.github.com/users/dzafman/gists{/gist_id}","starred_url":"https://api.github.com/users/dzafman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dzafman/subscriptions","organizations_url":"https://api.github.com/users/dzafman/orgs","repos_url":"https://api.github.com/users/dzafman/repos","events_url":"https://api.github.com/users/dzafman/events{/privacy}","received_events_url":"https://api.github.com/users/dzafman/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ceph/ceph/labels/pending-discussion","name":"pending-discussion","color":"f7c6c7"},{"url":"https://api.github.com/repos/ceph/ceph/labels/wip-zafman-testing","name":"wip-zafman-testing","color":"d4c5f9"}],"state":"open","locked":false,"assignee":{"login":"tchaikov","id":59071,"avatar_url":"https://avatars.githubusercontent.com/u/59071?v=3","gravatar_id":"","url":"https://api.github.com/users/tchaikov","html_url":"https://github.com/tchaikov","followers_url":"https://api.github.com/users/tchaikov/followers","following_url":"https://api.github.com/users/tchaikov/following{/other_user}","gists_url":"https://api.github.com/users/tchaikov/gists{/gist_id}","starred_url":"https://api.github.com/users/tchaikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tchaikov/subscriptions","organizations_url":"https://api.github.com/users/tchaikov/orgs","repos_url":"https://api.github.com/users/tchaikov/repos","events_url":"https://api.github.com/users/tchaikov/events{/privacy}","received_events_url":"https://api.github.com/users/tchaikov/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2016-05-07T04:02:04Z","updated_at":"2016-05-16T20:03:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/ceph/pulls/8983","html_url":"https://github.com/ceph/ceph/pull/8983","diff_url":"https://github.com/ceph/ceph/pull/8983.diff","patch_url":"https://github.com/ceph/ceph/pull/8983.patch"},"body":"\r\nThis is on top of pull another pull request.\r\n\r\nWe should discuss what the output format looks like."},"comment":{"url":"https://api.github.com/repos/ceph/ceph/issues/comments/219530835","html_url":"https://github.com/ceph/ceph/pull/8983#issuecomment-219530835","issue_url":"https://api.github.com/repos/ceph/ceph/issues/8983","id":219530835,"user":{"login":"dzafman","id":2467610,"avatar_url":"https://avatars.githubusercontent.com/u/2467610?v=3","gravatar_id":"","url":"https://api.github.com/users/dzafman","html_url":"https://github.com/dzafman","followers_url":"https://api.github.com/users/dzafman/followers","following_url":"https://api.github.com/users/dzafman/following{/other_user}","gists_url":"https://api.github.com/users/dzafman/gists{/gist_id}","starred_url":"https://api.github.com/users/dzafman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dzafman/subscriptions","organizations_url":"https://api.github.com/users/dzafman/orgs","repos_url":"https://api.github.com/users/dzafman/repos","events_url":"https://api.github.com/users/dzafman/events{/privacy}","received_events_url":"https://api.github.com/users/dzafman/received_events","type":"User","site_admin":false},"created_at":"2016-05-16T20:03:02Z","updated_at":"2016-05-16T20:03:02Z","body":"Sample of what the output changes.\r\n\r\nPrevious format:\r\n<pre> \r\n        {\r\n            \"name\": \"obj5\",\r\n            \"nspace\": \"\",\r\n            \"locator\": \"\",\r\n            \"snap\": \"0x00000007\",\r\n            \"oi_attr_missing\": true,\r\n            \"headless\": true\r\n        },\r\n        {\r\n            \"name\": \"obj2\",\r\n            \"nspace\": \"\",\r\n            \"locator\": \"\",\r\n            \"snap\": \"snapdir\",\r\n            \"ss_attr_missing\": true,\r\n            \"extra_clones\": true,\r\n            \"extra clones\": [\r\n                7,\r\n                4\r\n            ]\r\n        },\r\n</pre>\r\n\r\nNew format:\r\nNotice the snap id is just a decimal value.  The errors are an array of strings instead of each flag shown as true.\r\n<pre>\r\n        {\r\n            \"name\": \"obj5\",\r\n            \"nspace\": \"\",\r\n            \"locator\": \"\",\r\n            \"snap\": 7,\r\n            \"errors\": [\r\n                \"oi_attr_missing\",\r\n                \"headless\"\r\n            ]\r\n        },\r\n       {\r\n            \"name\": \"obj2\",\r\n            \"nspace\": \"\",\r\n            \"locator\": \"\",\r\n            \"snap\": \"snapdir\",\r\n            \"errors\": [\r\n                \"ss_attr_missing\",\r\n                \"extra_clones\"\r\n            ],\r\n            \"extra clones\": [\r\n                7,\r\n                4\r\n            ]\r\n        },\r\n</pre>"}}