{"action":"created","issue":{"url":"https://api.github.com/repos/lxc/lxd/issues/138","labels_url":"https://api.github.com/repos/lxc/lxd/issues/138/labels{/name}","comments_url":"https://api.github.com/repos/lxc/lxd/issues/138/comments","events_url":"https://api.github.com/repos/lxc/lxd/issues/138/events","html_url":"https://github.com/lxc/lxd/pull/138","id":54755973,"number":138,"title":"Image store","user":{"login":"kunalkushwaha","id":2758433,"avatar_url":"https://avatars.githubusercontent.com/u/2758433?v=3","gravatar_id":"","url":"https://api.github.com/users/kunalkushwaha","html_url":"https://github.com/kunalkushwaha","followers_url":"https://api.github.com/users/kunalkushwaha/followers","following_url":"https://api.github.com/users/kunalkushwaha/following{/other_user}","gists_url":"https://api.github.com/users/kunalkushwaha/gists{/gist_id}","starred_url":"https://api.github.com/users/kunalkushwaha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kunalkushwaha/subscriptions","organizations_url":"https://api.github.com/users/kunalkushwaha/orgs","repos_url":"https://api.github.com/users/kunalkushwaha/repos","events_url":"https://api.github.com/users/kunalkushwaha/events{/privacy}","received_events_url":"https://api.github.com/users/kunalkushwaha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-01-19T12:02:56Z","updated_at":"2015-01-21T12:00:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lxc/lxd/pulls/138","html_url":"https://github.com/lxc/lxd/pull/138","diff_url":"https://github.com/lxc/lxd/pull/138.diff","patch_url":"https://github.com/lxc/lxd/pull/138.patch"},"body":"A draft implementation of #7 .\r\n\r\nThis patch works for following commands.\r\n\r\n1. lxc list         <---------- default behavior.\r\n2. lxc list images:                       <----------------- list all servers from remote servers from registry server.\r\n3. lxc list images:lxc-images       <---------------- list all servers from lxc-images server.\r\n\r\n\r\nI have added all functionality of remote registry communication in registry.go file. https://github.com/kunalkushwaha/lxd/blob/image_store/registry.go\r\n\r\nProblems due to which, I am not able to integrate list functionality using client.go functionality (NewClient() and ListContainer() )\r\n1. Finger authentication fails for registry server.\r\n2. URLs at registry server is different then local server API builds the URL.\r\n\r\nListContainer() builds url as https://registry.linuxcontainers.org/1.0/list\r\n\r\nwhereas   actual url at server is https://registry.linuxcontainers.org/1.0/index.json\r\n\r\nPlease review the implementation and suggest the changes.\r\n"},"comment":{"url":"https://api.github.com/repos/lxc/lxd/issues/comments/70825421","html_url":"https://github.com/lxc/lxd/pull/138#issuecomment-70825421","issue_url":"https://api.github.com/repos/lxc/lxd/issues/138","id":70825421,"user":{"login":"kunalkushwaha","id":2758433,"avatar_url":"https://avatars.githubusercontent.com/u/2758433?v=3","gravatar_id":"","url":"https://api.github.com/users/kunalkushwaha","html_url":"https://github.com/kunalkushwaha","followers_url":"https://api.github.com/users/kunalkushwaha/followers","following_url":"https://api.github.com/users/kunalkushwaha/following{/other_user}","gists_url":"https://api.github.com/users/kunalkushwaha/gists{/gist_id}","starred_url":"https://api.github.com/users/kunalkushwaha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kunalkushwaha/subscriptions","organizations_url":"https://api.github.com/users/kunalkushwaha/orgs","repos_url":"https://api.github.com/users/kunalkushwaha/repos","events_url":"https://api.github.com/users/kunalkushwaha/events{/privacy}","received_events_url":"https://api.github.com/users/kunalkushwaha/received_events","type":"User","site_admin":false},"created_at":"2015-01-21T12:00:07Z","updated_at":"2015-01-21T12:00:07Z","body":"Thanks for detailed reply. It covers all my doubts that were there while implementation of this code.\r\n\r\nI have one implementation doubt. Is it okay, if remote server connection code is written outside client.go as it is part of registry.go in this patch?"}}