Re: Jenkins build became unstable: moose-latest-dev #710

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

abergel
Strange, the Fuel tests are green on my image and I have the last version of all packages.

Alexandre


On 23 Nov 2011, at 18:27, [hidden email] wrote:

> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>
>

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.





_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

Tudor Girba-2
Hi,

I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).

I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.

You added:
FAMIXPackage>>weightedMethodCount
        <MSEProperty: #weightedMethodCount type: #Number>
        <MSEComment: 'The sum of the complexity in a package'>
        ...

This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.

So, one solution is to add FAMIXPackage>>weightedMethodCount:

However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
FAMIXPackage>>weightedMethodCount
        <MSEProperty: #weightedMethodCount type: #Number>
        <MSEComment: 'The sum of the complexity in a package'>
        <derived>
        ...

Cheers,
Doru


On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:

> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>
> Alexandre
>
>
> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>
>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>
>>
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
www.tudorgirba.com

"We are all great at making mistakes."








_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

Santiago Vidal
Thanks for the clarification :D
Cheers,
 Santiago

2011/11/24 Tudor Girba <[hidden email]>
Hi,

I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).

I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.

You added:
FAMIXPackage>>weightedMethodCount
       <MSEProperty: #weightedMethodCount type: #Number>
       <MSEComment: 'The sum of the complexity in a package'>
       ...

This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.

So, one solution is to add FAMIXPackage>>weightedMethodCount:

However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
FAMIXPackage>>weightedMethodCount
       <MSEProperty: #weightedMethodCount type: #Number>
       <MSEComment: 'The sum of the complexity in a package'>
       <derived>
       ...

Cheers,
Doru


On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:

> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>
> Alexandre
>
>
> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>
>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>
>>
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
www.tudorgirba.com

"We are all great at making mistakes."








_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev



--
Santiago Vidal

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

abergel
In reply to this post by Tudor Girba-2
Thanks!
It makes perfect sense.

Alexandre


On 24 Nov 2011, at 04:20, Tudor Girba wrote:

> Hi,
>
> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).
>
> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.
>
> You added:
> FAMIXPackage>>weightedMethodCount
> <MSEProperty: #weightedMethodCount type: #Number>
> <MSEComment: 'The sum of the complexity in a package'>
> ...
>
> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.
>
> So, one solution is to add FAMIXPackage>>weightedMethodCount:
>
> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
> FAMIXPackage>>weightedMethodCount
> <MSEProperty: #weightedMethodCount type: #Number>
> <MSEComment: 'The sum of the complexity in a package'>
> <derived>
> ...
>
> Cheers,
> Doru
>
>
> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:
>
>> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>>
>> Alexandre
>>
>>
>> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>>
>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>>
>>>
>>
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> [hidden email]
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
> --
> www.tudorgirba.com
>
> "We are all great at making mistakes."
>
>
>
>
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.





_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

abergel
In reply to this post by Tudor Girba-2
I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins?

Cheers,
Alexandre


On 24 Nov 2011, at 04:20, Tudor Girba wrote:

> Hi,
>
> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).
>
> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.
>
> You added:
> FAMIXPackage>>weightedMethodCount
> <MSEProperty: #weightedMethodCount type: #Number>
> <MSEComment: 'The sum of the complexity in a package'>
> ...
>
> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.
>
> So, one solution is to add FAMIXPackage>>weightedMethodCount:
>
> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
> FAMIXPackage>>weightedMethodCount
> <MSEProperty: #weightedMethodCount type: #Number>
> <MSEComment: 'The sum of the complexity in a package'>
> <derived>
> ...
>
> Cheers,
> Doru
>
>
> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:
>
>> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>>
>> Alexandre
>>
>>
>> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>>
>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>>
>>>
>>
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> [hidden email]
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
> --
> www.tudorgirba.com
>
> "We are all great at making mistakes."
>
>
>
>
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.





_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

Tudor Girba-2
I explained above.

Perhaps what I did not say is that MSEProperty: pragmas only take
effect when you reset the meta description. So, I guess that you added
the MSEProperty and ran the tests without reseting the meta
descriptions. Given that the tests do not reset this meta description,
they worked locally. When the server started fresh, the meta
description got initialized and the new pragma took effect.

Cheers,
Doru

On Thu, Nov 24, 2011 at 2:59 PM, Alexandre Bergel
<[hidden email]> wrote:

> I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins?
>
> Cheers,
> Alexandre
>
>
> On 24 Nov 2011, at 04:20, Tudor Girba wrote:
>
>> Hi,
>>
>> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).
>>
>> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.
>>
>> You added:
>> FAMIXPackage>>weightedMethodCount
>>       <MSEProperty: #weightedMethodCount type: #Number>
>>       <MSEComment: 'The sum of the complexity in a package'>
>>       ...
>>
>> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.
>>
>> So, one solution is to add FAMIXPackage>>weightedMethodCount:
>>
>> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
>> FAMIXPackage>>weightedMethodCount
>>       <MSEProperty: #weightedMethodCount type: #Number>
>>       <MSEComment: 'The sum of the complexity in a package'>
>>       <derived>
>>       ...
>>
>> Cheers,
>> Doru
>>
>>
>> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:
>>
>>> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>>>
>>> Alexandre
>>>
>>>
>>> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>>>
>>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>>>
>>>>
>>>
>>> --
>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>> Alexandre Bergel  http://www.bergel.eu
>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Moose-dev mailing list
>>> [hidden email]
>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>> --
>> www.tudorgirba.com
>>
>> "We are all great at making mistakes."
>>
>>
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> [hidden email]
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>



--
www.tudorgirba.com
"Every thing has its own flow"

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

abergel
Ah yes!

Alexandre


On 24 Nov 2011, at 11:29, Tudor Girba wrote:

> I explained above.
>
> Perhaps what I did not say is that MSEProperty: pragmas only take
> effect when you reset the meta description. So, I guess that you added
> the MSEProperty and ran the tests without reseting the meta
> descriptions. Given that the tests do not reset this meta description,
> they worked locally. When the server started fresh, the meta
> description got initialized and the new pragma took effect.
>
> Cheers,
> Doru
>
> On Thu, Nov 24, 2011 at 2:59 PM, Alexandre Bergel
> <[hidden email]> wrote:
>> I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins?
>>
>> Cheers,
>> Alexandre
>>
>>
>> On 24 Nov 2011, at 04:20, Tudor Girba wrote:
>>
>>> Hi,
>>>
>>> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).
>>>
>>> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.
>>>
>>> You added:
>>> FAMIXPackage>>weightedMethodCount
>>>       <MSEProperty: #weightedMethodCount type: #Number>
>>>       <MSEComment: 'The sum of the complexity in a package'>
>>>       ...
>>>
>>> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.
>>>
>>> So, one solution is to add FAMIXPackage>>weightedMethodCount:
>>>
>>> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
>>> FAMIXPackage>>weightedMethodCount
>>>       <MSEProperty: #weightedMethodCount type: #Number>
>>>       <MSEComment: 'The sum of the complexity in a package'>
>>>       <derived>
>>>       ...
>>>
>>> Cheers,
>>> Doru
>>>
>>>
>>> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:
>>>
>>>> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>>>>
>>>> Alexandre
>>>>
>>>>
>>>> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>>>>
>>>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>>>>
>>>>>
>>>>
>>>> --
>>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>>> Alexandre Bergel  http://www.bergel.eu
>>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Moose-dev mailing list
>>>> [hidden email]
>>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>>
>>> --
>>> www.tudorgirba.com
>>>
>>> "We are all great at making mistakes."
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Moose-dev mailing list
>>> [hidden email]
>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> [hidden email]
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>
>
>
> --
> www.tudorgirba.com
> "Every thing has its own flow"
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.





_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Jenkins build became unstable: moose-latest-dev #710

tinchodias
Interesting! happily it's fixed!

Martín

On Thu, Nov 24, 2011 at 11:31 AM, Alexandre Bergel <[hidden email]> wrote:
Ah yes!

Alexandre


On 24 Nov 2011, at 11:29, Tudor Girba wrote:

> I explained above.
>
> Perhaps what I did not say is that MSEProperty: pragmas only take
> effect when you reset the meta description. So, I guess that you added
> the MSEProperty and ran the tests without reseting the meta
> descriptions. Given that the tests do not reset this meta description,
> they worked locally. When the server started fresh, the meta
> description got initialized and the new pragma took effect.
>
> Cheers,
> Doru
>
> On Thu, Nov 24, 2011 at 2:59 PM, Alexandre Bergel
> <[hidden email]> wrote:
>> I run the Fuel tests and they were green in my image. Any idea why they failed when run by Jenkins?
>>
>> Cheers,
>> Alexandre
>>
>>
>> On 24 Nov 2011, at 04:20, Tudor Girba wrote:
>>
>>> Hi,
>>>
>>> I reproduced the problem. It was due to FAMIXPackage>>weightedMethodCount :).
>>>
>>> I am describing the situation, because it will probably be useful for others that want to extend Moose with properties.
>>>
>>> You added:
>>> FAMIXPackage>>weightedMethodCount
>>>       <MSEProperty: #weightedMethodCount type: #Number>
>>>       <MSEComment: 'The sum of the complexity in a package'>
>>>       ...
>>>
>>> This means that this property is described. And it also means that the Fame import/export will deal with it as well. Now, the fuel tests simply export a model and then import it back. This property gets exported, but when Fame tries to import it it crashes because there is no setter.
>>>
>>> So, one solution is to add FAMIXPackage>>weightedMethodCount:
>>>
>>> However, in this case, we deal with a property that is derived from the information inside the model, so we do not want to save it in the first place. Thus, the proper solution is to mark it as derived:
>>> FAMIXPackage>>weightedMethodCount
>>>       <MSEProperty: #weightedMethodCount type: #Number>
>>>       <MSEComment: 'The sum of the complexity in a package'>
>>>       <derived>
>>>       ...
>>>
>>> Cheers,
>>> Doru
>>>
>>>
>>> On 23 Nov 2011, at 22:33, Alexandre Bergel wrote:
>>>
>>>> Strange, the Fuel tests are green on my image and I have the last version of all packages.
>>>>
>>>> Alexandre
>>>>
>>>>
>>>> On 23 Nov 2011, at 18:27, [hidden email] wrote:
>>>>
>>>>> See <http://hudson.moosetechnology.org/job/moose-latest-dev/710/>
>>>>>
>>>>>
>>>>
>>>> --
>>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>>> Alexandre Bergel  http://www.bergel.eu
>>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Moose-dev mailing list
>>>> [hidden email]
>>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>>
>>> --
>>> www.tudorgirba.com
>>>
>>> "We are all great at making mistakes."
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Moose-dev mailing list
>>> [hidden email]
>>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> [hidden email]
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>
>
>
> --
> www.tudorgirba.com
> "Every thing has its own flow"
>
> _______________________________________________
> Moose-dev mailing list
> [hidden email]
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.





_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev


_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev