Skip to content
Snippets Groups Projects

fix for failing unit test in AnalysisBase

Merged Nils Erik Krumnack requested to merge krumnack/athena:xaod_jet_test_fix into master
All threads resolved!

Basically just trying to ignore xAOD::Init line in log file, which will hopefully fix the remaining broken unit test in AnalysisBase. Unfortunately for some reason the test wasn't failing when I was running locally.

Merge request reports

Pipeline #1497749 passed

Pipeline passed for e1e8f8e9 on krumnack:xaod_jet_test_fix

Merged by Frank WinklmeierFrank Winklmeier 5 years ago (Mar 20, 2020 1:27pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • :negative_squared_cross_mark: CI Result FAILURE (hash 9d5d6e79)

    Athena AthSimulation AnalysisBase
    externals :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :o:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11182]

  • added 1 commit

    • e1e8f8e9 - change from EXTRA_PATTERNS to LOG_IGNORE_PATTERN

    Compare with previous version

  • This merge request affects 1 package:

    • Event/xAOD/xAODJet

    Adding @rbianchi as watcher

  • :white_check_mark: CI Result SUCCESS (hash e1e8f8e9)

    Athena AthSimulation AnalysisBase
    externals :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11199]

  • Edson Carquin Lopez resolved all threads

    resolved all threads

  • Looks good and already reviwed by experts, approving (L1)

  • mentioned in commit 37694f08

  • Please register or sign in to reply
    Loading