<div dir="ltr"><div><div>How about per PR build? Do they run in a production or dev setting?<br><br></div>Warm Regards,<br></div>Nimisha<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 14, 2017 at 12:15 AM, Max Andersen <span dir="ltr"><<a href="mailto:manderse@redhat.com" target="_blank">manderse@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Great you resolved the issue Michael!<br>
<br>
One thought about build being published when failing. Maybe it has same issue as devdoc had and script doesn't return the right return code.<br>
<br>
<br>
<br>
/max<br>
<a href="http://about.me/maxandersen" rel="noreferrer" target="_blank">http://about.me/maxandersen</a><br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> On 13 Jan 2017, at 09:58, Michael Kleinhenz <<a href="mailto:kleinhenz@redhat.com">kleinhenz@redhat.com</a>> wrote:<br>
><br>
> Hi Team,<br>
><br>
> the <a href="http://demo.almighty.io" rel="noreferrer" target="_blank">demo.almighty.io</a> service went down yesterday for a couple of<br>
> minutes. The problem is now fixed and the server is back up, but I<br>
> found two underlying, more general problems. Here's my findings in<br>
> bullet points:<br>
><br>
> - We had a problem on code level yesterday in master. That problem<br>
> should have been easily<br>
>  detected by our test chain. But it was not.<br>
><br>
> - currently, the functional tests are run against an intermediate<br>
> build. We're using the<br>
>  WebPack dev/debug function for that. Only after they succeed, the<br>
> final build is done<br>
>  and the output artifact created. The reason is that we don't need a<br>
> test-only http-server setup<br>
>  as WebPack already includes that.<br>
><br>
> - I found out that the error caused by the problem in master was<br>
> apperaring ONLY when<br>
>  doing a production build, NOT when doing a debug/dev build. The<br>
> result was, that the problem<br>
>  was completely invisible in a dev environment and for the functional<br>
> tests. I don't know yet, why<br>
>  the dev and prod build done by WebPack are behaving differently at a<br>
> such elementary level.<br>
><br>
> - The above WebPack mystery would have not affected demo.almighty<br>
> without a second problem:<br>
>  the build was deployed to demo.almighty althought it failed<br>
> building. Resulting in just index.html<br>
>  deployed to the server (!).<br>
><br>
> - I suspect that errors appearing inside the build container are<br>
> somehow not propagated to the<br>
>  outside Jenkins build. They may just be consumed somewhere.<br>
><br>
> - The second problem (build errors do not get propagated) is the more<br>
> severe one and also may<br>
>  affect other builds in other projects as we basically just re-used that setup.<br>
><br>
> Action points:<br>
><br>
> - We need to check if we can run the functional tests against the<br>
> production build instead of the<br>
>  intermediate build.<br>
>  -> <a href="https://github.com/fabric8io/fabric8-planner/issues/741" rel="noreferrer" target="_blank">https://github.com/fabric8io/<wbr>fabric8-planner/issues/741</a><br>
><br>
> - We need to check the build process to find out why errors are not<br>
> propagated to the outside<br>
>  world (or being recognized as a fail by Jenkins).<br>
>  -> <a href="https://github.com/fabric8io/fabric8-planner/issues/742" rel="noreferrer" target="_blank">https://github.com/fabric8io/<wbr>fabric8-planner/issues/742</a><br>
><br>
> - We need to fix the actual code problems that triggered the above.<br>
>  -> <a href="https://github.com/fabric8io/fabric8-planner/issues/732" rel="noreferrer" target="_blank">https://github.com/fabric8io/<wbr>fabric8-planner/issues/732</a><br>
><br>
> -- Michael<br>
><br>
> --<br>
> Michael Kleinhenz<br>
> Principal Software Engineer<br>
><br>
> Red Hat Deutschland GmbH<br>
> Werner-von-Siemens-Ring 14<br>
> 85630 Grasbrunn<br>
> Germany<br>
><br>
> RED HAT | TRIED. TESTED. TRUSTED.<br>
> Red Hat GmbH, <a href="http://www.de.redhat.com" rel="noreferrer" target="_blank">www.de.redhat.com</a>,<br>
> Registered seat: Grasbrunn, Commercial register: Amtsgericht München,<br>
> HRB 153243,<br>
> Managing Directors: Paul Argiry, Charles Cachera, Michael Cunningham,<br>
> Michael O'Neill<br>
><br>
> ______________________________<wbr>_________________<br>
> almighty-public mailing list<br>
> <a href="mailto:almighty-public@redhat.com">almighty-public@redhat.com</a><br>
> <a href="https://www.redhat.com/mailman/listinfo/almighty-public" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/almighty-<wbr>public</a><br>
><br>
><br>
<br>
______________________________<wbr>_________________<br>
almighty-public mailing list<br>
<a href="mailto:almighty-public@redhat.com">almighty-public@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/almighty-public" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/almighty-<wbr>public</a><br>
</div></div></blockquote></div><br></div>