Skip to content
Snippets Groups Projects

AthenaCommon + AthenaConfiguration: Don't disable MessageSvc default limit in MT jobs

Merged Alaettin Serhan Mete requested to merge amete/athena:main-msgsvc-drop-dftlmt into main
All threads resolved!

Based on the discussion we just had at the core software meeting, let's get rid of this (most likely) remnant of the MT commissioning configuration that disables the default message limit for the message service.

Merge request reports

Pipeline #6362679 passed

Pipeline passed for 38fad5d5 on amete:main-msgsvc-drop-dftlmt

Merged by Frank WinklmeierFrank Winklmeier 1 year ago (Oct 20, 2023 9:02am 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
  • added 1 commit

    Compare with previous version

  • Alaettin Serhan Mete resolved all threads

    resolved all threads

  • This merge request affects 2 packages:

    • Control/AthenaCommon
    • Control/AthenaConfiguration

    Affected files list will not be printed in this case

    Adding @ssnyder ,@gemmeren ,@maszyman as watchers

  • :white_check_mark: CI Result SUCCESS (hash 1ac53755)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :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: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST 71]

  • This looks like a pure Athena configuration update, approving for analysis release purposes.

    Nils [Analysis RC]

  • Jobs needs a restart due to couple of technical problems (details posted on Mattermost)

  • Jenkins please retry a build

  • This merge request affects 2 packages:

    • Control/AthenaCommon
    • Control/AthenaConfiguration

    Affected files list will not be printed in this case

    Adding @maszyman ,@gemmeren ,@ssnyder as watchers

  • :white_check_mark: CI Result SUCCESS (hash 38fad5d5)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :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: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST 91]

  • Thomas Strebler resolved all threads

    resolved all threads

  • Looks good, approving - L1

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

  • Frank Winklmeier mentioned in commit f63834b8

    mentioned in commit f63834b8

  • Please register or sign in to reply
    Loading