Skip to content

Fix EMCAL workflows in case DISABLE_ROOT_OUTPUT=0#15114

Open
shahor02 wants to merge 1 commit intoAliceO2Group:devfrom
shahor02:pr_dplwfemc
Open

Fix EMCAL workflows in case DISABLE_ROOT_OUTPUT=0#15114
shahor02 wants to merge 1 commit intoAliceO2Group:devfrom
shahor02:pr_dplwfemc

Conversation

@shahor02
Copy link
Collaborator

@shahor02 shahor02 commented Mar 3, 2026

No description provided.

@shahor02 shahor02 requested a review from a team as a code owner March 3, 2026 10:13
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@mhemmer-cern
Copy link
Contributor

mhemmer-cern commented Mar 3, 2026

Thanks Ruben, I think this is correct now.
Based on

( workflow_has_parameter AOD || [[ -z "$DISABLE_ROOT_OUTPUT" ]] || needs_root_output o2-emcal-cell-writer-workflow ) && ! has_detector_from_global_reader EMC && has_detector EMC && add_W o2-emcal-cell-recalibrator-workflow "--input-subspec 1 --output-subspec 0 --redirect-led"

The o2-emcal-reco-workflow produces the output in the spec that is used for the o2-emcal-cell-recalibrator-workflow as input. Since in the script the o2-emcal-cell-recalibrator-workflow always expects 1 as subspec for the input the o2-emcal-reco-workflow should have 1 as output.

@alibuild
Copy link
Collaborator

alibuild commented Mar 3, 2026

Error while checking build/O2/fullCI_slc9 for 243f067 at 2026-03-03 14:04:

No log files found

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants