deis / builder

Git server and application builder for Deis Workflow
https://deis.com
MIT License
40 stars 44 forks source link

fix(gitreceive/build.go): watch pods in the appropriate namespace #485

Closed mboersma closed 7 years ago

mboersma commented 7 years ago

Refs #483.

mboersma commented 7 years ago

I tested this and can confirm it fixes #483 that was broken before (for dockerfile apps as well). Thanks to @bacongobbler for isolating the problem!

My bad--this fixes things for Dockerfile-based apps, and helps buildpack-based apps get through builder, but there is a separate bug in controller that needs to be fixed now. I'll merge this and open an issue to track that.