BPHY CA Migration Part 3: Revenge of the SecVtx (ATLBPHYS-230)

• Add CA versions of BPHY10, BPHY15, BPHY16, BPHY21
• Fix file permissions
• Allow InDetConversionTrackSelectorTool to run without BeamspotCondAlg
• Various fixes

cc @bouhova @ranovotn

assigned to @abarton

This merge request affects 3 packages:

• InnerDetector/InDetRecTools/InDetTrackSelectorTool
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration

Affected files list will not be printed in this case

Adding @goetz ,@jcatmore ,@amorley ,@wwalko ,@sroe ,@abarton as watchers

added Derivation InnerDetector JetEtmiss master review-pending-level-1 labels

changed title from BPHY Derivations CA Migration Part 3: Revenge of the SecVtx (ATLBPHYS-230) to BPHY CA Migration Part 3: Revenge of the SecVtx (ATLBPHYS-230)

changed the description

CI Result SUCCESS (hash 04964e11)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 58203]

Just some suggestions related to the new configs introduced to move them closer to the existing configs of the same tools. If something of that order is anyway planned in the near future, feel free to resolve.

Best, Thomas - L1

added review-user-action-required label and removed review-pending-level-1 label

added 4 commits

• f3be235c - Add CA version of BPHY10 and BPHY15
• 320653b4 - allow InDetConversionTrackSelectorTool to run without BeamspotCondAlg
• 69eaae62 - Add CA of BPHY16 and BPHY21, fix permissions
• 5791c175 - Remove redundant Configfile

Compare with previous version

@tstreble Thanks for your comments I've moved and fixed a few things.

This merge request affects 5 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetRecTools/InDetTrackSelectorTool
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• Tracking/TrkVertexFitter/TrkV0Fitter

Affected files list will not be printed in this case

Adding @goetz ,@bouhova ,@amorley ,@jcatmore ,@wwalko ,@sroe ,@abarton ,@calpigia as watchers

added Tracking review-pending-level-1 labels and removed review-user-action-required label

resolved all threads

CI Result SUCCESS (hash 5791c175)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 58261]

Approving - L1

removed review-pending-level-1 label