Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

feat(api/grpc): add synchronization proto #332

Merged
merged 3 commits into from
Oct 17, 2023

Conversation

Freyskeyd
Copy link
Member

@Freyskeyd Freyskeyd commented Oct 9, 2023

https://img.shields.io/badge/2%20PR-grpc%20over%20p2p-blue

Description

Previous PR: #331
Next PR: #333

This PR adds the definition of the gRPC synchronization protocol, which contains various structs and also service definition.
This protocol will evolve as we implement and update the different kind of synchronization.

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@Freyskeyd Freyskeyd requested a review from a team as a code owner October 9, 2023 18:09
@Freyskeyd Freyskeyd requested review from gruberb and hadjiszs and removed request for a team October 9, 2023 18:09
@Freyskeyd Freyskeyd force-pushed the feature/create-grpc-behaviour branch from 496fcbd to 5534046 Compare October 9, 2023 18:12
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch 2 times, most recently from d2d7615 to f37e7e1 Compare October 9, 2023 19:36
@Freyskeyd Freyskeyd force-pushed the feature/create-grpc-behaviour branch from 5534046 to 88953eb Compare October 9, 2023 19:42
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from f37e7e1 to d71a995 Compare October 9, 2023 19:42
@Freyskeyd Freyskeyd force-pushed the feature/create-grpc-behaviour branch from 88953eb to 7ddfd88 Compare October 10, 2023 06:03
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from d71a995 to d96ddd4 Compare October 10, 2023 06:03
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (06af265) 60.77% compared to head (3a1cc4d) 60.73%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #332      +/-   ##
==========================================
- Coverage   60.77%   60.73%   -0.05%     
==========================================
  Files         235      231       -4     
  Lines       12934    12085     -849     
==========================================
- Hits         7861     7340     -521     
+ Misses       5073     4745     -328     
Files Coverage Δ
crates/topos-api/src/grpc/mod.rs 75.00% <100.00%> (+75.00%) ⬆️
crates/topos-api/tests/tce_layer.rs 100.00% <100.00%> (ø)
crates/topos-p2p/src/behaviour.rs 100.00% <ø> (ø)
crates/topos-p2p/src/network.rs 82.01% <ø> (-0.13%) ⬇️
crates/topos-p2p/src/runtime/handle_event.rs 42.22% <100.00%> (-1.87%) ⬇️

... and 12 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Freyskeyd Freyskeyd force-pushed the feature/create-grpc-behaviour branch 4 times, most recently from 5798914 to db66cf4 Compare October 10, 2023 12:49
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from d96ddd4 to 6116b37 Compare October 10, 2023 13:32
@Freyskeyd Freyskeyd force-pushed the feature/create-grpc-behaviour branch 2 times, most recently from f6f1936 to 2311ede Compare October 10, 2023 15:32
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch 4 times, most recently from 32cab65 to 93a4b64 Compare October 11, 2023 11:04
@Freyskeyd Freyskeyd marked this pull request as draft October 11, 2023 17:09
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch 2 times, most recently from 8e9941a to 896a217 Compare October 11, 2023 21:56
@Freyskeyd Freyskeyd marked this pull request as ready for review October 12, 2023 15:05
Copy link
Contributor

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

LGTM modulo nitpicks

@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from 896a217 to 8bfa1f1 Compare October 16, 2023 23:25
Base automatically changed from feature/create-grpc-behaviour to main October 17, 2023 10:50
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from 8bfa1f1 to 05a3192 Compare October 17, 2023 10:51
Copy link
Contributor

@gruberb gruberb left a comment

Choose a reason for hiding this comment

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

👍

@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from 05a3192 to 43dae86 Compare October 17, 2023 13:44
Signed-off-by: Simon Paitrault <simon.paitrault@gmail.com>
Signed-off-by: Simon Paitrault <simon.paitrault@gmail.com>
Signed-off-by: Simon Paitrault <simon.paitrault@gmail.com>
@Freyskeyd Freyskeyd force-pushed the feature/create-synchronization-protocol branch from 43dae86 to 3a1cc4d Compare October 17, 2023 14:41
@Freyskeyd Freyskeyd merged commit aa21dcc into main Oct 17, 2023
@Freyskeyd Freyskeyd deleted the feature/create-synchronization-protocol branch October 17, 2023 16:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants