Skip to content

use grpc++ instead of grpc++_unsecure#189

Merged
lkrcal merged 1 commit intoAliceO2Group:devfrom
knopers8:use-grpc-secure
Apr 24, 2025
Merged

use grpc++ instead of grpc++_unsecure#189
lkrcal merged 1 commit intoAliceO2Group:devfrom
knopers8:use-grpc-secure

Conversation

@knopers8
Copy link
Contributor

With the recent grpc bump, grpc causes troubles when both grpc and grpc_unsecure are loaded by the same binary. Here, it is the case when StfSender and StfBuilder load the OccLite plugin (which uses grpc). This change fixes both and allows them to be controlled by ECS again. I tested it on a VM setup with FLP suite.

I do not know if it might break DataDistribution in some other context, I hope that anyone reviewing it may know...

With the recent grpc bump, grpc causes troubles when both grpc and grpc_unsecure are loaded by the same binary.
Here, it is the case when StfSender and StfBuilder load the OccLite plugin (which uses grpc).
This change fixes both and allows them to be controlled by ECS again, tested on a VM setup with FLP suite.

I do not know if it might break DataDistribution in some other context, I hope that anyone reviewing it may know...
@knopers8
Copy link
Contributor Author

@lkrcal could you have a look please?

@vascobarroso vascobarroso requested a review from lkrcal April 24, 2025 09:27
@vascobarroso
Copy link
Member

@lkrcal could we get a tag with this ? Then we could install it and in case this breaks something we revert.

@lkrcal lkrcal merged commit ed15447 into AliceO2Group:dev Apr 24, 2025
@knopers8 knopers8 deleted the use-grpc-secure branch April 24, 2025 13:44
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.

3 participants