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

Draft: Checks using gtping #13

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Draft: Checks using gtping #13

merged 1 commit into from
Feb 13, 2024

Conversation

vjardin
Copy link
Contributor

@vjardin vjardin commented Feb 13, 2024

XXX do not merge

Thanks to:
https://github.com/ThomasHabets/gtping
target:

  • run gtp-guard per README.md's configuration file
  • gtping -c 3 128.0.0.1 -t 100 # should be ok

It cannot work yet since the former pull request is needed in order to set the pid file because we have the following constraint with github's action:

gtp-guard: pidfile_write : Can not open /var/run/gtp-guard.pid pidfile

Thanks to:
   https://github.com/ThomasHabets/gtping
target:
  - run gtp-guard per README.md's configuration file
  - gtping -c 3 128.0.0.1 -t 100 # should be ok
@vjardin vjardin marked this pull request as draft February 13, 2024 01:47
@acassen acassen marked this pull request as ready for review February 13, 2024 09:41
@acassen acassen merged commit c8cf077 into acassen:main Feb 13, 2024
3 of 4 checks passed
@vjardin vjardin deleted the gtping_test branch February 21, 2024 07:40
@vjardin vjardin restored the gtping_test branch April 30, 2024 05:59
@vjardin vjardin deleted the gtping_test branch April 30, 2024 07:02
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