pass frozen DAOD test when non-interested branches added/removed
For the frozen derivation test, in the case of removing/adding the branches that are not on the branches of interest list, we do not want to trigger the CI test to fail.
This MR added a mutual exclusion option in the diff_root. And by adding --inexact-branches
option, it allows us to change the message from error to info level in the above mentioned cases.
cc @dguest
Merge request reports
Activity
added Tools analysis-review-required main review-pending-level-1 labels
CI Result SUCCESS (hash f677aaba)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 77483]- Resolved by Jan-Eric Nitschke
Code changes look good. It also makes sense to me to enable this option when
branches_of_interest
is set.Is this the expectation that everyone using this option will have or does there need to be any discussion whether this should be enabled automatically with that option?
Cheers L1
CI Result SUCCESS (hash da221b53)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 77506]added review-approved label and removed review-pending-level-1 label
added analysis-review-approved label and removed analysis-review-required label
mentioned in commit 2546a7b8
added sweep:ignore label