{"action":"closed","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/4127","labels_url":"https://api.github.com/repos/openshift/origin/issues/4127/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/4127/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/4127/events","html_url":"https://github.com/openshift/origin/issues/4127","id":100609121,"number":4127,"title":"s2i incremental builds are broken","user":{"login":"bparees","id":4974046,"avatar_url":"https://avatars.githubusercontent.com/u/4974046?v=3","gravatar_id":"","url":"https://api.github.com/users/bparees","html_url":"https://github.com/bparees","followers_url":"https://api.github.com/users/bparees/followers","following_url":"https://api.github.com/users/bparees/following{/other_user}","gists_url":"https://api.github.com/users/bparees/gists{/gist_id}","starred_url":"https://api.github.com/users/bparees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bparees/subscriptions","organizations_url":"https://api.github.com/users/bparees/orgs","repos_url":"https://api.github.com/users/bparees/repos","events_url":"https://api.github.com/users/bparees/events{/privacy}","received_events_url":"https://api.github.com/users/bparees/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openshift/origin/labels/component/build","name":"component/build","color":"0052cc"},{"url":"https://api.github.com/repos/openshift/origin/labels/priority/P1","name":"priority/P1","color":"eb6420"}],"state":"closed","locked":false,"assignee":{"login":"bparees","id":4974046,"avatar_url":"https://avatars.githubusercontent.com/u/4974046?v=3","gravatar_id":"","url":"https://api.github.com/users/bparees","html_url":"https://github.com/bparees","followers_url":"https://api.github.com/users/bparees/followers","following_url":"https://api.github.com/users/bparees/following{/other_user}","gists_url":"https://api.github.com/users/bparees/gists{/gist_id}","starred_url":"https://api.github.com/users/bparees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bparees/subscriptions","organizations_url":"https://api.github.com/users/bparees/orgs","repos_url":"https://api.github.com/users/bparees/repos","events_url":"https://api.github.com/users/bparees/events{/privacy}","received_events_url":"https://api.github.com/users/bparees/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-08-12T18:46:00Z","updated_at":"2015-08-20T22:28:52Z","closed_at":"2015-08-20T22:28:52Z","body":"1) we don't pull the old version of the application image to the node unless forcePull is true.  We should always pull the old version of the image when incremental is true.  @mfojtik @soltysh @gabemontero \r\n2) we don't use credentials when pulling the old version of the image.  We need to use the push credentials (because we're pulling the image from the same place we're later going to push to) when we do the pull.\r\n\r\n@ncdc fyi.\r\n"}}