Skip to content

Feat/replication#146

Merged
lfbrehm merged 29 commits intodevfrom
feat/replication
Jan 25, 2024
Merged

Feat/replication#146
lfbrehm merged 29 commits intodevfrom
feat/replication

Conversation

@lfbrehm
Copy link
Copy Markdown
Member

@lfbrehm lfbrehm commented Jan 18, 2024

Replication

Adds a DataReplicationService that handles syncing objects to multiple data proxies.
Replication itself is done by the individual data proxies. Only endpoint status updates are managed by the service.

Changes

  • ReplicateProjectData: Updates complete projects with an additional endpoint and sends messages to respective data proxy event streams.
  • PartialReplicateData: Updates subtrees with an additional endpoint and sends messages to respective data proxies.
  • GetReplicationStatus/UpdateReplicationStatus: Gets endpoint status from objects/updates replication info for objects
  • DeleteReplication: Sends delete message to specified proxy (if at least one fullsync proxy remains) and sends update messages to all other proxies.
  • EndpointInfo in Object: Resources now have information of ReplicationStatus (if this is an Object) and ReplicationVariant.

@lfbrehm lfbrehm requested review from St4NNi and das-Abroxas January 18, 2024 15:22
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 18, 2024

Codecov Report

Attention: 606 lines in your changes are missing coverage. Please review.

Comparison is base (f56715a) 50.51% compared to head (11556e0) 48.65%.
Report is 4 commits behind head on dev.

Files Patch % Lines
src/middlelayer/replication_db_handler.rs 0.00% 307 Missing ⚠️
src/grpc/data_replication.rs 0.00% 92 Missing ⚠️
src/middlelayer/update_db_handler.rs 3.92% 49 Missing ⚠️
src/notification/natsio_handler.rs 0.00% 24 Missing ⚠️
src/database/dsls/object_dsl.rs 16.00% 21 Missing ⚠️
src/middlelayer/service_accounts_db_handler.rs 0.00% 19 Missing ⚠️
src/utils/database_utils.rs 0.00% 18 Missing ⚠️
src/middlelayer/create_request_types.rs 63.04% 17 Missing ⚠️
src/utils/conversions.rs 50.00% 17 Missing ⚠️
src/database/dsls/user_dsl.rs 0.00% 9 Missing ⚠️
... and 11 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #146      +/-   ##
==========================================
- Coverage   50.51%   48.65%   -1.86%     
==========================================
  Files          81       82       +1     
  Lines       12997    13615     +618     
==========================================
+ Hits         6565     6625      +60     
- Misses       6432     6990     +558     

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

Copy link
Copy Markdown
Contributor

@das-Abroxas das-Abroxas left a comment

Choose a reason for hiding this comment

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

The last feature for the 2.0 release is coming to an end. We'll see what the test instance has to say as the final word, as the PR is a little too big for a meaningful review.

Good work 👍

@lfbrehm lfbrehm merged commit 576494f into dev Jan 25, 2024
@St4NNi St4NNi deleted the feat/replication branch March 15, 2024 13:04
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.

2 participants