Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZDC: combine local and remote QC configuration into a single file #667

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

aferrero2707
Copy link
Contributor

@aferrero2707 aferrero2707 commented Oct 13, 2024

With the current ZDC workflows and QC configuration, the ZDC FLP is not only running the raw QC task, but also the corresponding checker and post-processing task.
While this is not forbidden, it would be preferable to move the checker and post-processing to the remote QC node, and only leave the task on the FLP.

Moreover, given that ZDC has only one FLP node, it seems more practical to combine all tasks in a single multi-node configuration file, to avoid duplicating parts of the configuration in multiple files.

Two new workflows, one local and one remote, are cloned from the original ones:

  • zdc-full-qcmn-local from zdc-raw-qc
  • zdc-full-qcmn-remote from zdc-rec-epn

Compared to the original workflows, the new ones have the following changes:

  • the parameters --local --host flp are added to the o2-qc command in the FLP workflow, to prevent the execution of the checker and post-processing tasks
  • the remote QC node workflow is modified to read the combined QC configuration file

In addition, I have appended the run_type and beam_type parameters to the apricot URLs in the new workflows, to allow extra flexibility in the QC configuration if needed.

More details are given in https://its.cern.ch/jira/browse/OCTRL-941

…tion file

Two new JIT workflows are added, for FLP and remote QC node, which allow
to run the local and remote QC tasks from a single configuration file
Copy link
Contributor

@Barthelemy Barthelemy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, thank you !

@Barthelemy Barthelemy merged commit b68ea06 into AliceO2Group:master Oct 14, 2024
1 check passed
@vascobarroso
Copy link
Member

@Barthelemy @aferrero2707 this has now been merged but it's not yet in production.
Please let me know when you would like to move this into production.

vascobarroso pushed a commit that referenced this pull request Oct 15, 2024
…tion file (#667)

Two new JIT workflows are added, for FLP and remote QC node, which allow
to run the local and remote QC tasks from a single configuration file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants