Skip to content
Snippets Groups Projects

Enable track reading in ACTS tracking Persistency test

Merged Tomasz Bold requested to merge tbold/athena:fix-measurments-recoring into main
All threads resolved!

This small changes enable trading of ACTS EDM backends in the ActsPersistifyEDM test.

Tagging @cvarni @wolter Marcin, this test basically verifies that we have readable content in output AODs.

Merge request reports

Pipeline #7254089 passed

Pipeline passed for f7d80548 on tbold:fix-measurments-recoring

Approval is optional

All merge request dependencies have been merged (1 merged)

Merged by Frank WinklmeierFrank Winklmeier 11 months ago (Apr 22, 2024 11:12am UTC)

Merge details

  • Changes merged into main with 3dba50d2 (commits were squashed).
  • Did not delete 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
  • Carlo Varni
  • Jenkins please retry a build

  • Restarted after the system glitch

  • :pencil: There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

  • :white_check_mark: CI Result SUCCESS (hash c0875c25)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 8296] (remote access info)

  • Tomasz Bold marked this merge request as draft

    marked this merge request as draft

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • Tomasz Bold marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 2 packages:

    • Tracking/Acts/ActsConfig
    • Tracking/Acts/ActsMonitoring

    Affected files list will not be printed in this case

    Adding @toyamaza ,@jojungge ,@tbold ,@pagessin ,@cvarni as watchers

  • Author Developer

    More motoring was added. It proves that we store properly TrackStates, TrackParameters and TrackMeasuremnts. Accessing surface info fails. This will be addressed in followup MR.

    Maybe people want to comment on what we could monitor. As you see it is quite straightforward to extend.

  • Tomasz Bold resolved all threads

    resolved all threads

  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 2 packages:

    • Tracking/Acts/ActsConfig
    • Tracking/Acts/ActsMonitoring

    Affected files list will not be printed in this case

    Adding @toyamaza ,@tbold ,@pagessin ,@jojungge ,@cvarni as watchers

  • :white_check_mark: CI Result SUCCESS (hash b641ecda)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 8341] (remote access info)

  • :white_check_mark: CI Result SUCCESS (hash f7d80548)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 8342] (remote access info)

  • Looks good from L1 perspective, approving.

    Cheers, Philipp (L1)

  • Marcin Wolter mentioned in merge request !70556 (merged)

    mentioned in merge request !70556 (merged)

  • mentioned in commit 3dba50d2

  • Please register or sign in to reply
    Loading