Skip to content

Disable TPCDigitContainer test temporarily#387

Merged
ktf merged 1 commit intoAliceO2Group:devfrom
dberzano:disable-test
May 19, 2017
Merged

Disable TPCDigitContainer test temporarily#387
ktf merged 1 commit intoAliceO2Group:devfrom
dberzano:disable-test

Conversation

@dberzano
Copy link
Copy Markdown
Contributor

@dberzano dberzano commented May 19, 2017

No description provided.

@dberzano
Copy link
Copy Markdown
Contributor Author

dberzano commented May 19, 2017

@sawenzel @matthiasrichter @ktf I have enabled make test in the O2 pull request checker and as you have seen the consequence is that we have a failing test. I am disabling it temporarily.

@ktf ktf merged commit 1a953f4 into AliceO2Group:dev May 19, 2017
@ktf ktf mentioned this pull request May 19, 2017
Closed
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented May 19, 2017

Error while checking build/O2/o2 for 6c609c2:

sw/BUILD/O2-latest/log
+ DEVEL_SOURCES=/mnt/mesos/sandbox/sandbox/O2
+ '[' /mnt/mesos/sandbox/sandbox/O2 '!=' /mnt/mesos/sandbox/sandbox/sw/SOURCES/O2/dev/0 ']'
+ perl -p -i -e 's|/mnt/mesos/sandbox/sandbox/sw/SOURCES/O2/dev/0|/mnt/mesos/sandbox/sandbox/O2|' compile_commands.json
+ ln -sf /mnt/mesos/sandbox/sandbox/sw/BUILD/b3b37fa96d85a0e858b2e6ac9dda2e680aec17de/O2/compile_commands.json /mnt/mesos/sandbox/sandbox/O2/compile_commands.json
+ [[ -n 1 ]]
+ make test
Running tests...
Test project /mnt/mesos/sandbox/sandbox/sw/BUILD/b3b37fa96d85a0e858b2e6ac9dda2e680aec17de/O2
      Start  1: testMessageList
 1/16 Test  #1: testMessageList ..................   Passed    0.00 sec
      Start  2: testDataHeader
 2/16 Test  #2: testDataHeader ...................   Passed    0.55 sec
      Start  3: testTimeStamp
 3/16 Test  #3: testTimeStamp ....................   Passed    0.55 sec
      Start  4: testBasicHits
 4/16 Test  #4: testBasicHits ....................   Passed    0.60 sec
      Start  5: TimeFrameTest
 5/16 Test  #5: TimeFrameTest ....................   Passed    0.68 sec
      Start  6: testTPCBase
 6/16 Test  #6: testTPCBase ......................   Passed    0.61 sec
      Start  7: testTPCCalDet
 7/16 Test  #7: testTPCCalDet ....................   Passed    0.73 sec
      Start  8: testTPCMapper
 8/16 Test  #8: testTPCMapper ....................   Passed    0.47 sec
      Start  9: testTPCSyncPatternMonitor
 9/16 Test  #9: testTPCSyncPatternMonitor ........   Passed    0.71 sec
      Start 10: testTPCAdcClockMonitor
10/16 Test #10: testTPCAdcClockMonitor ...........   Passed    0.68 sec
      Start 11: testTPCElectronTransport
11/16 Test #11: testTPCElectronTransport .........   Passed    0.89 sec
      Start 12: testTPCGEMAmplification
12/16 Test #12: testTPCGEMAmplification ..........   Passed   28.32 sec
      Start 13: testTPCSimulation
13/16 Test #13: testTPCSimulation ................   Passed    0.82 sec
      Start 14: run_flp2epn_distributed
14/16 Test #14: run_flp2epn_distributed ..........***Failed  Required regular expression not found.Regex=[acknowledged after
]  5.07 sec
      Start 15: testExampleModule1
15/16 Test #15: testExampleModule1 ...............   Passed    0.03 sec
      Start 16: O2MessageMonitorTest
16/16 Test #16: O2MessageMonitorTest .............   Passed    0.92 sec

94% tests passed, 1 tests failed out of 16

Total Test time (real) =  41.67 sec

The following tests FAILED:
	 14 - run_flp2epn_distributed (Failed)
Errors while running CTest

Full log here.

This was referenced May 19, 2017
Merged
Merged
knopers8 pushed a commit to knopers8/AliceO2 that referenced this pull request Sep 7, 2020
@lauraser
Fixed json file, included detector name in checks. (AliceO2Group#387)
58c0f20
mbroz84 pushed a commit to mbroz84/AliceO2 that referenced this pull request Mar 16, 2022
@TimoWilken
Fix PR auto-approval script (AliceO2Group#387)
9f47d42 
1. Parse the CODEOWNERS file properly -- `git check-ignore` uses slightly
   different syntax and precedence rules, so we can't use that.

2. If another codeowner has already approved the PR, then consider that OK for
   the relevant CODEOWNER lines.

   This means that if e.g. a PR authored by A changes files Fa and Fb, and A
   owns Fa, and B owns Fb, then auto-approval is still possible as long as B
   approves the PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dberzano @alibuild @ktf