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

Improving ipman service #207

Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Improving ipman service #207

wants to merge 2 commits into from

Conversation

DT3264
Copy link
Collaborator

@DT3264 DT3264 commented Sep 4, 2023

Improving ipman service

This PR improves the IPMan service to be more rock solid (restar automatically if failed and to not be re-runed if it was executed succesfully.


Stack created with Sapling. Best reviewed with ReviewStack.

@DT3264 DT3264 changed the title IPman improving Improving ipman service Oct 8, 2023
@DT3264 DT3264 force-pushed the pr207 branch 2 times, most recently from 2da3080 to 3a47f35 Compare October 9, 2023 00:00
@DT3264 DT3264 marked this pull request as ready for review October 9, 2023 00:25
DT3264 added 2 commits October 8, 2023 18:26
Updates IPMan to
- ping the directives server (if available) since sometimes the gateway might be blocked, it should not affect if the directives server is empty since a ping with an empty command just throws an error which is handled on the if
This PR improves the IPMan service to be more rock solid (restar automatically if failed and to not be re-runed if it was executed succesfully.
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.

1 participant