Skip to content
Snippets Groups Projects

Fix muon conditions scheduling issues related to overlay

Merged Tadej Novak requested to merge tadej/athena:overlay/muoncond into master
All threads resolved!

Fix some muon conditions scheduling issues related to overlay. If a folder/object is not used, the ReadHandleKey should not be initialised.

At some point we should probably not have the default keys (at least for MDT) and then configure them if needed, but I wanted to avoid changing old-style configuration for now.

/cc @rosati @nkoehler @stavrop

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • This merge request affects 3 packages:

    • MuonSpectrometer/MuonCnv/MuonByteStream
    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg
    • MuonSpectrometer/MuonConfig

    Adding @rosati ,@wleight ,@nkoehler as watchers

  • Tadej Novak resolved all threads

    resolved all threads

  • Tadej Novak added 1 commit

    added 1 commit

    • 6809f317 - Fix muon conditions scheduling issues related to overlay

    Compare with previous version

  • This merge request affects 3 packages:

    • MuonSpectrometer/MuonCnv/MuonByteStream
    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg
    • MuonSpectrometer/MuonConfig

    Adding @rosati ,@wleight ,@nkoehler as watchers

  • :white_check_mark: CI Result SUCCESS (hash 6809f317)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9866]

  • added review-approved label and removed review-pending-level-1 label

  • mentioned in commit 545cf167

  • Please register or sign in to reply
    Loading