Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 153
    • Merge requests 153
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !56350

Draft: AthDerivation: Make sure CA-based jobs can run + disable some tests

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Tadej Novak requested to merge tadej/athena:athderivation1 into master Aug 29, 2022
  • Overview 10
  • Commits 1
  • Pipelines 1
  • Changes 6

This MR adds a few more packages to AthDerivation so CA-based PHYS can run both for MC20 and Upgrade (and probably also for MC21). This only adds O(300) compilation units so it should not increase the build too much (it has almost 20k compilation units already).

By advice from @akraszna I also introduced ATLAS_DERIVATION_RELEASE so we can disable tests. For now I disabled

  • EventInfo overlay tests
  • InDetConfig tests as they all run ~full reconstruction
  • MuonConfig tests as they similarly run ~full reconstruction

I may look at some other tests in the next days. Before undrafting I want to confirm that this is not really tested and the CI will only confirm this does not break other projects (hopefully).

Tagging also @jcatmore and @calpigia for derivations, @npetters and @tstreble for upgrade.

Edited Sep 01, 2022 by Tadej Novak
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: athderivation1