-
Notifications
You must be signed in to change notification settings - Fork 450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Pipeline-sync] Pipeline build, started before Jenkins is deployed, gets deleted #139
Comments
@gabemontero @jimmidyson i can't imagine what is deleting these builds (but i was able to recreate). Are there any conditions under which the sync plugin would delete a build out of openshift? |
it certainly used to be that if you created the build before jenkins existed, it would simply get run when jenkins started up (though there were some issues reported about cases where it didn't run it, but it certainly didn't delete it) |
The sync plugin deletes builds if the |
Yep, the pipeline plugin does not delete builds. On Thu, Aug 11, 2016 at 10:31 AM, Jimmi Dyson notifications@github.com
|
QE has also just reported this issue here: I would have to assume the sync plugin is doing it, @jimmidyson is there some debug we can turn on or logging that would show up when the sync plugin deletes a build? |
@jimmidyson and it seems likely this is related: (old builds being deleted when the buildconfig is modified) |
It has been happening for at least a week or 2. I just got myself to properly investigate it today, sorry. Also, as @bparees mentioned, there was a problem even before Summit that the first build failed when triggered by ConfigChange, but we have removed that trigger so I am not sure if it's an issue anymore. |
Sorry about that - fixed now. |
If I create a pipeline BC and trigger it for the first time, it triggers jenkins deployment into that namespace and deletes the build. Starting it for the second time, after Jenkins is deployed, works.
Steps to reproduce:
you can see build nodejs-ex-pipeline-1
wait until Jenkins is deployed and ready
@bparees
The text was updated successfully, but these errors were encountered: