Page MenuHomePhabricator

[commtest] use shared/grpc_client for identity clients
ClosedPublic

Authored by varun on Oct 12 2023, 11:56 AM.
Tags
None
Referenced Files
F5889109: D9472.diff
Thu, Apr 17, 5:09 AM
Unknown Object (File)
Tue, Apr 1, 8:24 PM
Unknown Object (File)
Sat, Mar 22, 7:44 AM
Unknown Object (File)
Wed, Mar 19, 1:32 PM
Unknown Object (File)
Mar 15 2025, 8:12 AM
Unknown Object (File)
Mar 10 2025, 3:58 AM
Unknown Object (File)
Mar 10 2025, 3:58 AM
Unknown Object (File)
Mar 10 2025, 3:54 AM
Subscribers

Details

Summary

remove all custom clients and replace them with the grpc_clients library clients

Depends on D9471

Test Plan

ran cargo test and all tests but an unrelated backup integration test passed (wasn't running backup service)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Looks good. Conflicts a bit with my commtest changes, but I'll rebase as soon as you land this

services/commtest/build.rs
11–15 ↗(On Diff #31975)

This is not a perfect solution. It can stay for now but we should design something else for long term.
Can you create a follow-up task for this? E.g. under ENG-2311 or 4686

This revision is now accepted and ready to land.Oct 16 2023, 12:42 AM