{"action":"closed","issue":{"url":"https://api.github.com/repos/libstorage/libstoragemgmt/issues/167","repository_url":"https://api.github.com/repos/libstorage/libstoragemgmt","labels_url":"https://api.github.com/repos/libstorage/libstoragemgmt/issues/167/labels{/name}","comments_url":"https://api.github.com/repos/libstorage/libstoragemgmt/issues/167/comments","events_url":"https://api.github.com/repos/libstorage/libstoragemgmt/issues/167/events","html_url":"https://github.com/libstorage/libstoragemgmt/issues/167","id":164830334,"number":167,"title":"PLUGIN_BUG(2): Expected key missing from SmartArray hpssacli output:'Disk Name'","user":{"login":"cvedel","id":1119350,"avatar_url":"https://avatars.githubusercontent.com/u/1119350?v=3","gravatar_id":"","url":"https://api.github.com/users/cvedel","html_url":"https://github.com/cvedel","followers_url":"https://api.github.com/users/cvedel/followers","following_url":"https://api.github.com/users/cvedel/following{/other_user}","gists_url":"https://api.github.com/users/cvedel/gists{/gist_id}","starred_url":"https://api.github.com/users/cvedel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cvedel/subscriptions","organizations_url":"https://api.github.com/users/cvedel/orgs","repos_url":"https://api.github.com/users/cvedel/repos","events_url":"https://api.github.com/users/cvedel/events{/privacy}","received_events_url":"https://api.github.com/users/cvedel/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/libstorage/libstoragemgmt/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":{"login":"joehandzik","id":7061812,"avatar_url":"https://avatars.githubusercontent.com/u/7061812?v=3","gravatar_id":"","url":"https://api.github.com/users/joehandzik","html_url":"https://github.com/joehandzik","followers_url":"https://api.github.com/users/joehandzik/followers","following_url":"https://api.github.com/users/joehandzik/following{/other_user}","gists_url":"https://api.github.com/users/joehandzik/gists{/gist_id}","starred_url":"https://api.github.com/users/joehandzik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joehandzik/subscriptions","organizations_url":"https://api.github.com/users/joehandzik/orgs","repos_url":"https://api.github.com/users/joehandzik/repos","events_url":"https://api.github.com/users/joehandzik/events{/privacy}","received_events_url":"https://api.github.com/users/joehandzik/received_events","type":"User","site_admin":false},"assignees":[{"login":"joehandzik","id":7061812,"avatar_url":"https://avatars.githubusercontent.com/u/7061812?v=3","gravatar_id":"","url":"https://api.github.com/users/joehandzik","html_url":"https://github.com/joehandzik","followers_url":"https://api.github.com/users/joehandzik/followers","following_url":"https://api.github.com/users/joehandzik/following{/other_user}","gists_url":"https://api.github.com/users/joehandzik/gists{/gist_id}","starred_url":"https://api.github.com/users/joehandzik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joehandzik/subscriptions","organizations_url":"https://api.github.com/users/joehandzik/orgs","repos_url":"https://api.github.com/users/joehandzik/repos","events_url":"https://api.github.com/users/joehandzik/events{/privacy}","received_events_url":"https://api.github.com/users/joehandzik/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/libstorage/libstoragemgmt/milestones/3","html_url":"https://github.com/libstorage/libstoragemgmt/milestones/1.4","labels_url":"https://api.github.com/repos/libstorage/libstoragemgmt/milestones/3/labels","id":1701701,"number":3,"title":"1.4","description":"Planed features:\r\n * Add `lsm.Client.disk_ident_led_on` for plugin.\r\n * Change simc plugin to use sqlite which could share the same state file with sim plugin.\r\n * Change layout of c_binding to group function by resources/classes/structure.\r\n * Support SMART.\r\n * Support NVMe.\r\n * Spin down local disk.\r\n * Python 3 support.","creator":{"login":"tasleson","id":2520480,"avatar_url":"https://avatars.githubusercontent.com/u/2520480?v=3","gravatar_id":"","url":"https://api.github.com/users/tasleson","html_url":"https://github.com/tasleson","followers_url":"https://api.github.com/users/tasleson/followers","following_url":"https://api.github.com/users/tasleson/following{/other_user}","gists_url":"https://api.github.com/users/tasleson/gists{/gist_id}","starred_url":"https://api.github.com/users/tasleson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tasleson/subscriptions","organizations_url":"https://api.github.com/users/tasleson/orgs","repos_url":"https://api.github.com/users/tasleson/repos","events_url":"https://api.github.com/users/tasleson/events{/privacy}","received_events_url":"https://api.github.com/users/tasleson/received_events","type":"User","site_admin":false},"open_issues":6,"closed_issues":6,"state":"open","created_at":"2016-04-12T15:48:05Z","updated_at":"2016-07-21T05:13:23Z","due_on":"2016-08-31T16:00:00Z","closed_at":null},"comments":5,"created_at":"2016-07-11T12:55:33Z","updated_at":"2016-07-21T05:13:23Z","closed_at":"2016-07-21T05:13:23Z","body":"\r\nI get this message running the following command when a logical disk is failed:\r\n\r\n`lsmcli -u hpsa:// lv`\r\n\r\nAFAICS, the output of hpssacli does not contain the 'Disk Name' field in this case so that _hp_ld_to_lsm_vol fails with a KeyError at:\r\n\r\n`device = Device.from_device_file(_CONTEXT, hp_ld['Disk Name'])` \r\n\r\nA failed logical disk should not break the listing and it should still be listed so that it can be re-enabled once the physical disk has been replaced, i.e. this case is not equivalent to a DeviceNotFoundError.\r\n\r\nThanks\r\n"}}
{"actor":{"display_login":"cathay4t"}}