Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add config file watch service #1570

Merged
merged 4 commits into from
May 22, 2023
Merged

Conversation

K1li4nL
Copy link
Contributor

@K1li4nL K1li4nL commented May 16, 2023

This pr adds the ability for the system to react on the fly to changes in the config files added in #1519.

Limitations: Without greet_server implementation we are still susceptible to having double connections (A to B and B to A, over two websockets).

@K1li4nL K1li4nL added be2-scala decentralized Decentralized Communication Project labels May 16, 2023
@K1li4nL K1li4nL force-pushed the work-be2-lauener-add-peers-greetLao branch from 7f0939c to db5c785 Compare May 18, 2023 10:49
@K1li4nL K1li4nL force-pushed the work-be2-lauener-add-config-file-watch branch 2 times, most recently from 39ed6f5 to af53079 Compare May 18, 2023 13:26
@K1li4nL K1li4nL marked this pull request as ready for review May 18, 2023 13:31
@K1li4nL K1li4nL requested a review from a team as a code owner May 18, 2023 13:31
@K1li4nL K1li4nL force-pushed the work-be2-lauener-add-peers-greetLao branch from 2342b8b to f293b83 Compare May 19, 2023 10:49
@K1li4nL K1li4nL force-pushed the work-be2-lauener-add-config-file-watch branch from af53079 to 88f603a Compare May 19, 2023 10:50
Copy link
Contributor

@Jharaxus Jharaxus left a comment

Choose a reason for hiding this comment

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

Good job, the tests of FileMonitor are very nice !

@K1li4nL K1li4nL force-pushed the work-be2-lauener-add-config-file-watch branch from 88f603a to 454a91b Compare May 21, 2023 10:23
@sonarcloud
Copy link

sonarcloud bot commented May 21, 2023

[PoP - Be2-Scala] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

86.7% 86.7% Coverage
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented May 21, 2023

[PoP - Be1-Go] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarcloud
Copy link

sonarcloud bot commented May 21, 2023

[PoP - Fe2-Android] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarcloud
Copy link

sonarcloud bot commented May 21, 2023

[PoP - Fe1-Web] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Base automatically changed from work-be2-lauener-add-peers-greetLao to master May 22, 2023 11:46
Copy link
Contributor

@Ajkunas Ajkunas left a comment

Choose a reason for hiding this comment

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

LGTM

@K1li4nL K1li4nL merged commit db5d3da into master May 22, 2023
@K1li4nL K1li4nL deleted the work-be2-lauener-add-config-file-watch branch May 22, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
be2-scala decentralized Decentralized Communication Project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants