Skip to content
Snippets Groups Projects

Recent improvements to HLT2 monitoring

Merged Roel Aaij requested to merge spread_messages into master
All threads resolved!

Spread sending of messages over time to avoid passing high water marks. Increase high water marks. Fix bug in 2D histograms (y-axis was x-axis) Synchronise in batches to avoid timeout.

Edited by Roel Aaij

Merge request reports

Checking pipeline status.

Approval is optional

Merged by avatar (Apr 4, 2025 4:36am UTC)

Merge details

  • Changes merged into master with d5d8818f.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Roel Aaij marked as a Work In Progress

    marked as a Work In Progress

  • Roel Aaij changed the description

    changed the description

  • Author Maintainer

    Marked as WIP until I sort this out.

  • Roel Aaij added 2 commits

    added 2 commits

    • b72ba83f - Store histogram info also by hash and use shared_ptr to avoid duplication of memory.
    • c31e2fc1 - Improve sync mechanism and fix some compile errors on clang.

    Compare with previous version

  • Roel Aaij added 3 commits

    added 3 commits

    Compare with previous version

  • Author Maintainer

    Compilation is fixed and the warnings should be gone.

  • Roel Aaij unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Rosen Matev mentioned in merge request MooreOnline!39 (merged)

    mentioned in merge request MooreOnline!39 (merged)

  • Roel Aaij added 8 commits

    added 8 commits

    Compare with previous version

  • Sascha Stahl mentioned in merge request LHCb!713 (merged)

    mentioned in merge request LHCb!713 (merged)

  • @raaij can we merge this and get this into an Online release? So that we do not have to keep patching this into MooreOnline when making a release?

  • Roel Aaij added 1 commit

    added 1 commit

    • 4e5d3771 - Configure saver service to connect to registrar.

    Compare with previous version

  • Author Maintainer

    Yes, it can be merged, it's running in production and the nightlies on all platforms are also fine.

    There is one change to the configuration that is part of !115 (merged), which I have cherry-picked to ensure consistency.

    @frankm, any show-stoppers for merging this MR?

  • Roel Aaij resolved all discussions

    resolved all discussions

  • Roel Aaij added 9 commits

    added 9 commits

    • 4229ec3f - Update test scripts.
    • 8c83276e - Allow file descriptors to be registered to the python poller frontend and polled.
    • 0e61a9c2 - Update poller test scripts to test regular file descriptors and integrate into a single script.
    • 53a3d798 - Expose remote end of the pipe from Communicator so it can be used to poll from.
    • a1b248d1 - Add an optional pipe to the DIM forwarder to signal it when to instantiate and…
    • 8cba3c28 - Split start function in MonitoringJob in initialize/start to allow these to…
    • 466b1b0d - Make DiskMonitor runable as a DIM-steered job and add a test script for it.
    • d4532d64 - Fix histograms that are not supposed to be added, being to the from the version loaded from file.
    • 604d383a - Update Hlt2SaverSvc to check if the registrar is alive and attempt…

    Compare with previous version

  • Roel Aaij added 1 commit

    added 1 commit

    • 61d530b2 - Fix off-by-one error when ping socket was inserted. Registrar pings twice before…

    Compare with previous version

  • merged

  • Markus Frank mentioned in commit d5d8818f

    mentioned in commit d5d8818f

  • Please register or sign in to reply
    Loading