Skip to content

Conversation

@Barthelemy
Copy link
Collaborator

No description provided.

@Barthelemy Barthelemy requested a review from knopers8 as a code owner April 3, 2023 12:33
@Barthelemy
Copy link
Collaborator Author

depends on AliceO2Group/AliceO2#11253

@Barthelemy Barthelemy changed the title [WIP] [QC-953] non-critical tasks [QC-953] non-critical tasks Jun 2, 2023
@Barthelemy
Copy link
Collaborator Author

Barthelemy commented Jun 5, 2023

  • CheckRunners are all non-critical.
  • Tasks are critical by default. There is a parameter in the config.
  • Mergers are critical if the task they are merging is critical.
  • PP: by default critical. Parameter in config.

@Barthelemy Barthelemy changed the title [QC-953] non-critical tasks [WIP] [QC-953] non-critical tasks Jun 5, 2023
@ktf
Copy link
Member

ktf commented Nov 23, 2023

I just checked this and the topology builds fine now.

@Barthelemy
Copy link
Collaborator Author

Barthelemy commented Nov 28, 2023

After further discussion, we decided that:

  • DS is critical
  • Tasks are critical by default for the time being, can be marked expendable
  • output proxies are the same as the task they attach to
  • input proxies are the same as the task they attach to
  • mergers are resilient
  • checkers are resilient
  • aggregators are resilient
  • PP tasks are critical by default, can be marked as expendable
  • Documentation update

@Barthelemy
Copy link
Collaborator Author

all tests pass now.

@Barthelemy Barthelemy changed the title [WIP] [QC-953] non-critical tasks [QC-953] non-critical tasks Nov 30, 2023
@Barthelemy
Copy link
Collaborator Author

needs AliceO2Group/AliceO2#12354

@Barthelemy
Copy link
Collaborator Author

@knopers8 applied the changes

@Barthelemy Barthelemy merged commit b2f24f5 into AliceO2Group:master Dec 22, 2023
@Barthelemy Barthelemy deleted the critical-tasks branch December 22, 2023 07:41
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