Hi all, As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!). In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state. Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always. Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :) https://github.com/pharo-project/pharo/pull/184 You can follow up all the things we did in the issue also: https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70 As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea). Enjoy, Guille and Pablo -- Guille Polito Research Engineer French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr> *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13
And for the lazy ones, here is the link to the build (that can be also accessed from the PR) https://ci.inria.fr/pharo-ci-jenkins2/job/test%20multibranch%20pipeline/job/... On Sun, Aug 6, 2017 at 11:00 AM, Guillermo Polito <guillermopolito@gmail.com
wrote:
Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer
French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr>
*Web:* *http://guillep.github.io* <http://guillep.github.io>
*Phone: *+33 06 52 70 66 13 <+33%206%2052%2070%2066%2013>
-- Guille Polito Research Engineer French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr> *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13
Great job! Thank you very much. -- Pavel 2017-08-06 11:00 GMT+02:00 Guillermo Polito <guillermopolito@gmail.com>:
Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer
French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr>
*Web:* *http://guillep.github.io* <http://guillep.github.io>
*Phone: *+33 06 52 70 66 13 <+33%206%2052%2070%2066%2013>
Great guys, this is the way to go. On Sun, Aug 6, 2017 at 6:00 AM, Guillermo Polito <guillermopolito@gmail.com> wrote:
Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer
French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr>
*Web:* *http://guillep.github.io* <http://guillep.github.io>
*Phone: *+33 06 52 70 66 13
Excellent work! This is the kind of work that does not get much praise, but that provides the solid foundation we all need. Thanks! Doru -- www.tudorgirba.com "Every thing has its own flow"
On 6 Aug 2017, at 13:46, Gabriel Cotelli <g.cotelli@gmail.com> wrote:
Great guys, this is the way to go.
On Sun, Aug 6, 2017 at 6:00 AM, Guillermo Polito <guillermopolito@gmail.com> wrote: Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer French National Center for Scientific Research - http://www.cnrs.fr
Web: http://guillep.github.io Phone: +33 06 52 70 66 13
Yes this is great. Now I will be rerunning the pending fix and integrate the green ones. On Sun, Aug 6, 2017 at 11:00 AM, Guillermo Polito <guillermopolito@gmail.com
wrote:
Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer
French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr>
*Web:* *http://guillep.github.io* <http://guillep.github.io>
*Phone: *+33 06 52 70 66 13 <+33%206%2052%2070%2066%2013>
On Sun, Aug 6, 2017 at 6:00 AM, Guillermo Polito <guillermopolito@gmail.com> wrote:
Hi all,
As you all know, we are now building from github. Integrations are made through pull requests. And pull requests are validated using jenkins (that is back again, Yay!).
In this scenario, pull requests validations in github differentiate only between three states: Pased (green) and not passed (red). This of course conflicts with the three-state jobs of jenkins. By default jenkins transforms failing tests (yellow) to a red state. This means that we lose the extra information provided by the intermediate state.
Now, because of this, in order to really differentiate good builds from bad builds, we need to have green builds :). Always.
Yesterday we worked with Pablo on that and now we have a PR that fixes all tests and is green :)
https://github.com/pharo-project/pharo/pull/184
You can follow up all the things we did in the issue also:
https://pharo.fogbugz.com/f/cases/20288/Fix-failing-tests-in-pharo-70
As a summary, we could fix most tests by just touching some code here and there. Some of them were portability issues, some other encoding in the build slaves and so on. The only one we could not fix was a problem cause by a side effect of QualityAssistant-Tests, so we skipped it. Those two tests should be rewritten to not have side effects on other tests (In this case, Epicea).
Updates? anything has to be rewriten in the Epicea tests?
Enjoy, Guille and Pablo
--
Guille Polito
Research Engineer
French National Center for Scientific Research - *http://www.cnrs.fr* <http://www.cnrs.fr>
*Web:* *http://guillep.github.io* <http://guillep.github.io>
*Phone: *+33 06 52 70 66 13
participants (6)
-
Gabriel Cotelli -
Guillermo Polito -
Martin Dias -
Pavel Krivanek -
Stephane Ducasse -
Tudor Girba