Skip to content

[QC-648] Move to pub-sub for the channels between machines#855

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
knopers8:pubsub
Oct 4, 2021
Merged

[QC-648] Move to pub-sub for the channels between machines#855
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
knopers8:pubsub

Conversation

@knopers8
Copy link
Copy Markdown
Collaborator

@knopers8 knopers8 commented Oct 4, 2021

No description provided.

@knopers8 knopers8 requested a review from Barthelemy as a code owner October 4, 2021 08:05
@knopers8
Copy link
Copy Markdown
Collaborator Author

knopers8 commented Oct 4, 2021

@Barthelemy We should coordinate the FLP suite bump and QC bump on EPNs to avoid mismatching channel types.
@ktf

@ktf
Copy link
Copy Markdown
Member

ktf commented Oct 4, 2021

Can we hack this in at configuration level? Just to verify that it's indeed the issue we are observing.

@knopers8
Copy link
Copy Markdown
Collaborator Author

knopers8 commented Oct 4, 2021

We can, but that will take more time to implement.

@knopers8
Copy link
Copy Markdown
Collaborator Author

knopers8 commented Oct 4, 2021

@ktf If you are relying on the DDS topology files, then I suppose you could modify those channel config arguments by hand, that would be the quickest solution for you, I suppose.

@ktf
Copy link
Copy Markdown
Member

ktf commented Oct 4, 2021

We are now trying that with @shahor02.

@Barthelemy
Copy link
Copy Markdown
Collaborator

@knopers8 thank you for this very fast move.

@ironMann Please note the comment of Piotr above

@Barthelemy Barthelemy merged commit 8167d11 into AliceO2Group:master Oct 4, 2021
@ironMann
Copy link
Copy Markdown

ironMann commented Oct 4, 2021

Hi. A bunch of previously working FLP-QC workflows aren't doing well since the upgrade today. How soon can this be put in place?

@Barthelemy
Copy link
Copy Markdown
Collaborator

Is it related ? what are the symptoms ? I would propose to move the discussion to JIRA or by email.

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.

4 participants