Disable flp2epn-distributed test temporarily#390
Conversation
ktf
left a comment
ktf
left a comment
There was a problem hiding this comment.
As discussed we should merge this to avoid spurious errors and open a ticket about the test failing.
|
There seems to be a strange issue when running the unit tests in the CI. Whenever a test which has failed is disabled, another tests fails which had been ok before. Also the disabled here had been working before. |
|
Indeed, looks like a timeout or something like that. @dberzano can you look into it? |
|
Yep |
|
Hi @matthiasrichter, @ktf - I don't see the problem any longer, do you? |
|
Hi Dario, the last message from the Alice Builder is from 16:33 on this issue #370 (comment), I think the problem is still there. |
|
Follow up on this: there was a real problem with the failing unit test. But it looks like that the condition whether it was failing or not was depending on the history of unit tests as the test was not failing when the flp2epn-distributed test was failing. Can the actual error message from a failing test be part of the log? Right now that seems to be missing. |
|
I don't know from where this issue originated, but I would suggest to re-enable the test. If it is something unique to the flp2epn-distributed, then I will fix the origin of the fail. We shouldn't hide the problem by disabling the test. (Sorry if I am missing some parts of the discussion not present here!) I tried to reproduce any failures on MacOS/Linux but couldn't. @matthiasrichter Indeed, seeing the actuall failure message would be very useful :). |
…iceO2Group#390) * [QC-292] Make multi-node setups work nicely with ServiceDiscovery * Service Discovery - common names but unique IDs
* Add Tree variables * Fix clang format * apply the comments * Add tree branches related to Xic selection
4 participants
No description provided.