On Mon, 31 Dec 2018 at 02:22, Levente Uzonyi leves@caesar.elte.hu wrote:
Thanks Ben. I don't know what image version the CI uses,
The log provides the following info... Squeak5.1 latest update: #16551 Current Change Set: Unnamed1 Image format 6521 (32 bit)
but Nebraska-ul.54 in the Trunk should fix the issue with StringSocketTestCase.
Good to know, thanks Levente. Would Squeak 5.2 cover Nebraska-ul.54 ? It seems SQUEAK_VERSION could be updated here... https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI...
However SmalltalkCI README indicates it doesn't support 32-bit of latest images for Squeak or Pharo... https://github.com/hpi-swa/smalltalkCI#list-of-supported-images but is that info current? These files seem to indicate support... https://github.com/hpi-swa/smalltalkCI/blob/master/squeak/run.sh https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
@vm-dev... I find the specification of the test image too implicit here...
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI... For example, its hard to tell at a glance whether lowcode testing is enabled/disabled. Could the test image be explicitly defined in travis.yml ? (I'll submit a concept PR for review)
cheers -ben
Levente
On Sun, 30 Dec 2018, Ben Coman wrote:
An Opensmalltalk-vm Travis build job https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/473535608 looks like it failed with a socket error, probably external network. A bundle of failures look like this...
SocketStreamTest
7041f6_0fb1 ✗ #testClassComment (10003ms) TestFailure: Test timed out Process>>pvtSignal:list: [] in DelayWaitTimeout>>wait
<snip>
which is okay, but the following three with similar >10000ms timeout did not produce "TestFailure: Test timed out " but instead "MessageNotUnderstood" errors.
StringSocketTestCase
c43ad5_0fb1 ✗ #testBasics (10001ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
e360b0_0fb1 ✗ #testBogusInput1 (10000ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
e4f711_0fb1 ✗ #testBogusInput2 (10002ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
I'm not currently running Squeak, so I'm just reporting was I see in the
CI log
in case anyone was interested in touching-up such a corner case. Is it preferable to report on Mantis? Or not important :)
cheers -ben
Hi Ben,
On Mon, 31 Dec 2018, Ben Coman wrote:
On Mon, 31 Dec 2018 at 02:22, Levente Uzonyi leves@caesar.elte.hu wrote: Thanks Ben. I don't know what image version the CI uses,
The log provides the following info... Squeak5.1 latest update: #16551 Current Change Set: Unnamed1 Image format 6521 (32 bit) but Nebraska-ul.54 in the Trunk should fix the issue with StringSocketTestCase.
Good to know, thanks Levente. Would Squeak 5.2 cover Nebraska-ul.54 ?
No, it's in the Trunk, so I presume Squeak(64)-trunk would contain the fix. It's possible to backport the fix to 5.2, but I don't now if the CI script updates the image before running the tests.
Levente
It seems SQUEAK_VERSION could be updated here... https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI...
However SmalltalkCI README indicates it doesn't support 32-bit of latest images for Squeak or Pharo... https://github.com/hpi-swa/smalltalkCI#list-of-supported-images but is that info current? These files seem to indicate support... https://github.com/hpi-swa/smalltalkCI/blob/master/squeak/run.sh https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
@vm-dev... I find the specification of the test image too implicit here... https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI... For example, its hard to tell at a glance whether lowcode testing is enabled/disabled. Could the test image be explicitly defined in travis.yml ? (I'll submit a concept PR for review)
cheers -ben
Levente On Sun, 30 Dec 2018, Ben Coman wrote: > An Opensmalltalk-vm Travis build job > https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/473535608 > looks like it failed with a socket error, probably external network. > A bundle of failures look like this... > > SocketStreamTest > > 7041f6_0fb1 > ✗ #testClassComment (10003ms) > TestFailure: Test timed out > Process>>pvtSignal:list: > [] in DelayWaitTimeout>>wait > <snip> > > which is okay, but the following three with similar >10000ms timeout > did not produce "TestFailure: Test timed out " > but instead "MessageNotUnderstood" errors. > > > StringSocketTestCase > > c43ad5_0fb1 > ✗ #testBasics (10001ms) > MessageNotUnderstood: UndefinedObject>>destroy > UndefinedObject(Object)>>doesNotUnderstand: #destroy > StringSocketTestCase>>tearDown > > e360b0_0fb1 > ✗ #testBogusInput1 (10000ms) > MessageNotUnderstood: UndefinedObject>>destroy > UndefinedObject(Object)>>doesNotUnderstand: #destroy > StringSocketTestCase>>tearDown > > e4f711_0fb1 > ✗ #testBogusInput2 (10002ms) > MessageNotUnderstood: UndefinedObject>>destroy > UndefinedObject(Object)>>doesNotUnderstand: #destroy > StringSocketTestCase>>tearDown > > > I'm not currently running Squeak, so I'm just reporting was I see in the CI log > in case anyone was interested in touching-up such a corner case. > Is it preferable to report on Mantis? Or not important :) > > cheers -ben > >
On Mon, 31 Dec 2018 at 12:07 am, Ben Coman btc@openinworld.com wrote:
On Mon, 31 Dec 2018 at 02:22, Levente Uzonyi leves@caesar.elte.hu wrote:
Thanks Ben. I don't know what image version the CI uses,
The log provides the following info... Squeak5.1 latest update: #16551 Current Change Set: Unnamed1 Image format 6521 (32 bit)
but Nebraska-ul.54 in the Trunk should fix the issue with StringSocketTestCase.
Good to know, thanks Levente. Would Squeak 5.2 cover Nebraska-ul.54 ? It seems SQUEAK_VERSION could be updated here...
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI...
However SmalltalkCI README indicates it doesn't support 32-bit of latest images for Squeak or Pharo... https://github.com/hpi-swa/smalltalkCI#list-of-supported-images but is that info current? These files seem to indicate support... https://github.com/hpi-swa/smalltalkCI/blob/master/squeak/run.sh https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
Squeak[64]-trunk means Squeak-trunk (32bit) and Squeak64-trunk are supported. Eventually, we want to introduce Squeak32-trunk as well and change the default to 64bit. The same rule applies for Pharo. Maybe open an issue against smalltalkCI, so we don't forget to get rid of this confusion?
https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
@vm-dev... I find the specification of the test image too implicit here...
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI... For example, its hard to tell at a glance whether lowcode testing is enabled/disabled. Could the test image be explicitly defined in travis.yml ? (I'll submit a concept PR for review)
Please do, happy to review a PR. The current version is still the very first version, I'm sure it can use some improvements :)
Fabio
cheers -ben
Levente
On Sun, 30 Dec 2018, Ben Coman wrote:
An Opensmalltalk-vm Travis build job https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/473535608 looks like it failed with a socket error, probably external network. A bundle of failures look like this...
SocketStreamTest
7041f6_0fb1 ✗ #testClassComment (10003ms) TestFailure: Test timed out Process>>pvtSignal:list: [] in DelayWaitTimeout>>wait
<snip>
which is okay, but the following three with similar >10000ms timeout did not produce "TestFailure: Test timed out " but instead "MessageNotUnderstood" errors.
StringSocketTestCase
c43ad5_0fb1 ✗ #testBasics (10001ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
e360b0_0fb1 ✗ #testBogusInput1 (10000ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
e4f711_0fb1 ✗ #testBogusInput2 (10002ms) MessageNotUnderstood: UndefinedObject>>destroy UndefinedObject(Object)>>doesNotUnderstand: #destroy StringSocketTestCase>>tearDown
I'm not currently running Squeak, so I'm just reporting was I see in
the CI log
in case anyone was interested in touching-up such a corner case. Is it preferable to report on Mantis? Or not important :)
cheers -ben
On Mon, 31 Dec 2018 at 07:20, Fabio Niephaus lists@fniephaus.com wrote:
On Mon, 31 Dec 2018 at 12:07 am, Ben Coman btc@openinworld.com wrote:
However SmalltalkCI README indicates it doesn't support 32-bit of latest images for Squeak or Pharo... https://github.com/hpi-swa/smalltalkCI#list-of-supported-images but is that info current? These files seem to indicate support... https://github.com/hpi-swa/smalltalkCI/blob/master/squeak/run.sh https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
Squeak[64]-trunk means Squeak-trunk (32bit) and Squeak64-trunk are supported. Eventually, we want to introduce Squeak32-trunk as well and change the default to 64bit. The same rule applies for Pharo. Maybe open an issue against smalltalkCI, so we don't forget to get rid of this confusion?
https://github.com/hpi-swa/smalltalkCI/issues/407
@vm-dev... I find the specification of the test image too implicit here... https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI... For example, its hard to tell at a glance whether lowcode testing is enabled/disabled. Could the test image be explicitly defined in travis.yml ? (I'll submit a concept PR for review)
Please do, happy to review a PR. The current version is still the very first version, I'm sure it can use some improvements :)
cool.
btw, are we indeed not testing VMs on Windows? https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/.appveyor.yml#L12...
cheers -ben
On Mon, 31 Dec 2018 at 07:45, Ben Coman btc@openinworld.com wrote:
On Mon, 31 Dec 2018 at 07:20, Fabio Niephaus lists@fniephaus.com wrote:
On Mon, 31 Dec 2018 at 12:07 am, Ben Coman btc@openinworld.com wrote:
I find the specification of the test image too implicit here...
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI...
For example, its hard to tell at a glance whether lowcode testing is
enabled/disabled.
Could the test image be explicitly defined in travis.yml ? (I'll submit
a concept PR for review)
Please do, happy to review a PR. The current version is still the very
first version, I'm sure it can use some improvements :)
Here are my proposed changes... https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/331
Here it is a sample log of the first job... https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/473714437#L441 https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/473714437#L9438
So the goal was that you can hover over the environment variables to easily review the test configuration here... https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/builds/473714436
I'm not sure how the newspeak builds should be handled differently, since they use an alternate test script from here... https://github.com/OpenSmalltalk/opensmalltalk-vm/tree/Cog/tests
cheers -ben
btw, are we indeed not testing VMs on Windows?
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/.appveyor.yml#L12...
On Mon, Dec 31, 2018 at 12:46 AM Ben Coman btc@openinworld.com wrote:
On Mon, 31 Dec 2018 at 07:20, Fabio Niephaus lists@fniephaus.com wrote:
On Mon, 31 Dec 2018 at 12:07 am, Ben Coman btc@openinworld.com wrote:
However SmalltalkCI README indicates it doesn't support 32-bit of
latest images for Squeak or Pharo...
https://github.com/hpi-swa/smalltalkCI#list-of-supported-images but is that info current? These files seem to indicate support... https://github.com/hpi-swa/smalltalkCI/blob/master/squeak/run.sh https://github.com/hpi-swa/smalltalkCI/blob/master/pharo/run.sh
Squeak[64]-trunk means Squeak-trunk (32bit) and Squeak64-trunk are
supported. Eventually, we want to introduce Squeak32-trunk as well and change the default to 64bit.
The same rule applies for Pharo. Maybe open an issue against smalltalkCI, so we don't forget to get rid
of this confusion?
Fixed!
@vm-dev... I find the specification of the test image too implicit here...
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/tests/smalltalkCI...
For example, its hard to tell at a glance whether lowcode testing is
enabled/disabled.
Could the test image be explicitly defined in travis.yml ? (I'll submit
a concept PR for review)
Please do, happy to review a PR. The current version is still the very
first version, I'm sure it can use some improvements :)
cool.
btw, are we indeed not testing VMs on Windows?
https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/Cog/.appveyor.yml#L12...
I'm afraid you're right. If you could set that up in a similar, that'd be great. smalltalkCI does support AppVeyor: https://github.com/hpi-swa/smalltalkCI#appveyoryml-template
Fabio
cheers -ben
vm-dev@lists.squeakfoundation.org