{"action":"closed","issue":{"url":"https://api.github.com/repos/pachyderm/pachyderm/issues/1029","repository_url":"https://api.github.com/repos/pachyderm/pachyderm","labels_url":"https://api.github.com/repos/pachyderm/pachyderm/issues/1029/labels{/name}","comments_url":"https://api.github.com/repos/pachyderm/pachyderm/issues/1029/comments","events_url":"https://api.github.com/repos/pachyderm/pachyderm/issues/1029/events","html_url":"https://github.com/pachyderm/pachyderm/issues/1029","id":186674061,"number":1029,"title":"update-pipeline sometimes fails","user":{"login":"derekchiang","id":1535686,"avatar_url":"https://avatars.githubusercontent.com/u/1535686?v=3","gravatar_id":"","url":"https://api.github.com/users/derekchiang","html_url":"https://github.com/derekchiang","followers_url":"https://api.github.com/users/derekchiang/followers","following_url":"https://api.github.com/users/derekchiang/following{/other_user}","gists_url":"https://api.github.com/users/derekchiang/gists{/gist_id}","starred_url":"https://api.github.com/users/derekchiang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/derekchiang/subscriptions","organizations_url":"https://api.github.com/users/derekchiang/orgs","repos_url":"https://api.github.com/users/derekchiang/repos","events_url":"https://api.github.com/users/derekchiang/events{/privacy}","received_events_url":"https://api.github.com/users/derekchiang/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"derekchiang","id":1535686,"avatar_url":"https://avatars.githubusercontent.com/u/1535686?v=3","gravatar_id":"","url":"https://api.github.com/users/derekchiang","html_url":"https://github.com/derekchiang","followers_url":"https://api.github.com/users/derekchiang/followers","following_url":"https://api.github.com/users/derekchiang/following{/other_user}","gists_url":"https://api.github.com/users/derekchiang/gists{/gist_id}","starred_url":"https://api.github.com/users/derekchiang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/derekchiang/subscriptions","organizations_url":"https://api.github.com/users/derekchiang/orgs","repos_url":"https://api.github.com/users/derekchiang/repos","events_url":"https://api.github.com/users/derekchiang/events{/privacy}","received_events_url":"https://api.github.com/users/derekchiang/received_events","type":"User","site_admin":false},"assignees":[{"login":"derekchiang","id":1535686,"avatar_url":"https://avatars.githubusercontent.com/u/1535686?v=3","gravatar_id":"","url":"https://api.github.com/users/derekchiang","html_url":"https://github.com/derekchiang","followers_url":"https://api.github.com/users/derekchiang/followers","following_url":"https://api.github.com/users/derekchiang/following{/other_user}","gists_url":"https://api.github.com/users/derekchiang/gists{/gist_id}","starred_url":"https://api.github.com/users/derekchiang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/derekchiang/subscriptions","organizations_url":"https://api.github.com/users/derekchiang/orgs","repos_url":"https://api.github.com/users/derekchiang/repos","events_url":"https://api.github.com/users/derekchiang/events{/privacy}","received_events_url":"https://api.github.com/users/derekchiang/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2016-11-01T23:18:45Z","updated_at":"2016-11-04T20:00:29Z","closed_at":"2016-11-04T20:00:29Z","body":"The issue is that we have a `parentJob` function in PPS that finds the parent job for a job.  It does that by:\r\n\r\n1. Compute the input commits of the parent job\r\n2. Find the job that has the said input commits\r\n\r\nHowever, this function doesn't consider the fact that there can be more than one jobs that have the same set of input commits, which happens when you have updated a pipeline.  Therefore in step 2, `parentJob` might find more than one jobs that have the \"parent input commits\", and end up using the wrong job as the parent, which eventually leads to the new pipeline failing to start new jobs."}}
{"actor":{"display_login":"derekchiang"}}