{"action":"created","issue":{"url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/212","labels_url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/212/labels{/name}","comments_url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/212/comments","events_url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/212/events","html_url":"https://github.com/GENI-NSF/geni-ch/issues/212","id":78377195,"number":212,"title":"PGCH gives error looking up keys when you have none","user":{"login":"ahelsing","id":12136824,"avatar_url":"https://avatars.githubusercontent.com/u/12136824?v=3","gravatar_id":"","url":"https://api.github.com/users/ahelsing","html_url":"https://github.com/ahelsing","followers_url":"https://api.github.com/users/ahelsing/followers","following_url":"https://api.github.com/users/ahelsing/following{/other_user}","gists_url":"https://api.github.com/users/ahelsing/gists{/gist_id}","starred_url":"https://api.github.com/users/ahelsing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahelsing/subscriptions","organizations_url":"https://api.github.com/users/ahelsing/orgs","repos_url":"https://api.github.com/users/ahelsing/repos","events_url":"https://api.github.com/users/ahelsing/events{/privacy}","received_events_url":"https://api.github.com/users/ahelsing/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"ahelsing","id":12136824,"avatar_url":"https://avatars.githubusercontent.com/u/12136824?v=3","gravatar_id":"","url":"https://api.github.com/users/ahelsing","html_url":"https://github.com/ahelsing","followers_url":"https://api.github.com/users/ahelsing/followers","following_url":"https://api.github.com/users/ahelsing/following{/other_user}","gists_url":"https://api.github.com/users/ahelsing/gists{/gist_id}","starred_url":"https://api.github.com/users/ahelsing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahelsing/subscriptions","organizations_url":"https://api.github.com/users/ahelsing/orgs","repos_url":"https://api.github.com/users/ahelsing/repos","events_url":"https://api.github.com/users/ahelsing/events{/privacy}","received_events_url":"https://api.github.com/users/ahelsing/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/GENI-NSF/geni-ch/milestones/14","html_url":"https://github.com/GENI-NSF/geni-ch/milestones/1.4","labels_url":"https://api.github.com/repos/GENI-NSF/geni-ch/milestones/14/labels","id":1119892,"number":14,"title":"1.4","description":null,"creator":{"login":"ahelsing","id":12136824,"avatar_url":"https://avatars.githubusercontent.com/u/12136824?v=3","gravatar_id":"","url":"https://api.github.com/users/ahelsing","html_url":"https://github.com/ahelsing","followers_url":"https://api.github.com/users/ahelsing/followers","following_url":"https://api.github.com/users/ahelsing/following{/other_user}","gists_url":"https://api.github.com/users/ahelsing/gists{/gist_id}","starred_url":"https://api.github.com/users/ahelsing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahelsing/subscriptions","organizations_url":"https://api.github.com/users/ahelsing/orgs","repos_url":"https://api.github.com/users/ahelsing/repos","events_url":"https://api.github.com/users/ahelsing/events{/privacy}","received_events_url":"https://api.github.com/users/ahelsing/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":4,"state":"open","created_at":"2015-05-20T05:06:44Z","updated_at":"2015-05-20T05:18:05Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-05-20T05:18:05Z","updated_at":"2015-05-20T05:18:16Z","closed_at":null,"body":"If a user has no SSH keys, PGCH gives a noisy error, EG:\r\n```\r\n01/19/2014 12:48:38:WARNING :chapi:PGCH: piva.1532581@studenti.uniroma1.it: GetKeys: No entry for member urn:publicid:IDN+ch.geni.net+user+piva1532 from lookup_keys\r\n01/19/2014 12:48:38:ERROR   :amsoil:[pgchv1] [SERVER] SERVER_ERROR (GetKeys: No entry for member urn:publicid:IDN+ch.geni.net+user+piva1532 from lookup_keys)\r\n01/19/2014 12:48:38:ERROR   :amsoil:[pgchv1]   File \"/usr/share/geni-ch/chapi/AMsoil-gpo-0.3.2/src/plugins/pgch/PGCH.py\", line 100, in GetKeys\r\n    args)\r\n\r\n  File \"/usr/share/geni-ch/chapi/AMsoil-gpo-0.3.2/src/plugins/pgch/PGCH.py\", line 630, in GetKeys\r\n    raise CHAPIv1ServerError(msg)\r\n\r\n01/19/2014 12:48:38:INFO    :chapi:PGCH: piva.1532581@studenti.uniroma1.it: Result from GetKeys: {'output': '[SERVER] SERVER_ERROR (GetKeys: No entry for member urn:publicid:IDN+ch.geni.net+user+piva1532 from lookup_keys)', 'code': 101, 'value': None}\r\n```\r\n\r\nQuiet that down to just log a warning and return an empty list of keys\n\n _Imported from trac ticket [#212](http://trac.gpolab.bbn.com/chapi/ticket/212),  created by ahelsing on 01-22-2014 at 10:37, last modified: 01-28-2014 at 10:04_\n"},"comment":{"url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/comments/103763215","html_url":"https://github.com/GENI-NSF/geni-ch/issues/212#issuecomment-103763215","issue_url":"https://api.github.com/repos/GENI-NSF/geni-ch/issues/212","id":103763215,"user":{"login":"ahelsing","id":12136824,"avatar_url":"https://avatars.githubusercontent.com/u/12136824?v=3","gravatar_id":"","url":"https://api.github.com/users/ahelsing","html_url":"https://github.com/ahelsing","followers_url":"https://api.github.com/users/ahelsing/followers","following_url":"https://api.github.com/users/ahelsing/following{/other_user}","gists_url":"https://api.github.com/users/ahelsing/gists{/gist_id}","starred_url":"https://api.github.com/users/ahelsing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ahelsing/subscriptions","organizations_url":"https://api.github.com/users/ahelsing/orgs","repos_url":"https://api.github.com/users/ahelsing/repos","events_url":"https://api.github.com/users/ahelsing/events{/privacy}","received_events_url":"https://api.github.com/users/ahelsing/received_events","type":"User","site_admin":false},"created_at":"2015-05-20T05:18:16Z","updated_at":"2015-05-20T05:18:16Z","body":"To test: Do omni listkeys when you have no SSH keys registered. Look to see a nicer error message in omni, and no ugly error in the chapi logs. The fix commented out raising a CHAPIServerError, tweaked the error message, and made sure the return is an empty list of keys when there are no keys.\r\n\n\n_Trac comment by ahelsing on 01-28-2014 at 10:04_\n"}}