{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/4853","labels_url":"https://api.github.com/repos/openshift/origin/issues/4853/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/4853/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/4853/events","html_url":"https://github.com/openshift/origin/pull/4853","id":108907971,"number":4853,"title":"Allow importing tags from ImageStreams pointing to external registries","user":{"login":"soltysh","id":576341,"avatar_url":"https://avatars.githubusercontent.com/u/576341?v=3","gravatar_id":"","url":"https://api.github.com/users/soltysh","html_url":"https://github.com/soltysh","followers_url":"https://api.github.com/users/soltysh/followers","following_url":"https://api.github.com/users/soltysh/following{/other_user}","gists_url":"https://api.github.com/users/soltysh/gists{/gist_id}","starred_url":"https://api.github.com/users/soltysh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/soltysh/subscriptions","organizations_url":"https://api.github.com/users/soltysh/orgs","repos_url":"https://api.github.com/users/soltysh/repos","events_url":"https://api.github.com/users/soltysh/events{/privacy}","received_events_url":"https://api.github.com/users/soltysh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":26,"created_at":"2015-09-29T16:34:43Z","updated_at":"2015-10-21T14:05:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/4853","html_url":"https://github.com/openshift/origin/pull/4853","diff_url":"https://github.com/openshift/origin/pull/4853.diff","patch_url":"https://github.com/openshift/origin/pull/4853.patch"},"body":"This is the initial version of improved importer, to address [card 514](https://trello.com/c/Y2MnDbaI/514-ability-to-define-imagestreamtag-pointing-to-a-different-image-registry-each).\r\n\r\n@ncdc I know it's bit messy still, I'm working on getting it DRY-ed and most importantly fully tested, still I'd appreciate first raw pass, if this makes sense. The general flow is following:\r\n\r\n1. if no tags are defined import all from default spec\r\n2. if tags are defined import only the ones defined, both from default spec and from external repos\r\n\r\n@pweil- fyi"},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/149906483","html_url":"https://github.com/openshift/origin/pull/4853#issuecomment-149906483","issue_url":"https://api.github.com/repos/openshift/origin/issues/4853","id":149906483,"user":{"login":"soltysh","id":576341,"avatar_url":"https://avatars.githubusercontent.com/u/576341?v=3","gravatar_id":"","url":"https://api.github.com/users/soltysh","html_url":"https://github.com/soltysh","followers_url":"https://api.github.com/users/soltysh/followers","following_url":"https://api.github.com/users/soltysh/following{/other_user}","gists_url":"https://api.github.com/users/soltysh/gists{/gist_id}","starred_url":"https://api.github.com/users/soltysh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/soltysh/subscriptions","organizations_url":"https://api.github.com/users/soltysh/orgs","repos_url":"https://api.github.com/users/soltysh/repos","events_url":"https://api.github.com/users/soltysh/events{/privacy}","received_events_url":"https://api.github.com/users/soltysh/received_events","type":"User","site_admin":false},"created_at":"2015-10-21T14:05:16Z","updated_at":"2015-10-21T14:05:16Z","body":"@smarterclayton I figured out what is causing this problem. Due to my changes in the import the time needed for importing the image is a bit longer than previously. This infects the flow in [registry](https://github.com/openshift/origin/blob/master/pkg/dockerregistry/server/repositorymiddleware.go#L200-L209), where if IS is missing upon ISM creation we create new IS and right after that ISM, which is getting version mismatch error: `Error creating image stream mapping: imageStream \\\"ruby-20-centos7\\\" cannot be updated: the\r\n object has been modified; please apply your changes to the latest version and try again`. I've talked with @miminar and @ncdc I'm working on adding a retry to the ISM creation, that should solve our problem."}}